Добрый вечер. У меня небольшой вопрос насколько плохой тон писать контроллеры таким образом https://pastebin.com/mmjzEzsK Меня вроде как все устраивало , но мне кажется что я нарушаю какие то соглашения пихая часть логики в контроллер + читаемость у него не в лучшем уровне
P.s. стоит ли держать все в одном контроллере или нужно разделять методы по логике в разные?
Если делать эффективно, то все равно это будет set, хоть самописный. А так, в лоб - на каждом шаге проверяешь весь массив, встречается ли в нем такой элемент. Ну и дальше начинаешь это оптимизировать и приходишь к сету