комментарии (15)

• НЛО прилетело и опубликовало эту надпись здесь.
  • • 
    • zdanchik,
    • 30 августа 2010, 17:13
    • #
    • ↑
    • ↓
    • • 0
    Пока проект находится в закрытом тестировании. Здесь описан лишь алгоритм.
• • 
  • protagonist,
  • 30 августа 2010, 15:29
  • #
  • ↓
  • • 0
  Не понял смысл статьи.
  «У нас был файл, мы написали парсер, он распарсился».
  Ну я каждый день такое делаю, так и шо?
  • • 
    • zdanchik,
    • 30 августа 2010, 17:15
    • #
    • ↑
    • ↓
    • • 0
    Распарсить файл — практически всегда дело плевое… Однако в данном случае акцент нужно сместить в сторону: «как реализовать верный алгоритм». Ведь, впервые столкнувшись с подобной задачей, не мудрено наломать дров…
• • 
  • TheShock,
  • 30 августа 2010, 15:37
  • #
  • ↓
  • • +3
  позволите, у меня небольшое замечание по коду. все-же, это не очень элегантное решение:
  

  elseif (
   mb_strpos(mb_strtolower($addrItem, 'UTF-8'), 'просп. ')
|| mb_strpos(mb_strtolower($addrItem, 'UTF-8'), ' просп.')
|| mb_strpos(mb_strtolower($addrItem, 'UTF-8'), 'пр-т')
|| mb_strpos(mb_strtolower($addrItem, 'UTF-8'), 'проспект')
)


  
  Достаточно объявить всего две ф-ции:
  

  function contains ($in, $str) {
    return mb_strpos(mb_strtolower($in, 'UTF-8'), $str);
}
function containsOne ($in, array $array) {
    foreach ($array as $str) {
        if (contains($in, $str)) {
            return true;
        }
    }
    return false;
}

  
  и мы получаем намного более изящный результат, код уменьшается в раз 5 и становится намного более читаемым и расширяемым:
  

  elseif (containsOne($addrItem, array('просп. ', ' просп.', 'пр-т', 'проспект')))

  
  
  
  • • 
    • TheShock,
    • 30 августа 2010, 15:42
    • #
    • ↑
    • ↓
    • • +1
    но рефакторить код есть куда. очень много мест. сейчас он вообще плох.
    • • 
      • TheShock,
      • 30 августа 2010, 15:45
      • #
      • ↑
      • ↓
      • • +1
      еще непонятно, что будет, если ищущийся итем стоит в начале строки. например «г. Такой-то, просп. Другой-то». Корректно проверять как
      
      • • 
        • TheShock,
        • 30 августа 2010, 15:45
        • #
        • ↑
        • ↓
        • • +1
        *корректно проверять так:
        
        • • 
          • TheShock,
          • 30 августа 2010, 15:46
          • #
          • ↑
          • ↓
          • • +1
          та блин. вот так, то есть сравнивая c false:
          

          mb_strpos(mb_strtolower($in, 'UTF-8'), $str) === false;

          
          
  • • 
    • zdanchik,
    • 30 августа 2010, 17:21
    • #
    • ↑
    • ↓
    • • 0
    Да, согласен. Код писался на скорую руку и очень сыр. Пусть не будет звучать как оправдание, но целью данного кода был разовый разбор имеющихся данных. Цель оправдывает средства.
    • • 
      • TheShock,
      • 30 августа 2010, 18:30
      • #
      • ↑
      • ↓
      • • 0
      я бы постеснялся выкладывать столь сырой код, да еще и сообщая, что это библиотека.
      да и ведь неудобно так разрабатывать то было. задолбаться ведь можно
• • 
  • overPlumbum,
  • 30 августа 2010, 15:43
  • #
  • ↓
  • • 0

  Первый – самый простой и не надежный: скормить адрес имеющихся фирм геокодеру Яндекса, который и разберет адрес на кусочки. Но есть и большой минус этого способа – если в базе геокодера такого адреса по какой то причине не окажется, то он вернет ближайшее к указанному месту строение…
  

  
  а в чём тут проблема? он же сообщает с какой точностью найден объект: api.yandex.ru/maps/jsapi/doc/ref/reference/geocoderresult.xml#precision
  • • 
    • zdanchik,
    • 30 августа 2010, 17:12
    • #
    • ↑
    • ↓
    • • 0
    Дело в том, что случаются моменты, когда дома, находящиеся по одной улице, могут находится в совершенно разных районах. Был такой случай, что геокодер определял дом, который находится не в соответствующем районе
• • 
  • Covex,
  • 2 сентября 2010, 16:26
  • #
  • ↓
  • • 0

  нарисовать карты районов для всей России осилит не каждый

  Они все уже нарисованы и храняться в wikimapia. Можно подключить слой от WikiMapia в Google Earth и сохранять полигоны в kml-файлы.
  
  Я такое безобразие делал для сервиса частных объявлений о недвижимости. Способ, может быть и не совсем прямой, но для Нижнего его таки можно осилить =)
  
• • 
  • UshkiNaMakushke,
  • 13 июля 2011, 15:52
  • #
  • ↓
  • • 0
  Спасибо, ваша статья очень помогла, а то я уже мозг взорвала над тем где мне все таки получить внутригородские р-ны в кладаре, а их так и нет вовсе