Дневник капитана, желтая дАтА(2020, 3, 25)
Я часто страдаю перфекционизмом в разработке. Т.е. сразу продумываю архитектуру решения, чтобы было универсально, чтобы с заделом на будущее развитие и пишу чистый красивый код, даже если знаю, что его, скорее всего, никто не увидит. Но вот иногда задумываюсь, а не теряю ли я время, не перегибаю ли палку в этой погоне за мнимым совершенством?
Особенно задумываюсь, когда вижу код как на картинке снизу. Тут все прекрасно: форматирование (не понимаю, так сложно
Alt+Shift+F
нажать?), нейминг переменных, общий стиль... Кроме ошибок оформления, конечно, есть еще и реальные, которые могут влиять на производительность и надежность. Еще на формах встречаются такие перлы:
Процедура Команда16(Команда)
// Вставить содержимое обработчика.
Если не Элементы.Таблица1.ТекущиеДанные=неопределено Тогда Таблица1Выбор(Команда,Элементы.Таблица1.ТекущаяСтрока,Команда,Истина);
КонецЕсли;
КонецПроцедуры
Самое интересное, что эта
Команда16
привязана к элементу
Кнопка9
. Я вообще не понимаю, как это происходит. Ну и добивает то, что весь этот код встречается в конфе по 10, а то и больше раз с минимальными изменениями. Т.е. копипаст, никакого повторного использования.
Так вот, особенно я задумываюсь здесь потому, что это код из довольно популярного и коммерчески успешного продукта. И люди делают реально нужные и крутые вещи, идут на опережение. Работают часто в авральном режиме, чтобы сдать клиентам побыстрее, вот такое и выходит. Пользователи, вроде, довольны. Авторы в прибыли. Так может зря я тогда стараюсь, какой-то внутренней красоты добиваюсь?
Что вы думаете по этому поводу? Как пишете код вы? Ну и мини-конкурс. Пишите в группе, какие ошибки вы нашли в приведенном коде? Только канал у меня молодой, технику, электронику в качестве приза предложить не могу, поэтому можем сыграть только на интерес и чувство гордости за себя.