[C] Puntatore accede a locazioni sbagliate

[felpone]

Salve non capisco dove sia l'errore,mi sembra sia tutto ok con il puntatore...eppure il puntatore accede a locazioni sbagliate a quanto sembra. In questo caso la somma dovrebbe essere 0 invece da un numero folle.

#include <stdio.h>
#define LEN 5

int sum_two_dimensional_array(const int a[][LEN],int n){
 int *p,sum;
 
 for(p=&a[0];p<a[n]+LEN;p++){
    sum+=*p;                          
                              
 printf("%d\n",*p);
 
 }   
    
    
  return sum;  
    
 
    
}


int main (){
 int n=2;  
 const int c[][LEN]={0};
 
 printf("La somme e':%d",sum_two_dimensional_array(c,n));   
  system("pause");  
    
    
    }

[torn24]

Hai diversi errori
non inizializzi sum , quindi ha un valore indefinito al quale tu sommi e ottieni numeri strani

la condizione del for nella funzione , è errata dai a[n] n =2 ma la matrice non ha due
righe , non hai dato la prima dimensione

se non inizializzi la matrice con dei valori il risultato è sempre zero

codice con correzzioni :

int sum_two_dimensional_array(const int a[][LEN],int n){
 int *p,sum=0;//se non inizializzi sum a un valore indefinito 
 
 // n 0 puntatore al primo array
 // n 1 puntatore al secondo array
 for(p=a[n];p<a[n]+LEN;p++){
    
    sum+=*p;                          
                              
 printf("%d\n",*p);
 
 }   
    
    
  return sum;  
    
 
    
}


int main (){
 int n=1;  
 const int c[2][LEN]={{0,6,7,8,8},{10,23,20,6,1}};
 
 printf("La somme e':%d",sum_two_dimensional_array(c,n));   
  system("pause");  
    
    
    }

[Who am I]

Giusto, ma dal titolo della funzione penso che lui volesse ottenere la somma di entrambe gli array, non solo del secondo.Per cui ho modificato un' istruzione:

int sum_two_dimensional_array(const int a[][LEN],int n){
 int *p,sum=0;//se non inizializzi sum a un valore indefinito 
 
 // n 0 puntatore al primo array
 // n 1 puntatore al secondo array
 for(p=a[n];p<a[n]+LEN;p++){
    
    sum+=*p + *(p-LEN);                          
                              
 printf("%d\n",*p);
 
 }   
    
    
  return sum;  
    
 
    
}


int main (){
 int n=1;  
 const int c[2][LEN]={{0,6,7,8,8},{10,23,20,6,1}};
 
 printf("La somme e':%d",sum_two_dimensional_array(c,n));   
  system("pause");  
    
    
    }

PS: felpone, indenta meglio il codice.

[felpone]

Non è vero che non ho dato la prima dimensione della matrice,viene passata come argomento alla funzione (int n).

[oregon]

Originariamente inviato da felpone
Non è vero che non ho dato la prima dimensione della matrice

Come no? Non mi pare che in

const int c[][LEN]={0};

ci sia la prima dimensione.

E cosa c'entra il parametro della funzione?

[felpone]

ecco l'ho corretto e funziona perfettamente,come vedi la prima dimensione è passata alla funzione come argomento.

#include <stdio.h>
#define LEN 5

int sum_two_dimensional_array(const int a[][LEN],int n){
 int *p,sum=0;
 
 for(p=&a[0][0];p<&a[n-1][LEN];p++){
    sum+=*p;                          
                              
 printf("%d\n",*p);
 
 }   
    
    
  return sum;  
    
 
    
}


int main (){
 int n=2;  
 const int c[][LEN]={{2,0,0,0,0},{2,0,0,2,0}};
 
 printf("La somme e':%d",sum_two_dimensional_array(c,n));   
  system("pause");  
    
    
    }

[oregon]

Originariamente inviato da felpone
ecco l'ho corretto e funziona perfettamente,come vedi la prima dimensione è passata alla funzione come argomento.

Allora non hai capito quello che ho tentato di dirti.

Nel tuo primo post c'era scritto nel main

const int c[][LEN]={0};

mentre nell'ultimo

const int c[][LEN]={{2,0,0,0,0},{2,0,0,2,0}};

e ovviamente sono cose diverse.

Nella prima versione, non hai indicato al compilatore nulla che gli permettesse di capire quanto fosse grande la prima dimensione. Nella seconda versione invece, dai dati forniti per l'inizializzazione, hai fornito al compilatore l'indicazione che la prima dimensione è 2

E questo a prescindere dal parametro passato alla funzione in seguito.