не понимаю что не так

Да, тут есть одна особенность в массивах, счет как вы понимаете начинается с 0, то бишь написав "имяМассива.lenght" вы получаете число которое считает массив с единицы и выводи длину в цифрах, а не в индексах, а это число от числа индекс отличается, по этому для того чтобы получить нам индекс нужно написать "имяМассива.lenght-1"

Индексы массивов начинаются с нуля, а это значит, что в нём никогда нет элемента с индексом array.length. [0, 1, 2, ..., array.length-1] -> array.length штук Math.ceil у тебя ничего не делает, потому что у тебя происходит деление целочисленных операндов - там по определению нет остатка. Ну и даже если бы был - округлять вверх точно не требуется.