Замечания по FlipByte - Githubissues

RomanovAleksandr / oop

0 stars 0 forks source link

Замечания по FlipByte #4

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/04d4a8fdb4cb23189382b083698ae7dec7364738/lab1/flipbyte/flipbyte/main.cpp#L7-L10

[ ] В структуре должен находиться байт, а не строка

alexey-malov commented 4 years ago

[ ] Приложение падает, если передать через командную строку число, не помещающееся в int, например, 3 миллиарда. Надо обрабатывать исключения

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/04d4a8fdb4cb23189382b083698ae7dec7364738/lab1/flipbyte/flipbyte/main.cpp#L25-L40

[ ] объекты следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению
[ ] Незачем сканировать строку в поисках нецифровых символов. При помощи stoi можно сделать эти проверки надёжнее

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/04d4a8fdb4cb23189382b083698ae7dec7364738/lab1/flipbyte/flipbyte/main.cpp#L58-L69

[ ] Функция должна принимать и возвращать uint8_t