Работа с классами, Поиск max в матрице Опции

[Triplet]

Скажите, пожалуйста, почему программа не компилируется?
Необходимо найти максимум в матрице.

// T02(01).cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <iostream>
using namespace std;
#include <stdlib.h>
#include <math.h>

class MASS
{private:
int a[50][50];
public:
int n;int m;
	int i,j;
	MASS();
void getdata()
{cin >> n;
 cin >> m; 
};
void getmatrix()
{for (i=1;i<=n;i++)
{for (j=1;j<=m;j++)
{cin >> a[i][j];};};
};
void sort(int max, int i,int j,int n1,int m1)
{max=a[1][1];n1=n;m1=m;
for (i=1;i<=n1;i++)
{for (j=1;j<=m1;j++)
{if (a[i][j]>max){max=a[i][j];};
};
};
};
void putdata()
{cout << max;}
};

int _tmain(int argc, _TCHAR* argv[])
{MASS a;
 MASS n;MASS m;MASS max;
cout << "Enter n,m" << "\n";
n.getdata();
m.getdata();
cout << "Enter matrix" << "\n";
a.getmatrix();
max.putdata();
return 0;
}

[volvo]

Ну, вот ... А теперь посмотри, что ты делаешь... Вроде бы описан класс MASS, который и должен представлять эту самую матрицу, но почему-то m и n (которые уже есть внутри класса, и вроде бы задают размеры матрицы), в основной программе объявлены опять же как MASS... Не надо этого делать...

Ты уже определила их внутри класса, на этом и основан принцип инкапсуляции - класс MASS должен скрыть от пользователя детали реализации. В идеале - все, что пользователь должен сделать - это:

...
int main() {
  MASS mx(3, 3); // <--- Ну, или ввести размерность с клавиатуры

  cin >> mx;
  cout << "Максимум = " << mx.get_max() << endl;
  return 0;
}

Все остальное должно делаться внутри класса "матрица": и ввод значений элементов, и вычисление (при вызове get_max) максимума... Пользователю в основной программе совсем не обязательно знать, как это делается, и какие переменные (и каких типов) при этом используются...