170 похожих чатов

Доброго времени суток, скажите пожалуйста, является ли такое написание правильным

или стоило использовать интерфейсы со структурами? https://github.com/snapyzs/matrixuniq Благодарю

9 ответов

12 просмотров

Интерфейсы имеет смысл использовать, когда пишется полноценный сервис. А у вас просто небольшая программка написана

смотря для чего это написано... Width need more 0 and no minus > надо бы написать > Width needs to be more than 0

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

snap- Автор вопроса
Dmitriy
На мой взгляд не лучшее решение. Во-первых, код сл...

Этот код некому проверят, поэтому я и написал сюда, спасибо за фидбэк

А зачем два раза проверка на корректность? Вкусовщина, но я бы переименовал в correctCheck или isCorrect

snap- Автор вопроса
snap
Действительно излишне

https://github.com/snapyzs/matrixuniq/blob/second/main.go#L50 Вы понимаете, что w*h каждый раз будет вычисляться, может 1 раз вычислить в начале и подставлять?

snap
Действительно излишне

https://github.com/snapyzs/matrixuniq/blob/second/main.go#L45 Вот этот цикл не нужен, смысл его не пойму, ощущение, что сначало написали так, но увидели, что рандом может возвращать одинаковые и сверху сделали for. Уберите, ничего не изменится

snap- Автор вопроса
Maxim
https://github.com/snapyzs/matrixuniq/blob/second/...

Да вот здесь прочитали меня, как я действовал

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
Карта сайта