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

[alexey-malov]

1>C:\teaching\oop-labs\gladysheva\oop\labs\lab3\Calculator\Calculator\CalculatorControl.cpp(106): warning C4715: 'ToOperation': not all control paths return a value
1>C:\teaching\oop-labs\gladysheva\oop\labs\lab3\Calculator\Calculator\Calculator.cpp(172): warning C4715: 'Calculator::GetValue': not all control paths return a value

• [ ] Это предупреждение, на самом деле, очень серьезное. Фактически приводит к неопределенному поведению при передаче в функцию строки, задающей операцию, отличной от поддерживаемой

[alexey-malov]

https://github.com/AnyaGl/oop/blob/698fa18e7a8f803630115e57df49152eda51d4f4/labs/lab3/Calculator/Calculator/Function.h#L8-L15

• [ ] Nothing из списка возможных значений enum-а лучше убрать. Потому как если в качестве бинарной операции передать Nothing, то вряд ли это будет коррректно. Вместо этого сделайте операцию опциональной (внутри). Тогда унарные операции будут просто возвращать значение, а бинарные - выполнять действия. Либо можно передавать std::function вместо enum-а, а эта функция бы вычисляла значение.

[alexey-malov]

https://github.com/AnyaGl/oop/blob/698fa18e7a8f803630115e57df49152eda51d4f4/labs/lab3/Calculator/Calculator/Calculator.cpp#L139-L147 https://github.com/AnyaGl/oop/blob/698fa18e7a8f803630115e57df49152eda51d4f4/labs/lab3/Calculator/Calculator/Calculator.cpp#L149-L157

• [ ] Примените алгоритм transform (примечание используйте inserter для создания итератора)

[alexey-malov]

https://github.com/AnyaGl/oop/blob/698fa18e7a8f803630115e57df49152eda51d4f4/labs/lab3/Calculator/Calculator/Calculator.cpp#L123

• [ ] Подберите имя для функции так, чтобы код вызова функции лучше читался.

UpdateFunctionValues(varName); // Не понятно, что делает этот код

[alexey-malov]

• [ ] IsFunctionExist, IsVariableExist - неправильно построенные фразы с точки зрения английского языка.

[alexey-malov]

• [ ] Придумайте, как сделать так, чтобы не перебирать все функции при изменении значения единственной переменной