Форум программистов, компьютерный форум CyberForum.ru

программа виснет, не знаю что не так - C++

Восстановить пароль Регистрация
 
Greaple
1 / 1 / 0
Регистрация: 07.06.2012
Сообщений: 17
13.06.2012, 19:45     программа виснет, не знаю что не так #1
Дана квадратная целочисленная матрица. Определить, является ли она магическим квадратом. В магическом квадрате сумма чисел по всем горизонталям, вертикалям и двум диагоналям должна быть одна и та же.

#include<iostream>
#include <cmath>
using namespace std;
void vvodA(int **a,int n);
void summ(int **a,int n);
int main()
{
int n;
cout<<"n=";
cin>>n;
int **a=new int*[n];
vvodA(a,n);
summ(a,n);
cout<<endl;
return 0;
}
void vvodA(int **a,int n)
{
int i,j, sum=0;
cout << "Matriza A("<< n << "*" << n << "):" << endl;
for(i=0; i<n; i++)
for(j=0; j<n; j++)
{
cin>>a[i][j];
sum+=a[i][j];
}
}
void summ(int **a, int n)
{
bool flag=false;
for (int i=0; i<n; i++);
{
int i, j, sum, sum1=0,sum2=0,sum3=0,sum4=0;
for(j = 0; j < n; j++)
{
sum1+=a[i][j];
sum2+=a[j][i];
sum3+=a[j][n-1-j];
sum4+=a[n-1-j][i];
}
if (sum1==sum && sum2==sum && sum3==sum && sum4==sum) flag=true;
else flag=false;
}
if (flag) cout<<"Magic!!!\n";
else cout<<"Not magic\n";
delete []a;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
13.06.2012, 19:45     программа виснет, не знаю что не так
Посмотрите здесь:

C++ что не так в программе? Виснет при компиляции
C++ доброе время суток,подскажите что не так?программа в С++
C++ Как сделать так что я мог умножать не на два числа а на 3,4,5 и так далее?
C++ При решении программа выдаёт значение функции, равное 0 или выдаёт ошибку. Что не так?
C++ Не работает программа. не могу разобраться что в ней не так
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
13.06.2012, 20:27     программа виснет, не знаю что не так #2
форматируйте код, который сюда постите.
C++
1
2
3
4
5
6
7
//после строки
int **a=new int*[n];
//нужно еще повыделять немного массивом:
 for (int i = 0; i < n; ++i)
 {
   a[i] = new int[n];
 }
Ошибки в другой функции:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
void summ(int **a, int n)
 {
 bool flag=false;
 for (int i=0; i<n; i++); // тут лишная точка с запятой. странно что вас не смутило объявление еще одной переменной i чуть ниже
 {
 int i, j, sum, sum1=0,sum2=0,sum3=0,sum4=0; // тут объявление переменной sum. она не инициализирована
 for(j = 0; j < n; j++)
 {
 sum1+=a[i][j];
 sum2+=a[j][i];
 sum3+=a[j][n-1-j];
 sum4+=a[n-1-j][i];
 }
 // из-за того, что sum не нициализирована, тут будет краш при сравнении.
 // но чем ее инициализировать? если нулем, то корректно это работать не будет.
 // подозреваю вам проще всего завести глобальную переменную sum, которая вычисляется
 // в функции ввода, а тут идут сравнения с ней
 if (sum1==sum && sum2==sum && sum3==sum && sum4==sum) flag=true;
 else flag=false;
 }
 if (flag) cout<<"Magic!!!\n";
 else cout<<"Not magic\n";
 delete []a; // перед этим должно идти удаление дополнительных массивов. хотя вообще странно что это удаление происходит тут.
 }
Greaple
1 / 1 / 0
Регистрация: 07.06.2012
Сообщений: 17
13.06.2012, 20:36  [ТС]     программа виснет, не знаю что не так #3
Цитата Сообщение от DU Посмотреть сообщение
// из-за того, что sum не нициализирована, тут будет краш при сравнении.
*// но чем ее инициализировать? если нулем, то корректно это работать не будет.
*// подозреваю вам проще всего завести глобальную переменную sum, которая вычисляется
*// в функции ввода, а тут идут сравнения с ней
объясни плз
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
13.06.2012, 20:40     программа виснет, не знаю что не так #4
выделенное и есть объяснение проблемы. неинициализированная локальная переменная. если проинициализировать, то все равно это ничего не меняет, потому что внутри этой функции ее никто не изменяет и ее значение останется тем, которым ее инициализировали. подозреваю тут нужно то значение, которое считалось в функции vvod. так вот чтобы эта переменная была видна в двух разных функциях, самое простое для вас - сделать ее глобальной. боюсь что другие способы введут вас в ступор еще больше.
Greaple
1 / 1 / 0
Регистрация: 07.06.2012
Сообщений: 17
13.06.2012, 20:51  [ТС]     программа виснет, не знаю что не так #5
а как сделать её глобальной?
Andrey.K
 Аватар для Andrey.K
338 / 259 / 15
Регистрация: 14.11.2010
Сообщений: 480
13.06.2012, 21:28     программа виснет, не знаю что не так #6
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
#include <iostream>
#include <conio.h>
using namespace std;
void vvodA(int **a,int n)
{
    for(int i=0; i<n; i++)
    {
        for(int j=0; j<n; j++)
        {
            cout<<"Vvedite ["<<i<<"]["<<j<<"] element: ";
            cin>>a[i][j];
        }
    }
}
void vivodA(int** a,int n)
{
    cout << "Matriza A("<< n << "*" << n << "):" << endl;
    for(int i=0; i<n; i++)
    {
        for(int j=0; j<n; j++)
        {
            cout<<a[i][j]<<" ";
        }
        cout<<endl;
    }
}
void Magic(int** a,int* bufmas, int n)
{
    int pos=0;
    for(int i=0;i<n;i++)
    {
        bufmas[pos]=0;
        for(int j=0;j<n;j++)
        {
            bufmas[pos]+=a[i][j];
        }
        pos++;
    }
    for(int i=0;i<n;i++)
    {
        bufmas[pos]=0;
        for(int j=0;j<n;j++)
        {
            bufmas[pos]+=a[j][i];
        }
        pos++;
    }
    bufmas[pos]=0;
    for(int i=0;i<n;i++)
    {
        for(int j=0;j<n;j++)
        {
            if(i==j) bufmas[pos]+=a[i][j];
        }
    }
    pos++;
    bufmas[pos]=0;
    for(int i=n-1,j=0;i>-1;i--,j++)
    {
        bufmas[pos]+=a[i][j];
    }
    pos++;
    int count=bufmas[0];
    bool flag=true;
    for(int i=0;i<pos;i++)
    {
        if(bufmas[i]!=count) 
        {
            cout<<"Ne magicheskiy kvadrat."<<endl;
            flag=false;
            break;
        }
    }
    if(flag==true) 
    {
        cout<<"Magicheskiy kvadrat."<<endl;
    }
}
int main()
{
    int n;
    cout<<"n=";cin>>n;
    int **a=new int*[n];
    for(int i=0;i<n;i++)
        a[i]=new int[n];
    int* bufmas=new int[n*2+2];
    vvodA(a,n);
    vivodA(a,n);
    Magic(a,bufmas,n);
 
    for(int i=0;i<n;i++)
        delete[] a[i];
    delete[] a;
    delete[] bufmas;
 
    getch();
    return 0;
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
13.06.2012, 21:39     программа виснет, не знаю что не так
Еще ссылки по теме:

C++ Сделайте так что бы программа не закрывалась
C++ Сделать так, что бы при неверном значении программа сообщала об этом и запускала функцию второй раз
Программа - Имитация подбрасываний монеты. Что не так? C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Greaple
1 / 1 / 0
Регистрация: 07.06.2012
Сообщений: 17
13.06.2012, 21:39  [ТС]     программа виснет, не знаю что не так #7
спасибо!
Yandex
Объявления
13.06.2012, 21:39     программа виснет, не знаю что не так
Ответ Создать тему
Опции темы

Текущее время: 16:48. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru