¡Buenas!

XD, ahora te explico el codigo, que veo que te faltan algunos conocimientos.

1- primera = strtok(cadena," ");

Si lees el enlace a la referencia sobre strtok que te han dejado, veras que esta llamada ignora los primeros caracteres que se encuentran en los delimitadores y empieza a contar el primer token (no se si esta palabra tiene traducciona al castellano...) desde el primer caracter que no este entre los delimitadores hasta el primer caracter que encuentre en los delimitadores. En este punto marca el delimitador encontrado con el caracter '\0', y devuelve un puntero al primer token (las siguientes llamadas funcionan igual, pero con el primer parametro NULL).

Asi si tenemos la cadena {'a','b',' ','c','\0'}, despues de esta llamada cadena quedaria:  {'a','b','\0','c','\0'} y primera seria {'a','b','\0'}

Si no encontrase el delimitadore devolveria NULL, y aqui viene la segunda parte:

2- if(primera): Si hemos encontrado un delimitador, primera apuntara a alguna parte de la memoria, y tendra un valor distinto de NULL (o distinto de cero). Las condiciones logicas en C son valores numericos. Cero o NULL significa falso y distinto de cero verdadero. Por lo tanto estamos diciendo que ejecute el codigo del if si primera es distinto de NULL (cero).

3- En esta parte he tenido un error. Ahora lo corregimos.

resto = &cadena[strlen(primera) + 1]

Tendria que ser:

resto = &cadena[strlen(cadena) + 1];

¿Porque?

Strlen nos devuelve la longitud en caracteres de una cadena. Siguiendo el ejemplo anterior, como hemos cortado el primer delimitador con un caracter nulo, se tendra que strlen(cadena) = strlen({'a','b','\0','c','\0'}) = strlen({'a','b','\0'}) = 2. cadena[2] = '\0', asi que por eso le sumo 1, para pasar al siguiente caracter despues de donde hemos cortado la cadena original.

Y el porque deberia de ser cadena el parametro de strlen y no primero, es por lo que he dicho en el primer punto: Si una cadena contiene al principio caracteres que esten contenidos entre los delimitadores, strtok los ignora y contara el primer token a partir del primer caracter de la cadena que no este contenido entre los delimitadores. Si utilizasemos primera como referencia para "saltar" esos primeros caracteres y la cadena comenzase con delimitadores, estariamos contando menos caracteres de los necesarios, en cambio utilizando cadena como parametro para strlen, nos aseguramos de que esos primeros caracteres que pueden ser delimitadores tambien se esten contando.

entonces en este caso la expresion

resto = &cadena[strlen(cadena) + 1];

seria lo mismo que resto = &cadena[3];

& nos da la direccion en memoria de una variable, por lo que estamos asignando a resto la posicion en memoria de cadena[3], que sera la cadena restante de quitar la primera parte de cadena: {'c','\0'}

Si algo no te ha quedado claro no dudes en preguntar.

¡Saludos!

PD: El else no sobraba, en la cadena que has puesto tu si que hay espacios, pero una cadema cualquiera puede no tenerlos.

Ademas, tendrias que comprobar que, despues de "quitar" la primera palabra de la cadena, el resto no sea el caracter '\0', o una cadena compuesta por espacios en blanco, tabuladores..., ya que tendrias una cadena, en principio, sin ninguna informacion util...

??????

En lugar de tres asignaciones, una llamada a funcion (que posiblemente acceda mediante un bucle a las posiciones de la cadena para rastrear delimitadores) y una salida forzada de un bucle, estoy realizando una llamada a funcion, un desplazamiento de un puntero (y solo uno), el operador de direccion y una asignacion...

De todas maneras, voy a meter los dos codigos en bucles de 1000 repeticiones (o mas, para que tarde lo suyo) y a ver cual de los dos termina antes... Ahora pongo el codigo que utilizare y los resultados.

MODIFICADO:

El codigo:

#include <stdio.h>
#include <string.h>
#include <time.h>
 
#define ITER 134217728
 
int main ()
{
  char str[] ="Hola c";
  char str2[] = "Hola c"; /* para sobreescribir str en cada iteracion */
  char * pch;
  char * valor1;
  char * valor2;
  int i,inicio;
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        pch = strtok (str," "); // Empiezo a buscar la 1 coincidencia del espacio en blanco
        while (pch != NULL)
        {
            valor1 = (pch); // Aqui se almacena la 1 palabra de la frase separa por el espacio en blanco
            pch = strtok (NULL, ""); // Aqui ya no le pongo el espacio en blanco porque deseo el resto de la frase completa
            valor2 = (pch); // Almaceno el resto de la frase en la variable
            break; // Interrumpo el bucle pues como dije solo queria obtener la 1 palabra con el espacio como delimitador
        }
    }
 
    printf("Metodo 1: %d segundos.\n",time(NULL) - inicio);
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        pch = strtok(str," ");
 
        if(pch)
            valor2 = &str[strlen(str) + 1];
    }
 
    printf("Metodo 2: %d segundos.\n",time(NULL) - inicio);
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        pch = strtok(str," ");
 
        if (pch)
            valor2 = strtok(NULL, "");
    }
 
    printf("Metodo 3: %d segundos.\n",time(NULL) - inicio);
 
    return 0;
}
 

Y los resultados:

Compilado con gcc y ejecutado en un Dual-Core T4300 a 2.1 Ghz

Ejecutandolo varias veces el metodo de User ronda los 18 - 19 segundos y el mio esta entre los 10 y los 11 (en la maquina que he dejado indicada)

¡Saludos!

¡Buenas!

La cuestion es que si un codigo se ejecuta en menos tiempo que otro, significa que el coste (en tiempo) de las operaciones que realiza es menor. Por otra parte hay algo que me extraña y es que luego he hecho la prueba con el codigo de BlackZeroX:

for(i = 0 ; i < ITER ; i++)
{
    strcpy(CADENA, str2);
 
    ptr1 = strtok((char*)CADENA, ESPACIO);
    if (ptr1)
        ptr2 = strtok(NULL, END);
}
 

y me sigue saliendo que le cuesta tanto como al tuyo, por lo que resulta que es mas rapido valor2 = &str[strlen(str) + 1];  que ptr2 = strtok(NULL, END);. Y esto es lo que me resulta extraño, ya que con su codigo solamente se hace una llamada a funcion y una asignacion, por lo que la cuestion tiene que ser que strtok es mas lento que realizar todas las operaciones de &str[strlen(str) + 1]...

Que me lo expliquen... Si alguien de ingenieria inversa le puede hechar un vistazo a los binarios y aclararnoslo...

¡Saludos!

Acabo de dejar el codigo en el que he incluido el metodo de BlackZeroX por si quereis comprobar los tiempos.

¡Saludos!

Corrigiendo: Acabo de comprobar que si en una primera llamada no encuentra los delimitadores, devuelve la cadena completa, y si en alguna de las siguientes llamadas no encuentra delimitadores, tambien devuelve el resto de la cadena. Al parecer los unicos casos en los que strtok devuelve NULL es si se vuelve a llamar a la funcion cuando no se han encontrado delimitadores o si en algun momento solo quedan delimitadores en la cadena.

Pero si, el else estaba de mas, ya que si falla if(ptr1), es porque ptr1 ya es NULL, por lo que asignaerle NULL en el else es redundante.

Esto tampoco es correcto. A parte de que User ha dicho que todavia no ha estudiado la aritmetica de punteros, desplazar cadena stlen(ptr1) posiciones no te asegura pasar a la siguiente posicion despues del primer token (¿lees los post anteriores o solo les echas un vistazo por encima y luego posteas sin saber lo que se ha dicho anteriormente?)

Si cadena = "---hu-ha"; la primera llamada a strtok dejara las variables asi:

ptr1 = strtok(cadena,"-");

ptr1 = "hu";
cadena = "---hu'\0'ha"

Asi que cadena + strlen(ptr1) + 1 apuntara otra vez a "hu'\0'ha". En algun post anterior, ya he comentado que tiene que volver a utilizarse la misma cadena que se le ha pasado a strtok. Si miras como ha quedado cadena despues de la primera llamada a strtok veras el porque.

¡Saludos!

Si vas a criticar algo, lee lo que se postea primero y luego comentas, ¿ok?

Bien, estaba nervioso porque no entrabas en razon y he utilizado palabras que no deberia. Lo siento. Con este codigo puedes comprobar que no estas en lo cierto:

#include <stdio.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char cadena[] = "    Necesito repasar matematicas basicas";
    char *token,*resto,lector;
 
    token = strtok(cadena," ");
 
    printf("token: %s\ncadena + strlen(token) + 1: %s\ncadena + strlen(cadena) + 1: %s\n",
            token, cadena + strlen(token) + 1, cadena + strlen(cadena) + 1);
 
    while((lector = getchar()) != '\n' && lector != EOF);
 
    return 0;
}
 

Por lo menos tomate la molestia de comprobar si lo que dices es cierto o no.

¡Saludos!

PD: Esto si que es un error que he cometido yo. No he comprobado si realmente queda cadena o no despues de la primera llamada a strtok. Vosotros al utilizar strtok de nuevo, podeis saberlo mirando si el puntero devuelto es NULL o no. Aqui va el nuevo codigo y los resultados de los tiempos.

#include <stdio.h>
#include <string.h>
#include <time.h>
 
#define ITER 134217728
 
int main ()
{
  char str[] ="Hola como estas ?";
  char str2[] = "Hola como estas ?";
  char * pch;
  char * valor1;
  char * valor2;
  int i,inicio,longitud;
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        pch = strtok (str," "); // Empiezo a buscar la 1 coincidencia del espacio en blanco
        while (pch != NULL)
        {
            valor1 = (pch); // Aqui se almacena la 1 palabra de la frase separa por el espacio en blanco
            pch = strtok (NULL, ""); // Aqui ya no le pongo el espacio en blanco porque deseo el resto de la frase completa
            valor2 = (pch); // Almaceno el resto de la frase en la variable
            break; // Interrumpo el bucle pues como dije solo queria obtener la 1 palabra con el espacio como delimitador
        }
    }
 
    printf("Metodo 1: %d segundos.\n",time(NULL) - inicio);
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        longitud = strlen(str);
 
        pch = strtok(str," ");
 
        if(pch)
            if(strlen(str) != longitud)
                valor2 = &str[strlen(str) + 1];
    }
 
    printf("Metodo 2: %d segundos.\n",time(NULL) - inicio);
 
    inicio = time(NULL);
 
    for(i = 0 ; i < ITER ; i++)
    {
        strcpy(str,str2);
 
        pch = strtok(str," ");
 
        if (pch)
            valor2 = strtok(NULL, "");
    }
 
    printf("Metodo 3: %d segundos.\n",time(NULL) - inicio);
 
    return 0;
}
 

El tiempo del segundo metodo a subido algo...