fix metro iteration, refactor only_apartment and only_flat, min_balconies and metro_foot_minute params

Author: lenarsaitov

README.md

@@ -91,7 +91,7 @@ additional_settings = {
     "is_by_homeowner": True,
     "min_price": 1000000,
     "max_price": 10000000,
-    "have_balconies": True,
+    "min_balconies": 1,
     "have_loggia": True,
     "min_house_year": 1990,
     "max_house_year": 2023,
@@ -103,29 +103,31 @@ additional_settings = {
     "metro": "Московский",
     "metro_station": "ВДНХ",
     "metro_foot_minute": 45,
+    "flat_share": 2,
+    "only_flat": True,
+    "only_apartment": True,
     "sort_by": "price_from_min_to_max",
 }
 ```
 * __is_by_homeowner__ - объявления, созданных только собственниками
-* __min_price__ - цена от
-* __max_price__ - цена до
-* __have_balconies__ - наличие балкона
-* __have_loggia__ - наличия лоджи
+* __min_price__ - цена от (в рублях)
+* __max_price__ - цена до (в рублях)
+* __min_balconies__ - минимальное количество балконов
+* __have_loggia__ - наличие лоджи
 * __min_house_year__ - год постройки дома от
 * __max_house_year__ - год постройки дома до
 * __min_floor__ - этаж от
 * __max_floor__ - этаж до
 * __min_total_floor__ - этажей в доме от
 * __max_total_floor__ - этажей в доме до
-* __house_material_type__ - тип дома
-* __metro__ - название метрополитена
+* __house_material_type__ - тип дома (_см ниже возможные значения_)
+* __metro__ - название метрополитена (_см ниже возможные значения_)
 * __metro_station__ - станция метро (доступно при условии, что задано metro)
-* __metro_foot_minute__ - сколько минут до метро пешком (доступно при заданном параметре metro_station)
-* __metro_foot_minute_special__ - сколько минут до метро пешком (без указания конкретной станции)
-* __flat_share__ - с долями или без (2 - без долей, 1 - только доли)
-* __only_flat__ = 1 - без апартаментов
-* __apartment__ = 1 - только апартаменты
-* __sort_by__ - сортировка объявлений
+* __metro_foot_minute__ - сколько минут до метро пешком
+* __flat_share__ - с долями или без (1 - только доли, 2 - без долей)
+* __only_flat__ - без апартаментов
+* __only_apartment__ - только апартаменты
+* __sort_by__ - сортировка объявлений (_см ниже возможные значения_)
 
 Для **house_material_type** возможны следующие из значений
 - _1_ - киричный

cianparser/constants.py

@@ -9,7 +9,7 @@
 STUDIO = "&room9=1"
 WITHOUT_NEIGHBORS_OF_CITY = "&with_neighbors=0"
 IS_ONLY_HOMEOWNER = "&is_by_homeowner=1"
-HAVE_BALCONIES = "&min_balconies=1"
+MIN_BALCONIES = "&min_balconies={}"
 HAVE_LOGGIA = "&loggia=1"
 MIN_HOUSE_YEAR = "&min_house_year={}"
 MAX_HOUSE_YEAR = "&min_house_year={}"
@@ -25,8 +25,6 @@
 METRO_FOOT_MINUTE = "&only_foot=2&foot_min={}"
 METRO_ID = "&metro%5B0%5D={}"
 
-METRO_FOOT_MINUTE_SPECIAL = "&foot_min={}"
-
 FLAT_SHARE = "&flat_share={}"
 ONLY_FLAT = "&only_flat={}"
 APARTMENT = '&apartment={}'
@@ -567,6 +565,7 @@
         ['Апрелевка', '494'],
         ['Победа', '495'],
         ['Крёкшино', '496'],
+        ['Санино', '497'],
         ['Кокошкино', '498'],
         ['Толстопальцево', '499'],
         ['Лесной Городок', '500'],
@@ -596,7 +595,7 @@
         ['Физтех', '533'],
         ['Аэропорт Внуково', '535'],
         ['Пыхтино', '536'],
-        ['Марьина Роща,' '537'],
+        ['Марьина Роща', '537'],
     ],
     "Казанский": [
         ['Северный Вокзал', '314'],

cianparser/parser.py

@@ -99,8 +99,8 @@ def build_url(self):
         if self.additional_settings is not None:
             if "is_by_homeowner" in self.additional_settings.keys() and self.additional_settings["is_by_homeowner"]:
                 url += IS_ONLY_HOMEOWNER
-            if "have_balconies" in self.additional_settings.keys() and self.additional_settings["have_balconies"]:
-                url += HAVE_BALCONIES
+            if "min_balconies" in self.additional_settings.keys():
+                url += MIN_BALCONIES.format(self.additional_settings["min_balconies"])
             if "have_loggia" in self.additional_settings.keys() and self.additional_settings["have_loggia"]:
                 url += HAVE_LOGGIA
 
@@ -129,27 +129,24 @@ def build_url(self):
 
             if "metro" in self.additional_settings.keys():
                 if "metro_station" in self.additional_settings.keys():
-                    is_available_metro = False
                     if self.additional_settings["metro"] in METRO_STATIONS.keys():
                         for metro_station, metro_id in METRO_STATIONS[self.additional_settings["metro"]]:
                             if self.additional_settings["metro_station"] == metro_station:
                                 url += METRO_ID.format(metro_id)
-                                is_available_metro = True
 
-                    if "metro_foot_minute" in self.additional_settings.keys() and is_available_metro:
-                        url += METRO_FOOT_MINUTE.format(self.additional_settings["metro_foot_minute"])
-
-            if "metro_foot_minute_special" in self.additional_settings.keys():
-                url += METRO_FOOT_MINUTE_SPECIAL.format(self.additional_settings["metro_foot_minute_special"])
+            if "metro_foot_minute" in self.additional_settings.keys():
+                url += METRO_FOOT_MINUTE.format(self.additional_settings["metro_foot_minute"])
 
             if "flat_share" in self.additional_settings.keys():
                 url += FLAT_SHARE.format(self.additional_settings["flat_share"])
 
             if "only_flat" in self.additional_settings.keys():
-                url += ONLY_FLAT.format(self.additional_settings["only_flat"])
+                if self.additional_settings["only_flat"]:
+                    url += ONLY_FLAT.format(1)
 
-            if "apartment" in self.additional_settings.keys():
-                url += APARTMENT.format(self.additional_settings["apartment"])
+            if "only_apartment" in self.additional_settings.keys():
+                if self.additional_settings["only_apartment"]:
+                    url += APARTMENT.format(1)
 
             if "sort_by" in self.additional_settings.keys():
                 if self.additional_settings["sort_by"] == IS_SORT_BY_PRICE_FROM_MIN_TO_MAX: