C - Begrenzen Sie die Länge der Zeichenfolge

(Sorry für mein schlechtes Englisch !)

Schrieb ich ein Programm, das Sie auffordert, geben Sie ein Kennwort nicht mehr als eine bestimmte Zahl, acht Zeichen in diesem Fall. Die Zeichen, dass die Grenze, werden ausgeschnitten aus dem array:

#include <stdio.h>
#define MAXCHAR 8

main()
{
    char password[MAXCHAR];
    int i;
    char c;

    printf("Insert password: MAX 8 CHARS!\n\n");
    for(i = 0; i <= MAXCHAR; i++){
        c = getchar();

        if(i == MAXCHAR){
            break;
        }
        else{
            password[i] = c;
        }
    }

    printf("%s\n", password);
}

Damit das Programm funktioniert, ABER es ist ein "seltsames" problem. Wenn die Grenze IST ACHT, und ich geben Sie ein Kennwort mit mehr als acht Zeichen
(Beispiel: P455w0rds98)
die Ausgabe wird wie folgt Aussehen:

P455w0rd☺

So legt ein smiley am Ende und ich weiß nicht, warum. Es passiert nur, wenn a der Grenzwert ist auf acht.

  • Leichte Korrektur (keine straftat): meinst Du ", fragt Sie auf Typ ein Passwort" ("digit" hat eine andere Bedeutung und ist ziemlich irritierend hier).
  • Lesen Sie mehr über getchar.
  • Olaf, wenn Sie erwarten, dass eine präzise Terminologie angezeigt werden, auf diesem forum, Sie sind sich für die Einstellung Enttäuschung. Ich habe selten sehen auch die "Experten" verwenden Sie korrekte Terminologie konsistent, nachdenklich.
  • Warum würden Sie so etwas sagen ? "...computer-Programmierung möglicherweise nicht für Sie." Verdammt, Sie zerstört mich, ich war nur ein rookie, wenn einer dies geschrieben. Was Sie sagte, ist nicht gerade ermutigend an alle.
InformationsquelleAutor Claudio Pisa | 2016-08-16
  1. 3

    Müssen Sie die Länge zu drucken oder die Zeichenkette beenden. Andernfalls werden Sie aufrufen Undefiniertes Verhalten. Versuchen Sie dieses, in dem die letztere Methode ist implementiert.

    #include <stdio.h>
#define MAXCHAR 8

int main(void)
{
    char password[MAXCHAR + 1]; /* allocate one more element for terminating null-character */
    int i;
    char c;

    printf("Insert password: MAX 8 CHARS!\n\n");
    for(i = 0; i <= MAXCHAR; i++){
        c = getchar();

        if(i == MAXCHAR){
            break;
        }
        else{
            password[i] = c;
        }
    }
    password[MAXCHAR] = '\0'; /* terminate the string */

    printf("%s\n", password);
}

    Einige Leute sagen, dass die if(i == MAXCHAR){ break; } Teil nicht gut Aussehen, so ist hier ein anderes code-Beispiel:

    #include <stdio.h>
#define MAXCHAR 8

int main(void)
{
    char password[MAXCHAR + 1]; /* allocate one more element for terminating null-character */
    int i;

    printf("Insert password: MAX 8 CHARS!\n\n");
    /* read exactly 8 characters. To improve, breaking on seeing newline or EOF may be good */
    for(i = 0; i < MAXCHAR; i++){
        password[i] = getchar();
    }
    password[MAXCHAR] = '\0'; /* terminate the string */
    getchar(); /* to match number of call of getchar() to the original: maybe for consuming newline character after 8-digit password */

    printf("%s\n", password);
}
    • for(i = 0; i <= MAXCHAR; i++)..sind Sie sicher? ist es nicht, wie die 9 Elemente? Sie können loszuwerden, die ganze break; Dingens....
    • es ist ein hack, der innerhalb der Schleife zu umgehen 🙂 Update: Ja, Sie haben es gesehen. Ein klassisches Beispiel man einen patch ohne Verständnis der Ursache.
    • 9 Zeichen gelesen werden aber nur 8 Zeichen + null-Zeichen gespeichert werden. Es kann komisch sein-Spezifikation, aber ich glaube nicht, dass es ungültig ist.
    • Und es wird die Anwender nicht irreführen, denken, sein Passwort ist 9 Zeichen.
    • Es können 8 Zeichen + Zeilenumbruch, aber das Passwort mit 7 Zeichen oder weniger nicht erlaubt ist hier... Was ist "MAX" bedeuten?
    • Basiert auf Ihr code 8+newline. Sowieso, diese ganze if-break Sache ist Unsinn.
    • Ich habe zwei Frage nun: 1) Warum "if(i == MAXCHAR){ break; }" nicht gut Aussehen ? 2)Ok, aber warum die verdammte lächeln wird nur angezeigt, wenn der Grenzwert acht ? o.O Und warum erscheint Sie immer ?
    • 1) Denken Sie wieder, ich glaube nicht, dass der code ist so schlecht, weil, es verwendend, können Sie Zusammenführen von zwei anrufen von getchar() in einem. 2) Durch Zufall wenn man sich auf Undefiniertes Verhalten.
    • Warum denkst du, das ganze if-break Sache ist Unsinn.?

    InformationsquelleAutor MikeCAT
  2. 0

    Alle C-style-strings haben ein terminal \0 Zeichen (Wert 0). Dies ist eindeutig von einem anderen Charakter einen Wert, so kann es verwendet werden, um zu signalisieren, das Ende der Zeichenfolge. Das smiley-Gesicht Sie beobachten, ist nur ein Teil eines benachbarten Speicherblocks, der zufällig ein null-Zeichen nach dem ersten byte (damit es nur ein zusätzliches Zeichen). Die printf Funktion liest bytes aus der Zeichenkette gegeben, bis er sieht die \0. Um Ihr problem zu lösen, können Sie entweder schreiben

    password[MAXCHAR] = '\0';

    (Sie müssen reservieren, ein zusätzliches byte im array, für die \0).

    Oder Sie können die zero-out-array aus der get-go:

    char password[MAXCHAR + 1] = { };

    Oder mit memset:

    memset(password, '\0', sizeof password);
    InformationsquelleAutor Edward Karak
  3. 0

    Abgesehen von der Antwort, die Sie bereits erhalten von MikeCAT ein alternativer Ansatz wäre die Verwendung von fgets() zu Lesen die Eingabe des Benutzers.

    In diesem Fall , brauchen Sie nicht zu halten die Anzahl der einzelnen Zeichen eingegeben haben, können Sie geben Sie die max Größe und bekommen mit ihm getan. So etwas wie

     fgets(password, MAXCHAR, stdin);

    können den job zu erledigen für Sie, minus die Schleife und Zuordnung für jedes element.

    Einer Sache zu erinnern, ist jedoch für kürzere Eingaben als die gegebene Länge, fgets() liest und speichert die trailing newline auch, Sie benötigen, um loszuwerden, das manuell. Lesen Sie den verlinkten man-Seite für weitere Ideen.

    Sagte, main() ist ein sehr bad und fast nicht-standard - für gehostete Umgebungen. Sollten Sie verwenden int main(void) zumindest zu den standards.

    • fgetsder MAXCHAR Zeichen tatsächlich liest MAXCHAR - 1 Zeichen und verwendet die Letzte für das string-Ende. Es wird auch aufhören zu Lesen, nachdem die MAXCHAR-1TEN Zeichen gesehen wurde, unabhängig davon, ob es ein Zeilenumbruch ist oder nicht. Also mit fgets Sie vielleicht in das problem der Annahme von maximal 6 chars+newline oder 7 chars (mit einem Zeilenumbruch, die noch im stream), das macht es noch schwieriger, Sie zu schlagen, die genaue maximale Anzahl der Zeichen.
    • und warum denkst du das? 1) MAXCHAR muss angepasst werden, es ist nicht ein fester Wert. 2) Sie haben zu prüfen, und entfernen Sie die nachgestellte newline, falls vorhanden, dann überprüfen Sie die Länge der Zeichenfolge, worum geht es? 3) with a newline still remaining in the stream das ist eigentlich nicht viel zur Sache hier, ist es nicht?
    • Die Hauptsache ist: fgets wird nicht warten, für die Bestätigung des Passworts durch den Benutzer sobald er geliefert hat, die die maximale Menge von Zeichen - Das ist zumindest irritierend, kann aber dazu führen, dass alle möglichen Fehler. Funktioniert, aber ich würde nie verwenden, die in der Produktion von code.
    InformationsquelleAutor Sourav Ghosh
Schreibe einen Kommentar