Почему я не вижу результаты работы метода?

Question

[Tolstohabr]

work(), вроде, должен был вывести на экран два числа(maxt и maxm). Но он как буд-то не работает. Почему?

#include <iostream> 
using std::cin;
using std::cout;
using std::endl;

class matr
{
	int xsize, ysize;
	int** base;
public:
	int m;
	matr(int m, int n)
	{
		xsize = m;
		ysize = n;
		base = new int* [ysize];
		for (int i = 0; i < ysize; i++) {
			base[i] = new int[xsize];
		}
	}
	~matr()
	{
		for (int i = 1; i < ysize; i++) {
			delete[] base[i];
		}
		delete[] base;
		xsize = 0;
		ysize = 0;
	}

	void input();
	void output();
	void work();
	double get_xsize()
	{
		return (xsize);
	}
	double get_ysize()
	{
		return (ysize);
	}
	int& elem(int i, int j)
	{
		return (base[i][j]);
	}
};

void matr::input()
{
	cout << "vvod matr" << endl;
	for (int i = 0; i < xsize; i++)
		for (int j = 0; j < ysize; j++) {
			cout << "elem=";
			cin >> base[i][j];
		}
}

void matr::output()
{
	cout << "Vivod matr:\n";
	for (int i = 0; i < xsize; i++) {
		for (int j = 0; j < ysize; j++) {
			cout << base[i][j]<<" ";
		}
		cout << endl;
	}
}

void matr::work() {
	int maxt = 0;
	int maxm = 0;
	//писк минимального элемента в треугольнике
	for (int i = 0; i <= m / 2; i++) {
		for (int j = i; j < m - 1; j++) {
			if ((i == m && j == 0) || base[i][j] <= maxt) maxt = base[i][j];
		}
	}
	//поиск минимального элемента во всей матрице
	for (int i = 0; i < m; i++) {
		for (int j = i; j < m - 1; j++) {
			if ((i == m && j == 0) || base[i][j] <= maxm) maxm = base[i][j]; 
		}
	}
	cout << maxt << endl;
	cout << maxm << endl;
}

int main()
{
	int m;
	cout << "m=";
	cin >> m;
	matr w(m, m);
	w.input();
	w.output();
	w.work();
	system("PAUSE");
	return 0;
}

Comments

[Roman]

public:
  int m;

m == ???

Tolstohabr,

Но он как буд-то не работает. Почему?

Потому что неициализированая m имеет значение от балды.

//писк минимального элемента в треугольнике
  for (int i = 0; i <= m / 2; i++) {
    for (int j = i; j < m - 1; j++) {
      if ((i == m && j == 0) || base[i][j] <= maxt) maxt = base[i][j];
    }
  }
  //поиск минимального элемента во всей матрице
  for (int i = 0; i < m; i++) {
    for (int j = i; j < m - 1; j++) {
      if ((i == m && j == 0) || base[i][j] <= maxm) maxm = base[i][j]; 
    }
  }

Answer 1

[Mercury13]

Потому что сначала output, потом work.

Comments

[Roman]

work(), вроде, должен был вывести на экран два числа(maxt и maxm).

void matr::work() {
// ...
  cout << maxt << endl;
  cout << maxm << endl;
}

[Mercury13]

Да, посмотрел ещё код, и дело тут вот в чём. Что делает этот самый m? У нас же для этого xsize и ysize.

НЕ СОВЕТУЮ называть так габариты матрицы, спутаешь. Ведь первый габарит матрицы — СТРОКА, то есть y. Лучше nRows/nCols.