Python-сообщество

Rodegast
2) Не надо считать пробелы.

Rodegast
Вот я сейчас попробовал это проделать в Geany, по нажатию на Tab он как и положено ставит табуляцию

Rodegast
в итоге пробелы и табуляция смешивается что не допустимо

Rodegast
3) На табуляцию я не переходил. Я ей постоянно пользовался.

Отредактировано py.user.next (Апрель 29, 2018 00:05:27)

druidich92
можете подсказать, как этот код можно сделать более читаемый?

Отредактировано py.user.next (Апрель 29, 2018 00:18:52)

> Табуляция выглядит по-разному в разных редакторах.

Меня это не напрягает.

Rodegast
1) Geany адекватен? …
2) Это нормальная практика когда на свои серверы люди притаскивают свои конфиги vim/emacs.

>> Меня это не напрягает.

Слушай, да для тебя и английский не нужен, и еще куча всего того странного что ты на этом форуме высказал. Ты здесь представитель “иного” мышления, и я очень сильно подбирал слова.

Мое мнение что ты не адекватен. Посему зарекаюсь больше не вступать с тобой в дискуссии. Да да тебе оно и не надо и вообще фиолетово. Я больше сказал это что бы публично было.

Rodegast
1) У тебя не функция, а метод.
2) Документация не соответствует действительности
3) Давай переменным короткие имена
4) Для отступов используй табуляцию
5) Используй переносы в словарях и списках
6) Изменяемое состояние это зло. Не надо его так агрессивно использовать! В место списков izo_paraphine_data и norm_paraphine_data создай 4 списка и нормально заполняй их не пользуясь индексами.
P.S. PEP8 это анахронизм не используй его.

> Geany адекватен? …

А ХЗ, это первое что мне попалось. Смысл в том что я не привязан к редактору/IDE. Я хоть в блокноте могу исходники редактировать и у меня с отступами проблем не будет.

> Это нормальная практика когда на свои серверы люди притаскивают свои конфиги vim/emacs.

На свои перетаскивают, а на чужие нет.

> Ты здесь представитель “иного” мышления

Совершенно верно. Я не мыслю шаблонами! Вот тебе какой-то “дядя” дал установку что отступ должен быть только пробелами и ты этому слепо следуешь, хотя табы дают ряд преимуществ. Надо ко всему относится критически.

> 3- встречал наоборот суждение, что лучше давать нормальные осознанные имена.

Имя должно быть в первую очередь осмысленным, по этому тебе и писали что например не надо использовать однобуквенные имена. Но длинные имена тоже плохи, они более сложны для восприятия. По этому необходимо давать осмысленные имена минимальной длины. Например у izo_paraphine_data и norm_paraphine_data можно отбросить общую часть имени и будет просто izo и norm.

> 6 - не понимаю что за изменяемое состояние.

Это в первую очередь изменяемые объекты. Я не буду рассказывать почему это зло (вот можешь тут подробности почитать http://fprog.ru/2009/issue1/eugene-kirpichov-fighting-mutable-state/ ), но в императивном программировании без него не обойтись и работать с ним надо предельно аккуратно. Например ты создаёшь список содержащий 2 списка, потом их изменяешь обращаясь по индексам, а потом “вытаскиваешь” эти списки из общего списка и формируешь результат. Очевидно что эту конструкцию лучше заменить четырьмя одномерными списками.

Отредактировано Rodegast (Апрель 29, 2018 13:56:38)

Rodegast
а можно еще совета попросить. Уже в плане алгоритма.

     def paraphine_distribution(self):
        '''
            распределение всего состава по 3 группам: нормальные парафины, 
            изопарафины и водород. Учитываются компоненты, частота появления 
            которых не равно нулю
            :param data: входные данные
            :type data: calculate_file
        '''
        izo_component = []
        izo_quantity = []
        norm_component = []
        norm_quantity = []
        h2_component = [x for x in np.linspace(-300, -250, 6)]
        h2_quantity = [random.randint(0, 30) for x in range(0, 6)]
        h2_quantity[2] = 100
        for i, item in enumerate(self.full_components):
            if self.full_composition[-1][i] != 0:
                if item in self._criterion_norm_paraphine:
                    norm_component.append(item)
                    norm_quantity.append(self.full_composition[-1][i])
                elif item in self._criterion_izo_paraphine:
                    izo_component.append(item)
                    izo_quantity.append(self.full_composition[-1][i])
        izo_composition = {
            'x': np.array(izo_component),
            'y': np.array(izo_quantity)
        }
        norm_comosition = {
            'x': np.array(norm_component),
            'y': np.array(norm_quantity)
        }
        h2_composition = {
            'x': np.array(h2_component),
            'y': np.array(h2_quantity)
        }
        return izo_composition, norm_comosition, h2_composition

данный метод у меня почти дублируется. только в одном случае он работает когда

 if self.full_composition[-1][i] != 0:

а в другом когда

 if self.full_composition[-1][i] == 0:

а все остальное - одно и тоже…. как с этим можно поступить ?

Самое очевидное решение этой проблемы это создать предикат и передавать его в метод как параметр:

 def paraphine_distribution(self, test):
        ....
       if test(i):
              ....
        ....
 
self.paraphine_distribution(lambda i: self.full_composition[-1][i])

Отредактировано Rodegast (Май 1, 2018 18:58:24)

Rodegast
Самое очевидное решение этой проблемы это создать предикат и передавать его в метод как параметр:

Выкладывай. Помогу чем смогу…