0
char linea[] = "15,16777215,-,-"; 

Tengo que convertir el primer y segundo valor en dos variables de esta cadena, una que sea igual a 15 y la otra igual a 16777215.

Este es mi código pero no me guarda el segundo valor.

 #include <iostream>
#include <string.h>
#include <conio.h>
using namespace std;
int main(){
    
  char linea[] = "15,46777215,-,-";
  cout<<linea<<endl;
  
  int i = 0;
  int j = 0;
  char nro[10]; 
  int numero1 = 0;
  int numero2 = 0;

    while(linea[i] != ','){

        nro[j++] = linea[i++];
        nro[j] = '\0';
        numero1 = atoi(nro);
        

         
     }

      j = 0;

    while(linea[i] != ','){

        nro[j++] = linea[i++];
        nro[j] = '\0';
        numero2 = atoi(nro);
        

     } 
     
     cout<<numero1<<endl;
     cout<<numero2<<endl;


    return 0;
 }

Pero solo me esta guardando el primer valor.

COMPILACION

16
  • Al hacer linea[i++] estas incrementando el valor de i y más abajo nuevamente incrementar su valor i++ por lo que al comparar linea[i]!=',' dará false ya que i vale 2
    – Christian
    Commented el 15 abr. 2022 a las 15:00
  • Estas incrementando i 2 veces. Tendria que ser solo linea[i]. Commented el 15 abr. 2022 a las 15:06
  • @Pablochaches osea que quito el i++? Commented el 15 abr. 2022 a las 15:29
  • Estás en C++. Considera usar strings de C++ a menos que usar strings de C sea una necesidad. También cuenta con conversiones: en.cppreference.com/w/cpp/string/basic_string/stol
    – Mateo
    Commented el 15 abr. 2022 a las 16:08
  • O quitas el i++ al final de loop, o cambias linea[i++] por linea[i]. Commented el 15 abr. 2022 a las 16:40

1 respuesta 1

0

Después del primer while(), i queda apuntando a la posición de la coma que rompe ese bucle, al revisarlo el segundo while, sigue siendo una coma lo que encuentra, por lo que no se ejecuta su contenido.

Agrega i++; despues de j=0; justo antes del segundo while()

Et voilá

3
  • Si la idea es analizar toda la cadena, tu código puede mejorar mucho. Un solo while() para revisar toda la cadena, dentro un if() para validar si encontró una coma, mientras tanto, guardar todo lo encontrado en el arreglo nro[], si hay coma, agregar el terminador \0, y guardar la conversión de nro[] a int en un arreglo int de Numeros con indices j.
    – DEVX75
    Commented el 16 abr. 2022 a las 4:21
  • gracias,hay alguna otra forma de hacerlo integrando el i++; en el algoritmo para que no tenga que quedar por fuera del bucle,osea que automaticamente me quede despues de la coma Commented el 16 abr. 2022 a las 16:00
  • Podrías adelantar ++i en el segundo while j = 0; while(linea[++i] != ','){ nro[j++] = linea[i]; nro[j] = '\0'; numero2 = atoi(nro); } PERO.... ojo con esto, no estás validando el largo de la cadena y solo incrementas i, en algún momento, llegarás al final de la cadena, si no encuentra otra coma antes del final.
    – DEVX75
    Commented el 18 abr. 2022 a las 22:28

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.