3dsoundfromtagan / ea-course

0 stars 0 forks source link

Review #1 #1

Open VulpesCorsac opened 4 years ago

VulpesCorsac commented 4 years ago

1) Сделай Base шаблонным. Сможешь?) 2) Sign через Enum 3) Конструктор не должен менять переменные, которые в него подаются. Хочешь менять - заведи временные 4) У тебя, по сути, конструкторы от чисел очень похожи. Самое большое - unsigned long long. Сделай нормально функцию для получения LongNum из него, а все остальные приводи к нему, просто правильно сохраняя знак 5) Не хочешь париться с char* - переведи его в std::string и отправь тоже в отдельную функцию 6) Соответственно нет перегрузки всех (а вот реально всех) арифметических операторов. Правда, тебе сложение и вычитание придётся писать отдельно. Лучше всего - написать две вспомогательные функции: сложение и вычитание из большего меньшего, а дальше при перегрузке следить за знаком результата лапками

3dsoundfromtagan commented 4 years ago

Разве заведение временных переменных не затормозит сильно работу программы?

VulpesCorsac commented 4 years ago

Дополнительный int она впроне переживёт. Создание string из char - тоже. А вот сам string модифицировать, в принципе, и не особо-то обязательно (хотя, это тоже не так уж и критично, это скорее вопрос рациональности). Мы пока не в highload backend