Пестов Владимир Алексеевич#5
Conversation
czertyaka
left a comment
czertyaka
left a comment
There was a problem hiding this comment.
Задание отлично выполнено, принимаю. Замечания минорные
PR вливать не надо
| #include <iostream> | ||
| #include <random> | ||
|
|
||
| int16_t GenerateRandomNumber() { |
There was a problem hiding this comment.
А почему для хранения чисел от 0 до 9 вы выбрали именно 2 байтное знаковое? Если эконоить на размере, то можно же выбрать более подходящий тип
There was a problem hiding this comment.
Потому что int8_t начинал парситься в char, что заставляло
либо использовать inp - '0' в качестве числа, либо принимать int и делать static_cast
Оба варианта показались не очень красивыми, решил оставить этот (хотя, конечно, в таком виде неочевидно, почему 16)
| std::random_device random; | ||
|
|
||
| std::default_random_engine engine(random()); | ||
| std::uniform_int_distribution<int16_t> uniform_dist(0, 9); |
There was a problem hiding this comment.
Отлично, что используете более современный подход. srand в задании я предложил использовать для простоты, но вообще в настоящем коде правильнее было бы использовать типы из <random>
| } | ||
|
|
||
| int main() { | ||
| auto random_number = GenerateRandomNumber(); |
There was a problem hiding this comment.
Можно сделать константой
| auto random_number = GenerateRandomNumber(); | ||
|
|
||
| while (true) { | ||
| auto user_input = ReceiveUserInput(); |
There was a problem hiding this comment.
В рамках одной итерации тоже может быть константой)
3 participants
No description provided.