added solution#302
Conversation
| } else { | ||
| result = a / b; | ||
| } | ||
| break; |
There was a problem hiding this comment.
| } else { | |
| result = a / b; | |
| } | |
| break; | |
| } | |
| return result = a / b; |
| result = Math.pow(a,b); | ||
| break; |
There was a problem hiding this comment.
| result = Math.pow(a,b); | |
| break; | |
| return Math.pow(a,b); |
| private double operand1; | ||
| private double operand2; | ||
| private double actual; | ||
| private char ch; |
There was a problem hiding this comment.
| private char ch; | |
| private char operation; |
| private double operand1; | ||
| private double operand2; | ||
| private double actual; |
There was a problem hiding this comment.
так як ці змінні різні для кожного тест кейсу їх логічніше винести в окремі методи і не оголошувати на рівні класу
There was a problem hiding this comment.
в принципі, ці змінні можна взагалі не створювати, а передавати відразу в метод цифри та char, чи не так?
There was a problem hiding this comment.
так, загаом можна одразу передавати, неймінг потрібен тількки якщо неочевидно чому те чи інше значення було передано (наприклад k як invalidOperation
|
|
||
| @Test | ||
| void calculate_IllegalOperation_NotOk() { | ||
| if (ch != '+' && ch != '-' && ch != '*' && ch != '/' && ch != '^') { |
There was a problem hiding this comment.
цей тест кейс краще зробити більш читаймим: оголосит невалідну операцію в методі, передати її в calculate і тоді очікувати ексепшн
| operand2 = 3; | ||
| ch = '/'; | ||
| actual = calculator.calculate(operand1,operand2,ch); | ||
| assertEquals(operand1 / operand2,actual); |
There was a problem hiding this comment.
цей код не належить до випадку, коли ми ділимо на 0 краще його прибрати/перенести в інший тест кейс якщо в цьому є потреба
|
|
||
| @Test | ||
| void calculate_IllegalOperation_NotOk() { | ||
| char operation = ')'; |
There was a problem hiding this comment.
| char operation = ')'; | |
| char illegalOperation = ')'; |
| double operand1 = 6; | ||
| double operand2 = 7; | ||
| char operation = '+'; | ||
| double actual = calculator.calculate(operand1,operand2,operation); |
There was a problem hiding this comment.
а тут так, можна прередавати значення одразу в calculate
2 participants
No description provided.