Всё та же ошибка сегментации

[edw1n]

Дано квадратная матрица [m][m].
Надо посчитать сумму эл-тов в зарисованной области.
Безимени-2.jpg

#include <iostream>
int main (void)
{ int m, i, j, s; int a[m][m];
printf ("Enter m\n");
scanf ("%d", &m);
printf ("Enter massive elements\n");
for (i=0; i<m; i++)
    for (j=0; j<m; j++)
        scanf ("%d", &a[i][j]);
for (i=0; i<m; i++)
    { for (j=0; j<m; j++)
          printf ("%5d", a[i][j]);
          printf ("\n");
    }
for (i=0; i<m/2; i++)
    for (j=0; j<=i; j++)
        s+=a[i][j]+a[m-i-1][j]+a[i][m-j-1]+a[m-i-1][m-j-1];
    if (m%2==1)
       for (j=0; j<m; j++)
           s+=a[m/2][j];
       printf ("SUM - %d", s);
system ("pause");
return 0;
}

Программу делал препод. Попробовал запустить дома - вызывает ошибку сегментации. В чем проблема ? Помогите, пожалуйста.

[Stilet]

Ну а ты отладчиком проходил?
P.S. s=0 пропиши там где переменные описываешь.

[alexander13]

int m, i, j, s; 

int a[m][m];

[Stilet]

alexander13

Поясни... Я чет не понял...

[alexander13]

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

Поясни... Я чет не понял...

Моветон так массивы объявлять.
Размер массива определяется у него переменной m, которая объявлена шагом раньше. Что в этой переменной? У меня, например, при тестовом запуске только что, было 2130567168 =)
Лучше уж константы использовать.. Или, если так хочется, чтобы размер массива задавал пользователь, то динамику.
А тут получается непонятно.. Я допускаю, что в какой-нибудь среде такой код отработает нормально.. Может такое быть, наверное.. Но это плохой стиль и вообще неправильно.

[Stilet]

Моветон так массивы объявлять.

А-а-а. Ну да. В ученическом компиляторе это может и пройдет (хотя не знаю даже в каком) а на практике конечно чревато.
Я то подумал что ты что-то в синтаксисе углядел такое...

[edw1n]

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

Моветон так массивы объявлять.
Размер массива определяется у него переменной m, которая объявлена шагом раньше. Что в этой переменной? У меня, например, при тестовом запуске только что, было 2130567168 =)
Лучше уж константы использовать.. Или, если так хочется, чтобы размер массива задавал пользователь, то динамику.
А тут получается непонятно.. Я допускаю, что в какой-нибудь среде такой код отработает нормально.. Может такое быть, наверное.. Но это плохой стиль и вообще неправильно.

Да. Спасибо, не подумал об этом. Позже отпишу что получается.

[edw1n]

В общем убрал я int m. Компилятор начал ругать из-за того что такое делать нельзя. Вернул назад int m, как ни странно всё заработало. Не знаю почему тогда выдавало ошибку сегментации. В любом случае спасибо за помощь, товарищи