Ayuda

#include <stdio.h>
#include <string.h>
#define N 50
typedef struct {
char eng[N];
char esp[N];
int lleno;
}traductor;
traductor trad[N];
void vacio();
void anadir();
void traducir();
void tradu(int op);
void cambio (char palabra[N]);
int main() {
int op;
char key;
vacio();
void anadir();
do {
do {
printf ("Elige que deseas hacer:\n");
printf ("(1) Anadir palabras al traductor.\n");
printf ("(2) Buscar traduccion de la palabra en el traductor.\n");
scanf ("%i", &op);
} while (op < 1 || op > 2);
switch (op) {
case 1:
anadir();
break;
case 2:
traducir();
break;
}
printf ("Introduce si deseas realizar otra operacion. (S/N)");
scanf ("%c", &key);
} while(key == 'S' || key == 's');
system ("pause");
return 0;
}
void vacio() {
int i;
for (i=0 ; i<N ; i++) {
trad[i].lleno = 0;
}
}
void anadir() {
int i, aux;
aux = 0;
for (i=0 ; i<N && aux == 0 ; i++) {
if (trad[i].lleno == 0) {
printf ("Introduce la palabra en ingles.\n");
fflush(stdin);
fgets (trad[i].eng,N,stdin);
cambio (trad[i].eng);
printf ("Introduce la palabra en espanol.\n");
fflush(stdin);
fgets (trad[i].esp,N,stdin);
cambio (trad[i].esp);
trad[i].lleno = 1;
aux = 1;
}
}
}
void traducir() {
int op;
do {
printf ("Elige que deseas hacer:\n");
printf ("(1) Traducir de Ingles a Espanol.\n");
printf ("(2) Traducir de Espanol a Ingles.\n");
scanf ("%i", &op);
} while (op < 1 || op > 2);
switch (op) {
case 1:
tradu(op);
break;
case 2:
tradu(op);
break;
}
}
void tradu(int op) {
int i, j, temp;
char aux[N];
temp = 0;
printf ("Introduce la palabra que desea buscar: \n");
fflush(stdin);
fgets (aux,N,stdin);
cambio (aux);
if (op == 1) {
for (i=0 ; i<N && temp==0 ; i++) {
j = strcmp (aux, trad[i].eng);
if (j == 0) {
printf ("La traduccion de %s es %s.\n", trad[i].eng, trad[i].esp);
temp = 1;
}
}
}
else {
for (i=0 ; i<N && temp==0 ; i++) {
j = strcmp (aux, trad[i].esp);
if (j == 0) {
printf ("La traduccion de %s es %s.\n", trad[i].esp, trad[i].eng);
temp = 1;
}
}
}
}
void cambio (char palabra[N]) {
int i;
for (i=0 ; i<N ; i++) {
if (palabra[i] == '\n') {
palabra[i] = '\0';
}
}
}

Hola:
- Esto puede ser el problema que queres decir que solo traduce una palabra y no mas; despues que hace la opcion de añadir o traducir la variable "op" vale 1 o 2, y aunque decidas traducir otra palabara poniendo "s", op va a valer 1 o 2 y como no cumple la condicion de la linea 32 no entra en ese bucle nunca mas. Tenes que reiniciar "op" en 0 por ejemplo en la linea 42.

- En el procedimiento traducir() no tiene sentido un 'switch' porque elija traducir de una manera u otra hace lo mismo (llama al mismo metodo con el mismo parametro.
En vez de:

Cita de: Maxmorningstar en 18 Junio 2022, 02:21 am

Código:

 switch (op) {
  case 1:
   tradu(op);
  break;
  case 2:
      tradu(op);
  break;
 }

pone solo esto

Código:

tradu(op)

- Nunca traduce una 'oracion' porque lo hiciste para solo traducir por palabras. Tendria que hacer alguna manera de que ingrese una oracion y que compares palabra por palabra. Puede que hallas agregado una 'oracion' y busques esa exactamente (todas las letras, espacios o cualquier caracter coincida); ahi si funcionaria pero no es para nada practico y no es coherente. *

- Por el tema de la repeticion de palabras es pedir que palabra queres introducir y en que idioma esta, entonces busca en la correspondiente variable de la estructura. Si esta no la agregas, y si no esta como ya sabes la palabra en un idioma lo agregas y pedis que ingrese en el otro idioma esa palabra.

Y cuando busques una palabra para traducir deberias fijarte si hay mas palabras en el "diccionario", para eso deberia usarse la variable lleno; o no seria necesario que exista esa variable porque preguntas por alguna cadena (eng o esp) si es nulo (creo que todos las cadenas son nulas por defecto sino usarias "lleno" solamente) termina de recorrer el diccionario. En este caso para hacer eso, cuando recorres en la condicion del 'for' agregas la condicion trad[n].eng==NULL. **

Un consejo: no te obsesiones con usar condiciones; pensa si fueras a un ATM y te pregunte cada vez que ingrese a una opcion si desea seguir.
Es solo un consejo

*Si lo que quieres es traducir oraciones, seria algo bastante mas complejo, porque pensa que quieras traducir diferentes oraciones con las mismas palabras, tienen diferentes significados por ende las combinaciones posibles serias muchisimas y ademas traducidas. Si lo que quieres es hacer una programa realmente funcional, es un trabajo mucho mas dificil

**Aclaracion: viendo que usas la variable lleno en otro procedimiento quedaria mejor usar eso y no lo de NULL, pero tendrias un costo minimo de momoria inceseario.

Saludo!