Этого треда уже нет.
Это копия, сохраненная 29 марта 2020 года.

Скачать тред: только с превью, с превью и прикрепленными файлами.
Второй вариант может долго скачиваться. Файлы будут только в живых или недавно утонувших тредах. Подробнее

Если вам полезен архив М.Двача, пожертвуйте на оплату сервера.
Screen Shot 2020-03-21 at 03.18.33.png175 Кб, 510x424
сделайте ревью моего кода писал тестовое задание в геймдев контору https://pastebin.com/hex1UzJh 216536813 В конец треда | Веб
сделайте ревью моего кода
писал тестовое задание в геймдев контору

https://pastebin.com/hex1UzJh
2 216536860
Бамп
3 216536931
>>6813 (OP)
это сишарп? говнина редкостная. В коде у тебя анальное ооп и уныние, не надо так!.
4 216536954
>>6931
вообще я бы не стал использовать методы типа void, потому что если ты можешь вернуть error вместо нихуя, лучше так и делать
5 216536981
>>6954
Что ты несешь
6 216537228
>>6981
иди сюда, мой сладенький!
в общем, основной засадой типа void является то, что ты не возвращаешь нихуя (или возвращаешь указатель на какую-то хрень, как в С, но тогда ты пишешь void * ). Да, ты можешь где-то выкинуть эксепшон, но он не всегда может быть обработан/ Короче, если я вызываю метод "прочитай эту хуйню", я ожидаю, что мне вернётся либо эта хуйня, либо какое-то абстрактное значение, которое означает "хуйня не вернулась, потому что потому"
7 216537338
>>6813 (OP)
Покатит, ну я конечно не знаю сути задания, но так в глаза ничё не бросилось вроде.
8 216537361
>>6813 (OP)
Говно
9 216537403
>>7228
А нахера для этого отдельный Error заводить? Чем тебе выбрасывание исключений не угодило? Просто следи за их иерархией и незачем дублировать уже имеющийся функционал языка.
10 216537411
>>6813 (OP)
Если это у тебя класс, который реализует управление с двух разных устройств, почему он не разделен на интерфейс и два класса, keyboard и gamepad?
11 216537688
Если я правильно понял предназначение класса, то я бы сделал иначе:
Для каждого вида устройств ввода делаешь свой детектор, отдельно для клавиатуры, отдельно для мыши, отдельно для геймпада, а потом регистрируешь все эти детекторы в ControllerTypeDetector, в котором уже просто в цикле проходишься по зарегистрированным детекторам и возвращаешь тип того детектора, который задетектился.
12 216537753
в дополнение >>7688
по итогу у тебя получается набор легковесных классов, каждый из которых в отдельности легко читается, ввиду своего размера, и выполняет только одну функцию.
13 216539891
Комментарии убери.
14 216539983
>>6813 (OP)
Фигурные скобки говно. Уже давно выяснили что надо их над друг другом ставить.
Тред утонул или удален.
Это копия, сохраненная 29 марта 2020 года.

Скачать тред: только с превью, с превью и прикрепленными файлами.
Второй вариант может долго скачиваться. Файлы будут только в живых или недавно утонувших тредах. Подробнее

Если вам полезен архив М.Двача, пожертвуйте на оплату сервера.
« /b/В начало тредаВеб-версияНастройки
/a//b//mu//s//vg/Все доски