Замечания по программе Calculator

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/Calculator.h#L28-L29

• [ ] Лучше назвать DeclareFunction. Имя метода, как правило, должно быть глаголом, а не существительным

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/Calculator.h#L39-L40

• [ ] Do Ñot use Portuguese letters in method Ñames :)

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/Calculator.cpp#L157-L176

• [ ] Методы, которым не требуется доступ к состоянию класса следует объявляться статическими методами класса, либо свободными функциями (в безымянном пространстве имен внутри .cpp файла)

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/Calculator.cpp#L196-L199

• [ ] find может не найти значение, код должен быть рассчитан на данную ситуацию

[alexey-malov]

• [ ] При вводе Ctrl+Z код не должно выводиться сообщение "Unknown command"

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/Calculator.h#L38

• [ ] Выбранное имя подходит для переменной, но не для типа

[alexey-malov]

https://github.com/IrinaZagaynova/oop/blob/815d86c57d1149d29595e6ce24e8dacadabe62ad/lab3/Calculator/Calculator/CalculatorControl.h#L6-L28

• [ ] В этом задании нет телевизора

• [ ] В программе "Телевизор" класс назывался CRemoteControl (пульт дистанционного управления), и слово Control в имени было оправданным. В классе CCalculatorControl что слово Control точно подходящее?