или стоило использовать интерфейсы со структурами? https://github.com/snapyzs/matrixuniq Благодарю
Интерфейсы имеет смысл использовать, когда пишется полноценный сервис. А у вас просто небольшая программка написана
смотря для чего это написано... Width need more 0 and no minus > надо бы написать > Width needs to be more than 0
На мой взгляд не лучшее решение. Во-первых, код слегка переусложненный, реализация не очень. Во-вторых, сложность подобного алгоритма вызывает вопросы. Но это мой скромный взгляд, который может не совпадать с тем, кто будет этот код проверять
Этот код некому проверят, поэтому я и написал сюда, спасибо за фидбэк
А зачем два раза проверка на корректность? Вкусовщина, но я бы переименовал в correctCheck или isCorrect
Действительно излишне
https://github.com/snapyzs/matrixuniq/blob/second/main.go#L50 Вы понимаете, что w*h каждый раз будет вычисляться, может 1 раз вычислить в начале и подставлять?
https://github.com/snapyzs/matrixuniq/blob/second/main.go#L45 Вот этот цикл не нужен, смысл его не пойму, ощущение, что сначало написали так, но увидели, что рандом может возвращать одинаковые и сверху сделали for. Уберите, ничего не изменится
Да вот здесь прочитали меня, как я действовал
Обсуждают сегодня