Пособие по рефакторингу кода из 1975 года

 Публичный пост

Короткий фильм "Critical Program Reading", с восстановленным цветом. Несмотря на то, что там почти 18 минут, сейчас, конечно, такое видео на Ютубе можно растянуть настолько только если впихнуть туда рассказ о своём детстве или пару рекламных интеграций пельменей. А когда-то такие материалы же действительно смотрели в аудиториях и лекториях, и с настройкой оборудования это наверняка занимало половину академического часа.

Забавно видеть, что большинство концепций в целом можно показать текущим джунам и мидлам, и это всё ещё будет полезно.

Здесь есть прописные истины "не называйте переменные км, называйте их количество_мешков", и более продвинутые мысли "избегайте излишних вложений (nesting)".

Да, тут всё ещё magic constants, и scope визуальный мы получаем сейчас "бесплатно" автоформатерами, но на фоне типичных тем "ого, как же быстро движется айти" - приятно удвляет, что некоторые вещи практически не изменились за 50 лет.

P.S. За замену "<= 0.33" на "< 0.33" и ">= 0.2" на "> 0.2" я бы конечно на ревью завернул!

3 комментария 👇
Egor Suvorov Программист/преподаватель C++ 3 января 2023

О, я это недавно в очередном выпуске Technology Connections видел, но не справился найти оригинал.

  Развернуть 1 комментарий

А где ты увидел !< 0.2?

Конец рефакторинга: `<0.20` и `<0.33`
Конец рефакторинга: `<0.20` и `<0.33`

К тому же исходный код:

IF RR IS LESS THAN .20
THEN GO TO END-OF-ANALYSIS.
IF RR IS LESS THAN OR EQUAL TO .33
THEN ADD ONE TO HRC,

В данном коде проверки тоже < 0.20 и <= 0.33

  Развернуть 1 комментарий

@Mysh,

вот конец
вот конец

раньше при <.2 прыгало в EOF, при >=.2 исполнялся бранч
теперь при >.2 - опять поменяли логику работы

на всякий случай чуть поправил исходный пост

  Развернуть 1 комментарий

😎

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

Что вообще здесь происходит?


Войти  или  Вступить в Клуб