задача с матрицОй )

[Ponch_Begginer]

Задача:
Задана матрица размером NxN. Поменять местами строку с максимальным элементом матрицы и строку с минимальным эле-ментом.

Мой код:

#include <iostream>

using namespace std;

const int NUM(4); 

int main()
{
    int mat[NUM][NUM];

    int _max(0), _min(0), mxline(0), mnline(0);

    for(int i(0);i<=NUM-1;i++)		
        for(int j(0);j<=NUM-1;j++)	
        {
            cout << "mat[" << i << "][" <<j<< "] = ";
            cin >> mat[i][j];
            if(i==0)
               _min = _max = mat[0][0];		
            else
            {
                if(mat[i][j]>_max)	
                {
                    _max = mat[i][j];
                    mxline = i;
                }
                if((mat[i][j]<_min))	
                {
                    _min = mat[i][j];
                    mnline = i;
                }
            }

    	}

    	    for(int i(0);i<=NUM-1;i++)
    {
        for(int j(0);j<=NUM-1;j++)
            cout << mat[i][j] << " | ";
        cout << endl;
	}

    int temp;

    for(int i(0);i<=NUM-1;i++)	
    {
        temp = mat[mxline][i];
        mat[mxline][i] = mat[mnline][i];
        mat[mnline][i] = temp;
    }

    for(int i(0);i<=NUM-1;i++)
    {
        for(int j(0);j<=NUM-1;j++)
            cout << mat[i][j] << " | ";
        cout << endl;
	}
    return 0;
}

Правильно-ли я сделал?
Что исправить?
Готов к разумной критике .

[temaps]

сходу:

if(i==0&&j==0)
_min = _max = mat[0][0];

глубоко не разбирался, но на первый взгляд остальное верно

[min@y™]

Я надеюсь, ты учёл, что строк с минимальным и/или максимальным элементом матрицы может быть несколько?

[Ponch_Begginer]

Спасибо.
Меня интересуют не только ошибки, но советы по улучшению.

[min@y™]

и не надо пихать всё хозяйство в main(), ибо свыше 20 строк - уже жуть и смотреть страшно. разнеси по функциям, выполняющим свои чиста-конкретные задачи.

[Ponch_Begginer]

Я не стал учитывать возможность нескольких строк, так-как в условии ничего на счет этого ничего не говорится)

[Stilet]

Улучшений хочешь? - Сократи код:

#include <iostream>

using namespace std;

const int NUM(4); 

void Write(int **a, int N){

 for(int i(0);i<N;i++)
    {
        for(int j(0);j<N;j++)  cout << mat[i][j] << " | ";
        cout << endl;
     }
 cout << endl;
}

int main()
{
    int mat[NUM][NUM];

    int _max(0), _min(0), mxline(0), mnline(0);

    for(int i(0);i<NUM;i++)		
        for(int j(0);j<NUM;j++)	
        {
            cout << "mat[" << i << "][" <<j<< "] = ";
            cin >> mat[i][j];
            if(mat[i][j]>mat[_max][j]) _max=i;
            if(mat[i][j]<mat[_min][j]) _min=i;
    	}
    Write(&mat,NUM);

    int temp;
    for(int i(0);i<NUM;i++)	
    {
        temp = mat[_max][i];
        mat[_max][i] = mat[_min][i];
        mat[_min][i] = temp;
    }

    Write(&mat,NUM);

    cin.get();
    return 0;
}

[min@y™]

Я не стал учитывать возможность нескольких строк, так-как в условии ничего на счет этого ничего не говорится)

однако, в условии и обратное не оговорено. если препод нетрадиционно сексуально ориентированный - может сыграть с тобой в игру "я начальник - ты дурак". ибо в реальных условиях таки максимумов/минимумов может быть больше 1, как ни крути.

Мне-то пох, тебе сдавать-то...

[Ponch_Begginer]

Да спасибо, учту на будущее)