首页 > 解决方案 > 为什么我的 strlen 似乎在这个 for 循环中返回了两个值?

问题描述

基本上,我正在尝试使用我对 C 的(基本)知识创建 d&d 骰子滚轮。在for应该循环输入并返回骰子中边数的 -loop 中,strlen()似乎返回两个单独的值:1。正确的值,2.一个非常大的数字。我不知道为什么它会返回第二个数字,这会弄乱for-loop。

我尝试分配strlen()给另一个名为 length 的变量,并在我使用的函数中调用它strlen()。每次我strlen()在输入dice_inp存在的函数中打印时,它都会打印正确的值,但是一旦我将它传递给它,sides()它就会返回非常大的数字。

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

int num();
int sides();

int main()
{
    printf("This is a dice rolling simulator for D&D!\n");
    int num_dice, num_sides;
    num_dice = num();
    num_sides = sides();
    printf("num %d sides %d", num_dice, num_sides);
}

int num()
{
    int i, keynum, dice_num = 0, length;
    char dice_inp[20];

    printf("Input the number of dice(n), then d,then the number of sides of each dice(s) so the input appears n the form 'nds'!\n");
    scanf("%s", dice_inp);
    length = abs(strlen(dice_inp));

    keynum = strcspn(dice_inp, "d");

    for (i = 0; i < keynum; i++)
    {
        char c = dice_inp[keynum - 1 - i];
        c = c - '0';
        c = round(c*pow(10, i));
        dice_num += c;
    }

    if (dice_num == 1)
    {
        printf("There is %d dice being rolled.\n", dice_num);
        Sleep(500);
    }

    else if (dice_num == 0)
    {
        printf("You cannot roll 0 dice.\n");
        Sleep(500);
        num();
    }

    else if (dice_num > 1)
    {
        printf("There are %d dice being rolled.\n", dice_num);
        Sleep(500);
    }

    Sleep(500);
    sides(length, dice_inp, keynum);
    return dice_num;
}

int sides(int length, char dice_inp[], int keynum)
{
    int i, dice_sides = 0;

    for (i = 0; i < abs((length - keynum - 1)); i++)
    {
        char c = dice_inp[strlen(dice_inp) - 1 - i];
        c = c - '0';
        c = round(c*pow(10, i));
        dice_sides += c;
        printf("a");
    }

    if (dice_sides == 1)
    {
        printf("A dice cannot have one side. Please try again!\n");
        Sleep(500);
        num();
    }

    else if (dice_sides == 0)
    {
        printf("A dice cannot have 0 sides. Please try again.\n");
        Sleep(500);
        num();
    }
    else if (dice_sides > 1)
    {
        printf("The dice will have %d sides.\n", dice_sides);
        Sleep(500);
    }

    printf("%d", dice_sides);
}

标签: cfor-loopstrlen

解决方案


你的代码有很多问题。首先,函数“sides”的定义如下:

int sides(int length, char dice_inp[], int keynum);

但是你不带参数调用它

num_sides = sides();

当你得到一个无效的输入时,在函数“num”中,你再次递归调用它,这会导致堆栈的浪费。最好使用循环。在同一个函数中,当您递归调用它时,您会丢失返回值。这意味着最后你得到了错误的值。该功能中的“睡眠”功能对我来说毫无意义。

我猜测 strlen 的问题与输入字符串不包含终止符有关。尝试使用 printf 在 strlen 之前跟踪输入。

我在这里描述的问题并不是唯一的。


推荐阅读