и уменьшения когнитивной нагрузки
2. AddPhrase нарушает инкапсуляцию и только усложняет интерфейс бота, вместо KeyValuePair можно сразу передавать key и value.
3. Почему для AddPhrase отлавливается любой Exception, а для RemovePhrase конкретные доменные исключения?
4. Однострочные try\catch субъективно хуже понимать
5. Если ожидается ввод с консоли, то ReadLine не вернёт null, надо проверять на пустую строку.
6. Вложенный трай\кэтч в другой кэтч - попахивает. Если у тебя подразумевается, что пользователь может ввести команду, а может ключ для поиска, то это не надо разруливать исключениями. Это стандартное поведение, исключения должны быть исключительными, а не использоваться для контроля потока выполнения программы. Может надо добавить третью комнаду - GetPhrase.
да он рофлит, че ты
Обсуждают сегодня