C++ и функции

[Santic]

Дана целочисленная квадратная матрица. Определить:
2) минимум среди сумм модулей элементов диагоналей, параллельных побочной диагонали матрицы.

Этот пункт задания надо оформить в виде функции.

Написал код без функций, всё пашет:

#include <iostream.h>
#include <stdlib.h>
#include <conio.h>

int x[50][50], y[50][50], i, j, k, n, sum, sum1 = 0, sum2 = 0, min = 0;

void main()
{
	randomize(); clrscr();

	cout<<"Enter size of matrix: "; cin>>k; cout<<"\n";

	for (i = 0; i < k; i++)
	{
		cout<<"\n"<<"  ";
		for (j = 0; j < k; j++)
			{
			x[i][j] = (random(15)-5.2)/2.;
			cout<<x[i][j]<<"    ";
			}
	}

	cout<<"\n"<<"\n";
	
	for (i = 0; i < k; i++)
		{
		for (j = 0; j < k - i; j++)
			{
			y[i][j] = x[k - j - 1][k - i - 1];
			}
		}

	cout<<"sum:"<<"\n"<<"\n";

	for (j = 0; j < k; j++)
	{ sum = 0;
		for (i = 0; i < k; i++)
		{
	if (x[i][j] < 0) {sum = 0; break;}
		else
		sum += x[i][j];
		}
		cout<<"   "<<sum;
	}

	cout<<"\n"<<"\n";

//__________________________________________________________
//transponirovanie
		cout<<"matrix y:"<<"\n";

		for (i = 0; i < k; i++)
       		{
       		cout<<"\n"<<"  ";
       		for (j = 0; j < k; j++)
			{
			cout<<y[i][j]<<"    ";
			}
		}

//___________________________________________________________

	cout<<"\n"<<"\n";
	for (n = k; n > 0; n--)
	{
		for (i = 0; i < k - 1; i++)
			{
			 sum1 += x[i][k - i - 2];
			 sum2 += y[i][k - i - 2];
			}
			k--;
	 }
		if (sum1 < sum2) min=sum1;
		else min=sum2;

	cout<<"up: "<<sum1<<"   "<<"down: "<<sum2<<"\n"<<"\n";
	cout<<"min sum: "<<abs(min);

	getch();
}

Оформил это же задание в виде функций:

#include <iostream.h>
#include <stdlib.h>
#include <conio.h>

/*void size(int k)
{
cout<<"Enter size of matrix: "; cin>>k; cout<<"\n";
}
*/

int  enter_matrix(int **x, const int k)
{
int i = 0, j = 0;
	for (i = 0; i < k; i++)
	{
		cout<<"\n"<<"  ";
		for (j = 0; j < k; j++)
			{
			x[i][j] = (random(15)-5.2)/2.;
			cout<<x[i][j]<<"    ";
			}
	}
return x[i][j];
}

int transponirovanie(int **x, int **y, const int k)	
{	
int i = 0, j = 0;
	for (i = 0; i < k; i++)
		{
		for (j = 0; j < k - i; j++)
			{
			y[i][j] = x[k - j - 1][k - i - 1];
			}
		}
return y[i][j];
}

int sum_i_vivod_stolbc(int **x, const int k)
{	

int sum = 0, i = 0, j = 0;

	for (j = 0; j < k; j++)
	{ sum = 0;
		for (i = 0; i < k; i++)
		{
	if (x[i][j] < 0) {sum = 0; break;}
		else
		sum += x[i][j];
		}
		cout<<"   "<<sum;
	}
return sum;
}
		
int sum_elements_diagonaley_deistvia(int **x, int **y, int k) 
{

int i = 0, n = 0, sum1 = 0, sum2 = 0, min = 0;

	for (n = k; n > 0; n--)
	{
		for (i = 0; i < k - 1; i++)
			{
			 sum1 += x[i][k - i - 2];
			 sum2 += y[i][k - i - 2];
			}
			k--;
	 }

	if (sum1 < sum2) min=sum1;
	else min=sum2;

	cout<<"up: "<<sum1<<"   "<<"down: "<<sum2<<"\n"<<"\n";
	cout<<"min sum: "<<abs(min);

return sum1, sum2, min;
}

int main()
{
int i = 0, j = 0, k = 0;
int **x;
int **y;

	randomize(); clrscr();

	cout<<"Enter size of matrix: "; cin>>k; cout<<"\n";

	enter_matrix(x, k);
	cout<<"\n"<<"\n";
	
	transponirovanie(x, y, k);

	cout<<"sum:"<<"\n"<<"\n";
	sum_i_vivod_stolbc(x, k);
	cout<<"\n"<<"\n";

	sum_elements_diagonaley_deistvia(x, y, k);

	cout<<"\n"<<"\n";

	getch();

return 0;
}

Проблема состоит в том, что функция sum_elements_diagonaley_deistvia выдаёт up и down не правильные если квадратная матрица имеет более 3 строк и столбцов(хотя без функций всё ок). Где искать корень этой ошибки? Заранее спасибо!

[kaljan775]

а программа все ок обрабатывает, которая без функция?

какие у вас интересные функции, по 3 значения возвращают

[Santic]

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

а программа все ок обрабатывает, которая без функция?

Та что без функций - работает исправно, проверял много раз.

какие у вас интересные функции, по 3 значения возвращают

Разве не правильно возвращать так?

[kaljan775]

ну, по идее так
int faunc(int x,int y,int z) {return x+y+z;}

и вызывается она вот так
x=faunc(1,2,3);

а чтобы менять значения в переменных обычно используется void-функция, которой сообщается адрес пересенной, которую требуется изменить

[Santic]

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

ну, по идее так
int faunc(int x,int y,int z) {return x+y+z;}

и вызывается она вот так
x=faunc(1,2,3);

а чтобы менять значения в переменных обычно используется void-функция, которой сообщается адрес пересенной, которую требуется изменить

Сути не меняет, баг не отловлен.

Как же быть??? Об клаву бьюсь, не считает правильно и всё!!!!

[ACE Valery]

Дебаггером пользовались? Говорят, помогает

[Santic]

Сообщение от ACE Valery

Дебаггером пользовались? Говорят, помогает

Если честно, то ни разу не пользовался Пишу на Borland C :D

Всё, проблема решена. Вот код:

#include <iostream.h>
#include <stdlib.h>
#include <conio.h>


int transponirovanie(int **x, int **y, const int k)	
{	
int i = 0, j = 0;
	for (i = 0; i < k; i++)
		{
		for (j = 0; j < k - i; j++)
			{
			y[i][j] = x[k - j - 1][k - i - 1];
			}
		}
return 0;
}

int sum_i_vivod_stolbc(int **x, const int k)
{	

int sum = 0, i = 0, j = 0;

	for (j = 0; j < k; j++)
	{ sum = 0;
		for (i = 0; i < k; i++)
		{
	if (x[i][j] < 0) {sum = 0; break;}
		else
		sum += x[i][j];
		}
		cout<<"   "<<sum;
	}
return sum;
}
		
int sum_elements_diagonaley_deistvia(int **x, int **y, const int k) 
{
int i = 0, n = 0, m = k, sum1 = 0, sum2 = 0, min = 0;

	for (n = m; n > 0; n--)
	{
		for (i = 0; i < m - 1; i++)
			{
			 sum1 += x[i][m - i - 2];
			 sum2 += y[i][m - i - 2];
			}
			m--;
	 }

	if (sum1 < sum2) min=sum1;
	else min=sum2;

	cout<<"up: "<<sum1<<"   "<<"down: "<<sum2<<"\n"<<"\n";
	cout<<"min sum: "<<abs(min);

return sum1+sum2+min;
}

int main()
{
int i = 0, j = 0, k = 0;
int **x;
int **y;

	randomize(); clrscr();

	cout<<"Enter size of matrix: "; cin>>k; cout<<"\n";
	

//new fuuuuu massive :D

	x = new int* [k];
	for (i = 0; i < k; i++)
		x[i] = new int [k];
	for (i = 0; i < k; i++)
		for (j = 0; j < k; j++)		
			x[i][j] = (random(15)-5.2)/2.;
			
//new fuuuuu2 massive :DDD
				
	y = new int* [k];
	for (i = 0; i < k; i++)
		y[i] = new int [k];
	for (i = 0; i < k; i++)
		for (j = 0; j < k; j++)		
			y[i][j] = x[i][j] ;

//vivod fuuuu massive

	for (i = 0; i < k; i++)
	{
		cout<<"\n"<<"  ";
		for (j = 0; j < k; j++)
			{
			cout<<x[i][j]<<"    ";
			}
	}
	
	cout<<"\n"<<"\n";
	
	transponirovanie(x, y, k);

	cout<<"sum:"<<"\n"<<"\n";
	sum_i_vivod_stolbc(x, k);
	cout<<"\n"<<"\n";
	
	sum_elements_diagonaley_deistvia(x, y, k);

	cout<<"\n"<<"\n";

	getch();

return 0;
}

Что было не так: Матрица не должным образом была сформирована; Функция не может возвращать матрицы :D