c - 用不同的编程结构替换 `goto`

Question

我正在尝试用防御性编程来完成这个小程序，但我很难处理这个避免 Loop-Goto 的问题，因为我知道这是糟糕的编程。我尝试过 while 和 do...while 循环，但在一种情况下我没有问题。当我要再做一次时，问题就开始了……而对于第二种情况(“不插入空格或单击输入按钮”)。我试过嵌套 do...while 但这里的结果更复杂。

#include <ctype.h>
#include <stdlib.h>
#include <string.h>

int main()
{
    int i;
    int length;
    char giventext [25];        
    Loop:

    printf("String must have 25 chars lenght:\n");
    gets(giventext);

    length = strlen(giventext);

    if (length > 25) {
        printf("\nString has over %d chars.\nMust give a shorter string\n", length);
        goto Loop;
    }
    /* Here i trying to not give space or nothing*/
    if (length < 1) {
        printf("You dont give anything as a string.\n");
        goto Loop;
    } else {
        printf("Your string has %d\n",length);
        printf("Letter in lower case are: \n");

        for (i = 0; i < length; i++) {
            if (islower(giventext[i])) {                            
                printf("%c",giventext[i]);
            }
        }
    }
    return 0;
}

Answer 1

请注意，您的代码根本不是防御性的。你无法避免缓冲区溢出，因为，

1. 在将字符串输入程序后检查字符串的长度，以便在缓冲区溢出已经发生之后
2. 您使用了 gets()，它不检查输入长度，因此很容易发生缓冲区溢出。

改用 fgets() 并丢弃多余的字符。

我认为您需要了解 strlen() 不计算输入的字符数，而是计算字符串中的字符数。

如果你想确保插入的字符少于N那么

int
readinput(char *const buffer, int maxlen)
{
    int count;
    int next;

    fputc('>', stdout);
    fputc(' ', stdout);

    count = 0;
    while ((next = fgetc(stdin)) && (next != EOF) && (next != '\n')) {
        // We need space for the terminating '\0';
        if (count == maxlen - 1) {
            // Discard extra characters before returning
            // read until EOF or '\n' is found
            while ((next = fgetc(stdin)) && (next != EOF) && (next != '\n'))
                ;
            return -1;
        }
        buffer[count++] = next;
    }
    buffer[count] = '\0';
    return count;
}

int
main(void)
{
    char string[8];
    int result;

    while ((result = readinput(string, (int) sizeof(string))) == -1) {
        fprintf(stderr, "you cannot input more than `%d' characters\n", 
                            (int) sizeof(string) - 1);
    }
    fprintf(stdout, "accepted `%s' (%d)\n", string, result);
}

注意，通过使用一个函数，这个程序的流程控制清晰简单。这正是 goto 不被鼓励的原因，不是因为它是邪恶的东西，而是因为它可能像您一样被滥用。