Печать цифр, присутствующих в строке

Печать цифр, присутствующих в строке.

Предположим, у меня есть такой ввод

Input: {1,32,33,41,59}

Результат должен выглядеть так

Output: 1 32 33 41 59

Мой код

#include<stdio.h>
#include<string.h>
#include<stdlib.h>
int main()
{
char input[200],words[10][10];
int length=0,i=0,j=0,k=0,t;

fgets(input,200,stdin);


//counting the length of str
while(input[length] != '\0')
{
    length++;
}

for(i=1;i<=length;i++)
{
   if (input[i] == ',' || input[i] == '}')
        {
            words[j][k] = '\0';
            j++;
            k=0;
        }
    else
        {
            words[j][k] = input[i];
            k++;
        }
}
int temp[j];
for(i=0;i<j-1;i++)
{
    temp[i] = atoi(words[i]);
    printf("%d\n",temp[i]);
}

return 0;
}

Мой код просто печатает первую цифру в строке. Я не могу понять почему.

Индексы начинаются с 0.

Vlad from Moscow 14.04.2018 17:18

Я начал с 1, чтобы исключить "{"

Syed Ali 14.04.2018 17:19
i<j-1 должен быть i<j во время печати.
Achal 14.04.2018 17:22

"Мой код просто печатает первую цифру в строке." Выражение Yout неверно. Он просто не печатает номер последний ввода.

Paul Ogilvie 14.04.2018 17:29
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
1
4
257
4
Перейти к ответу Данный вопрос помечен как решенный

Ответы 4

Используйте j вместо j-1

#include<stdio.h>
    #include<string.h>
    #include<stdlib.h>
    int main()
    {
    char input[200],words[10][10];
    int length=0,i=0,j=0,k=0,t;

    fgets(input,200,stdin);


    //counting the length of str
    while(input[length] != '\0')
    {
        length++;
    }

    for(i=1;i<=length;i++)
    {
       if (input[i] == ',' || input[i] == '}')
            {
                words[j][k] = '\0';
                j++;
                k=0;
            }
        else
            {
                words[j][k] = input[i];
                k++;
            }
    }
    int temp[j];
    for(i=0;i<j;i++)
    {
        temp[i] = atoi(words[i]);
        printf("%d\n",temp[i]);
    }

    return 0;
    }

Я внес несколько изменений в ваш код и считаю, что он работает так, как вы хотите. Я прокомментировал изменения, смотрите ниже.

 #include<stdio.h>
 #include<string.h>
 #include<stdlib.h>
 int main()
 {
     char input[200],words[10][10];
     int length=0,i=0,j=0,k=0,t;

     fgets(input,200,stdin);


     while(input[length] != '\0')
     {   
         length++;
     }   

     for(i=1;i<=length;i++)
     {   
         if (input[i] == ',' || input[i] == '}')
         {
             words[j][k] = '\0';
             j++;
             k=0;
         }
         else
         {
             words[j][k] = input[i];
             k++;
         }
     }   
     int temp[j];
     //Iterate through all elements in the array
     //0 ---> j-1 is j elements 
     for(i=0;i < j ;i++)
     {   
         temp[i] = atoi(words[i]);
         //print on the same line
         printf("%d ",temp[i]);
     }   
     //newline here
     printf("\n");

     return 0;
 }
Ответ принят как подходящий

Ваш подход слишком сложен и поэтому содержит ошибки.

Нет необходимости определять символьный двумерный массив для вывода сохраненных в виде строки чисел. Достаточно с самого начала использовать стандартную функцию, например, strtoull.

Вот, пожалуйста.

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

int main(void) 
{
    enum { N = 200 };
    char input[N];

    while ( 1 )
    {
        printf( "Input: " );

        if ( !fgets( input, sizeof( input ), stdin ) || input[0] == '\n') break;

        for ( char *p = input; *p; )
        {
            if ( isdigit( ( unsigned char )*p ) )
            {
                char *endptr;

                unsigned long long int num = strtoull( p, &endptr, 10 );
                printf( "%llu\n", num );

                p = endptr;
            }
            else
            {
                ++p;
            }
        }
    }

    return 0;
}

Вывод программы может выглядеть так:

Input: {1,32,33,41,59}
1
32
33
41
59
Input: 

@SyedAli Например, в SO. :)

Vlad from Moscow 14.04.2018 18:03

и что ? ;) Я не могу тебя достать

Syed Ali 14.04.2018 18:04

@SyedAli SO - это StackOverflow.

Vlad from Moscow 14.04.2018 18:05

Вы издеваетесь;) Есть ли способ связаться с вами как гугл-тусовки? так что я мог задать свои сомнения в с

Syed Ali 14.04.2018 18:08

@SyedAli Здесь вы можете задать свои вопросы. Есть много квалифицированных программистов.

Vlad from Moscow 14.04.2018 18:09

Предлагаю вам это:

#include<stdio.h>
#include<string.h>
#include<stdlib.h>
const char *const s = ",";
int main()
{
    char input[200], *ptri, *ptr;

    fgets(input, 200, stdin);

    ptr = input;
    while (*ptr != '{' && *ptr != '\0')
    ptr++;
    if (*ptr == '\0') {
    return -1;      /* not found */
    }
    ptr++;
    ptri = ptr;
    while (*ptr != '}' && *ptr != '\0')
    ptr++;
    if (*ptr == '\0') {
    return -1;      /* not found */
    }
    *ptr = '\0';

    ptr = strtok(ptri, s);

    while (ptr != NULL) {
    printf("%s ", ptr);
    ptr = strtok(NULL, s);
    }
    printf("\n");

    return 0;
}

Другие вопросы по теме