Fonte de char à char tableau ou tableau de char en char?

strcat traite son argument est un pointeur vers un nul de terminaison de chaîne. Le moulage d'un char à un char * est juste dangereux, et je ne peux imaginer aucune raison de ne jamais être utile (ne signifie pas que vous êtes stupide pour essayer - tout le monde fait des erreurs stupides quand ils apprennent. C'est pourquoi nous sommes ici.)

La raison en est qu'elle interpréterait le seul octet char, ainsi que les extra sizeof(char*) - sizeof(char) (généralement 3) octets environnantes qui char, comme un pointeur, ce qui serait le point de... n'importe où. Vous n'avez aucun moyen de savoir d'où il points de, depuis 3 de ces octets sont hors de votre contrôle, et donc aucun moyen de savoir si c'points de données valides.

Vous pouvez prendre cela comme une deuxième approche:

strcat(inp, &c);

Ce temps, vous feriez mieux, depuis &c est une expression de type char * et pas de castes sont nécessaires. Mais encore une fois, strcat suppose que c'est l'argument est nul de terminaison de chaîne, et puisque vous n'avez aucun moyen de garantir un octet nul après votre char de données, ce n'est pas cela.

La meilleure façon de le faire:

size_t len = strlen(inp); //this may already be calculated somewhere else
...
inp[len++] = c; //add c as the last character, and adjust our len
inp[len] = '
size_t len = strlen(inp); //this may already be calculated somewhere else
...
inp[len++] = c; //add c as the last character, and adjust our len
inp[len] = '\0'; //add a new nul terminator to our string
'; //add a new nul terminator to our string

Mise à JOUR:

En fait, j'ai menti. Le meilleur façon est d'utiliser la bibliothèque standard de la fonction fgets, qui semble faire plus ou moins ce que vous essayez de faire. Honnêtement je l'ai oublié, mais si c'est un devoir à la maison de votre professeur ne peut pas vous voulez à l'aide de fgets de sorte que vous pouvez apprendre comment le faire manuellement. Toutefois, si ce n'est pas de devoirs, fgets fait exactement ce que vous cherchez. (En fait, la troisième approche est bien sur son chemin à réimplanter fgets ou fgets-comme fonctionnalité.)

Je voudrais également ajouter d'autres commentaires sur votre input fonction:

1. char* inp = ""; aura point de données en lecture seule. Une faille dans le C standard (pour la rétro compatibilité) permet de littéraux de chaîne pour être affecté à char* plutôt les types de const char * types comme ça (à mon humble avis) devraient l'être.

   Il y a plusieurs façons d'aborder cette, mais le mieux est de l'allocation dynamique. Utiliser le malloc fonction de réserve de certaines données, de garder trace de combien vous avez utilisé dans votre fonction et l'utilisation du realloc fonction si vous avez besoins de plus d'espace pour les stocker. Si vous avez besoin d'aide avec ça, je suis sûr que vous serez de retour ici (j'espère ne pas trop bientôt) avec une autre question. 🙂

2. getchar() retourne un int, parce que EOF est définie à l'extérieur de l'intervalle normal d'une char. Afin de bien distinguer entre les char et EOF, il est préférable de faire c un int. Sinon un bon caractère pourrait être le signal d' EOF. Assurez-vous de jeter c vers le bas pour un char lorsque vous l'ajoutez à la chaîne de caractères.

OriginalL'auteur Chris Lutz

Si vous voulez concaténer un seul char sur une chaîne, vous pouvez utiliser strncat() et spécifier le nombre de caractères à concaténer comme 1. Mais notez que:

• Le tampon de destination doit avoir suffisamment d'espace; et
• En C89, vous devez disposer d'une réelle char objet de prendre l'adresse du (de sorte que vous ne pouvez pas directement concaténer le résultat de getchar().

Par exemple:

char str[5] = "foo";
char c;
int ch;

if ((ch = getchar()) != EOF) {
    c = ch;
    strncat(str, &c, 1);
}

En C99, vous pouvez utiliser un littéral composé et la deuxième restriction ne s'applique pas:

char str[5] = "foo";
int ch;

if ((ch = getchar()) != EOF) {
    strncat(str, &(char){ ch }, 1);
}

OriginalL'auteur caf

Tout d'abord, vous êtes en retournant un pointeur vers une pile allouée tableau. Ce comportement est indéfini. Ensuite, vous essayez d'écrire des caractères dans la mémoire non allouée. Troisièmement, le type de inp est const char* pas char*.

Vous voulez quelque chose comme:

char* input(char *dest) {
  strcpy(dest,"");
  while(1){
  char c=getchar();
  if(c){
   if(c=='\n'||c==EOF){
    break;
   }else{
    strcat(inp,&c);
   }
  }
 }
 return dest;
}

OriginalL'auteur KitsuneYMG

Un moyen simple est d'utiliser snprintf:

char newString[50+1];
snprintf(newString, sizeof(newString), "%s%c", "ljsdflusdfg", getchar());

C'est une solution simple, mais exige que vous avez environ connaissons la taille de la chaîne va être à l'avance.

Note: j'utilise le +1 pour me rappeler qu'il doit y avoir de la place pour la fin null, et snprintf est mieux que sprintf que c'est compte tenu de la taille de la mémoire tampon pour empêcher la saturation de la mémoire tampon.

OriginalL'auteur Daniel