|
Top-level
 Пост интересный, но одно место вызывает вопросы) > Так делать не нужно. По двум причинам: если это π, оно никогда не изменится, и его никто не спутает с ФИО генерального. Не соглашусь. Однако бывают такие случаи, когда нужно положить явную константу, не предполагаемую к изменению через «настройки», потому что это может, например, сломать логику или привести к каким-нибудь странным сайдэффектам. 11 comments
@mudasobwa тебе не нужно, ты её помнишь [сейчас], а человеку, который будет потом читать этот код - нужно  @rubikoid зачем? Это не праздный вопрос. Нет ни единой причины, по которой он должен знать, что вот это число называется вот так. @mudasobwa потому что человеку, который [потом] будет работать с этим кодом, надо полностью понимать, что этот код делает? Добавлю абсурда: при таком подходе можно не именовать локальные переменные - зачем, по ним же и так всё видно. @rubikoid я эти клише слышал миллион раз, но спросить не поленюсь: как именование константы поможет понять, что этот код делает? Не общими словами только. И вообще, сколько раз проблемы с пониманием кода упирались в магическую константу без имени? Типа, знал бы, как зовут вот этот коэффициент — сразу бы все понял, а так — как в тумане? @mudasobwa как минимум, название константы может гуглиться. Причем с ненулевой вероятностью - лучше, чем сама константа. Кроме того, ассоциативная связь название константы -> алгоритм сильнее, чем сама константа -> алгоритм. @rubikoid это все рассказанная людьми, которые никогда всерьез не программировали, теория. А вот практика: сколько раз лично вам доводилось гуглить константу из чужого кода? Это простой вопрос, но него можно ответить прямо, без цитат из вредных мотивационных книг. @rubikoid это хороший пример, он действительно живой, вот только он полностью закрыт моим предложением вынести такую настройку в конфиг. Потому что она должна адекватно проверять stdout, и не печатать ничего, если он не терминал, а пайп, например. > Я нахожу сомнительным тот факт, что кому-либо вообще может понадобится конфигурировать то, каким цветом в консоли рисуется OK. Да ладно. Мои приложения не только разными цветами логи пишут, но еще и каждое свой эмоджик использует. @rubikoid если у меня половина вывода из других мест зеленая — то на синий может захотеться. Но в любом случае, я не воевал за копипасту, если оно используется в трех местах — надо избавиться от изменения в трех местах заранее, это не про читаемость вообще. |
Gregory's Server
@rubikoid и если ее не нужно менять — чем определение по месту хуже? Да, я знаю, что принято считать, будто бы читать `x *= 2.3456` сложно, а `COEFF_OF_GUAZILLIATION = 2.3456 […] x *= COEFF_OF_GUAZILLIATION` просто, но это ведь растиражированная ерунда.
Если это константа, я ее либо узна́ю в лицо, либо мне не нужно знать, что она — коэффициент гуазиллизации.