只允许八进制数

Question

我的代码所做的是：

• 用户选择从十进制到八进制或八进制到十进制的转换。
• 用户输入一个数字
• 用户获取转换结果

现在我只想在从八进制转换为十进制时允许 otcal 数字，如果数字不是八进制，则会发出错误消息。 如何实现？

#include <stdio.h>
#include <math.h>

int UmrechnungDezimalZuOktal(int dezimalzahl), UmrechnungOktalZuDezimal(int oktalzahl);
int main()
{
    char auswahl;
    int dezimalzahl, oktalzahl, nichtBeendet = 1, c;
    

    while (nichtBeendet) {
            printf("\n\n  ##################  AUSWAHL  ##################\n"
                "  #                                             #\n"
                "  #     1) Konvertierung Dezimal -> Oktal       #\n"
                "  #     2) Konvertierung Oktal -> Dezimal       #\n"
                "  #     3) Abbruch                              #\n"
                "  #                                             #\n"
                "  ###############################################\n\n");

            printf("Auswahl: ");
            scanf("%c", &auswahl);

            switch (auswahl) {
            case '1': {                                                                                      /* Konvertierung Dezimal -> Oktal */
                printf("Geben Sie eine positive, ganze Dezimalzahl ein: ");
                scanf("%d", &dezimalzahl);

                if (dezimalzahl >= 0)
                {
                printf("Die Dezimalzahl %d entspricht der Oktalzahl %d", dezimalzahl, UmrechnungDezimalZuOktal(dezimalzahl));
                }
                else
                {
                printf("Bitte nur positive Zahlen eingeben!");
                }
                break;
            }
            case '2': {                                                                                      /* Konvertierung Oktal-> Dezimal */
                printf("Geben Sie eine positive, ganze Oktalzahl ein: ");
                scanf("%d", &oktalzahl);

                if (oktalzahl >= 0)
                {
                printf("Die Oktalzahl %d entspricht der Dezimalzahl %d", oktalzahl, UmrechnungOktalZuDezimal(oktalzahl));
                 }
                else
                {
                    printf("Bitte nur positive Zahlen eingeben!");
                }
                break;

            }
            case '3': {                                                                                      /* Abbruch */
                nichtBeendet = 0;
                break;
            }
            default: {                                                                                       /* Ungültige Eingabe*/
                printf("Ungueltige Eingabe!");
                break;
            }
            }
            while ((c = getchar()) != '\n' && c != EOF) {};
        }
}

int UmrechnungDezimalZuOktal(int dezimalzahl)                                                               /* Rechnung Dezimal -> Oktal */                                                         
{
    int oktalzahl = 0, i = 1;

    while (dezimalzahl != 0)
    {
        oktalzahl += (dezimalzahl % 8) * i;
        dezimalzahl /= 8;
        i *= 10;
    }

    return oktalzahl;
} 

int UmrechnungOktalZuDezimal(int oktalzahl)                                                                 /* Rechnung Oktal -> Dezimal*/
{
    int dezimalzahl = 0, i = 0;

    while (oktalzahl != 0)
    {
        dezimalzahl += (oktalzahl % 10) * pow(8, i);
        ++i;
        oktalzahl /= 10;
    }

    i = 1;

    return dezimalzahl;
}

Answer 1

我会用fgets 和strtol 替换所有scanf 和硬编码函数：

enum {oct = 8, dec = 10, hex = 16};

int to_number(int base, bool *match)
{
    char str[32];
    long num = 0;

    *match = false;
    if (fgets(str, sizeof str, stdin))
    {
        char *ptr;

        num = strtol(str, &ptr, base);
        *match = (*ptr == '\n'); // match is true if strtol stops scanning
                                 // at the newline left by fgets
    }
    return (int)num;
}

...

bool match;
int num = to_number(oct, &match);

printf("%d %s an octal\n", num, match ? "is" : "is not");

编辑：

正如 @klutt 在 cmets 中指出的那样，由于 strtol 返回一个 long，理想情况下，该函数还应该返回 long 以避免强制转换，那么，如果你真的需要一个 int，你可以强制转换函数的返回：

long to_number(int base, bool *match);

int num = (int)to_number(oct, &match);