[recovery mode] Призрак локомотива или биржевой рынок через призму корреляций28.07.2017 11:33

Комментарии (6)

  • 28 июля 2017 в 07:21 (комментарий был изменён)

    +1

    ↑

    ↓

    Можно поинтересоваться, почему не использовались конструкции типа string.format или string.Template или в нескольких фрагментах вида: 


    page = urllib.urlopen('http://export.finam.ru/'+str(row['code'])+'_'+str(year_start)+str(month_start)+str(day_start)+'_'+str(year_end)+str(month_end)+str(day_end)+str(e)+'?market='+str(row['id_exchange_2'])+'&em='+str(row['em']).....

    • 28 июля 2017 в 09:20

      0

      ↑

      ↓

      Ответ, думаю, прост: автор совершенно не озабочен чистотой кода, а про PEP-8, видимо, вообще не слышал, отсюда и такие нечитаемые конструкции. Хотя в остальном статья интересная.

      • 28 июля 2017 в 09:30

        0

        ↑

        ↓

        Конструкции очень даже читабельные советую больше не только писать, но и читать не свой код. За «интересная» — спасибо; здорово, если материал окажется вам полезным.
        Вообще, буду рад сообщениям и письмам — какие направления были бы интересны.
        Возможно, они будут темами следующих статей, я открыт для общения.

        • 28 июля 2017 в 10:22

          0

          ↑

          ↓

          безумные строки это еще полдела, в коде куча штук типа: 
          if:
    много много много строк
else:
    continue

          или unlink на файл в каждой ветке двух вложенных if.
          Очень рекомендую кому-то дать свой код на ревью перед публикацией, текущий вариант показывать нельзя.

          • 28 июля 2017 в 10:34

            0

            ↑

            ↓

            Ок, спасибо за рекомендацию. Просто таким образом логика программы максимально наглядна. Даже для начинающих. Исходил из этого.

  • 28 июля 2017 в 09:20

    0

    ↑

    ↓

    Надо учитывать введение санкций и другие новости. Они могут двигать весь рынок, т.к. в РФ он маленький.

© Habrahabr.ru