victor hsu 1B Ответов: 2

Есть ли у этого кода проблема? Мой учитель сказал, что у меня есть забавная проблема в этом(помогите)

#include <stdio.h>

//function to calculate length of the string using recursion
int stringLength(char *str)
{
    static int length=0;
    if(*str!=NULL)
    {
        length++;
        stringLength(++str);
    }
    else
    {
        return length;
    }
}
int main()
{
    char str[100];
    int length=0;

    printf("Enter a string: ");
    gets(str);

    length=stringLength(str);

    printf("Total number of characters (string length) are: %d\n",length);

    return 0;
}


Что я уже пробовал:

запустить его, но он работает так что я не знаю где это неправильным.

PIEBALDconsult

Я не вижу, как это может сработать; вы уверены, что это так?
О, подождите, я вижу, что длина статична-так что вы обманываете, не делайте этого.

2 Ответов

Рейтинг:
2

PrafullaVedante

Я вижу здесь множество проблем.
1)проблема здесь заключается в статической переменной длине. Он будет инициализирован до нуля только один раз в жизни программы.

Если вы вызовете функцию еще раз, она даст вам неправильный результат.

2) Вы передаете неинициализированный массив str в функцию. Это может привести к зависанию или сбою программы, если она не найдет нулевой символ в последующих ячейках памяти.

Простое решение этой проблемы-найти NULL в заданных пределах.

Здесь str объявляется как
char str[100];

Поэтому мы должны проверить NULL только в 100 ячейках памяти.

Вы должны передать длину входного буфера строки в качестве проверки привязки.

Рейтинг:
1

Patrice T

Попробуйте этот код, чтобы увидеть, что смешного

int main()
{
    char str[100];
    int length=0;
 
    printf("Enter a string: ");
    gets(str);
    length=stringLength(str);
    printf("Total number of characters (string length) are: %d\n",length);
 
    printf("Enter a string: ");
    gets(str);
    length=stringLength(str);
    printf("Total number of characters (string length) are: %d\n",length);
 
    return 0;
}

-----
Этот код даже не должен компилироваться.
Цитата:
запустить его, но он работает так что я не знаю где это неправильным.

Что вы подразумеваете под "работой"? если он не разобьется, хорошо. если он дает правильный ответ, это еще более удивительно.
stringLength не хватает половины логики. Что вы делаете с результатом, когда возвращаетесь из рекурсии?
Используйте отладчик, чтобы точно видеть, что делает ваш код, обратите внимание на переменные.

Существует инструмент, который позволяет вам видеть, что делает ваш код, его имя отладчик Это также отличный инструмент обучения, потому что он показывает вам реальность, и вы можете увидеть, какие ожидания соответствуют реальности.
Когда вы не понимаете, что делает ваш код или почему он делает то, что он делает, ответ таков: отладчик.
Используйте отладчик, чтобы увидеть, что делает ваш код. Просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не находит ошибок, он просто помогает вам. Когда код не делает того, что ожидается, вы близки к ошибке.