Обновить
14

Химик и программист.

32
Подписчики
Отправить сообщение

Ok. Договорились, что комментарии нужны (в разумных объемах).

И чтобы узнать, что делает этот код нужно лезть в отдельную процедуру.

а вы в этом случае комментарий сверху тоже обновите?

Конечно.

decrypt м.б. нетривиальный по новому алгоритму и
loadFromArchive может быть новым. Может новый драйвер на асм.

Пустые строки сродни комментам?


Кроме обработки ошибок, потом может новый код добавиться.

Ну ИМХО это уже философский вопрос: насколько реальность отвечает нашим ожиданиям, Статья про разработку новых алгоритмов, и, в этом случае, комменты особо важны: читаем, что хотел автор, и смотрим код: что у него получилось.

Завозили, но речь не о том. Речь про отказ от комментов и про использование только значимых имен. Чем больше строк между такими именами — тем труднее читать, тем привлекательней комменты.

Если две строки рядом: одна за другой, то читать легко. А если:


procedure loadFile;
var
 err : integer;
begin
 err:= loadFromArchive;
if err<0 then
begin
 if err=-1 then
  print ('I/O error')
else
 if err=-2 then
 print ('File not found')
else
 print ('Unknown error');
exit
end;
decrypt;
  //и так далее
end

Можно записать коммент в определение процедуры loadFile:


procedure loadFile;
//загрузить из архивного файла, записи для 
//товарных вагонов, расшифровать их, пустые 
//вагоны добавить в таблицу свободных вагонов
begin

В Вашем решении не понял: что делают методы freight и free?

В процедуре loadFile; просто формат Хабра не позволяет разместить этот вызов и следующий комментарий в одну строку. На практике такой проблемы нет.

Cоотносится, как и всякий комментарий. Нпр.:


begin
loadFile;
//загрузить из архивного файла, записи для 
//товарных вагонов, расшифровать их, пустые 
//вагоны добавить в таблицу свободных вагонов
nextProc;
end;

Я показал:


//загрузить из архивного файла, записи для 
//товарных вагонов, расшифровать их, пустые 
//вагоны добавить в таблицу свободных вагонов
Если что-то обновляется — то я только рад.

Интересно, а если Ваша банковская карта будет обновляться каждую неделю, то каждый понедельник Вы радостно побежите в банк за новой?


Больше применения в 2021 году я ей найти не могу.

Т.е. Вы не учитесь на опыте старых мастеров?


Я уверен, что человеку, который не может зарядить телефон по причине "полевых условий" не нужен ни Хабр ни Винда 11

К сожалению, любой человек может попасть в авиакатастрофу. Любой человек может заблудиться в лесу. Если выжил, то нужен телефон, чтобы вызвать спасателей. Кроме того есть туристы и геологи, где они ходят розеток иногда нет.


И Вы не ответили на вопрос о новом дизайне Хабра: он Вам нравится?

Программисты — они понятливые, но вместо того, чтобы в одной строке коммента прочесть, программисту нужно бегать по разным функциям. Что быстрее? эргономичнее?

Есть: загрузить из архивного файла, записи для товарных вагонов. Это может значить найти и загрузить.


Пойнт в том, что для получателя, которому нужен список свободных вагонов, пофиг, пришлось вам их расшифровывать, или нет.

Согласен, что конечному юзеру пофиг. А программисту который пришел на место разраба очень важно понять, почему в методе добавитьИзАрхивногоФайла после чтения идет много строк кода. Напомню, эта ветка обсуждения началась с идеи отказа от комментов, и замены их именами.

Не значит. Нпр., для ускорения поиска по БД поле "тип записи" можно не шифровать. Если тип записи = "ТВ", то это запись товарного вагона.

Очень большая. С шифрованными записями прога не сможет работать.

Я не критикую новый дизайн...

Не опасаетесь, что многие воспримут некритичный подход, как минус? А старый дизайн критикуете? — Если да, то интересно услышать Вашу критику на картинку в начале статьи.
И новый дизайн Хабра Вам нравится больше старого?


Да и Винда 11 мне нравится больше MSDOS.

ИМХО Вы сравнили несравнимое. Давайте сравнивать GUI OS. Что Вам нравится больше Винда 11 или MacOS?


Я думаю, очевидный факт, что калькулятор явно лучше счёт.

Это когда доступна розетка для периодической зарядки. А в полевых условиях, без бортового эелектрогенератора автомобиля, калькулятор скоро станет бесполезен. Придется на счетах или на бумажке.

Ну и как впихнуть в значимое имя фразу: загрузить из архивного файла, записи для товарных вагонов, расшифровать их, пустые вагоны добавить в таблицу свободных вагонов? Так чтобы было понятно какие действия совершает прога.

Спасибо,
попробую. Но в общем придерживаюсь принципа KISS («Keep it simple, stupid»). Если легко могу без чего-то обойтись, то режу бритвой Оккама.

Информация

В рейтинге
Не участвует
Зарегистрирован
Активность