Pido Corrección: Conversor Informática

Iniciado por Fluflu, 29 Agosto 2010, 17:45 PM

0 Miembros y 1 Visitante están viendo este tema.

Imprimir
Ir Abajo Páginas1
Acciones del Usuario

Fluflu

29 Agosto 2010, 17:45 PM Ultima modificación: 30 Agosto 2010, 00:45 AM por Fluflu
Hola, estoy intentando aprender a programar y para poner en práctica unas cuantas cosas que he estudiado he hecho este programa que convierte cantidades de bits, megabits, bytes, kilobytes, megabytes, gigabytes y terabytes. Se que seguramente habrá código inútil y lo debo de haber hecho demasiado largo para tan poca cosa. Viendo el nivel que tengo ¿Me podríais dar algunas críticas constructivas? Remarco lo  de mi nivel, cosas que pueda entender. Bueno, supongo que leyéndolo se verá que está desarrollado en entorno Linux. Gracias :)

Código (c) [Seleccionar]
#include <stdio.h>
#include <string.h>

int main() {

char bit[] = "bits";
char megabit[] = "Mb";
char byte[] = "bytes";
char kbyte[] = "KB";
char megabyte[] = "MB";
char gigabyte[] = "GB";
char terabyte[] = "TB";

int opc1;
int opc2;

/* Guarda la opción escogida por el usuario sobre el tipo de dato en las variables opc1 y opc2 */
system("clear");
printf("\n1.bit\n2.megabit\n3.byte\n4.kbyte\n5.megabyte\n6.gigabyte\n7.terabyte\n\nIntroduce el tipo de dato origen: ");
scanf("%i", &opc1);

system("clear");
printf("\n1.bit\n2.megabit\n3.byte\n4.kbyte\n5.megabyte\n6.gigabyte\n7.terabyte\n\nIntroduce el tipo de dato destino: ");
scanf("%i", &opc2);

/* Variables para el número introducido por el usuario y el resultado del cálculo */
float num;
float result;

/* Variables para indicar el tipo de dato original y final. */
char ori[6];
char fin[6];

/* Cálculo del resultado */
system("clear");
printf("\nIntroduce la cifra: ");
scanf("%f", &num);

switch(opc1)
{
case 1:
strcpy(ori, bit);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num;
break;

case 2:
strcpy(fin, megabit);
result = num / 1000000;
break;

case 3:
strcpy(fin, byte);
result = num / 8;
break;

case 4:
strcpy(fin, kbyte);
result = num / 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num / 1024 / 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num / 1024 / 1024 / 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num / 1024 / 1024 / 1024 / 1024;
break;
}
break;

case 2:
strcpy(ori, megabit);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 1000000;
break;

case 2:
strcpy(fin, megabit);
result = num;
break;

case 3:
strcpy(fin, byte);
result = num * 1000000 / 8;
break;

case 4:
strcpy(fin, kbyte);
result = num * 1000000 / 8 / 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num * 1000000 / 8 / 1024 / 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num * 1000000 / 8 / 1024 / 1024 / 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num * 1000000 / 8 / 1024 / 1024 / 1024 / 1024;
break;
}
break;

case 3:
strcpy(ori, byte);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 8;
break;

case 2:
strcpy(fin, megabit);
result = num * 8 / 1000000  ;
break;

case 3:
strcpy(fin, byte);
result = num;
break;

case 4:
strcpy(fin, kbyte);
result = num / 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num / 1024 / 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num / 1024 / 1024 / 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num / 1024 / 1024 / 1024 / 1024;
break;
}
break;

   case 4:
strcpy(ori, kbyte);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 1024 * 8;
break;

case 2:
strcpy(fin, megabit);
result = num * 1024 * 8 / 1000000  ;
break;

case 3:
strcpy(fin, byte);
result = num *1024;
break;

case 4:
strcpy(fin, kbyte);
result = num;
break;

case 5:
strcpy(fin, megabyte);
result = num / 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num / 1024 / 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num / 1024 / 1024 / 1024;
break;
}
break;

case 5:
strcpy(ori, megabyte);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 1024 * 1024 * 8;
break;

case 2:
strcpy(fin, megabit);
result = num * 1024 * 1024 * 8 / 1000000  ;
break;

case 3:
strcpy(fin, byte);
result = num * 1024 * 1024;
break;

case 4:
strcpy(fin, kbyte);
result = num * 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num;
break;

case 6:
strcpy(fin, gigabyte);
result = num / 1024 / 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num / 1024 / 1024 / 1024;
break;
}
break;

case 6:
strcpy(ori, gigabyte);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 1024 * 1024 * 1024 * 8;
break;

case 2:
strcpy(fin, megabit);
result = num * 1024 * 1024 * 1024 * 8 / 1000000  ;
break;

case 3:
strcpy(fin, byte);
result = num * 1024 * 1024 * 1024;
break;

case 4:
strcpy(fin, kbyte);
result = num * 1024 * 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num * 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num;
break;

case 7:
strcpy(fin, terabyte);
result = num / 1024;
break;
}
break;

case 7:
strcpy(ori, terabyte);
switch(opc2)
{
case 1:
strcpy(fin, bit);
result = num * 1024 * 1024 * 1024 * 1024 * 8;
break;

case 2:
strcpy(fin, megabit);
result = num * 1024 * 1024 * 1024 * 1024 * 8 / 1000000  ;
break;

case 3:
strcpy(fin, byte);
result = num * 1024 * 1024 * 1024 * 1024;
break;

case 4:
strcpy(fin, kbyte);
result = num * 1024 * 1024 * 1024;
break;

case 5:
strcpy(fin, megabyte);
result = num * 1024 * 1024;
break;

case 6:
strcpy(fin, gigabyte);
result = num * 1024;
break;

case 7:
strcpy(fin, terabyte);
result = num;
break;
}
break;
}

/* Impresión en pantalla del resultado. */
system("clear");
printf("\n");
printf("\n%f %s es igual a: %f %s\n", num, ori, result, fin);
printf("\n");

return 0;

}
La puerta más segura es la que no necesita ser cerrada...

pucheto

#1
30 Agosto 2010, 03:50 AM
Te tiro una pista para q quede mas lindo, podes hacerlo como ejercicio.

   char bit[] = "bits";
   char megabit[] = "Mb";
   char byte[] = "bytes";
   char kbyte[] = "KB";
   char megabyte[] = "MB";
   char gigabyte[] = "GB";
   char terabyte[] = "TB";

Todas esas variables que declaraste, comparten tipo, y ademas podemos decir q comparten "función u objetivo" (no se me ocurre una manera prolija de decirlo).
Proba metiendolas en un array, de manera q la opcion "i" te manda a la unidad en la i-esima posicion. Eso simplificaría el switch, (algo similar también para la conversión).

clodan

#2
30 Agosto 2010, 17:23 PM
mmm yo no le veo nada mal, eso si, tal como dijo nuestro compañero, las variables que tengan igual tipo es mejor agruparlas en 1 sola linea, pero eso depende de tus gustos...

mmm despues... ahh... dejo aca para que los que tengan windows puedan tambien ver los problemas sin inconvenientes...

mmm no se en linux, pero al final tenes qe poner un system pause, o getchar () para que muestre el resultado, sino el programa se cierra y por la velocidad a la que pasa todo es imposible ver el resultado, pero el programa funciona barbaro...

si queres hacer para ambas versiones, windows y linux, lo que podes hacer es una funcion que diga

if system("cls")=error
system("clear")
else
system("cls")

el problema es que justo ahora no me acuerdo como era para escribir error de comando...

para windows:
Código (c) [Seleccionar]
#include <stdio.h>
#include <string.h>
int main() {
      char bit[] = "bits";
      char megabit[] = "Mb";
      char byte[] = "bytes";
      char kbyte[] = "KB";
      char megabyte[] = "MB";
      char gigabyte[] = "GB";
      char terabyte[] = "TB";
      int opc1;
      int opc2;
/* Guarda la opción escogida por el usuario sobre el tipo de dato en las variables opc1 y opc2 */

system("cls");
printf("\n1.bit\n2.megabit\n3.byte\n4.kbyte\n5.megabyte\n6.gigabyte\n7.terabyte\n\nIntroduce el tipo de dato origen: ");
scanf("%i", &opc1);
system("cls");
printf("\n1.bit\n2.megabit\n3.byte\n4.kbyte\n5.megabyte\n6.gigabyte\n7.terabyte\n\nIntroduce el tipo de dato destino: ");
scanf("%i", &opc2);

/* Variables para el número introducido por el usuario y el resultado del cálculo */

float num;
float result;

/* Variables para indicar el tipo de dato original y final. */

char ori[6];
char fin[6];

/* Cálculo del resultado */

system("cls");
printf("\nIntroduce la cifra: ");
scanf("%f", &num);

switch(opc1) {
              case 1:
                   strcpy(ori, bit);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num / 1000000;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num / 8;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num / 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num / 1024 / 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num / 1024 / 1024 / 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num / 1024 / 1024 / 1024 / 1024;
                                      break;
                                      }
                   break;
              case 2:
                   strcpy(ori, megabit);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 1000000;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num * 1000000 / 8;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num * 1000000 / 8 / 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num * 1000000 / 8 / 1024 / 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num * 1000000 / 8 / 1024 / 1024 / 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num * 1000000 / 8 / 1024 / 1024 / 1024 / 1024;
                                 break;
                                 }
                   break;
              case 3:
                   strcpy(ori, byte);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 8;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num * 8 / 1000000  ;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num / 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num / 1024 / 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num / 1024 / 1024 / 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num / 1024 / 1024 / 1024 / 1024;
                                      break;
                                      }
                   break;    
              case 4:
                   strcpy(ori, kbyte);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 1024 * 8;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num * 1024 * 8 / 1000000  ;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num *1024;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num / 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num / 1024 / 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num / 1024 / 1024 / 1024;
                                      break;
                                      }
                   break;
              case 5:
                   strcpy(ori, megabyte);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 1024 * 1024 * 8;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num * 1024 * 1024 * 8 / 1000000  ;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num * 1024 * 1024;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num * 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num / 1024 / 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num / 1024 / 1024 / 1024;
                                      break;
                                      }
                   break;
              case 6:
                   strcpy(ori, gigabyte);
                   switch(opc2) {
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 1024 * 1024 * 1024 * 8;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num * 1024 * 1024 * 1024 * 8 / 1000000  ;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num * 1024 * 1024 * 1024;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num * 1024 * 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num * 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num / 1024;
                                      break;
                                      }
                   break;
              case 7:
                   strcpy(ori, terabyte);
                   switch(opc2) {                   
                                 case 1:
                                      strcpy(fin, bit);
                                      result = num * 1024 * 1024 * 1024 * 1024 * 8;
                                      break;
                                 case 2:
                                      strcpy(fin, megabit);
                                      result = num * 1024 * 1024 * 1024 * 1024 * 8 / 1000000  ;
                                      break;
                                 case 3:
                                      strcpy(fin, byte);
                                      result = num * 1024 * 1024 * 1024 * 1024;
                                      break;
                                 case 4:
                                      strcpy(fin, kbyte);
                                      result = num * 1024 * 1024 * 1024;
                                      break;
                                 case 5:
                                      strcpy(fin, megabyte);
                                      result = num * 1024 * 1024;
                                      break;
                                 case 6:
                                      strcpy(fin, gigabyte);
                                      result = num * 1024;
                                      break;
                                 case 7:
                                      strcpy(fin, terabyte);
                                      result = num;
                                      break;
                                      }
                   break;
              }
/* Impresión en pantalla del resultado. */
system("cls");
printf("\n");
printf("\n%f %s es igual a: %f %s\n", num, ori, result, fin);
printf("\n");
system("PAUSE");
return 0;

}

Fluflu

#3
31 Agosto 2010, 01:45 AM
Gracias por responder, os haré caso con lo del array, respecto a lo de system("pause"); al hacerlo en linux no lo necesito (al menos de momento) porque ejecuto los programas desde la terminal, de forma que el resultado queda visible, pero lo tendré en cuenta si programo en windows y no me interesa abrirlo desde la consola.

Y gracias también por esto:

Citarif system("cls")=error
system("clear")
else
system("cls")

no se me había ocurrido. ;-)

Si alguien quiere seguir añadiendo cosas lo agradeceré también.
La puerta más segura es la que no necesita ser cerrada...

clodan

#4
31 Agosto 2010, 02:38 AM
Cita de: Fluflu en 31 Agosto 2010, 01:45 AM
Gracias por responder, os haré caso con lo del array, respecto a lo de system("pause"); al hacerlo en linux no lo necesito (al menos de momento) porque ejecuto los programas desde la terminal, de forma que el resultado queda visible, pero lo tendré en cuenta si programo en windows y no me interesa abrirlo desde la consola.

Y gracias también por esto:

Citarif system("cls")=error
system("clear")
else
system("cls")

no se me había ocurrido. ;-)

Si alguien quiere seguir añadiendo cosas lo agradeceré también.

igual mira qe ese codigo no existe ehh!! lo vi aca una vez pero no me acuerdo como se ponia cuando un comando da error... por eso...

carlitos_jajajajaja

#5
31 Agosto 2010, 05:07 AM
Una forma mas sencilla de hacerlo es  usando macros...


Despues de los includes, colocas esto
Código (c) [Seleccionar]

#ifdef WINDOWS
     #define CLEAR_COMMAND "cls"
#elif defined(UNIX)
     #define CLEAR_COMMAND "clear"
#else
     #error "No programas ni en Windows ni en Linux?? Eres raro!!!"
#endif


Las macros WINDOWS y UNIX las define normalmente tu compilador de preferencia, no necesariamente con esos nombres, pero averigua q nombres usa

Luego, a la hora de hacer la llamada al sistema para limpiar pantalla:
Código (c) [Seleccionar]

system(CLEAR_COMMAND);


Y tienes codigo limpio portable y elegante :D




Merodeo los foros, esperando el momento de decir algo inteligente...

Se necesita paciencia

clodan

#6
31 Agosto 2010, 05:53 AM
Cita de: carlitos_jajajajaja en 31 Agosto 2010, 05:07 AM
Una forma mas sencilla de hacerlo es  usando macros...


Despues de los includes, colocas esto
Código (c) [Seleccionar]

#ifdef WINDOWS
     #define CLEAR_COMMAND "cls"
#elif defined(UNIX)
     #define CLEAR_COMMAND "clear"
#else
     #error "No programas ni en Windows ni en Linux?? Eres raro!!!"
#endif


Las macros WINDOWS y UNIX las define normalmente tu compilador de preferencia, no necesariamente con esos nombres, pero averigua q nombres usa

Luego, a la hora de hacer la llamada al sistema para limpiar pantalla:
Código (c) [Seleccionar]

system(CLEAR_COMMAND);


Y tienes codigo limpio portable y elegante :D



CLAAAA!!!! eso es lo que buscaba!!! jaajaja gracias!!!
Imprimir
Ir Arriba Páginas1
Acciones del Usuario