Cast char zu char-Arrays oder char-array char?

strcat behandelt das argument als ein Zeiger auf eine null-terminierte Zeichenfolge. Casting eine char zu einem char * ist einfach gefährlich, und ich kann mir keinen Grund, es wäre schon mal hilfreich (nicht zu unterstellen, dass Sie dumm sind es zu versuchen - jeder macht dumme Fehler, wenn Sie lernen. Das ist, warum wir hier sind.)

Der Grund ist, dass es interpretieren würde die single-byte -char, plus die zusätzlichen sizeof(char*) - sizeof(char) (Regel 3) bytes Umgebung, die char, als einen Zeiger, der auf das um... überall. Sie haben keine Möglichkeit zu wissen, wo es Punkte, da 3 dieser bytes sind außerhalb Ihrer Kontrolle, und somit keine Möglichkeit zu wissen, ob es Punkte auf gültige Daten.

Dass Sie vielleicht nehmen Sie dies als einen zweiten Ansatz:

strcat(inp, &c);

Dieser Zeit, Sie würden besser sein, da &c ist ein Ausdruck vom Typ char * und keine casts notwendig sind. Aber wieder strcat davon ausgegangen, dass es argument ist einen nul-terminierten string, und da Sie keine Möglichkeit haben, garantiert ein nul-byte nach Ihrem char Daten, die dies nicht tun.

Der beste Weg ist dieser:

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

UPDATE:

Eigentlich habe ich gelogen. Die besten Möglichkeit ist die Verwendung der standard-Bibliothek-Funktion fgets, das scheint zu tun, mehr oder weniger, was Sie zu tun versuchen. Wahrhaftig, ich vergaß es, aber wenn dies ist eine Hausaufgabe Ihr professor kann nicht wollen, dass Sie mit fgets so dass Sie lernen können, wie man es manuell zu tun. Allerdings, wenn dies nicht Hausaufgaben, fgets macht genau das, was du bist suchen für. (Eigentlich der Dritte Ansatz ist wohl auf dem Weg zu reimplementing fgets oder fgets-ähnliche Funktionalität.)

Ich würde auch hinzufügen, einige andere Kommentar auf Ihrem input Funktion:

1. char* inp = ""; auf nur-lese-Daten. Ein Fehler in der C-standard (für rückwärts-Kompatibilität) erlaubt eine string-Literale zugeordnet werden char* Arten statt const char * Arten wie es (IMHO) sein sollte.

   Es gibt mehrere Möglichkeiten, dies zu nähern, aber das beste ist die dynamische Zuordnung. Verwenden Sie die malloc Funktion zu behalten einige der Daten, verfolgen Sie, wie viel Sie verwendet haben, in Ihrer Funktion, und verwenden Sie die realloc Funktion, wenn Sie am Ende brauchen mehr Raum, um es zu speichern. Wenn Sie dabei Hilfe benötigen, ich bin sicher, du wirst wieder hier sein (hoffentlich nicht zu bald) mit einer anderen Frage. 🙂

2. getchar() gibt ein int, weil EOF definiert werden, die außerhalb der normalen Bandbreite eines char. Um zu unterscheiden zwischen char und EOF, ist es am besten zu machen c eine int. Ansonsten ein absolut gültigen Charakter kann das signal EOF. Werden Sie sicher, dass cast c unten, um eine char wenn Sie es hinzufügen, um die Zeichenkette, obwohl.

InformationsquelleAutor Chris Lutz

Wenn Sie möchten, verketten Sie eine einzelne char auf eine Zeichenfolge, die Sie verwenden können strncat() und geben Sie die Anzahl der Zeichen zum verketten, als 1. Aber beachten Sie, dass:

• Der Ziel-Puffer muss genügend Platz haben; und
• In C89 muss eine tatsächliche char Objekt die Adresse (also können Sie nicht direkt verketten, das Ergebnis getchar().

Beispiel:

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

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

In C99 können Sie eine compound literal und die zweite Einschränkung gilt nicht für:

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

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

InformationsquelleAutor caf

First off, sind Sie wieder ein Zeiger auf ein stack-array zugewiesen. Dies ist undefiniert bahavior. Zweitens, Sie versucht, zu schreiben chars auf nicht zugeordneten Speicher. Drittens, die Art der inp ist const char* nicht char*.

Du willst so etwas wie:

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;
}

InformationsquelleAutor KitsuneYMG

Eine einfache Möglichkeit ist die Verwendung von snprintf:

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

Es ist eine einfache Lösung, aber erfordert, dass Sie in etwa weiß wie groß der string sein wird, vor der Zeit.

Hinweise: I verwenden Sie die +1-um mich daran zu erinnern, dass es muss Raum sein für die null-Terminierung, und snprintf ist besser als sprintf, wie es gegeben ist, die Größe des Puffers zu verhindern, Pufferüberlauf.

InformationsquelleAutor Daniel