Ну, одна проблема в том, что если val
больше всех текущих значений, вы получите maxval = 0
.Вы можете исправить это, инициализируя maxval
в values.length - 1
для начала, возможно ...
Другая проблема заключается в том, что вы фактически теряете исходное значение values[values.length - 1]
- оно перезаписывается с более раннимзначение, но ничто не копирует это значение в другом месте.Какое здесь желаемое поведение?