Как получить результат функции в main при работе с матрицами?

Question

[moneymakerXA]

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

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int sum (int n, int matrix[n][n]) {
    int sum = 0;
    for (int j = 0; j < n; ++j){
        for (int i = 0; i < n; ++i){
            if (j == i){
                sum += matrix[j][i];
            }
        }
    }
    return sum;
}

int main ()
{
    int n, i, j;
    int sum_of = 0;
    printf("n \n");
    scanf("%d",&n);
    int matrix[n][n];

    srand(time(0));
    for (i = 0; i < n; i++){
        for (j = 0; j < n; j++){
            matrix[i][j] = rand()%10;
        }
    }

    for (i = 0; i < n; i++){
        for (j = 0; j < n; j++){
            printf("%d ", matrix[i][j]);
        }
        printf("\n");
    }

    sum_of = sum(matrix[][]);

    printf("%d", sum_of);

    return 0;
}

Answer 1

[jcmvbkbc]

sum_of = sum(matrix[][]);

должно стать
sum_of = sum(n, matrix);

int sum (int n, int matrix[n][n]) {
    int sum = 0;
    for (int j = 0; j < n; ++j){
        for (int i = 0; i < n; ++i){
            if (j == i){
                sum += matrix[j][i];
            }
        }
    }
    return sum;
}

Это, конечно, работает, но лучше написать так:

int sum (int n, int matrix[n][n])
{
    int sum = 0;
    for (int i = 0; i < n; ++i) {
        sum += matrix[i][i];
    }
    return sum;
}

Answer 2

[dedalqq]

Что то я сомневаюсь, что у вас программа работает. Вы пытаетесь в стеке задать масив произвольной длинны. В стеке можно создавтаь масивы только фиксированной длинны, так как под них выделяется память во время компиляции. В вашем же случае нужно использовать кучу (см функции malloc() и free()) https://en.wikipedia.org/wiki/C_dynamic_memory_all...

Это если мы говорим про C если же вы пишете это на C++ (пожалуйста различайте эти языки) то вам в помощь вектора

Comments

[res2001]

Если мы говорим о Си, то начиная с С99 есть VLA - это как раз массивы произвольной длины в стеке.
Но до сих пор не все компиляторы поддерживают эту возможность, например в msvc до сих пор нет, gcc и clang - работает на ура.

Answer 3

[CityCat4]

Программа полностью работает

Сумнительно.

Вот так:

printf("n \n");
    scanf("%d",&n);
    int matrix[n][n];

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

Выделение памяти нужно делать таким образом:

int *matrix;

matrix = calloc( n * n, 1);

после чего matrix - указатель на int и всю программу соответственно нужно переделать под адресную арифметику (подозреваю, что задание было на изучение адресной арифметики :) )

По-хорошему, результат calloc нужно проверить - выделилась ли память и перед завершением программы сделать
free(matrix);

Comments

[jcmvbkbc]

Вот так:

printf("n \n");
    scanf("%d",&n);
    int matrix[n][n];

делать нельзя. Статический массив строится в момент компиляции,

Здесь нет статического массива, и в С начиная с С99 так делать можно.

Answer 4

[res2001]

Обратите внимание на то как объявлена функция:
int sum (int n, int matrix[n][n])
И на то как вы ее вызываете:
sum_of = sum(matrix[][]);
Надеюсь дальше разберетесь сами.