Comment lire une chaîne de caractères à partir du clavier à l'aide de C?

Je veux lire une chaîne de caractères saisie par l'utilisateur. Je ne sais pas la longueur de la chaîne. Comme il n'existe pas de chaînes de caractères en C, j'ai déclaré un pointeur:

char * word;

et utilisé scanf pour lire l'entrée à partir du clavier:

scanf("%s" , word) ;

mais j'ai une erreur de segmentation.

Comment puis-je lire l'entrée à partir du clavier en C lorsque la durée est inconnue ?

InformationsquelleAutor mainajaved | 2011-10-10
  1. 51

    Vous n'avez pas de stockage alloué pour word - c'est juste une balançant pointeur.

    Changement:

    char * word;

    à:

    char word[256];

    Noter que 256 est un choix arbitraire, ici la taille de ce tampon doit être supérieure à la plus grande chaîne possible que vous pourriez rencontrer.

    Note également que fgets est mieux (plus sûr), alors scanf pour la lecture des chaînes de longueur arbitraire, en ce qu'elle prend un size argument, ce qui aide à prévenir les dépassements de tampon:

     fgets(word, sizeof(word), stdin);
    • -1: Il a dit que la longueur est inconnue, et votre choix de 256 est entièrement arbitraire, de sorte que vous (bien, à 2½ ans) lui a appris à écrire des dépassements de mémoire tampon et d'énormes failles de sécurité. Félicitations.
    • Vous remercier pour le (un peu tardive) de la critique - j'ai mis à jour la réponse à couvrir non seulement les balançant pointeur, mais aussi la recommandation d'utiliser fgets plutôt que scanf de longueur arbitraire de la chaîne d'entrée.
    • Ah, merveilleux. 🙂 +1. Pour ce qui est tardive, pour être juste, glglgl dit ci-dessous, à seulement trente-quatre minutes après avoir posté la réponse! 😉
    InformationsquelleAutor Paul R
  2. 17

    Je ne vois pas pourquoi il y a une recommandation de l'utilisation de scanf() ici. scanf() est sûr que si vous ajoutez des paramètres de restriction de la chaîne de format comme %64s ou donc.

    Une bien meilleure façon est d'utiliser char * fgets ( char * str, int num, FILE * stream );.

    int main()
{
    char data[64];
    if (fgets(data, sizeof data, stdin)) {
        //input has worked, do something with data
    }
}

    (non testé)

    • Faire que stdin, pas STDIN.
    InformationsquelleAutor glglgl
  3. 15

    Lors de la lecture d'entrée à partir de n'importe quel fichier (stdin inclus) où vous ne connaissez pas la longueur, il est souvent préférable d'utiliser getline plutôt que scanf ou fgets parce que getline va gérer l'allocation de la mémoire pour votre chaîne automatiquement tant que vous fournir un pointeur null pour recevoir la chaîne entrée. Cet exemple permet d'illustrer:

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

int main (int argc, char *argv[]) {

    char *line = NULL;  /* forces getline to allocate with malloc */
    size_t len = 0;     /* ignored when line = NULL */
    ssize_t read;

    printf ("\nEnter string below [ctrl + d] to quit\n");

    while ((read = getline(&line, &len, stdin)) != -1) {

        if (read > 0)
            printf ("\n  read %zd chars from stdin, allocated %zd bytes for line : %s\n", read, len, line);

        printf ("Enter string below [ctrl + d] to quit\n");
    }

    free (line);  /* free memory allocated by getline */

    return 0;
}

    Les parties pertinentes en cours:

    char *line = NULL;  /* forces getline to allocate with malloc */
size_t len = 0;     /* ignored when line = NULL */
/* snip */
read = getline (&line, &len, stdin);

    Réglage line à NULL causes getline pour allouer de la mémoire automatiquement. Exemple de sortie:

    $ ./getline_example

Enter string below [ctrl + d] to quit
A short string to test getline!

  read 32 chars from stdin, allocated 120 bytes for line : A short string to test getline!

Enter string below [ctrl + d] to quit
A little bit longer string to show that getline will allocated again without resetting line = NULL

  read 99 chars from stdin, allocated 120 bytes for line : A little bit longer string to show that getline will allocated again without resetting line = NULL

Enter string below [ctrl + d] to quit

    Donc, avec getline vous n'avez pas besoin de deviner combien de temps de l'utilisateur de votre chaîne sera.

    • Ah! quelqu'un a eu son fgets dans son getline.... Bonne prise.
    • Re fgets() v. getline(). Le illimité getline(), l'OMI est trop près d'un hacker exploiter, car il permet à l'externe fourni des données pour saturer un programme. Encore une mémoire tampon fixe (fgets()) est trop restrictif. J'ai joué avec m'allocating un énorme tampon pour fgets() et en s'appuyant sur *retard de répartition de l' de contrôler les choses.
    • Correct vous êtes. Comme un pinson, j'ai utilisé getline à la mémoire tampon d'un 868789 chaîne de caractères -- pas de plaintes. Je vais creuser dans votre retard allocation lien et voir ce que les perles/les risques sont là. Il n'y a tout simplement pas de remplacement pour la souplesse getline offre. J'ai bricolé avec l'écriture de remplacement à l'aide de fgetc, mais la fin de la réécriture getline plus ou moins. La protection en getline est le fait qu'elle continue de les répartir, à empêcher un dépassement, mais il met du fardeau de la preuve sur vous pour vérifier la taille du résultat de la vraisemblance...
    InformationsquelleAutor David C. Rankin
  4. 4
    #include<stdio.h>

int main()
{
    char str[100];
    scanf("%[^\n]s",str);
    printf("%s",str);
    return 0;
}

    d'entrée: lire la chaîne

    sortie: imprimer la chaîne

    Ce code affiche la chaîne de caractères avec des lacunes, comme indiqué ci-dessus.

    InformationsquelleAutor Ananth Reddy
  5. 1

    Vous avez besoin d'avoir le pointeur quelque part à l'utiliser.

    Essayez ce code:

    char word[64];
scanf("%s", word);

    Cela crée un tableau de caractères de longueur 64 et le lit à l'entrée à elle. Notez que si l'entrée est de plus de 64 octets du mot tableau déborde et que votre programme n'est fiable.

    Comme Jens a fait remarquer, il serait préférable de ne pas utiliser scanf pour lire des chaînes de caractères. Ce serait à l'abri de la solution.

    char word[64]
fgets(word, 63, stdin);
word[63] = 0;
    • C'est pourquoi vous ne devez pas utiliser scanf en premier lieu de lire une chaîne de caractères; utilisation fgets à la place qui prend un argument de longueur.
    • Vous avez raison et je devrais avoir honte de ne pas fournir la meilleure solution au premier abord. J'ai édité ma réponse au cas où quelqu'un vient de le lire.
    • fgets déjà de zéro met fin à la chaîne d'entrée. Aussi, contrairement à *scanf, elle ne représente que l'utilisation de la pleine taille de la matrice est idiomatiques: fgets(word, sizeof word, stdin)
    • fgets(word, 63, stdin); word[63] = 0; --> fgets(word, sizeof word, stdin);
    InformationsquelleAutor Juho