POV: ты новый разраб, которому дали отрефакторить эту функцию на строке 6061 / Новый разработчик сталкивается с непростой задачей рефакторинга функции на строке 6061. / программирование :: geek :: картинка с текстом :: pov :: код

Кусок_Быдла

4 нед.

программированиеgeekкартинка с текстомpovкод

Preview

О посте

Отличный комментарий!

А сама функция обфусцирована..

liberum

07.08.2026, 15:36

ссылка

+9.5

Там проверка на четность

Demonter

07.08.2026, 17:19

ссылка

+43.1

Комментарии

35

07.08.26, 15:33

+41.1

скотство какое то

faustoun

4 нед. назад

07.08.2026, 15:34

ссылка

+10.4

А сама функция обфусцирована..

liberum

4 нед. назад

07.08.2026, 15:36

ссылка

+9.5

Я такого слова не знаю, ну чувствую что оно ведёт куда-то далеко.

JonsiRei

4 нед.

07.08.2026, 16:41

ссылка

+19.7

От такого сразу в штаны обфусцируешься.

Onanim 2015

4 нед.

07.08.2026, 17:08

ссылка

+1.3

Там проверка на четность

Demonter

4 нед.

07.08.2026, 17:19

ссылка

+43.1

Построчно уже давно не платят... Зачем так?

mr.Leonard

4 нед.

07.08.2026, 17:31

ссылка

+0.5

Думаю, тут замешаны наркотики и свободное время.

Кусок_Быдла

4 нед.

07.08.2026, 17:37

ссылка

+8.9

В таком случае было бы что-то типа
If (number in (2,4,6...)) return true;
else if (number > nmax) raise Exception("Number too large");
return false;

А от этого кода так и несет индусятиной.

frenzy

4 нед.

07.08.2026, 17:58

ссылка

+1.3

То есть деление по модулю мы принципиально не будем использовать?

Домовой Сыч

4 нед.

07.08.2026, 19:20

ссылка

+3.6

Это каждый дурак умеет, нет пространства для творчества и выебонов.

VasiliyRed

4 нед.

07.08.2026, 19:29

ссылка

+3.5

Комментов: 1

а это сохранилось с тех времён
просто никто так и не заглядывал

akss

4 нед.

07.08.2026, 18:23

ссылка

+3.9

Явно какой-то джун писал. Сеньоры для этого используют внешние библиотеки - https://www.npmjs.com/package/is-odd для проверки на нечётность и https://www.npmjs.com/package/is-even для проверки на чётность.

koka

4 нед.

07.08.2026, 22:56

ссылка

+3.9

И дна нет, а верх запаян

хуйняизподконя

4 нед.

08.08.2026, 03:58

ссылка

+0.0

claude, do refactoring of function on line 19517, run tests, do not stop until it will pass them, make no mistakes, be professional

banderlog

4 нед. назад

07.08.2026, 16:12

ссылка

+2.1

Мне как-то иишница написала тест, в котором было Assert.Equals(true, true)
Код его успешно проходил.

gwarlock

4 нед.

07.08.2026, 16:39

ссылка

+2.5

IMPORTANT: **DO NOT CHANGE TESTS CODE**

banderlog

4 нед.

07.08.2026, 16:45

ссылка

+5.9

Поздно спохватился

คຖ໓rē ¢hค໐Ş

4 нед.

07.08.2026, 16:50

ссылка

+0.8

Пф, `git restore .`
No limit to number of tries

banderlog

4 нед.

07.08.2026, 16:51

ссылка

+2.1

I'm sorry, banderlog, I'm afraid I can't do that

คຖ໓rē ¢hค໐Ş

4 нед.

07.08.2026, 16:55

ссылка

+3.1

Есть лимит. Кредит токенов называется.

wataru

4 нед.

07.08.2026, 17:46

ссылка

+0.9

Тебе сверху пришла отмашка "осваивать новый ИИ инструмент в обязательном порядке" - вот ты и осваиваешь его до предела.

frenzy

4 нед.

07.08.2026, 18:00

ссылка

+2.3

Это только для бедных

banderlog

4 нед.

08.08.2026, 06:47

ссылка

0.0

Комментов: 1

а тестов-то и нет

alphashooter

4 нед.

07.08.2026, 17:02

ссылка

0.0

Угу, это при условии, что тесты покрывают все ветки кода в этой функции. И что тесты вообще есть.

BoobSound

4 нед.

07.08.2026, 17:33

ссылка

+0.1

В нормальной IDE всегда есть подсветка coverage, так что можно прогнать тесты и посмотреть что они покрывают. Но учитывая этот кейс тестов либо нет, либо они были написаны, когда функция заканчивалась на 6080й строке.

frenzy

4 нед.

07.08.2026, 18:02

ссылка

-2.1

Сначала вспомнил анекдот про программистов и пользователей* и хотел спросить а, что тут собственно? А потом увидел, что там 13,5 тысяч строк кода спрятано, понял, что не такой наверное я и пользователь...

* ну и сам анекдот естественно

Едут в одном вагоне четверо программистов и четверо пользователей.
У пользователей — четыре билета на четверых, а у программистов — один.
Приходит пора предъявлять билеты.
Программисты запираются в туалете, приходит контролер. Стучится, из сортира высовывается рука и протягивает билет. Контролер уходит.
Пользователи все видят, и им завидно.
Едут все те же пользователи и программисты обратно. Но на этот раз у пользователей — один билет на четверых, а у программистов — ни одного.
Приходит пора проверки билетов.
Пользователи запираются в сортир.
Приходит один из программистов и стучится в дверь. Ему высовывают билет. Он берет билет, программисты запираются в другом сортире. Приходит контролер..
Мораль: не всякий алгоритм, разработанный системным программистом, подходит для рядового пользователя.

BigZed

4 нед. назад

07.08.2026, 16:53

ссылка

+22.7

Наличие 6к строк в классе уже сигнал, что открывать его не следует

xirurr

4 нед. назад

07.08.2026, 17:49

ссылка

+1.4

И тем не менее, класс на 6к строк всё ещё радикально лучше 6к классов по 1 строке значимого кода, которые вместе описывают сложную сущность. В одном классе разобраться трудно, в мешанине, в которой скрыт бизнес процесс за миллионом классов - невозможно.

Оху́евший

4 нед.

08.08.2026, 05:27

ссылка

0.0

закидываешь код в ИИ, спрашиваешь, что делает этот код? Вуаля

DemonInside

3 д.

02.09.2026, 09:59

ссылка

0.0

Да, сейчас это проще. Можно даже бинарник закинуть, сказать декомпиль это и разберись. ИИшке посрать, разбираться в коде, или ассемблере.

Оху́евший

3 д.

02.09.2026, 14:43

ссылка

0.0

1) просим ИИ написать 500 тестов 2) просим ИИ зарефакторить чтоб тесты прошли 3)profit

dimasik_memasik

4 нед. назад

07.08.2026, 18:19

ссылка

-0.4

на секунду представил что это кнопка "открыть комменты под постом"

obolon

4 нед. назад

07.08.2026, 19:53

ссылка

+1.2

Класс, дайте пару месяцев

DrXak

4 нед. назад

07.08.2026, 22:24

ссылка

+0.1

Чтобы написать коммент, необходимо залогиниться