[C] prodotto fra matrici.

[oregon]

Ho messo a posto un po' di indici nei cicli ... ti mostro tutte le parti modificate e quelle eliminate in quanto inutili ...

Per il main, parto da dopo l'input di righe e colonne fino alla fine

    matrice = (int**) malloc((righe * sizeof(int*)));
    for (i = 0; i < righe; i++) 
        matrice[i] = (int*) malloc(colonne * sizeof(int));
    
    trasp = (int**) malloc((colonne * sizeof(int*)));
    for (i = 0; i < colonne; i++) 
        trasp[i] = (int*) malloc(righe * sizeof(int));
    
    ris = (int**) malloc((righe * sizeof(int*)));
    for (i = 0; i < righe; i++) 
        ris[i] = (int*) malloc(righe * sizeof(int));
    
    for (i = 0; i < righe; ++i) {
        for (j = 0; j < colonne; ++j) {
            printf("Inserisci elemento riga %d e colonna %d: ", i, j);
            fflush(stdout);
            scanf("%d", &matrice[i][j]);
            printf("\n");
            fflush(stdout);
        }
    }
    
    trasposta(trasp, matrice, righe, colonne);
    moltiplicazione(ris, trasp, matrice, righe, colonne);
    
    for (i = 0; i < righe; ++i) {
        for (j = 0; j < colonne; ++j) {
            printf("matrice[%d][%d]= %d ", i, j, matrice[i][j]);
            fflush(stdout);
        }
        printf("\n");
        fflush(stdout);
    }




    for (i = 0; i < colonne; ++i) {
        for (j = 0; j < righe; ++j) {
            printf("trasp[%d][%d]= %d ", i, j, trasp[i][j]);
            fflush(stdout);
        }
        printf("\n");
        fflush(stdout);
    }




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

per le funzioni

void trasposta(int **trasp, int **matrice, int righe, int colonne) 
{
    int i, j; 
    for (i = 0; i < righe; i++) {
        for (j = 0; j < colonne; j++) {
            trasp[j][i] = matrice[i][j];
        }
    }
}


void moltiplicazione(int **ris, int **trasp, int **matrice, int righe, int colonne)
{
    int i, j, k;
    for (i = 0; i < righe; i++) 
        for (j = 0; j < righe; j++) 
        {
            ris[i][j] = 0;
            for (k = 0; k < colonne; k++) 
                ris[i][j] += matrice[i][k] * trasp[k][j];
        }
}

[mariella_88]

usando questo metodo, le ultime due colonne danno dei valori sballati, ecco il risultato:
Righe........: 3
Colonne......: 2


Inserisci elemento riga 0 e colonna 0: 1


Inserisci elemento riga 0 e colonna 1: 2


Inserisci elemento riga 1 e colonna 0: 3


Inserisci elemento riga 1 e colonna 1: 1


Inserisci elemento riga 2 e colonna 0: 2


Inserisci elemento riga 2 e colonna 1: 3


variabile temporanea 1
variabile temporanea 2
variabile temporanea 3
variabile temporanea 6
variabile temporanea 12
variabile temporanea 18
variabile temporanea 6
variabile temporanea 12
variabile temporanea 18
variabile temporanea 1
variabile temporanea 2
variabile temporanea 3
matrice[0][0]= 1 matrice[0][1]= 2
matrice[1][0]= 3 matrice[1][1]= 1
matrice[2][0]= 2 matrice[2][1]= 3
trasp[0][0]= 1 trasp[0][1]= 3 trasp[0][2]= 2
trasp[1][0]= 2 trasp[1][1]= 1 trasp[1][2]= 3
ris[0][0]= 3 ris[0][1]= 18 ris[0][2]= 1566414912
ris[1][0]= 18 ris[1][1]= 3 ris[1][2]= 1566414912
da cosa può dipendere?!
Non dovrebbe essere un problema di puntatoti, e allo stesso tempo, la variabile temporanea assume valori che risultano corretti

[oregon]

Hai usato il codice che ti ho mostrato io ? Non mi pare perché non visualizzo la variabile temporanea ...

[mariella_88]

Si l'ho usato, l'unica cosa che ho dovuto cambiare (per generalizzare il prodotto visto che non mi serve solo per il prodotto con la trasposta) è che ho inserito righe e colonne di entrambi le matrici.. E visto che non mi trovavo, ho stampato il valore intermedio per capire dove fosse il problema.. Ti riporto il for:

void moltiplicazione(int **ris, int **matr1, int **matr2, int righe1,        int colonne1,int righe2,int colonne2){
    int i, j, k;
    for (i = 0; i < righe1; i++)
        for (j = 0; j < colonne2; j++)
        {
            ris[i][j] = 0;
            for (k = 0; k < colonne1; k++)
                ris[i][j] += matr1[i][k] * matr2[k][j];
        }


}

[oregon]

Non sono necessari tutti i dati di righe e colonne ma solamente 3 (infatti non è necessario il parametro righe2). La funzione potrebbe essere

void moltiplicazione(int **ris, int **matr1, int **matr2, int righe1, int colonne1, int colonne2)
{
    int i, j, k;
    for (i = 0; i < righe1; i++)
        for (j = 0; j < colonne2; j++)
        {
            ris[i][j] = 0;
            for (k = 0; k < colonne1; k++)
                ris[i][j] += matr1[i][k] * matr2[k][j];
        }
}

Come chiami la funzione?

[mariella_88]

moltiplicazione(ris, trasp, matrice,colonne,righe,righe, colonne);

[mariella_88]

Ho tolto il parametro che non serviva.. ma visto che devo calcolare A(trasp)*A ho invertito nella chiamata alla funzione le duematrici.. Il problema è che mi escono numeri del tipo 5308612 nell'ultima riga

[oregon]

EDIT

Io ho scritto

moltiplicazione(ris, matrice, trasp, righe, colonne, righe);

[oregon]

Se la moltiplicazione è trasp x matrice allora ris deve essere allocato così

ris = (int**) malloc((colonne * sizeof(int*)));
for (i = 0; i < colonne; i++) 
        ris[i] = (int*) malloc(colonne * sizeof(int));

e ugualmente la visualizzazione di ris

[mariella_88]

Adesso funziona perfettamente grazie.