Задача видимо не совсем понята правильно, суть в чём:
Создаётся массив 5,5 (у тебя это есть), далее заполняется числами от 5 до 100. Заполнение есть, то что оно выходит за радиус 100 - не обговаривалось в задаче, поэтому то что ты заполнил до 125 - не проблема.
Далее идёт чтение номера, получается в коде ты получаешь номер, и далее находишь по числу (значение в массиве) индекс. По идее верно, с одним условием, что если мы изменим массив немного, получится что вычисления не будут корректными, т.к. подразумевается что массив всегда соответствует тому что был создан (условно "статичен").
Только первая часть задания была выполнена.
Но также, исходя из задачи, чередование было сперва по y затем x, в твоём решении наоборот, хотя результат от этого не меняется, лишь "логика" хранения данных, что порой может сбивать с толку.
Вторая часть задания не совсем была понята корректно, для ясности я внёс в первый пост пример вывода, который должен получиться.
Насчёт деталей. Они правда в основном на внимательность, и средство разработки поправит чаще всего, но всё же прокомментирую:
Двумерный массив объявляется так:
Остальное весьма корректно.
По идее всё более менее конечно, если быть не грубым, но такой подход цикла не совсем корректен с математической и логической точки зрения, т.к. основывается на определённых условиях задания, это относится к числу 25 - как ограничение в цикле, что было тобою вычислено исходя из количества элементов в массиве и длине шага (5), 5*5 = 25 - столько элементов, в задании было сказано от 5 до 100. Как уже говорил не столь важно от части. Но при изменении размера массива, придётся пере вычислять его размер, и т.п. хотя в более корректном варианте всё равно будет тоже зависимость от количества элементов.
А так весьма неплохое начало.
имхо