funcion compleja

[richard_h]

buenas a todos, soy nuevo en el foro. Tengo 15 años y llevo un tiempo aprendiendo c, y ahora me he pasado a c++, pero tengo un codigo en c que tengo dudas.

Estoy haciendo un programa que muestre un verbo en español y yo lo tenga que traducir al ingles. Si esta mal, da la opcion de volver a intentarlo o ir al menu principal.

Pues bien, todos los verbos los reparto en cases, ya que el programa elige mostrar un verbo aleatoriamente. He intentado busca una funcion, pero no me sale.

Aqui dejo la funcion, a ver que tengo mal y me pueden ayudar :)

Código C:

case 1:
                       verb_i = betray;
                       verb_e = traicionar;                       
                       system("cls");
                       do{
                       printf("\n Traduzca %s: ",verb_e);
                       scanf(" %[^\n]s",&verbo);
                       
                       if(strcmp(verbo,verb_i)==0)
                       { 
                          system("cls");
                          printf("\n Muy bien.\n\n");
                          do{
                          printf(" 1-. Otro verbo\n");
                          printf("\n 2-. Menu\n");
                          printf("\n\n Que desea hacer ahora?: ");
                          scanf(" %d",&opcion_verbo); 
                          
                          switch(opcion_verbo)
                          {
                              case 1:
                                   system("cls");
                                   goto aleatorio_espanol;
                                   break;
                              case 2:
                                   system("cls");
                                   break;  
                                
                              default:
                                   system("cls");
                                   printf("\n Opcion incorrecta, intentelo de nuevo: \n\n");
                                   break;                  
                           }
                          }while(opcion_verbo != 1 && opcion_verbo != 2);
                       }
                       else
                       {
                          system("cls");                          
                          printf("\n Lo siento, traduccion incorrecta\n");
                          do{
                          printf("\n 1-. Volver a intentarlo\n");                          
                          printf("\n 2-. Menu\n");
                          scanf(" %d",&opcion_mal);
                                                    
                          switch(opcion_mal)
                          {
                              case 1:
                                   system("cls");
                                   break;
                              case 2:
                                   system("cls");
                                   break; 
                                     
                              default:
                                   system("cls");
                                   printf("\n Opcion no valida, intentelo de nuevo\n");
                                   break;                                              
                          }
                          }while(opcion_mal != 1 && opcion_mal != 2);  
                       }
                         
                         }while(opcion_mal == 1);
 

[L3m0n]

Podrias poner el programa completo, es que no comprendo bien lo que hace.
Tienes diversos verbos en diferentes cases? Pon una porcion más grande porfa.

Ara te daría unas recomendaciones para arreglar tus programas:

1. No uses goto (etiquetas) si no es absolutamente necesario, según los expertos siempre hay otra opcion, ya que si otra persona tiene que ver tu codigo, con goto es una pesadilla.

2. No uses la funcion scanf, para numeros aun no se comporta del todo "mal", pero para cadenas NUNCA hay que usarla, usa la funcion fgets.

Si tienes alguna duda pregunta :)

[cesar_casla]

Hola!! una pregunta L3m0n xq decis "No uses la funcion scanf, para numeros aun no se comporta del todo "mal", pero para cadenas NUNCA hay que usarla, usa la funcion fgets. ", Tanto para numeros como para cadenas funciona perfectamente(hasta donde yo se) ¿Podrias ampliar? en todo caso fgets me parece mas conveniente usarla cuando se quiere leer una linea y no una palabra.
Con respecto al codigo richard_h, no entiendo bien cual es tu problema lo unico que veo mal son las lineas 2 y 3, betray y traicionar deberian estar entre comillas si es que son cadenas

Código C:

verb_i="betray";
verb_e="traicionar";
 

Ademas coincido con L3m0n en no usar goto no es un crimen usarlo pero atenta con la forma de programacion estructurada como lo es C... Amplia el codigo x favor

Saludos

[L3m0n]

Scanf tiene un problema que es bastante gordo y es con el buffer de entrada, ya que esta función suele dejar mucha basura.

Ahora bien la función, leyendo cadenas tiene el mismo problema que gets, que no tiene límite de entrada.
Me explico: Tu puedes tener una cadena de ponle 10 caracteres, pero si usas gets o scanf el programa pedira hasta que haya un \n y los guardara en algun lugar de la memoria, lo cual es muy peligroso ya que puedes tener cualquier cosa en la memoria y estas funciones te lo pueden cambiar.

Si tienes alguna otra duda pregunta no dudes en hacerla, la intentaré responder lo mejor que pueda :)

[cesar_casla]

Gracias por la aclaracion L3m0n! no me habia puesto a pensar que pasaba con los datos que se escribia demas y mucho menos sabia que podia ser tan peligroso

Saludos

[richard_h]

Vale, ahora pondre el programa completo pero solo con un case. Y si, hay muchos verbos y enc ada case hay uno. LLamo a random, y segun el numero aleatorio que salga, sera un case, y ese es el verbo que tendre que traducir.
Sobre los goto ya lo arreglo, pero ahora lo que me interesa es poner el case como una funcion, ya que cada vez que hay un verbo nuevo, tengo que copiar el mismo trozo de codigo solo que cambaido algunas variables segun el verbo que sea.

Código C:

#include <stdio.h>
 
int main()
{   //variables    
    int cont=0;
    int aleatorio_ingles=0, aleatorio_espanol=0, opcion_mal, opcion_verbo;
       
    char verbo[15];
    
    //pongo chars con los verbos
    char betray[8]="betray";
    char traicionar[15]="traicionar";
    
    //.......
    //Ahora aqui estaria el menu principal que elegiria 
    //entre traducir a espanol o al ingles.
    //......
    
    //Aqui es la opcion traducir al ingles
    printf("\n\n\t ***Espanol - Ingles***\n\n");
               aleatorio_espanol:
               cont++;
               srand(time(NULL));//random
               aleatorio_espanol2:
               aleatorio_espanol=152+rand()%209;//se saca un numero random
                                                // del 152 al 209
               //esto una cosa rara porque algnas veces salia otro numero raro,
               //jaja. Simplemente asegurar que el numero es el valido
               if(aleatorio_espanol < 152 || aleatorio_espanol > 209)
                   goto aleatorio_espanol2;
               
               switch(aleatorio_espanol)// un switch segun el numero random
               
               case 1://por ejemplo el case 1, que es el verbo traicionar
               // esta es la parte que quiero convertir en funcion, porque
               //cuando hay un verbo neuvo, copio este mismo codigo y hago otro 
               //case, solo que cambiando el nombre de las variables al nuevo verbo                     
                       system("cls");
                       do{
                       printf("\n Traduzca traicionar: ");
                       scanf(" %[^\n]s",&verbo);
                       
                       if(strcmp(verbo,betray)==0)
                       { 
                          system("cls");
                          printf("\n Muy bien.\n\n");
                          do{
                          printf(" 1-. Otro verbo\n");
                          printf("\n 2-. Menu\n");
                          printf("\n\n Que desea hacer ahora?: ");
                          scanf(" %d",&opcion_verbo); 
                          
                          switch(opcion_verbo)
                          {
                              case 1:
                                   system("cls");
                                   goto aleatorio_espanol;
                                   break;
                              case 2:
                                   system("cls");
                                   break;  
                                
                              default:
                                   system("cls");
                                   printf("\n Opcion incorrecta, intentelo de nuevo: \n\n");
                                   break;                  
                           }
                          }while(opcion_verbo != 1 && opcion_verbo != 2);
                       }
                       else
                       {
                          system("cls");                          
                          printf("\n Lo siento, traduccion incorrecta\n");
                          do{
                          printf("\n 1-. Volver a intentarlo\n");                          
                          printf("\n 2-. Menu\n");
                          scanf(" %d",&opcion_mal);
                                                    
                          switch(opcion_mal)
                          {
                              case 1:
                                   system("cls");
                                   break;
                              case 2:
                                   system("cls");
                                   break; 
                                     
                              default:
                                   system("cls");
                                   printf("\n Opcion no valida, intentelo de nuevo\n");
                                   break;                                              
                          }
                          }while(opcion_mal != 1 && opcion_mal != 2); 
                                                  
                          
                       }
                         
                         }while(opcion_mal == 1);
                        
                                
                       break;
    
getchar();
return 0;    
}
 

[L3m0n]

Osea que lo que quieres es crear la misma funcion para todos los verbos no?

Fácil.

Tienes que crear una funcion que reciba como parametros el verbo que el usuario ingresa y el verbo correcto en ingles. I que devuelva si es correcto o no.

Cómo se hace esto, fácil:

Código C:

int comprovar(char verbointroducido[], char verbocorrecto[]){
//miras si coinciden o no, dependiendo de eso dices que devuelva una cosa o otra
if(strcmp (verbointroducido,verbocorrecto)==0) return 0;
else return 1;
}
 

Ahora en la funcion main puedes ir, poner la funcion y segun si devuelve una cosa o otra pues sigues con el programa, no se si me he explicado bien.

Si algo no te ha quedado claro pregunta :)

[richard_h]

muchisimas gracias por la rapidez, es increible, y por tus conocimientos y ayuda tambn, que estas aportando :)

Pero lo que yo querria es poner TODO el case en una funcion, porque en todos los cases es el mismo codigo. Tu lo que has hecho es una funcion para saber si el verbo es igual o no. Asi solo llamo a la funcion una vez en cada verbo nuevo, no se si me explico, quiero que en la funcion este los printf y eso, con los if else, yque en el case solo este la llamada a la funcion

es eso posible? y gracias de antemano, sois de gran ayuda :))

[L3m0n]

Ah ya entiendo, osea que quieres hacer una funcion desde case hasta el break final.

Puedes hacer una cosa que solía y suelo hacer, una "solución rápida por llamarlo de alguna manera"

Como?

Fácil, es tan simple como crear una funcion void con todo el código.
Si no te queda claro dimelo y te escribo el codigo pero ya te digo es copiar lo que tienes en una funcion void.

[richard_h]

ya intente hacer eso, pero no me funciono. No se por que es, ya que lo copie como dijiste pero nada. La cosa es que la variable de un verbo, en este caso traicionar, habría que añadirla como variable dentro de la funcion, no? es que no se como se hace para que una funcion utilize una variable del main.

si me ayudas sera de gran utilidad.

[L3m0n]

Creo que tienes mal el concepto, no tienes que hacer que la funcion use una variable de main.
Lo que tienes que hacer es crear una variable global, que es eso? Pues es una variable que puede ser accedida para todas las funciones.

Como se hace?

Muy sencillo, solo tienes que hacer la declaracion fuera del main.

Antes del main declaras las variables que quieres que sean globales:

Código C:

int variable; //aqui declaras las variables globales
int main(void){
...
}
 

Si dudas en algo pregunta :)

[richard_h]

Muchas gracias, eso no lo sabia :P

Ya lo hice y me salio a la perfeccion. Pero al intentar hacer otra funcion para la parte español-ingles, al intentar probarlo me daba error. Tras hacer muchas pruebaas, decidi borrar esa funcion y probar el programa. Y cual fue mi sorpresa que dio error.

Es decir, hice la primero funcion, me salio, y despues de hacer otra funcion y borrarla, ahora la primera funcion me da error, sin apenas haberla tocado. Que esta pasando¿

me dice: "passing arg 1 of strcmp' makes pointer from integer without a cast", eso en cada variable. Muy extraño.

Os dejo el programa, haber que pasa:

Código C:

int cont=0;
    int aleatorio_ingles=0,opcion,opcion_espanol,opcion_mal,opcion_verbo;
    char verbo;    
    int aleatorio_espanol=0,opcion_verbo2;
    
char *verb, *verb2, *verb3, *verb4, *verb5, *verb6, *verb7;
 
  void e_verbo()
                  {
                  system("cls");
                       do{
                       printf("\n Traduzca %s: ",verb2);
                       scanf(" %[^\n]s",&verbo);
                       
                       if(strcmp(verbo,verb)==0)//el primer error me lo da aqui
                       //,no se porque
                       { 
                          system("cls");
                          printf("\n Muy bien.\n\n");
                          do{
                          printf(" 1-. Otro verbo\n");
                          printf("\n 2-. Menu\n");
                          printf("\n\n Que desea hacer ahora?: ");
                          scanf(" %d",&opcion_verbo); 
                          
                          if(opcion_verbo < 1 || opcion_verbo > 2)
                          {
                                   system("cls");
                                   printf("\n Opcion incorrecta, intentelo de nuevo: \n\n");
                           }
                          }while(opcion_verbo != 1 && opcion_verbo != 2);
                       }
                       else
                       {
                          system("cls");                          
                          printf("\n Lo siento, traduccion incorrecta\n");
                          do{
                          printf("\n 1-. Volver a intentarlo\n"); 
                          printf("\n 2-. Ver traduccion\n");                          
                          printf("\n 3-. Menu\n");
                          scanf(" %d",&opcion_mal);
                         
                         switch(opcion_mal)
                         {
                             case 1:
                                  system("cls");
                                  break;
                             
                             case 2:
                                  system("cls");
                                  if(verb3==NULL)
                                  printf("\n La traduccion del verbo %s es: %s\n\n", verb, verb2);
                                  
                                  else
                                  {
                                  if(verb4==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s\n\n", verb, verb2, verb3);
                                  
                                  else
                                  {
                                  if(verb5==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s\n\n", verb, verb2, verb3, verb4);    
                                  
                                  else
                                  {
                                  if(verb6==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s, %s\n\n", verb, verb2, verb3, verb4, verb5);    
                                  
                                  else
                                  {
                                  if(verb7==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s, %s, %s\n\n", verb, verb2, verb3, verb4, verb5, verb6);    
                                  
                                  }}}}
                                  
                                  break;
                                  
                             case 3:
                                  break;
                                  
                             default:
                                      system("cls");
                                   printf("\n Opcion no valida, intentelo de nuevo\n");
                                     break;              
                         }
                         
                       }while(opcion_mal != 1 && opcion_mal != 2 && opcion_mal != 3); 
                                                  
                          
                       }
                         }while(opcion_mal == 1);
                         }
    
    int main(){   
                          
  case 1:         //os paso directamente al case
                        verb = betray;
                        verb2 = traicionar;                        
                        verb3, verb4, verb5, verb6, verb7;
                        
                        e_verbo();// llamo a la funcion
                        
                      
                       break;
                       
                       return 0;
                       }
 

[cesar_casla]

Es xq a la variable verbo la declaraste como char (linea 3) y no como char *, x eso la funcion strcmp te tira error

Saludos

[richard_h]

NO es eso, por favor, no digas cosas sin saber. En los otros he puesto * porque se va a asignar a otro char, pero verbo es solo para comparar. Sin mala intencion :)

[starfix]

proba cambiando la linea 15 por lo siguiente:

Código C:

if(strcmp(&verbo,verb)==0)
 

solamente agregue el &

[cesar_casla]

Sabes el uso de la funcion strcmp?? sabes la diferencia entre un char y una cadena??? si declaras char verbo; xq en el scanf pones %s (linea 13)?? no deberia ser %c???
El strcmp recibe dos char * como argumentos, ES UN ERROR PASERLE UN CHAR! al menos pasale la direccion de verbo para que funcione

[richard_h]

a ver, cuando el char es de una letra, por ejemplo 'a', se pone %c
pero cuando el char tiene mas de un caracter se convierte en CADENA por lo que se pone %s.

[cesar_casla]

A si!! ahora el tipo de dato char, definido como char verbo; se puede usar como una cadena?? Decime en que lenguaje estamos hablando xq eso no es C?? sabias que con char solo podes almacenar un byte osea un solo caracter?? como haces con un char para almacenar una cadena, me gustaria saberlo

Bueno no voy a discutir mas con vos, este foro esta para ayudar y se ve que vos no queres aprender, yo solo me fije en tu codigo y respondi segun a eso. pero que vos digas "NO es eso, por favor, no digas cosas sin saber" esta muy mal toda respuesta se agradece

[richard_h]

ok perdoname por favor por haber ofendido, simplemente es que me estoy frusrando demasiado, lo siento :S

A ver si tengo esto bien o estoy equivocado. a un char si quieres poner por ejemplo la palabra alvaro, haces:

Código C:

char[7] = "alvaro"
 

". Asi es una cadena, no?

Perdon otra vez por la tonta discusion :)

[richard_h]

y starfix, con eso que dijiste de ñadir el & me funciona ya bien, lo malo es que ahora me manda un mensaje porque tengo que añadirselo a los otros verbos, que son mas de 200 :/

pero lo extraño es que antes no hacia falta poner eso, y tras hacer una funcion aparte y borrarla y dejar el programa como estaba, me da error, no lo entiendo

[starfix]

Iniciado por richard_h

y starfix, con eso que dijiste de ñadir el & me funciona ya bien, lo malo es que ahora me manda un mensaje porque tengo que añadirselo a los otros verbos, que son mas de 200 :/

cuales son los otros 200?, son los que pondras mas adelante en los cases dentro de main?

[richard_h]

starfix: no, ya los tengo puestos. LA opcion que os he enseñado, el verbo traicionar, era el case numero 209, asi que......

Pero es que lo EXTRAÑO es que antes no necesitaba ese & y funcionaba perfectamente, y ahora no se que he hecho que me da error. Además he añadido una nueva opcion que es mostrar la traduccion del verbo, y es muy extraño porque a al principio me sale bien, le doy a menu y al inentarlo no me va, despues si, y asi bastantes veces, no se que pasa.

Os paso el codigo con el case del verbo traicionar.
(por cierto ya se que hay algunos goto, pero por ahora mi problema es que funcione la funcion, mas adelante rematare eso :).

Código C:

int cont=0;
    int aleatorio_ingles=0,opcion,opcion_espanol,opcion_mal,opcion_verbo;
    char verbo[];
    char *verb, *verb2, *verb3, *verb4, *verb5, *verb6, *verb7;
    
    int aleatorio_espanol=0,opcion_verbo2;
    
    
    char traicionar[15]="traicionar", betray[8]="betray";
    
    
    void e_verbo()
                  {
                  system("cls");
                       do{
                       printf("\n Traduzca %s: ",verb2);
                       scanf(" %[^\n]s",&verbo);
                       if(strcmp(verbo,verb)==0)
                       { 
                          system("cls");
                          printf("\n Muy bien.\n\n");
                          do{
                          printf(" 1-. Otro verbo\n");
                          printf("\n 2-. Menu\n");
                          printf("\n\n Que desea hacer ahora?: ");
                          scanf(" %d",&opcion_verbo); 
                          
                          if(opcion_verbo < 1 || opcion_verbo > 2)
                          {
                                   system("cls");
                                   printf("\n Opcion incorrecta, intentelo de nuevo: \n\n");
                           }
                          }while(opcion_verbo != 1 && opcion_verbo != 2);
                       }
                       else
                       {
                          system("cls");                          
                          printf("\n Lo siento, traduccion incorrecta\n");
                          do{
                          printf("\n 1-. Volver a intentarlo\n"); 
                          printf("\n 2-. Ver traduccion\n");                          
                          printf("\n 3-. Menu\n");
                          scanf(" %d",&opcion_mal);
                         
                         switch(opcion_mal)
                         {
                             case 1:
                                  system("cls");
                                  break;
                             
                             case 2:
                                  system("cls");
                                  if(verb3==NULL)
                                  printf("\n La traduccion del verbo %s es: %s\n\n", verb2, verb);
                                  
                                  else
                                  {
                                  if(verb4==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s\n\n", verb2, verb, verb3);
                                  
                                  else
                                  {
                                  if(verb5==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s\n\n", verb2, verb, verb3, verb4);    
                                  
                                  else
                                  {
                                  if(verb6==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s, %s\n\n", verb2, verb, verb3, verb4, verb5);    
                                  
                                  else
                                  {
                                  if(verb7==NULL)
                                  printf("\n La traduccion del verbo %s es: %s, %s, %s, %s, %s\n\n", verb2, verb, verb3, verb4, verb5, verb6);    
                                  
                                  }}}}
                                  
                                  break;
                                  
                             case 3:
                                  break;
                                  
                             default:
                                      system("cls");
                                   printf("\n Opcion no valida, intentelo de nuevo\n");
                                     break;              
                         }
                         
                       }while(opcion_mal != 1 && opcion_mal != 2 && opcion_mal != 3); 
                                                  
                          
                       }
                         }while(opcion_mal == 1);
    }
  
  int main(){  
                          
  case 210://paso directamente al case:
                        verb = rattle;
                        verb2 = sonar;                        
                        verb3 = vibrar, verb4, verb5, verb6, verb7;
                        
                        e_verbo();
                        
                        if (opcion_verbo == 1){  
                        system("cls");                        
                        goto aleatorio_espanol;}
                                                
                        if(opcion_verbo == 2 || opcion_mal == 3){system("cls");}
                        
                        if(opcion_mal == 2) goto menu;
                                   
                                                     
                       break;
 

[L3m0n]

Pon comillas cuando asignas cadenas a variables.

Código C:

verb = "rattle";
verb2 = "sonar";
 

Ahora dime una cosa, el compilador te da algun problema o el problema sucede cuando el programa ya esta en marcha?

[richard_h]

Pon comillas cuando asignas cadenas a variables.

Pero estoy asignando una variable a puntero(verb), no una cadena. Por eso sale * de puntero que apunta a la memoria de un verbo.

Pero no se si eso esta bien, hasta ahora me ha funcionado. Acabo de cambiar eso por entre comillas y no me funciona. Podrias pegar el codigo como tu aconsegas?

gracias :)

[L3m0n]

Ah, ya entiendo, tienes rattle declarado como cadena anteriormente no? Es que en el codigo que has puesto no se ve bueno supongo que será así. De todos modos lo de hacerlo con punteros me parece un poco extraño, nose cada programador piensa como quiere, ahora bien recuerda inicializar los punteros.

Código C:

char *puntero = NULL;
 

Así el compilador entiende que ese puntero no apunta a ninguna parte, si los declaras sinn inicializar, estos punteros pueden apuntar a cualquier lado, cosa que no es muy recomendable.

[richard_h]

ah, perdona, se me olvido ponerlo en el codigo :P

yo soy novato en esto, asi que quiero seguir tu consejo, me he liado demasiado con los punteros.

Lo que tu dices seria hacer verb = "rattle", no? pero entonces a verb habria que quitarle el * de puntero, no?

Si es eso, lo he intentado y al poner la traduccion y darle al enter, el programa no responde no se por que.

Ademas en TODOS los cases, que son unos cuantos, me poner error con el strcmp: passing arg 1 of strcmp makes pointer from integer without a cast.

Que hago? ayudaaa ;)

[L3m0n]

Yo creo que te complicas mucho, yo de ti haria el mismo programa de otra manera:

Creas una variable global verbo de ponle 20 caracteres, que no creo que haya muchos verbos mas grandes que eso.

Código C:

char verbo[20];
 

Y ahora para cada case asignas a la variable una cadena, me explico:

Código C:

case 1: verbo = "run";
                ...
case 2: verbo= "start";
               ...
 

Y asi en todos los cases. Lo de los punteros no lo encuentro muy util nose. Lo has hecho por alguna razon en concreto?

[richard_h]

estoy desesperado, no se que le pasa al codigo. Hice lo que me dijiste pero me dice incopatyble type in assignment, cuando pongo el [20] y le quito el *.

Y sobre la variable verbo, hay tres tipos. Uno es verbo, que guarda el verbo que escribo, y los otros dos dos son verb y verb2, que son los verbos en ingles y español para compararlos con el que he ingresado.

Ademas he visto que no se puede hacer

Código C:

verbo="run";
 

porque da error. habria que hacerlo caracter por caracter.

Código C:

verbo[0]='r';
    verbo[1]='u';
    verbo[2]='n';
    verbo[3]='\0';
 

Asi que, que hago?

gracias :)

[Fw190]

En C no puedes asignar directamente un valor a una cadena de caracteres. Utiliza

Código C:

strcpy(verbo, "run");
 

[L3m0n]

Iniciado por Fw190

En C no puedes asignar directamente un valor a una cadena de caracteres. Utiliza

Código C:

strcpy(verbo, "run");
 

Cierto, es que estaba programando con python y a veces se me cuelan xD

Mea culpa.