Imprimir Página - problema con el siguiente codigo [C]

Código:

#include <stdio.h>

int esVocal (char c);
char sigVocal (char c);

int main ()
{
char c;
do{
    if(esVocal(c)) sigVocal(c);
    printf("%c", c);
        }while(scanf("%c", &c)!= EOF);

return 0;
}


int esVocal (char c){
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}

char sigVocal (char c){
if(c=='a')c='e';
if(c=='e')c='i';
if(c=='i')c='o';
if(c=='o')c='u';
if(c=='u')c='a';

return c;
}

El problema es que no cambia la vocal:

si la entrada es:

Citar

hola que tal

la salida es:

Citar

hola que tal

cuando deberia ser:

Código:

 Hule qui tel

Una vez lo consiga hacer con la entrada por teclado y la salida por monitor el siguiente paso seria hacerlo con ficheros. salu2

Cita de: SoyelRobert en 3 Enero 2014, 14:56 pm

Estoy intentado hacer un programa al cual se le introduce un texto por teclado y que vaya cambiando las vocales por su siguiente vocal:

Si lee una 'a' devuelve una 'e'.

Si lee una u devuelve una 'a'.

No se tienen en cuenta mayusculas ni vocales acentuadas.
...................................................................
Una vez lo consiga hacer con la entrada por teclado y la salida por monitor el siguiente paso seria hacerlo con ficheros. salu2

Antes que nada procura indentar o sangrar el código para que sea más legible y elige las etiquetas de C++, aunque el código sea en C. Salen de esa forma coloreado y es más fácil leerlo.

Y en tema pon una breve descripción de lo que buscas, no aclara nada eso de "problema con el siguiente codigo [C]"

Te pongo el código con las correciones que me han parecido y que no se aparten de tu código original. Por cierto, ya veras cómo salir del bucle. ;) ;) ;).

Código

#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main (){
    char c;
    do{
        puts ("Introduzca vocal:");
        scanf("%c",&c);
        while (getchar ()!='\n');
        if(esVocal(c)!=0)
            printf("%c\n", sigVocal(c));
    }while(c!= EOF);
    return 0;
}
 
int esVocal (char c){
    int si_o_no=0;
    if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u')
        si_o_no=1;
    else si_o_no=0;
    return si_o_no;
}
 
char sigVocal (char c){
    if(c=='a')
        c='e';
    else if(c=='e')
        c='i';
    else if(c=='i')
        c='o';
    else if(c=='o')
        c='u';
    else if(c=='u')
        c='a';
    return c;
}

¡¡¡¡ Saluditos! ..... !!!!

:rolleyes: ;) ;) ;) :rolleyes:

hola leosansan, no se lo que quiere decir indentar un cogido la verdad...

y respecto a lo que has programado, se sale un poco de lo que en un principio buscaba programar, mira:

el asunto era crear un programa en C que al tu introducir un texto cambie todas las vocales que aparezcan por la siguiente vocal que le toque, no se trata de introducir una única vocal para que la modifique sino introducir un texto entero donde aparezcan blancos, saltos de linea, etc (cualquier caracter hasta que se acabe el fichero, EOF)

salu2 y gracias

Código:

#include <stdio.h>

int esVocal (char c);
char sigVocal (char c);

int main ()
{
char c;
do{
    if(esVocal(c)){
         sigVocal(c);
    }
    printf("%c", c);
    return 0;
}
while(scanf("%c", &c)!= EOF);

int esVocal (char c){
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}

char sigVocal (char c){
if(c=='a')c='e';
if(c=='e')c='i';
if(c=='i')c='o';
if(c=='o')c='u';
if(c=='u')c='a';

return c;
}

Nose si asi funciona, solo meti los claudators al if, porque vamos, a medida que lee las letras queres que si es una vocal la canvie segun la funcion sigVocal. no?

Cita de: joanj94 en 3 Enero 2014, 16:28 pm

Código:

#include <stdio.h>

int esVocal (char c);
char sigVocal (char c);

int main ()
{
char c;
do{
    if(esVocal(c)){
         sigVocal(c);
    }
    printf("%c", c);
        }while(scanf("%c", &c)!= EOF);

return 0;
}


int esVocal (char c){
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}

char sigVocal (char c){
if(c=='a')c='e';
if(c=='e')c='i';
if(c=='i')c='o';
if(c=='o')c='u';
if(c=='u')c='a';

return c;
}

Nose si asi funciona, solo meti los claudators al if, porque vamos, a medida que lee las letras queres que si es una vocal la canvie segun la funcion sigVocal. no?

si metes el printf dentro del if solo pintara en caso de que c sea una vocal, por lo tanto, no pintria consonantes.

Código:

Código:
#include <stdio.h>

int esVocal (char c);
char sigVocal (char c);

int main ()
{
char c;
do{
    if(esVocal(c)){
         sigVocal(c);
    }//fin if
    printf("%c", c);
}// fin dentro del do
while(scanf("%c", &c)!= EOF);

return 0;
}//fin main

int esVocal (char c){
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}// fin esVocal

char sigVocal (char c){
if(c=='a')c='e';
if(c=='e')c='i';
if(c=='i')c='o';
if(c=='o')c='u';
if(c=='u')c='a';

return c;
}//fin sigVocal

El printf esta fura del if

Indentar es poner las tabulaciones, vamos poner el código bonito.

Este es tú codigo sin indentar:

Código

#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main ()
{
char c;
do{
    if(esVocal(c)) sigVocal(c);
    printf("%c", c);
        }while(scanf("%c", &c)!= EOF);
 
return 0;
}
 
 
int esVocal (char c){
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}
 
char sigVocal (char c){
if(c=='a')c='e';
if(c=='e')c='i';
if(c=='i')c='o';
if(c=='o')c='u';
if(c=='u')c='a';
 
return c;
}

Este es el código indentado:

Código

#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main ()
{
    char c;
    do
    {
        if(esVocal(c)) sigVocal(c);
        printf("%c", c);
    }
    while(scanf("%c", &c)!= EOF);
 
    return 0;
}
 
int esVocal (char c)
{
    if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}
 
char sigVocal (char c)
{
    if(c=='a')c='e';
    if(c=='e')c='i';
    if(c=='i')c='o';
    if(c=='o')c='u';
    if(c=='u')c='a';
 
    return c;
}

En cuanto al fallo de tú código, scanf lee hasta un espacio o salto de línea. Para leer letra a letra puedes usar getch() de la librería conio.h, decirte que no es estandar pero cumple su propósito.

Otro error que tienes es:

Código

int esVocal (char c)
{
    if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return;
}// fin esVocal

No devuelves nada?

Bien he supuesto que el texto termina con un punto y te he codificado esto, y funciona bien

Código

#include <stdio.h>
 
void sigVocal ();
 
char c;
 
int main (){
	c=getchar();
	do{
		sigVocal();
		printf("%c", c);
		c=getchar();
	}// fin dentro del do
	while(c!= '.');
	printf("\n");
}//fin main
 
void sigVocal (){
	if(c=='a'){c='e';}
	else if(c=='e'){c='i';}
	else if(c=='i'){c='o';}
	else if(c=='o'){c='u';}
	else if(c=='u'){c='a';}
	else {c=c;}
}//fin sigVocal

exactamente, justo lo acabo de sacar yo tambien:

Código

#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main ()
{
char c;
do{
    if (esVocal(c)) printf("%c", sigVocal(c));
    else printf("%c", c);
 
        }while(scanf("%c", &c)!=EOF);
 
return 0;
}
int esVocal (char c){
 
return
(c=='a'||c=='e'||c=='i'||c=='o'||c=='u');
 
}
char sigVocal (char c){
 
switch(c){
case 'a': c='e'; break;
case 'e': c='i'; break;
case 'i': c='o'; break;
case 'o': c='u'; break;
case 'u': c='a'; break;
}
return (c);
}

;-) gracias

EDIT: no se como ponerlo bonito la verdad :(

Cita de: joanj94 en 3 Enero 2014, 17:35 pm

Bueno yo no se meter los colores, pero lo tabulo jajaja

PD: Un consejo los break, goto, y uso de librerias estilo conio.h evitalos si no son requeridos, lo digo porque si es un ejercicio de universidad o lo que sea, muchas veces te pueden penalizar por eso ;) (ademas de garantizar asi una maxima compatibilidad en todos los OS)

exactamente, de hecho todos mis examenes empiezan asi:

(http://i42.tinypic.com/2d0xpy0.png)

sin embargo por breaks y goto no me penalizan.

Cita de: SoyelRobert en 3 Enero 2014, 17:31 pm

exactamente, justo lo acabo de sacar yo tambien

Todavía hay un error en el programa, cuando se ejecuta la primera iteracion del bucle:

Código

int main()
{
   char c;
 
   do {
      if (esVocal(c))
         printf("%c", sigVocal(c));
      else 
         printf("%c", c);
   }while (scanf("%c", &c) != EOF);
 
   return 0;
}

El valor inicial de la variable "c" es no definido o basura, para procesar la variable solo después de almacenar en ella un valor se debe cambiar el bucle por uno con la condición al principio de este:

Código

int main()
{
   char c;
 
   while (scanf("%c", &c) != EOF)
      if (esVocal(c))
         printf("%c", sigVocal(c));
      else 
         printf("%c", c);
 
   return 0;
}

Cita de: joanj94 en 3 Enero 2014, 17:35 pm

PD: Un consejo los break, goto, y uso de librerias estilo conio.h evitalos si no son requeridos

En el caso del programa de SoyelRobert las sentencias "break;" en la función "sigVocal" son necesarias para tener el efecto deseado, si se eliminaran la función siempre retornaría 'a' como la siguiente vocal.

Un saludo

Cita de: SoyelRobert en 3 Enero 2014, 16:17 pm

.............................................
el asunto era crear un programa en C que al tu introducir un texto cambie todas las vocales que aparezcan por la siguiente vocal que le toque, no se trata de introducir una única vocal para que la modifique sino introducir un texto entero donde aparezcan blancos, saltos de linea, etc (cualquier caracter hasta que se acabe el fichero, EOF)

En todo caso hasta que se acabe la cadena o frase, es decir hasta \0. Lo otro sería para ficheros, que no es el caso, supongo.

Citar

Introduzca frase:
leosansan salio una tarde que llovia.

leusensen selou ane terdi qai lluvoe.

Código

/*hay que modificar la condicion del while*/
#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main (){
    int i;
    char c,frase[100];
    puts ("Introduzca frase:");
        scanf("%100[^\n]s",frase);
        while (getchar ()!='\n');
    do{
        if(esVocal(frase[i])!=0)
            frase[i]=sigVocal (frase[i]);
        i++;
    }while(frase[i]!= '\0');
    printf("\n%s\n",frase);
    return 0;
}
 
int esVocal (char c){
    int si_o_no=0;
    if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u')
        si_o_no=1;
    else si_o_no=0;
    return si_o_no;
}
 
char sigVocal (char c){
    if(c=='a')
        c='e';
    else if(c=='e')
        c='i';
    else if(c=='i')
        c='o';
    else if(c=='o')
        c='u';
    else if(c=='u')
        c='a';
    return c;
}

¡¡¡¡ Saluditos! ..... !!!!

(http://st.forocoches.com/foro/images/smilies/aaaaa.gif)

Cita de: leosansan en 3 Enero 2014, 18:28 pm

En todo caso hasta que se acabe la cadena o frase, es decir hasta \0. Lo otro sería para ficheros, que no es el caso, supongo.

sip jaja exactante, queria hacerlo para sentencias para la entrada estandar y para textos en caso de tratarse de ficheros:

Código

#include <stdio.h>
 
int esVocal (char c);
char sigVocal (char c);
 
int main ()
{
char c;
FILE *fe;
FILE *fs;
 
fe= fopen("C:\entrada.txt", "r");
fs= fopen("C:\salida.txt", "a");
 
if(fe == NULL) puts("error en apertura de fichero");
else {
    while (fscanf(fe,"%c", &c) != EOF)
        if (esVocal(c))
            fprintf(fs,"%c", sigVocal(c));
        else
            fprintf(fs,"%c", c);
 
}
   return 0;
}
int esVocal (char c){
 
 
if(c=='a'||c=='e'||c=='i'||c=='o'||c=='u') return 1;
else return 0;
 
}
char sigVocal (char c){
 
switch(c){
case 'a': c='e'; break;
case 'e': c='i'; break;
case 'i': c='o'; break;
case 'o': c='u'; break;
case 'u': c='a'; break;
}
return (c);
}

::)

Insisto SoyelRobert:

Código

int main ()
{
char c;
FILE *fe;
FILE *fs;
 
fe= fopen("C:\entrada.txt", "r");
fs= fopen("C:\salida.txt", "a");
 
//...
 
}

Deja 4 espacios después de una llave, asi se pueden identificar bloques fácilmente:

Código

int main ()
{
    char c;
    FILE *fe;
    FILE *fs;
 
    fe= fopen("C:\entrada.txt", "r");
    fs= fopen("C:\salida.txt", "a");
 
    //...
}

Eso en un editor de código es seleccionar las líneas y apretar TAB. O bien usar el plugin que indenta el código automaticamente (en el Codeblocks se llama "Source code formatter (AStyle)").

Estás prácticas son prácticamente obligadas para cualquier progamador, de hecho algunos lenguajes (como el python) no tienen llaves y se identifican los bloques con la indentación.

Cita de: amchacon en 3 Enero 2014, 19:55 pm

gracias :)

Cita de: SoyelRobert en 3 Enero 2014, 18:32 pm

sip jaja exactante, queria hacerlo para sentencias para la entrada estandar y para textos en caso de tratarse de ficheros:
...........................................

Que sirva de precedente, aclaren bien las preguntas que hagan y así no perdemos el tiempo en buscarte soluciones que no les servirán. Y esto lo digo de forma general, no por tí en particular.¡¡¡ Pero es que resulta frustante !!!, da ganas de dejarlo. ;)

¡¡¡¡ Saluditos! ..... !!!!

(http://st.forocoches.com/foro/images/smilies/aaaaa.gif)

Foro de elhacker.net

Programación => Programación C/C++ => Mensaje iniciado por: SoyelRobert en 3 Enero 2014, 14:56 pm