El código que funciona sería así:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define PROMPT "$"
int parse_args(char **args, char *line){
char *token;
int n=0;
token=strtok(line," "); //aquí da warning: assignment makes pointer from integer without a cast (porque faltaba string.h)
while(token!=NULL){
printf("token%i: %s\n",n,token);
*args=token;
n++;
*args++;
token=strtok(NULL," ");//aquí da warning: assignment makes pointer from integer without a cast (porque faltaba string.h)
}
printf("token%i: %s\n",n,token);
*args=token;
return n;
}
char *read_line(char **line){ //aquí me dice que espera un char* pero que recibe un char** - lo correcto es que sea char**
printf("%s ",PROMPT);
*line = (char *) malloc(500 * sizeof(char));//aquí da warning: assignment makes pointer from integer without a cast // porque malloc devuelve (void *) y no (char *) - por otra parte sizeof(500) devuelve el tamaño de un int, tú quieres 500 char (500 por el tamaño de un char)
fgets(*line,500,stdin);
return *line;
}
int execute_line(char *line){//aquí me dice que espera un char* pero que recibe un char**
char **args;
parse_args(args,line);
check_internal(args);
return 0;
}
int check_internal(char **args){
if( strcmp(*args, "cd")==0 ){
internal_cd();
} else{
if( strcmp(*args, "export")==0 ){
internal_export();
}else{
if( strcmp(*args, "source")==0 ){
internal_source();
}else{
if( strcmp(*args, "jobs")==0 ){
internal_jobs();
}else{
printf("%s","pasa los ifelse\n");
return 0;
}
}
}
}
}
int internal_cd(char **args){
printf("%s","cambio de directorio\n");
return 1;
}
int internal_export(char **args) {
printf("%s","éste es el export\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;
while(read_line(&line)){//warning: tipo de puntero incompatible - ahora read_line recive char **
execute_line(line);//warning: tipo de puntero incompatible - aquí no necesitas el prefijo porque line ya es una dirección como la espera excecute_line
}
free(line);
}
En este otro ejemplo, definiendo un nuevo tipo, las cosas quedan mucho más claras:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define PROMPT "$"
typedef char* string;
int parse_args(string *args, string line){
string token;
int n=0;
token=strtok(line," "); //aquí da warning: assignment makes pointer from integer without a cast
while(token!=NULL){
printf("token%i: %s\n",n,token);
*args=token;
n++;
*args++;
token=strtok(NULL," ");//aquí da warning: assignment makes pointer from integer without a cast
}
printf("token%i: %s\n",n,token);
*args=token;
return n;
}
string read_line(string *line){ //aquí me dice que espera un char* pero que recibe un char**
printf("%s ",PROMPT);
*line = malloc(500 * sizeof(char));//aquí da warning: assignment makes pointer from integer without a cast
fgets(*line,500,stdin);
return *line;
}
int execute_line(string line){//aquí me dice que espera un char* pero que recibe un char**
string *args;
parse_args(args,line);
check_internal(args);
return 0;
}
int check_internal(string *args){
if( strcmp(*args, "cd")==0 ){
internal_cd();
} else{
if( strcmp(*args, "export")==0 ){
internal_export();
}else{
if( strcmp(*args, "source")==0 ){
internal_source();
}else{
if( strcmp(*args, "jobs")==0 ){
internal_jobs();
}else{
printf("%s","pasa los ifelse\n");
return 0;
}
}
}
}
}
int internal_cd(string *args){
printf("%s","cambio de directorio\n");
return 1;
}
int internal_export(string *args) {
printf("%s","éste es el export\n");
return 1;
}
int internal_source(string *args) {
printf("%s","éste es el source\n");
return 1;
}
int internal_jobs(string *args){
printf("%s","éste es el jobs\n");
return 1;
}
void main(){
string line;
while(read_line(&line)){//warning: tipo de puntero incompatible
execute_line(line);//warning: tipo de puntero incompatible
}
}
Porque en la función read_line
, tú necesitas modificar el contenido de line
sin embargo no está permitido porque line
pertenece a main
entonces lo haces a través de su dirección. En read_line
modificas el contenido donde apunta line
. El mismo criterio debes aplicarlo a parse_args
. Es un caso análogo al ejemplo de intercambio de enteros del libro de K&R. La función fgets
no tiene límite, lee hasta encontrar un EOF
o un \n
pero sólo guarda los primeros n carácteres. Los máximos dependerán del sistema donde lo corras. Lo último que te sucede es normal porque fgets
incluye el \n
que el usuario ingresó en stdin
. Para solucionarlo debes reemplazar el último carácter por \0
. Por último no olvides liberar la memoria cuando ya no la uses.
Entonces el programa completo queda así:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define PROMPT "$"
typedef char* string;
typedef string* list;
int parse_args(list *args, string line){
string token;
int n=0;
*args = malloc(500 * sizeof(string)); // máximo 500 elementos
token=strtok(line," ");
while(token!=NULL){
printf("token%i: %s\n",n,token);
*(*args+n)=token;
n++;
token=strtok(NULL," ");
}
printf("token%i: %s\n",n,token);
*(*args+n)=token;
free(token); // liberar, no se usa más
return n;
}
string read_line(string *line){
printf("%s ",PROMPT);
*line = malloc(500 * sizeof(char));
fgets(*line,500,stdin);
*(*line + strlen(*line) - 1) = '\0'; // quitar el '\n' final
return *line;
}
int execute_line(string line){
list args;
parse_args(&args,line);
check_internal(args);
free(args); // liberar, no se usa más
return 0;
}
int check_internal(list args){
if( strcmp(*args, "cd")==0 ){
internal_cd(args);
} else{
if( strcmp(*args, "export")==0 ){
internal_export(args);
}else{
if( strcmp(*args, "source")==0 ){
internal_source(args);
}else{
if( strcmp(*args, "jobs")==0 ){
internal_jobs(args);
}else{
printf("%s","pasa los ifelse\n");
return 0;
}
}
}
}
}
int internal_cd(list args){
printf("%s","cambio de directorio\n");
return 1;
}
int internal_export(list args) {
printf("%s","éste es el export\n");
return 1;
}
int internal_source(list args) {
printf("%s","éste es el source\n");
return 1;
}
int internal_jobs(list args){
printf("%s","éste es el jobs\n");
return 1;
}
void main(){
string line;
while(read_line(&line)){
execute_line(line);
free(line); // liberar, no se usa más
}
}