Mi programa crashea al usar Strtok

[Lunethlion]

Buenas tardes, soy nuevo en el foro así que me disculpo por adelantado debido a cualquier error en la redacción de este, y además soy estudiante novato de C++, de modo que lamento mi ignorancia.

El tema es, en concreto estoy realizando un programa cuyo funcionamiento gira en torno a un prompt al cual se le proporcionan instrucciones que el programa reconocerá y en consecuencia ejecutará una determinada función. Utilizo gets para que se le introduzca por teclado la información necesaria, y strtok para dividirla en tokens que puedan ser procesados. Por ejemplo, para realizar una prueba, si le introduzco el comando "prueba" imprimirá: "Prueba 1 completada"; y si se le proporciona "prueba 2" la respuesta será además de la primera, "Prueba 2 completada".

El problema reside en el caso en el que se le introduzca un solo comando reconocido en una función que pide más, crashea...

Este es mi programa:

Código [Seleccionar] Expandir

#include<iostream>
#include<stdio.h>
#include<stdlib.h>
#include <string.h>

using namespace std;

int i=1, a=1;
char prompt[256]="[*]:>", comando[256];
string token[12];


int main()
{
while(i==1)
   {
    a=1;
   cout << endl << prompt;
   gets(comando);
   token[a]=strtok(comando," ");

   if(token[a]=="prueba")
       {
       cout << "prueba 1 completada.\n";
       a++;
       token[a]=strtok(NULL," ");
       if(token[a]=="2"){cout << "prueba 2 completada.\n";}
       }
   }
}


El caso es, en la función prueba (o mejor dicho, la que en un futuro sería una función) "token[a]=strtok(comando," ")"; divide la cadena "comando" en tokens: y "token[a]=strtok(NULL, " ")" continúa dividiendola por el sitio donde lo dejó el anterior strtok. El problema es que cuando strtok(NULL," ") se ejecuta intentando seguir dividiendo una cadena que no existe (por ejemplo "prueba" la cual empieza y termina de dividirse en el primer strtok) hace que el programa crashee, porque la función prueba para ejecutarse requiere 2 valores: "Prueba" seguido de "2".

Como podría solucionar este problema? Hay algo que me he perdido, hecho mal o usado incorrectamente?

Muchas gracias por vuestra atención.

[AlbertoBSD]

Antes de interactuar con el contenido del token tienes que evaluar que no sea NULL

Código (c) [Seleccionar] Expandir

if(token[a] != NULL){
//Acciones
}

Saludos!

[Lunethlion]

Antes de interactuar con el contenido del token tienes que evaluar que no sea NULL

Código (c) [Seleccionar] Expandir

if(token[a] != NULL){
//Acciones
}

Saludos!

He probado a añadir tu sugerencia, pero ahora el programa no me compila...

Código [Seleccionar] Expandir

if(token[a]=="prueba")
       {
       cout << "prueba 1 completada.\n";
       if(token[a] != NULL)
           {
           a++;
           token[a]=strtok(NULL," ");
           if(token[a]=="2"){cout << "prueba 2 completada.\n";}
           }
       }

concretamente el error es:

E:\Onedrive\Mis Programas\PFunPro.cpp|36|error: no match for 'operator!=' (operand types are 'std::string {aka std::basic_string<char>}' and 'int')|

[ivancea96]

strtok trabaja con cadenas de caracteres (char*), no con string. El retorno, es un char*.

Veo que estás usando cabeceras de C y a la vez, C++. Las cabeceras de C++ son como las de C, pero con una 'c' al principio, y sin el ".h": <math.h> - <cmath>

Si quieres trabajar con string, puedes usar métodos de la clase striong como find() o substr(), para encontrar el índice de un caracter y para obtener una sub-cadena, respectivamente.

[Lunethlion]

strtok trabaja con cadenas de caracteres (char*), no con string. El retorno, es un char*.

Veo que estás usando cabeceras de C y a la vez, C++. Las cabeceras de C++ son como las de C, pero con una 'c' al principio, y sin el ".h": <math.h> - <cmath>

Si quieres trabajar con string, puedes usar métodos de la clase striong como find() o substr(), para encontrar el índice de un caracter y para obtener una sub-cadena, respectivamente.

Gracias por responder. Find me servirá de lujo para una futura función que debo implementar, pero no veo como me puede ayudar concretamente aqui. Además, quisiera aprender el funcionamiento de Strtok para segmentar cadenas de carácteres.

Añado que he probado a cambiar token de string a char*, pero ahora no funciona. Probablemente no he entendido a que te referías.

[AlbertoBSD]

Código (c) [Seleccionar] Expandir

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

int main() {
int i=1, a;
char *prompt="[*]:>";
char comando[256];
char *token[12];
while(i==1) {
printf("%s",prompt);
fgets(comando,256,stdin);
a = 0;
token[a]=strtok(comando," ");
a = 1;
while(token[a-1] != NULL && a < 12){
printf("Token: %s\n",token[a-1]);
token[a]=strtok(NULL," ");
a++;

}
memset(token,0,sizeof(char *)*12);
memset(comando,0,256);
}
}

Cambie un poco el codigo.

Saludos!

[Lunethlion]

Código (c) [Seleccionar] Expandir

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

int main() {
int i=1, a;
char *prompt="[*]:>";
char comando[256];
char *token[12];
while(i==1) {
printf("%s",prompt);
fgets(comando,256,stdin);
a = 0;
token[a]=strtok(comando," ");
a = 1;
while(token[a-1] != NULL && a < 12){
printf("Token: %s\n",token[a-1]);
token[a]=strtok(NULL," ");
a++;

}
memset(token,0,sizeof(char *)*12);
memset(comando,0,256);
}
}

Cambie un poco el codigo.

Saludos!

Siguiendo tu ejemplo, no me deja usar NULL. El compilador salta y dice...

error: no match for 'operator!=' (operand types are 'std::string {aka std::basic_string<char>}' and 'int')|"

he puesto: while(token != NULL){------}

[HardForo]

El codigo de AlbertoBSD compilalo con gcc (u otro compilador de C), lo estas compilando con un compilador de C++

Si usas Dev-C++ o CodeBlocks al menos que la extension del programa sea .c y no .cpp

[engel lex]

Siguiendo tu ejemplo, no me deja usar NULL. El compilador salta y dice...

error: no match for 'operator!=' (operand types are 'std::string {aka std::basic_string<char>}' and 'int')|"

he puesto: while(token != NULL){------}

fijate el código y lo que estás haciendo... tu error es

Código [Seleccionar] Expandir

error: no match for 'operator!=' (operand types are 'std::string {aka std::basic_string<char>}' and 'int')|

es decir estás comparando un string con NULL

el codigo de AlbertoBSD no compara nunca con string...

linea 16

Código [Seleccionar] Expandir

token[a-1] != NULL && a < 12

y si vemos la decharación en la linea 9

Código [Seleccionar] Expandir

char *token[12];

no estás haciendo nada de lo que te dicen, por eso falla... no puedes comprar string con NULL, tienes que pasar a char *, recuerda char *, no string... (y por dejarlo claro) string no se puede comprar con NULL, es char *

luego de char* si puedes pasar a string, pero no antes