Task13 #817
Task13 #817
Conversation
| int stepsCount = 0; | ||
| int leftover = wateringCanVolume; | ||
| for (int i = 0; i < plants.length; i++) { | ||
| if (plants[i] < 1 || plants[i] > wateringCanVolume) { |
There was a problem hiding this comment.
уже писал - валидацию и логику стоит разделять
| return -1; | ||
| } | ||
|
|
||
| if (leftover >= plants[i]) { |
There was a problem hiding this comment.
этот if-else можно упростить. Фактически, ты все описал верно. Но есть мнение, что увеличение шага на один и затраты на полив происходят всегда (на каждой итерации цикла). И лишь дополнительными действиями являются поход к реке и наполнение лейки. То есть тут фактически нет разветвления на два сценария, есть лишь небольшой крюк при определенном условии:)
| return stepsCount; | ||
| } | ||
|
|
||
| static boolean isValid (int plant, int wateringCanVolume) { |
There was a problem hiding this comment.
лишний пробел после имени метода
| int stepsCount = 0; | ||
| int leftover = wateringCanVolume; | ||
| for (int i = 0; i < plants.length; i++) { | ||
| if (!isValid(plants[i], wateringCanVolume)) { |
There was a problem hiding this comment.
валидация все еще не выдела из логики. Где гарантия, что на plants[plants.length - 1] мы не обнаружим не валидное число? Получится, что все итерации до этого затратили лишние ресурсы процессора
There was a problem hiding this comment.
думала, лучше всё одним циклом сделать, теперь понимаю, почему не лучше 😄
| stepsCount += i * 2; | ||
| leftover = wateringCanVolume - plants[i]; | ||
| } | ||
| stepsCount += 1; |
2 participants
No description provided.