Коротин Егор #3
Коротин Егор #3
Conversation
|
@ARBUZNIJ @SosnovskayaUlyana |
ARBUZNIJ
left a comment
ARBUZNIJ
left a comment
There was a problem hiding this comment.
Разве мы можем выставлять ограничение "унарный минус должен быть отделён скобками от другой операции"?
Длинную строку не принимает
| void resize() { | ||
| size *= 2; | ||
| T* tmp = new T[size]; | ||
| std::copy(mas, mas + size, tmp); |
There was a problem hiding this comment.
Копируется size элементов, хотя значения лежат только в size/2 эл-тов
| throw std::out_of_range("There are no elements in this Stack"); | ||
| } | ||
| void clear() { | ||
| count = 0; |
There was a problem hiding this comment.
Не ошибка, но всё же: при очистке лучше откатить stack к заводским, т.е. снова сделать его размер маленьким и очистить лишнюю память
| TEST(TPostfix, count_is_correct) { | ||
| TPostfix str("3+10-2*6/(-3)+((5))-(-sin(1))+tan(3)-cos(2)*cot(1)+log(2)*exp(1)"); | ||
| EXPECT_EQ(true, str.count()-(3 + 10 - 2 * 6 / (-3) + 5 - (-sin(1)) + tan(3) - cos(2) * 1 / tan(1) + log(2) * exp(1))< pow(10,-8)); | ||
| } |
There was a problem hiding this comment.
Как это нет? есть
Или я тебя не понял
There was a problem hiding this comment.
Есть на проверку корректности ввода переменных, но на подсчёт выражения с переменными - нет
There was a problem hiding this comment.
Проблема в том, что: чтобы ввести переменные необходим их ввод пользователем, разве мы такое можем реализовать в тестах? + мы до этого так не делали, всё, что касается ввода пользователем не проверялось
There was a problem hiding this comment.
Тестируй для известного количества переменных, и тогда можно затолкнуть какие-нибудь значения в переменные при помощи специального метода, минуя ввод с консоли.
|
Обвинения в том, что программа не принимает слишком длинную строку, снимаю: ошибка не в твоей реализации |
include/arithmetic.h
Outdated
|
|
||
| int get_prior(const std::string& s) noexcept; //Method that return the priority of an operation. If it is not an operation returns 0 | ||
|
|
||
| int TPostfix::get_prior(char c) noexcept; //similar method for characters |
There was a problem hiding this comment.
Зачем здесь прямое указание поля имён TPostfix::? Аналогично ниже
|
@ValentinV95 |
ValentinV95
left a comment
ValentinV95
left a comment
There was a problem hiding this comment.
Лабораторная и отчет приняты.
В отчете неправильный теитульный лист
3 participants
No description provided.