Les chaînes extraites de l'utilisateur en C sont brouillées

Question

J'ai écrit le code C suivant pour obtenir une liste de chaînes de l'utilisateur. Mais les chaînes stockées donnent des valeurs bizarres.

#include <stdio.h> 
#include <stdlib.h> 


#define MAX_STRING_LENGTH 50 


void readInStrings(char* arr[],int n) 
{ 
    int i=0; 
    char line[MAX_STRING_LENGTH]; 

    for(i=0;i<n;i++){ 
    arr[i]=malloc(MAX_STRING_LENGTH); 
    printf("Enter another string : "); 
    scanf("%s",&arr[i]); 
    //fgets(&arr[i],MAX_STRING_LENGTH,stdin); 
    } 



    printf("Strings read in correctly.... \n"); 

    printf("Displaying out all the strings: \n"); 
    for(i=0;i<n;i++){ 
    printf("%s\n",&arr[i]); 
    } 
} 



void testStringInputs() 
{ 
    printf("Enter the number of entries : "); 
    int n; 
    scanf("%d",&n); 

    char* strings[n]; 
    readInStrings(strings,n); 
} 

Exemple d'entrée:

Entrez le nombre d'entrées: 3
Entrez une autre chaîne: Alladin
Entrez une autre chaîne: Barack Obama
Entrez une autre chaîne: Les chaînes lues correctement .. ..
Affichage de toutes les chaînes:
AllaBaraObama
BaraObama
Obama

Problèmes: 1) Pourquoi une chaîne ne prend pas en tant qu'entrée du tout?
2) Pourquoi les chaînes affichées sont-elles brouillées comme ça?

Le problème est le même si j'utilise gets() ou fgets() à la place de scanf().

Answer 1

Retrait de la & (comme le premier answerer noté) dans scanf("%s",&arr[i]); et printf("%s\n",&arr[i]); a fait l'affaire pour moi. Notez également que si vous avez compilé avec des avertissements au maximum, votre compilateur vous aurait immédiatement indiqué que le & était égaré.

Answer 2

arr [i] est déjà un pointeur, vous n'avez pas besoin &

Answer 3

Il est préférable d'utiliser un tableau de tableaux (deux dimensions) au lieu de tableau de pointeurs. J'ai eu du mal à corriger votre code. Donc, j'ai changé le code à ce

#include <stdio.h> 
#include <stdlib.h> 
#define MAX_STRING_LENGTH 50 
void readInStrings(char (*arr)[MAX_STRING_LENGTH],int n) 
{ 
    int i; 
    for(i = 0 ; i< n+1; ++i) 
    fgets(*(arr+i),MAX_STRING_LENGTH,stdin); 
    printf("Strings read in correctly.... \n"); 
    printf("Displaying out all the strings: \n"); 
    for(i=0;i< n+1;i++){ 
    printf("%s",arr[i]); 
    } 
} 
int main() 
{ 
    printf("Enter the number of entries : "); 
    int n; 
    scanf("%d",&n); 
    char strings[n][MAX_STRING_LENGTH]; 
    readInStrings(strings,n); 
    return 0; 
}