Столкнувшись с проблемами с функцией Atoi, если используется повторно - PullRequest
0 голосов
/ 12 декабря 2018

Мне нужно найти частоту цифр {0,1,2,3,4,5,6,7,8,9} в данной строке, я использую функцию atoi дляпреобразовать символ в целое число, и у меня возникли проблемы с функцией atoi , когда входная строка велика (пробовал это с разными тестами различной длины), например,

, есливходная строка -

1v88886l256338ar0ekk

мой код работает правильно, и ответ -

1 1 1 2 0 1 2 0 5 0

, где первая цифра указывает частоту от 0 и т. д. до 9,

но если входная строка равна

9139f793308o0lo66h6vc13lgc697h0f6c32lu84445972k0o0l033od17c083yn5051d6j319hyo8j939n28d913015ns6zx5653x01211x12ch2526o65sg7xw6302141q9203s22l336319ll9yx4b597mr318a7943906750j4u152067nq83ne9f24thu96yd05173l47c803roxci45615f0w53i1sz913jj6za733l73tw6r66mq6p44sfhjr26h8e801z8zlcx2l1e65r2879xj3w3acv216196uq158o663y7oz2i5378v0v5w17762451t424352m23026r9o202i9785382o159e4gu1c8561157z5f1vqs5755465b8u728u956434mv944885li456628a994u7j5278m269n1pk8e46940q834h06il6h447888tr7ig72z10fe09k5g98h9bgt6z40v42s16pt6k3l3v45i83i01b9448g554741w766f2q7v31i085488h060e710p53076c6nm98pi946g8j2n6j8x29qa1ad48172y0u4818121p686bud89741201p54087u56g8scerv9pvhuo09re477zfb224i2c1325bj58jx4bk7b009f6446j5i95474p266i503r670n631x6940gwl71ejbx47imx576129248901765rnpu6l80084t0j1839f5y3409w2n403fu6ogw1170jmb6o5l520vg0703e0

по достижении конца строки, функция atoi возвращает неправильные значения

, например,

myкод использует atoi для преобразования текстового символа в целое число и сохраняет его в int num

; в начале функция работает нормально,

text is 9 num is 9 
text is 1 num is 1 
text is 3 num is 3 
text is 9 num is 9 
text is 7 num is 7 
text is 9 num is 9 
text is 3 num is 3 
text is 3 num is 3 
text is 0 num is 0 
text is 8 num is 8 
text is 0 num is 0 
.
.
.

и при приближении к самому концу строки функция возвращает

.
.
. 
text is 2 num is 2 
text is 4 num is 4 
text is 0 num is 0 
text is 3 num is 30 
text is 6 num is 60 
text is 1 num is 10 
text is 1 num is 10 
text is 7 num is 70 
text is 0 num is 0 
text is 6 num is 61 
text is 5 num is 51 
text is 5 num is 51 
text is 2 num is 21 
text is 0 num is 1
text is 7 num is 71  
text is 0 num is 1 
text is 0 num is 1 
text is 3 num is 31 

Если я заменю int num = atoi(&text) на int num = text - '0' моей программыram отлично работает для всех тестовых случаев,

, поэтому кто-то может сказать мне, что пошло не так и правильно ли я использовал функцию.Пожалуйста, имейте в виду, я просто хочу знать, почему atoi не работает, поэтому я не ищу замены для функции.

Я включил фрагмент моего кода ниже

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

int main() {
    int arr[10] = {0};
    char text;
   text = getchar();
   while(text != EOF)
   {
       if(isdigit(text))
       {
           printf("text is %c ",text);
           int num = atoi(&text);
           printf("num is %d\n ",num);
           for(int i =0; i<10;i++)
           {
               if(num==i)
               {
                   arr[i]++;
                   //printf("arr[%d] is %d\n", i,arr[i]);
                   break;
               }
           }
       }
       text = getchar();
   }
for(int i=0; i<10;i++)
{
    printf("%d ",arr[i]);
}  
    return 0;
}

Заранее спасибо, что нашли время, чтобы прочитать и ответить на мой вопрос

Ответы [ 2 ]

0 голосов
/ 12 декабря 2018
  • вам не нужно atoi(), поскольку вы работаете с одиночными символами, а не со строками
  • getc () возвращает int [0 .. 255 для реальных символов, -1 для EOF]
  • Вы можете избежать многих {} фигурных скобок, используя break и continue

#include <stdio.h>
#include <ctype.h>

int main() {
    int arr[10] = {0};
    int text, num;
   while(1) {
        text = getchar();
        if (text == EOF) break;
        if (!isdigit(text)) continue;
        printf("text is %c ", text);
        num = text - '0'
        printf("num is %d\n ",num);
        arr[num]++;
        //printf("arr[%d] is %d\n", num, arr[num]);
    }
    for(int i=0; i<10;i++)
    {
        printf("%d ",arr[i]);
    }

    return 0;
}
0 голосов
/ 12 декабря 2018

Per документация atoi() в стандарте C :

Функции atoi, atol и atoll преобразуют начальную часть строки , на которую указываетnptr в int, long int и long long int, соответственно.

Обратите внимание на выделенную жирным шрифтом часть.

Учитывая

char text;

этот кодвызывает неопределенное поведение, потому что адрес, переданный atoi(), не является адресом строки:

int num = atoi(&text);

Одно исправление будет:

char text[2];
text[1] = '\0';

// getchar() returns int, not char, in order
// to handle EOF properly
int input = getchar();
while(input != EOF)
{
    text[0] = input;
    if(isdigit(text[0]))
    {
        printf("text is %s ",text);
        int num = atoi(text);

Это гарантирует, что строка (нуль-завершенная серия char) передается в atoi().

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...