[C] Problema con acquisizione stringhe da file

[wFree]

Stavo provando ad acquisire dei dati da file in una stringa e poi portarli in una matrice ma non riesco. Arriva fino alla terza riga e poi si pianta. Dov'è sbagliato l'algoritmo? Grazie

file in questione:
.GATTO.....LINO
.A.O....A..A..A
.S.RITORNO.U...
...I....N..R...
...N...SOGNO...
POMODORO......

 while(fgets(str,N,ff)!=NULL){
        len=strlen(str);
            if(str[len-1]=='\n'){
                str[len-1]='\0';
                len=len-1;}

        for(i=0;i<N+2;i++)
            matrix[righe+1][i]='.';

        for(i=0;i<len;i++){
            matrix[righe+1][i]=str[i];
        }
    righe++;}

[Scara95]

Stavo provando ad acquisire dei dati da file in una stringa e poi portarli in una matrice ma non riesco. Arriva fino alla terza riga e poi si pianta. Dov'è sbagliato l'algoritmo? Grazie

file in questione:
.GATTO.....LINO
.A.O....A..A..A
.S.RITORNO.U...
...I....N..R...
...N...SOGNO...
POMODORO......

 while(fgets(str,N,ff)!=NULL){
        len=strlen(str);
            if(str[len-1]=='\n'){
                str[len-1]='\0';
                len=len-1;}

        for(i=0;i<N+2;i++)
            matrix[righe+1][i]='.';

        for(i=0;i<len;i++){
            matrix[righe+1][i]=str[i];
        }
    righe++;}

Potresti postare anche come hai inizializzato le variabili?
In ogni caso mi sembra che tu ti stia complicando eccessivamente la vita!

[wFree]

Sì, ecco!

#define N 50

    FILE* ff;
    int righe=0;
    char matrix [righe+1][N+1];
    char str[N+2]={0};
    int i=0, len=0;

    ff=fopen("cruciverba.txt", "r");

[Scara95]

Sì, ecco!

#define N 50

    FILE* ff;
    int righe=0;
    char matrix [righe+1][N+1];
    char str[N+2]={0};
    int i=0, len=0;

    ff=fopen("cruciverba.txt", "r");

char matrix [righe+1][N+1];

questa riga è sbagliata!

In ogni caso è più semplice usare una memset e una strncpy

#include <stdio.h>
#include <string.h>

#define N 50

int main()
{
   FILE *fp;
   char buff[N];
   char mat[N][N];
   memset(mat, '.', N*N);
   int righe;
   
   fp = fopen("input.txt", "r");
   if( fp != NULL ){
      righe = 0;
      while ( fgets(buff, N, fp) != NULL ){
         if(buff[strlen(buff)-1] == '\n') buff[strlen(buff)-1] = '\0';
         strncpy(mat[righe], buff, strlen(buff));
         ++righe;
      }
      fclose(fp);
   }
   
   return 0;
}

[wFree]

Ok, ho capito grazie!
Mi puoi solo spiegare perchè la riga in questione è sbagliata?

char matrix [righe+1][N+1];

Non si può fare una cosa del genere? Perchè la utilizzo praticamente sempre

[Scara95]

Da C99 in poi sì, si può fare, ma non è questo il punto, il punto è che stai riservando spazio per per una matrice [1][N+2] e poi sicuramente leggerai più di 1 riga...
Puoi risolvere chiedendo il numero delle righe all'inizio, sovra-allocando il numero delle righe (se sei sicuro che queste basteranno sempre) oppure attraverso l'allocazione dinamica della memoria.

[wFree]

Chiaro, grazie mille!

[Scara95]

Non c'è di che