ninazino (ninazino) wrote,
ninazino
ninazino

Categories:

Дыбр рабочий

Пришлось тут подибажить питонский скрипт.  Это — пестня.

Есть массив токенов (в просторечии — слов, поскольку ни один из остальных полей класса токена нигде не используется, только token.text), которые надо как-то нормализовать.  Есть двадцать примерно методов, каждый из которых делает что-то свое: кто-то преобразовывает даты, кто-то числа, кто-то аббревиатуры, и т.д.

Так вот, каждому из этих методов отсылается весь массив токенов, и каждый из методов проверяет токен  на пригодность  для этого метода преобразования.  То есть, двадцать (или сколько там по количеству вызываемых методов) loops.  Как это, кстати, по-русски?  Ну не «петель» же?  Или таки петель?

Но и это еще не самый главный грех.  Вишенка на торте: метод сначала спрашивает некую вспомогательную функцию, типа, может это быть датой?  Вспомогательная функция на основе самых примитивных критериев отвечает, что может.  После этого недоверчивый метод проверяет, а вдруг число, репрезентирующее месяц, больше 12-ти, а число, представляющее день, больше 31-го (шепотом: проверки на то, что эти числа могут быть меньше единицы — нет!  А в массивах такие варианты — есть).  

И вот, получив токен, который был заподозрен в том, что он — дата, но оказался неспособным соответствовать критерию, метод возвращает все, ВСЕ токены обратно, не проверяя, есть ли в массиве после этой неудачной даты другие, возможно, удачные.

А теперь гендерное.  Я, к примеру, ни одной секунды не сомневаюсь, что этот кривой питонский скрипт написан... как бы это помягше?

This entry was originally posted at https://ninazino.dreamwidth.org/2118995.html
Subscribe

  • Никакой войны не будет

    Раисся сама по себе вообще не способна раскрутить войну: Вторую Миоровую ей помогли, а сама она ни на что не способна. Да нет, по-тихому отжать в…

  • И нет, я не отниму у либерализма

    …такой сладкой конфетки, как очередной омикрон. Как, скажите, как от нас убежали все те буквы греческого алфавита, которые между дельтой и…

  • Все сосем не так, как на самом деле:))

    Но на самом деле -- нет никаких "леваков" против "праваков" (кстати, этого слова я не слыхала, хотя "леваками" весь русскоязычный интернет засран).…

  • Post a new comment

    Error

    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded 

  • 21 comments

  • Никакой войны не будет

    Раисся сама по себе вообще не способна раскрутить войну: Вторую Миоровую ей помогли, а сама она ни на что не способна. Да нет, по-тихому отжать в…

  • И нет, я не отниму у либерализма

    …такой сладкой конфетки, как очередной омикрон. Как, скажите, как от нас убежали все те буквы греческого алфавита, которые между дельтой и…

  • Все сосем не так, как на самом деле:))

    Но на самом деле -- нет никаких "леваков" против "праваков" (кстати, этого слова я не слыхала, хотя "леваками" весь русскоязычный интернет засран).…