Size: a a a

2021 October 16

EA

Egene Avdeev in CODE BLOG / C#
источник

EA

Egene Avdeev in CODE BLOG / C#
Никаких снейк кейс
источник

EA

Egene Avdeev in CODE BLOG / C#
Математика это не ислам
источник

EA

Egene Avdeev in CODE BLOG / C#
Дальше?
источник

t

tavvi in CODE BLOG / C#
да
источник

EA

Egene Avdeev in CODE BLOG / C#
Field это сущность, её бы хорошо вынести в отдельный класс, у неё ширина, высота. Потом эту сущность использовать и в генераторе, и везде где нужны границы.

Сейчас тебе приходится передавать переменные
источник

t

tavvi in CODE BLOG / C#
согласен, я поздно заметил как Field распространился по всему коду, надо бы его вынести в отдельную структуру
источник

EA

Egene Avdeev in CODE BLOG / C#
Если хранишь промежутки времени, храни их не как int delay
источник

Q

QuotLy in CODE BLOG / C#
источник

EA

Egene Avdeev in CODE BLOG / C#
delay в секундах, минутах, миллисекундах?

Бери TimeSpan
источник

EA

Egene Avdeev in CODE BLOG / C#
Для этого есть подходящая структура данных
источник

t

tavvi in CODE BLOG / C#
там всегда миллисекунды
источник

t

tavvi in CODE BLOG / C#
но про этот тип я знаю, я поменяю
источник

EA

Egene Avdeev in CODE BLOG / C#
Все равно, тебе же надо код правильный писать
источник

EA

Egene Avdeev in CODE BLOG / C#
А не коду быть правильным
источник

EA

Egene Avdeev in CODE BLOG / C#
Ну и структурно все слои перемешаны
источник

EA

Egene Avdeev in CODE BLOG / C#
То есть у тебя бизнес логика с представлением перемешивается
источник

EA

Egene Avdeev in CODE BLOG / C#
Почитай про MVC например, поймёшь о чем речь
источник

t

tavvi in CODE BLOG / C#
что тут будет контроллером? Тут же не asp
источник

EA

Egene Avdeev in CODE BLOG / C#
Не asp, но подход с разделением слоев ты должен понять
источник