Problema con mi codigo en c

Ratchetmdt512Hace 9 años9

[b]Ok revisare el codigo entonces, gracias   [/b]

Ratchetmdt512Hace 9 años10

[b]En principio uso b para guardar a al final del trayecto para compararla con el siguiente para saber si he de sumar o restar.[/b]

Neo35871706Hace 9 años11

No quiero criticarte, pero tú código es bastante malo (obviando los fallos que pueda haber), entre las funciones globales innecesarias, tener que escanear los caracteres uno a uno, el medio millón de if-else, que se pueden hacer con un switch o un bucle si te lo montas bien, y la cantidad de código repetido.....

Me he montado un código en un par de minutos que debería funcionar correctamente:


#include  

// Sí, lo he escrito en inglés, algo que debería hacerse siempre para que el código lo entienda cualquier persona

int main(void) {
  
  char strScales[] = "IVXLCDM";
  int  scalesValues[] = {1,5,10,50,100,500,1000};
  
  printf("Introduce un numero en romano: ");
  char strNumber[20];
  scanf("%s", strNumber);
  
  int i = 0, value = 0, lastValue = 0;
  
  while(strNumber[i] != 0) {
    
    int scale = 0;
    
    // Check scale value of the selected char
    while(strNumber[i] != strScales[scale]) {
      if(strScales[scale++] == 0) { 
        printf("Invalid number");
        return 0;
      }
    }
    
    value += scalesValues[scale];
    
    if(i++) 
      if(lastValue < scalesValues[scale])
        value -= lastValue*2;
    
    lastValue = scalesValues[scale];
  }
  
  printf("%d", value);
  
  getchar();
  return 0;
}




Por cierto, el foro de 3Djuegos no deja poner barra invertida y se me come los espacios.
EDITADO EL 15-11-2014 / 05:21 (EDITADO 1 VEZ)

CyberBoy1257Hace 9 años12

@Neo3587 Parece que no leíste lo que dijo. No puede usar arrays (arreglos).

@Ratchetmdt Está bien el uso que le das a "b", pero al inicio de todo, "b" aun no almacena nada y asumes que tiene valor cero. Creo que los compiladores de ahora tienen eso solucionado pero antes, algunos reconocían basura si no lo inicializabas, es por eso que se recomienda que lo inicialices en cero en la declaración de variables. En todo caso, tampoco es que sea obligatorio si te funciona bien en tu compilador.

Neo35871706Hace 9 años13

@CyberBoy 
Mostrar cita
@Neo3587 Parece que no leíste lo que dijo. No puede usar arrays (arreglos).

@Ratchetmdt Está bien el uso que le das a "b", pero al inicio de todo, "b" aun no almacena nada y asumes que tiene valor cero. Creo que los compiladores de ahora tienen eso solucionado pero antes, algunos reconocían basura si no lo inicializabas, es por eso que se recomienda que lo inicialices en cero en la declaración de variables. En todo caso, tampoco es que sea obligatorio si te funciona bien en tu compilador.
Cierto, es que empecé a leer un poco su código y acabé pasando del resto ya que me parecía bastante ilegible, un asco que los profesores limiten la creatividad del alumno de esta forma.

Tactica2302Hace 9 años14

@Neo3587 
Mostrar cita
Por cierto, el foro de 3Djuegos no deja poner barra invertida y se me come los espacios.
[code]
/*
Ejemplos de printf()

printf("Muy buenas \\n");

printf("Un espacio: Dos espacios:  Tres espacios:   Cuatro espacios:   \\n");
*/
[/code]

Dejar te deja, lo que pasa es que tienes que poner dos juntas porque en cada edición o previsualización desaparece una.

Y por cierto, tu programa también es mejorable, lo de andar declarando e inicializando variables en medio del código es muy feo 

  Cita:
Mostrar cita
un asco que los profesores limiten la creatividad del alumno de esta forma.
Se llama aprender a hacer las cosas conforme a unos requerimientos, hay toneladas de ejercicios similares en los mejores libros de C. El ejercicio que le piden está chupado, sólo necesita crear una función en lugar de repetir el mismo código 100 veces. Yo lo intenté pero ya digo, al dejarlo de lado ya me he olvidado 

@Yasinator

Espero que seáis conscientes de este fallo y se corrija alguna vez, es especialmente grave que [code] tenga el mismo fallo y no sea un equivalente directo del PRE de HTML.
EDITADO EL 15-11-2014 / 13:41 (EDITADO 1 VEZ)

Tactica2302Hace 9 años15

@CyberBoy 
Mostrar cita
@Ratchetmdt Está bien el uso que le das a "b", pero al inicio de todo, "b" aun no almacena nada y asumes que tiene valor cero. Creo que los compiladores de ahora tienen eso solucionado pero antes, algunos reconocían basura si no lo inicializabas, es por eso que se recomienda que lo inicialices en cero en la declaración de variables. En todo caso, tampoco es que sea obligatorio si te funciona bien en tu compilador.
Que yo sepa GCC lo sigue haciendo, y en mi opinión es una buena idea. Quien no inicializa variables como es debido se merece sufrir todos los tormentos