lio con programa

[Amagoia686]

tengo que hacer un programa para clase y no me termina de salir. Consiste en meter una palabra y que a continuación me cambie las posiciones pares a mayúsculas y las impares a minúsculas y no me sale. Os dejo lo que he hecho hasta ahora que debe de estar mal con el cacao que tengo ahora mismo.
Saludos y grasias

Código:

#include <conio.h>
#include <stdio.h>
#include <conio.h>
#include <string.h>
#include <ctype.h>

void pasar_a_mayus(char cad[]);  /*prototipo funcion que me hace el cambio a mayúsculas o a minúsculas*/

void main(void)
{
char txt[20];


printf("\nIntroduce una palabra: ");
flushall();
gets(txt);
pasar_a_mayus(txt);
clrscr();
printf("\n\nLa Nueva palabra es %s",txt);
getch();
}

void pasar_a_mayus(char cad[])
{
 int i,j,longi;
 longi=strlen(cad);

 for(i=0;i<=longi;i++)
  {
   cad[i+2]=tolower(cad[i+2]);
   for(j=1; j<=longi; i++);
    {
    cad[j+2]=toupper(cad[i+2]);
    }
  }
 return;
}

[Instru]

Bueno, hisiste un relajo para algo tan simple.

Mira, te recomiendo que uses punteros para tu funcion void pasar_a_mayus(char cad[]); para que asi pases por referencia.

Por otro lado.

Esta bien tu plantemiento de esa funcion solo que ahi te va unas correcciones.

EN el primer for dale algo como esto:

for(i=0; i<longi; i=i+2)
{
cad[i]=tolower(cad[i]);
cad[i+1]=toupper(cad[i+1]);
}
Y con eso ya te debe convertir sin hacer tanta cosa como pusiste.

Saludos

[TolaWare]

ademas, dentro del bucle, nunca te convierte la primera letra de la cadena. Ademas corres el peligro de que trates de acceder a una posicion del vector que no existe.

[MaxExtreme]

Iniciado por TolaWare

ademas, dentro del bucle, nunca te convierte la primera letra de la cadena. Ademas corres el peligro de que trates de acceder a una posicion del vector que no existe.

Ya sabemos que estaba mal... No se lo restrieges ;)

PD:No es vector, es array.


Si además quieres acortarlo, "i=i+2" puedes poner "i+=2"

[spike_jr]

for(i=0; i<longi; i=i+2)
{
cad[i]=tolower(cad[i]);
cad[i+1]=toupper(cad[i+1]);
}

A mi parecer esto no puede funcionar, ya que accedes a una posicion más del array que no es correcta.

[Instru]

Bueno, con un poco de condiciones para no acceder a esos de espacios de memoria no preparada quedaria bastante bien.

SAludos

[MaxExtreme]

Es fácil: En vez de "i<longi", "(i+2)<longi"

[spike_jr]

Como no me gusta hacer operaciones dentro de los [] te propongo esta solucion.

for(i=0; i<longi; i++)
{
if(i%2==0)cad[i]=tolower(cad[i]);
else cad[i]=toupper(cad[i]);
}

[MaxExtreme]

Realmente esas operaciones se transforman en inmediatas en assembler, por lo que sí es recomendable hacerlas. En cambio un módulo al menos es una operación separada.

Por otro lado, es mejor:

Código:

for(i=0; i<longi; i++)
	cad[i] = (i%2==0) ? tolower(cad[i]) : toupper(cad[i]);

[spike_jr]

Me puedes explicar esa sintaxis?

[MaxExtreme]

Deberías conocerla, es parte del estándar C. Aquí está explicado:

http://www.zator.com/Cpp/E4_9_6.htm

No es más que el operador condicional (similar a un if), pero que permite hacer cosas como las que has visto arriba. Tienden a optimizarse mejor.

[spike_jr]

Ok. Pero es estandar de C, no de C++ verdad?

[spike_jr]

Disculpa no habia leido bien :p, es de C++

[spike_jr]

Me sonaba de algo q lei, pero no le di importancia a su utilidad. De hecho no lo he visto nunca en codigos a excepcion de ejemplos en libros.

[MaxExtreme]

Iniciado por spike_jr

Disculpa no habia leido bien :p, es de C++

No... es de C y de C++. Que el manual que te pase sea de C++ da igual, simplemente era la primera referencia en Google.

[MaxExtreme]

Iniciado por spike_jr

Me sonaba de algo q lei, pero no le di importancia a su utilidad. De hecho no lo he visto nunca en codigos a excepcion de ejemplos en libros.

Será porque no has mirado códigos suficientes ;)

[spike_jr]

nunca se ven los codigos suficientes... jeje