Wake up, Neo!
Нашёл в просторах многолетнего рабочего проекта 😂
А расскажите, пожалуйста, непрограммисту, что сей кусок кода делает? Точнее в каком случае она начинает будить Нео?
раскрыть ветку (32)
Вероятно, в прошлых версиях этого кода что-то шло не так. Программист написал специальную проверку на "что-то не так", и если оно опять случилось - то получает в консоли сообщение для самого себя - типа "чувак, ты опять не осилил сделать правильно".
раскрыть ветку (6)
Там и сейчас может быть ошибка. Если find не найдёт объект, но свойство name будет запрашиваться.
раскрыть ветку (5)
Что забавно, даже id несуществующего города незалогирован )) Так они дооолго отлаживаться на ПРОМЕ будут ))
раскрыть ветку (4)
раскрыть ветку (3)
Похоже, "Сбербанк разраб детектед"))
Недавно работал с их API и у них прод называется "пром" - "промышленный".
раскрыть ветку (2)
Конструкция try-catch в программировании обычно означает, что если произошла какая то ошибка, сделать то что написано в catch.
Конкретно тут - если не получилось найти город, который указан для текущего юзера, то напишем текст ошибки и мемную фразу в логи
Конкретно тут - если не получилось найти город, который указан для текущего юзера, то напишем текст ошибки и мемную фразу в логи
раскрыть ветку (5)
раскрыть ветку (2)
Если ты не можешь объяснить человеческим языком то, что делаешь, значит ты не умеешь это делать, а просто копируешь что то что видел раньше.
раскрыть ветку (1)
Конкретно здесь, надпись в консоль будет выведена в случае возникновения ошибки фильтрации списка городов по идентификатору.
Но доставляет не это, а наличие таких комментариев в коде ><
раскрыть ветку (16)
Тут скорее поиск, а не фильтрация. Ошибка скорее всего произойдёт, если один из объектов окажется не определённым или нужное свойство будет отсутствовать
раскрыть ветку (13)
ну так тогда костыльная вся программа, а не эта ф-ция. Интересно там сделана работа с данными, если она в рандомный момент может выплюнуть исключение.
Или весь "костыль" заключается в кетче?
раскрыть ветку (12)
Да, сама функция норм. Ловить исключение в ней для поиска имеет смысл только если ты реально подозреваешь, что объекты какие-то не те будут. Но это, действительно, из пушки по воробья.
Даже если чувак просто для прикола это написал, чтоб скрин сделать, то выглядит убого. Как не посмотри на этот код - мудатория.
Но такое на проде может быть. Мне начальник рассказывал, что они в 1с подобные штуки по молодости писали в виде:
Если a=b тогда
...
ИначеЕсли a<>b Тогда
...
Иначе
Сообщить("Ошибка! Разбей стекло! Еби гусей! ");
КонецЕсли
раскрыть ветку (4)
Если бы ради шутки - реальный коммент прямо в коде. И не знаю, то ли смеяться, то ли плакать...
А вообще да. Как-то видел участок "китайского" кода, где на вход передавалось число и, в зависимости от него, нужно было выполнить какое-то действие. За давностью лет не помню что за действие, но примерно так было:
function get(id) {
if (id === 1) return 'a';
if (id === 2) return 'b';
if (id === 3) return 'c';
if (id === 4) return 'd';
if (id === 5) return 'e';
//... и так далее на ~300 строк
}
раскрыть ветку (3)
Лень было сделать две проверки, но не лень писать комменты и тупой try catch. Увольняйте нафиг этого школьника.
(return тоже тут лишний, как и скобки)
(return тоже тут лишний, как и скобки)
раскрыть ветку (6)
Где лишние скобки, почему ретёрн лишний, и почему трайкетч тупой, если он делает что должен? Мб тебе проспаться надо?
раскрыть ветку (5)
раскрыть ветку (4)
А, и ещё, две проверки на что? Откуда ты знаешь, какие исключения могут вылезти? Их там может быть и 10 разных, а трайкетч отработает все
раскрыть ветку (3)
раскрыть ветку (2)
Выглядит так, будто ты напился и решил повыёбываться своими навыками программирования, которые ты получил на первом курсе универа, изучая турбопаскаль. Повыёбываться не получилось, у тебя нет этих навыков
раскрыть ветку (1)
Да ладно, не расстраивайся, у всех когда-то был говонокод, может научишься еще программировать, пока лучше прислушайся к советам взрослых
Так много чего может быть,
cities может быть нулл
может не найтись элемента вообще
куча всего еще
если без натягивания совы на глобус:
разбудит если у юзера с какого-то перепугу в профиле стоит ГородКоторогоНет™ в хранилище программы
например, если из списка городов какой-то город удалили, а пользователь по-прежнему на него ссылается. т.е. ссылается с некоторых пор вникуда.
разбудит если у юзера с какого-то перепугу в профиле стоит ГородКоторогоНет™ в хранилище программы
например, если из списка городов какой-то город удалили, а пользователь по-прежнему на него ссылается. т.е. ссылается с некоторых пор вникуда.
IT-юмор
5.7K постов52.5K подписчиков
Правила сообщества
Не публикуем посты:
1) с большим количеством мата
2) с просьбами о помощи
3) не относящиеся к IT-юмору