arrays — Program To Test, является ли матрица 4×4 магическим квадратом, используя функции в переполнении стека

Вот что у меня есть. У меня есть вывод большинства сумм только для того, чтобы я мог проверить там значения. Я думаю, что проблема со значением элементов в массиве, хранящем суммы столбцов.
Буду очень признателен за любые отзывы.

#include <iostream>
#include <cmath>
using namespace std;

void fillMatrix(int matrix[][4],const int SIZE);
int rowSum(int matrix[][4],const int SIZE,int row[]);
int columnSum(int matrix[][4], const int SIZE, int column[]);
bool isMagic(int matrix[][4], const int SIZE,int row[],int column[]);

int main()
{
const int SIZE=4;
int matrix[SIZE][SIZE];
int row[4],column[4];//arrays to be filled with row and column sums.
char response=0;

cout<<"This program determines whether or not a 4x4 square matrix is a magic square.\n";
do
{
fillMatrix(matrix,SIZE);
rowSum(matrix,SIZE,row);
columnSum(matrix,SIZE,row);if(isMagic(matrix,SIZE,row,column))
cout<<"This is a magic square.\n\n";
else {
cout<<"This is not a magic square.\n\n";
}
cout<<"To end this program, enter q. To check another matrix, enter any other letter.\n";
cin>>response;
}while(response!='q'&&response!='Q');

return 0;
}

void fillMatrix(int matrix[][4],const int SIZE)
{
for(int i=0;i<4;i++)
{
cout<<"Enter four values for row "<<i+1<<".\n";
for(int j=0;j<4;j++)
{
cin>>matrix[i][j];
}
}
}

int rowSum(int matrix[][4],const int SIZE,int row[4])
{
int i=0;
int rowsum=0;
for(i=0;i<4;i++)
{
rowsum=0;
for(int j=0;j<4;j++)
{
rowsum+=matrix[i][j];
}
row[i]=rowsum;
cout<<row[i]<<endl;
}
return row[i];

}
int columnSum(int matrix[][4], const int SIZE, int column[4])
{
int j=0;
int columnsum=0;
for(j=0;j<4;j++)
{
columnsum=0;
for(int i=0;i<4;i++)
{
columnsum+=matrix[i][j];
}
column[j]=columnsum;
cout<<column[j]<<endl;
}
return column[j];
}

bool isMagic(int matrix[][4], const int SIZE,int row[4],int column[4])
{
int rightdiagonalsum=0, leftdiagonalsum=0, check;

for(int i=0;i<4;i++)
{
rightdiagonalsum+=matrix[i][i];
}
cout<<rightdiagonalsum<<endl;
for(int i=0;i<4;i++)
{
leftdiagonalsum+=matrix[i][3-i];
}
cout<<leftdiagonalsum<<endl;
for(int i=1;i<4;i++)
{
if (row[i]==row[i-1])
{
check=row[i];
}
else {
return false;
}
}
for(int j=0;j<4;j++)
{
if (column[j]!=check)
{
cout<<column[j]<<"*****";//For some reason, the value of column[j] is 0.
return false;
}
}

if (rightdiagonalsum!=check||leftdiagonalsum!=check)
{
return false;
}

return true;

}

-1

Решение

Этот код:

rowSum(matrix,SIZE,row);
columnSum(matrix,SIZE,row);

должно быть:

rowSum(matrix,SIZE,row);
columnSum(matrix,SIZE,column);

Итак column массив в вашем коде имеет нулевые значения по довольно обычной причине, что вы никогда не инициализировали его.

Более того, вы получаете доступ к концу массива здесь:

return row[i];

и здесь:

return column[j];

В обоих этих точках i а также j иметь значения 4, Вы бы избежали такой ошибки, если бы вы объявили переменные цикла внутри for заявление. Это ограничило бы область применения этих переменных.

Чтобы исправить проблему, верните rowsum а также columnsum соответственно.

Мне интересно, с какой целью различные SIZE объявления служат, поскольку ваш жесткий код имеет значение 4 повсюду.

0

Другие решения

Других решений пока нет …

По вопросам рекламы [email protected]