Задача с матрицей с++. Не могу найти причину неправильной работы программы.

[Azzzza]

Матрицу М(m, n) заполнить натуральными числами от 1 до m*n по спирали, начинающейся в левом верхнем углу и закрученной почасовой стрелке
1 2 3 4 5
16 17 18 19 6
15 24 25 20 7
14 23 22 21 8
13 12 11 10 9
Вот моя программа, но она не работает, как надо, не могу исправить

#include<iostream.h>
#include<stdlib.h>
#include<conio.h>
#include<math.h>
main()
{
int n,m;
int i,imax,imin,j,jmax,jmin,k;
cin>> m;
cin >> n;
int**a;
a=new int*[m];
for(i=0; i<m; i++)
a[i]=new int[n];
jmin=1;
jmax=n;
imin=2;
imax=m;
k=0;
for(j=jmin; j<jmax; j++)
{
k++;
a[imin-1][j]=k;
}
for(i=imin; i<imax; i++)
{
k++;
a[i][jmax]=k;
}
imax--;
for(i=imax; i>imin; i--)
{
k++;
a[i][jmin]=k;
}
imin++;
jmin++;
while(k>=m*n)
for(i=1; i<m; i++)
{
cout<<endl;
for(j=1; j<n; j++)
cout<<a[i][j]<<"";
}
getch();
}

[Alt_Shift]

шапка странная какая то. Попробуйте вот так

#include<iostream>
#include<stdlib.h>
#include<conio.h>
#include<math.h>
using namespace std;
void main()
{

[Azzzza]

Изменила, программа вобще не запускается, выдает ошибку: Namespace name expected

[Alex11223]

Какой компилятор? В VS 2010 запускается после добавления using namespace std;
Но результат не выводится, где-то тут ошибка вероятно

while(k>=m*n)
		for(i=1; i<m; i++)
		{
			cout<<endl;
			for(j=1; j<n; j++)
				cout<<a[i][j]<<"";
                }

[Mandrivnyk]

Сообщение от Alex11223

Какой компилятор? В VS 2010 запускается после добавления using namespace std;
Но результат не выводится, где-то тут ошибка вероятно

while(k>=m*n)
		for(i=1; i<m; i++)
		{
			cout<<endl;
			for(j=1; j<n; j++)
				cout<<a[i][j]<<"";
                }

Ошибка в том, что while -- это вторая часть блока do -- while, вот только do в коде нету -)
Кроме этого, сам алгоритм немного неверный...
Сейчас гляну внимательнее.

[Mandrivnyk]

Ну, вот как-то так...

#include<iostream>
#include<cstdlib>

using namespace std;

int main()
{
    int columns, rows;
    int rs, re, cs, ce, k;
    
    cout << "Введите количество строк: ";
    cin >> rows;
    cout << endl;
    cout << "Введите количество столбцов: ";
    cin >> columns;
    
    int dimension = rows * columns;
    
    int **Array = new int *[rows];
    for(int i = 0; i < rows; i++)
         Array[i] = new int[columns];
    
    rs = 0;                            
    re = rows-1;
    cs = 0;
    ce = columns-1;
    k = 1;
    
    do 
    {
        // слева направо по столбцам
        if (k > dimension) break;
        for(int ci = cs; ci <= ce; ci++)
            Array[rs][ci] = k++;
        rs++;
        
        
        // сверху вниз по строчкам
        if (k > dimension) break;
        for(int ri = rs; ri <= re; ri++)
            Array[ri][ce] = k++;
        ce--;
        
        // справо налево по столбцам
        if (k > dimension) break;
        for(int ci = ce; ci >= cs; ci--)
            Array[re][ci] = k++;
        re--;
        
        // снизу вверх по строчкам
        if (k > dimension) break;
        for(int ri = re; ri >= rs; ri--)
            Array[ri][cs] = k++;
        cs++;
        
    } while (k < dimension);
    
    // вывод массива
    for(int ri = 0; ri < rows; ri++)
    {
        for(int ci = 0; ci < columns; ci++)
            cout << Array[ri][ci] << "\t";
        cout << endl;
    }
}

Оптимизировать бы его еще -)

[Mandrivnyk]

А, да!
По-хорошему, еще надо бы и память освободить. После вывода массива:

    for (int i = 0; i < rows; i++) delete[]columns[i];
    delete[]rows;

[Azzzza]

Опять та же ошибка: Namespace name expected, удаляю строку"using namespace std", программа запускается, ввожу кол-во строк, столбцов, нажимаю Enter, консольное окно закрывается...

[Mandrivnyk]

Попробуй везде перед cout и cin поставить std::
А вообще, это странно, конечно же...
Повторю вопрос коллеги -- какой компилятор?

[Azzzza]

Очень старый, видимо...
Borland C++ 5.02