// Мне немного трудно перестроиться на другой язык, пишу на C#
int m[,]=new m[5,5];
int x;
for(int c=0;c<25;c++){
m[c%5][c/5]=(c+1)*5;
}
x=Сonvert.ToInt32(Concole.Read()); // Не помню правильный синтаксис.
Console.WriteLine("m[{0},{1}]={2}",((x/5)-1)%5,((x/5)-1)/5,x);

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

Насчёт деталей. Они правда в основном на внимательность, и средство разработки поправит чаще всего, но всё же прокомментирую:
Двумерный массив объявляется так:Остальное весьма корректно.

По идее всё более менее конечно, если быть не грубым, но такой подход цикла не совсем корректен с математической и логической точки зрения, т.к. основывается на определённых условиях задания, это относится к числу 25 - как ограничение в цикле, что было тобою вычислено исходя из количества элементов в массиве и длине шага (5), 5*5 = 25 - столько элементов, в задании было сказано от 5 до 100. Как уже говорил не столь важно от части. Но при изменении размера массива, придётся пере вычислять его размер, и т.п. хотя в более корректном варианте всё равно будет тоже зависимость от количества элементов.

А так весьма неплохое начало. :)

имхо

int mw=5,mh=5,x;
int[,] m=new int[mw,mh]; // Тут меня всегда поправляет студия, хотя я знаю как правильно =)
// Предполагаю, что массив заданных размеров способен вместить 100 чисел. Но можно условие присобачить.
for(int c=0;c<100;c++){
m[c%mw][c/mw]=(c+1)*5; // Здесь я наверное что-то не понимаю. Я заполняю массив "книжкой", т.е. построчно,
// справа налево, сверху вниз. А как надо?
}
// Массив возможно изменяется, предполагаю, что числа могли перемещаться любым случайным (или нет) образом. Переменные mw,mh принимают новое значение.
// Тут мне не приходит в голову других решений кроме поиска по числу.

for(int c=0;c<5;c++){
for(int a=0;a<mw*mh;a++){
if(m[a%mw][a/mw]==(c+1)*20){ Console.WriteLine("m[{0},{1}]={2}",(a%mw),(a/mw),(c+1)*20); break; }
}
}

// В остальных случаях можно подправить мое первое решение под поиск по новым размерам, или сохраненным старым.
// (Если новый массив числа заполнят по тому же принципу, или останутся "в уголке" с теми же координатами, что и в массиве старого размера)

Первое заполнение массива было правильным в плане итераций, лишь поменяны местами индексы. Не "m[c%5,c/5]", а "m[c/5,c%5]". Тогда будет перелистывать все y для каждого x.
В новом примере, первый цикл теперь выходит за пределы массива, т.к. цикл до 100, а по формуле "c/mw" будут варианты больше 5, что вылезет как runtime ошибка.
Если поставить ограничение в первом цикле на 25, то программа продолжится.
Вторая часть программы с вложенным циклом верна, за исключением коррекции в первом (поменять вычисления для индексов местами, для y будет с модулем.

Результат правильный, с коррекциями.

Теперь относительно вложенного цикла (for (int a=0;a<mw*mh;a++) {). Посчитай количество итераций, с учётом первого цикла. В данном случае вложенный цикл почти не отличается от первого цикла, где ты заполняешь сам массив, это смекалка для данной задачи - ты бы мог вывезти результат при заполнении массива, плюс за смекалку, минус за то что меньше математики и дополнительного цикла, в данном бы случае, стоило бы узнать у тестируемого, как бы он разделил приложение, сделал бы второй цикл, для проверки математики.

В первом цикле ты устанавливаешь результат, но ты ведь его уже приписал в массиве, он уже есть в массиве, и его не нужно получать. Закономерность запрашиваемых циклов в том что они чередуются 20, думаю это ещё одна подсказка, что поможет упростить.
Второй цикл, можно разобрать для читаемости на два, как цикл по x и по y, но и этот вариант хорош, лишь менее читабелен и немного менее адаптируемый для модификаций.