Skip to main content
3 de 4
Como máximo, MAX_LINE / 2 ordenes, suponiendo que sea tipo 'a b c d e ...'
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Tu código contenía bastantes errores.

Esta es mi versión, manteniendo la interfaz de tus funciones intactas.

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

#define PROMPT "$"
#define MAX_LINE 512

#define DELIMS " \t\r\n\v\f"

int parse_args( char **args, char *line){
  int n = 0; // Contador de palabras.
  char *token = strtok( line, DELIMS );

  while( token ) {
    args[n] = token;
    ++n;
    token = strtok( NULL, DELIMS );
  }
  return n;
}

char *read_line( char *line ){
  printf( "%s%s ", getenv( "USER" ), PROMPT );
  fflush( stdout );
  return fgets( line, MAX_LINE, stdin );
}

int execute_line( char *line ){
  char *args[MAX_LINE/2]; // NO puede haber mas de MAX_LINE / 2 ordenes.

  memset( args, 0, MAX_LINE ); // Por seguridad.
  parse_args( args, line );

  return check_internal( args );
}

int check_internal( char **args ) {
  if( !strcmp( args[0], "cd" ) ) {
    return internal_cd( args );
  } else
  if( !strcmp(args[0], "export" ) ) {
    return internal_export( args );
  } else
  if( !strcmp( args[0], "source" ) ) {
    return internal_source();
  } else
  if( !strcmp(args[0], "jobs" ) ) {
    return internal_jobs( );
  } else
  return -1;
}

int internal_cd( char **args ){
  char buff[MAX_LINE];
  int ret;

  if( !args[1] ) {
    printf( "Debe indicar una ruta.\n" );
  } else {
    if( chdir( args[1] ) == -1 ) {
      fprintf( stderr, "Error: %s\n", strerror( errno ) );
      return -1;
    }
    getcwd( buff, MAX_LINE );
    printf( "Estas en la ruta %s\n", buff );
  }
  return 1;
}

int internal_export( char **args ) {
  char buff[MAX_LINE];
  char *old;
  int ret;

  if( args[1] && args[2] ) {
    old = getenv( args[1] );
    printf( "Valor antiguo: %s\n", old ? old : "SIN ASIGNAR" );
    if( setenv( args[1], args[2], 1 ) == -1 ) {
      fprintf( stderr, "Error: %s\n", strerror( errno ) );
      return -1;
    }
    printf( "Valor nuevo: %s\n", args[2] );
  } else {
    fprintf( stderr, "Error. Uso: export <nombre> <valor>\n" );
  }
  return 1;
}

int internal_source(char **args) {
  printf("%s","éste es el source\n");
  return 1;
}

int internal_jobs(char **args){
    printf("%s","éste es el jobs\n");
    return 1;
}

void main( ) {
  char line[MAX_LINE]; 

  while( read_line( line ) ) {
    execute_line( line );
  }
}

No te detallo todas las modificaciones porque, sinceramente, han sido bastantes y no las recuerdo.
Como se mantiene la interfaz de tus funciones, puedes compararlas.

EDITO

El gran truco está en strtok( str, delims ). Esta función modifica str, colocando 0 en las posiciones en las que encuentre delimitadores. No es necesario reservar memória ni nada, usamos directamente la línea leída por fgets( ).

Trauma
  • 26.9k
  • 5
  • 42
  • 68