slacky.eu

DaD ha scritto:Non riesco a capire perchè questo codice mi restituisca un warring.
Il codice è questo;

Codice: Seleziona tutto

#include <stdio.h>
#include "matrix.h"

int main(void){
   int a[2][3] = {{2, 3, 4},
                  {3, 4, 5}};
   printf("L'elemento massimo è %d\n", max_matrice(a, 2, 3));
   getchar();
   return 0;
}

/* matrix.h */
int max_matrice(int **a, int n, int m){
   int i, j, max = a[0][0];
   for(i = 0; i <= n; i++){
      for(j = 0; j <= m; j++){
          if(a[i][j] > max)
             max = a[i][j];
      }
   }
}

Il warring è questo;

Codice: Seleziona tutto

warning: passing argument 1 of 'max_matrice' from incompatible pointer type

In pratica da quello che ho capito sbaglio a passargli la matrice, ma non riesco a capire l'errore dove sia ...

int **a

int *a

gallows ha scritto:In C non puoi convertire un T[][] in T**, se devi implementare una matrice ti consiglio una struttura opaca, che peraltro ti permette una facile gestione degli errori.

nicop ha scritto: Dad non ne sono tanto sicuro,ma se non ricordo male usando questo dichiarazione

Codice: Seleziona tutto

int **a

gli passi come parametro un puntatore ad un puntatore.Prova semplicemte a passare come parametro il puntatore,ovvero così

Codice: Seleziona tutto

int *a

spero di non aver detto una cavolata...
ciaoo

DaD ha scritto:

gallows ha scritto:In C non puoi convertire un T[][] in T**, se devi implementare una matrice ti consiglio una struttura opaca, che peraltro ti permette una facile gestione degli errori.

Scusa la mia ignoranza gallows ma cosa intendi per "struttura opaca"?
Intendi dire di inzializzare la matrice in questo modo: a[n][m]?

typedef unsigned int   uint;
typedef int            mx_t;

typedef struct _Matrix Matrix;

struct _Matrix {
    mx_t*  data;
    uint   cols;
    uint   rows;
};

/* con una libreria come la seguente: */
/* (codice non testato) */

Matrix* matrix_init(uint rows, uint cols)
{
    Matrix* mx = malloc( sizeof(Matrix) );

    if ( rows && cols )
        mx->data = malloc( rows * cols * sizeof(mx_t) );
    else
        mx->data = NULL;

    mx->rows = rows;
    mx->cols = cols;

    return mx;
}

void matrix_destroy(Matrix* mx)
{
    free( mx->data );
    free( mx );
}

mx_t matrix_get_cell(const Matrix* mx, uint i, uint j)
{
    if (i >= mx->rows || j >= mx->cols)
        /* gestisci l'errore */

    return mx->data[mx->cols * i + j];
}

int matrix_set_cell(Matrix* mx, uint i, uint j, mx_t v)
{
    if (i >= mx->rows || j >= mx->cols)
        return -1;

    mx->data[mx->cols * i + j] = v;

    return 0;
}

DaD ha scritto:Non riesco a capire perchè questo codice mi restituisca un warring.
Il codice è questo;

Codice: Seleziona tutto

/* matrix.h */
int max_matrice(int **a, int n, int m){
   int i, j, max = a[0][0];
   for(i = 0; i <= n; i++){
      for(j = 0; j <= m; j++){
          if(a[i][j] > max)
             max = a[i][j];
      }
   }
}

lamarozzo ha scritto:Sei vuoi passare un array bidimensionale ad una funzione basta fare

Codice: Seleziona tutto

//dichiarazione
void funzione (int *matrice[2]);

int main()
{
  int a[3][2];
  funzione(a);
}  

Ricordati che nella lista di parametri devi sempre dichiarare la grandezza (in questo caso 2) dell'ultima dimensione dell'array.

Gli array multidimensionali sono la soluzione più comoda se ne conosci le dimensioni al tempo di compilazione. Altrimenti devi ricorrere all'allocazione dinamica. Se con la matrice in questione devi fare operazioni intensive (calcoli) devi fare in modo che occupi uno spazio contiguo in memoria.

Ad esempio seguendo il codice di Gallows. Soltanto che io eviterei di implementare le funzioni set e get per accedere alla matrice per non appesantire troppo il programma.

Gallows suggeriva di usare i void* per scrivere codice generico. Il problema di quella soluzione, a mio avviso, è la type safety. C'è un'altra soluzione: usare le macro del preprocessore. I fortunati utilizzatori del c++ hanno invece i potentissimi template.

A presto.

P.S. Corrado ha ragione

int   myvect[3][4] = {{1, 2, 3, 4}, {3, 2, 0, 1}, {5, 8, 1, 1}};

int max_matrice(int **a, int n, int m);

int maxmatrice(int *a[], int n, int m);

int max_matrice(int *a[4], int n, int m);

int *a[4]

(*a)[4]

int max_matrice(int (*a)[3], int n, int m);

/* matrix.h */
int max_matrice(int (*a)[3], int n, int m){
	int i, j, max = a[0][0];
	for(i = 0; i < n; i++){
		for(j = 0; j < m; j++){
			if(a[i][j] > max)
				max = a[i][j];
		}
	}
	return max;
}

phobos3576 ha scritto: Lamarozzo ... attenzione alla sintassi ...

phobos3576 ha scritto:P.S. A parte questo, il codice scritto da DaD per calcolare il valore massimo mi sembra sbagliato; la versione giusta dovrebbe essere:

Codice: Seleziona tutto

/* matrix.h */
int max_matrice(int (*a)[3], int n, int m){
	int i, j, max = a[0][0];
	for(i = 0; i < n; i++){
		for(j = 0; j < m; j++){
			if(a[i][j] > max)
				max = a[i][j];
		}
	}
	return max;
}

DaD ha scritto:Si ma int m, a questo punto non serve visto che la matrice sarà sempre N X 3, giusto?