[C] Acquisizione e visualizzazione di stringhe

[SwagLighning]

Buonasera, devo svolgere un semplice esercizio di acquisizione e visualizzazione di stringhe utilizzando printf e scanf, ma non riesco a farlo funzionare. Potreste gentilmente indicarmi cosa ho sbagliato nel mio codice? Premetto di essere alle prime armi con la programmazione in generale.
Vi ringrazio anticipatamente per eventuali risposte.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define D 21


int main()
{
    int N, i;
    char stringa[N][D];
    const char* s;
    
    printf("Inserisci numero stringhe: ");
    scanf("%d", &N);
    while ((N<1) || (N>99))
    {printf("\nNumero non consentito! Reinserire numero stringhe: ");
    scanf ("%d", &N);}
    printf("\n\n");
    for(i=0;i<N;i++)
    {printf("Inserisci stringa %d: ", i+1);
    scanf("%s", s);
    if (strlen(s)==0)
        { printf("Vuota - ripeti\n");
        i--;}
    else 
        {if(strlen(s)>D)
            {printf("Troppo lunga! Max %d caratteri\n", D-1) ;
            i--;}
        else
            {strcpy(stringa[i], s);}}
    }
    printf("\n\nLe stringhe inserite sono: \n");
    for(i=0;i<N;i++)
    {printf("Stringa %d = %s\n", i+1, stringa[i]);}
    return 0;
}

[zacca94]

perchè è multidimensionale questa?
char stringa[N][D];

[SwagLighning]

perchè è multidimensionale questa?
char stringa[N][D];

Perché l'ho intesa come se fosse una particolare matrice, essendo un vettore di vettore di caratteri; infatti N corrisponde al numero di stringhe totali, specificate dall'utente, mentre D al numero di caratteri massimi consentiti. Ho per caso sbagliato qui?

[boots]

Ci sono due problemi:

1. Quando dichiari stringa

char stringa[N][D];

La dimensione deve essere nota al momento della compilazione, perchè è definito nello stack. Tu N lo valorizzi solo in seguito, di conseguenza lo "spazio" assegnato a stringa è ignoto. Se N è 0 (ad esempio) stringa non ha memoria disponibile per contenere le tue stringhe. Di conseguenza quando vai a scrivere sull'array vai fuori dai limiti ed il programma crasha
Quindi o definisci N come hai fatto per D (e quindi poi vai ad usare solo una parte di questo array),

#define D 21
#define MAXN 99

int main()
{
    int N, i;
    char stringa[MAXN][D];
...

oppure "sposti" nell heap stringa:

int main()
{
    int N, i;
    char** stringa;
    char s[D] ;
    printf("Inserisci numero stringhe: ");
    scanf("%d", &N);
    while ((N<1) || (N>99))
    {printf("\nNumero non consentito! Reinserire numero stringhe: ");
    scanf ("%d", &N);}
    printf("\n\n");
    stringa = (char**)malloc(sizeof(char*) * N);
    for(i=0;i<N;i++)
    {
     stringa[i] = (char*)malloc(sizeof(char) * D);
     printf("Inserisci stringa %d: ", i+1);
    scanf("%s", s);
    if (strlen(s)==0)
        { printf("Vuota - ripeti\n");
        i--;}
    else 
        {if(strlen(s)>D)
            {printf("Troppo lunga! Max %d caratteri\n", D-1) ;
            i--;}
        else
            {strcpy(stringa[i], s);}}
    }
    printf("\n\nLe stringhe inserite sono: \n");
    for(i=0;i<N;i++)
    {printf("Stringa %d = %s\n", i+1, stringa[i]);}
    return 0;
}

2. Come vedi nel codice qui su ho cambiato s.
Come hai fatto tu, s è un puntatore che "punta" non si sa sa dove. Quando lo passi alla scanf questa tenta di scrivere nella zona di memoria a cui fa riferimento s che però essendo ignota porta al crash del programma (se sei fortunato). Ricorda che un puntatore DEVE far sempre riferimento ad una zona di memoria valida

[SwagLighning]

Ti ringrazio boots, anche per la spiegazione dettagliata; finalmente mi hai fatto venire a capo dell'esercizio!