Считается ли это хард-кодом?🧐

я бы сказал, что это говнокод скорее, чем просто хардкод ничего личного 😂

как минимум, объедини общие условия (которые ведут к одному результату) И вместо увеличения вложенности, лучше отваливаться раньше - будет сильно читабельнее Т.е. принцип такой, что основная ветка логики всегда на минимальном уровне вложенности

ща попробую что нибудь сделать, пока просто прототип механики накидал, чтобы хоть как-то работало

norlin
ну и вот такой адочек при нормальном рефакторинге ...

спасибо, ща займусь)

norlin
ну и вот такой адочек при нормальном рефакторинге ...

Contains вроде null exception дает при пустом массиве

во первых, не должно во-вторых, причём тут это?)

Омг, мужик, сделать свою структуру более плоской, инвертируй условия и используй continue

не должно то я согласен. я про то, что если должно, то он вынужден первую проверку делать перед contains.

я не понял, сорян)

В if проверка выполняется последовательно через &&, нет смысла так все усложнять абсолютно ... Можно еще проверять через ! и выходить сразу, если что-то не так ...

Да не вызывает он никаких null expection, это какие-то слухи на уровне что нельзя nullptr кастовать

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

ну не говнокод... это яркий пример Индуского кода, кмк

USE &&, Luke

Use separate method to check conditions Luke

это уже следующий шаг )

скинь пример о чем вы) просто метод сделать с этим условием, а зачем?

нет, много методов

много if застаканных убери в один и норм

один try catch на все действия 🤣

что-то на джаваскриптовом