Foros del Web - Ver Mensaje Individual

antoniog · #2 (**permalink**) 17/09/2008, 20:58

Hola,

Seria bueno que pusieras el error que te marca.

Lo que si es que el error te lo marca por un erro con las llaves {}, te equivocaste por ahi.

Notas para tu codigo:

Estas dejando fuera valores de compra como 1000.5 (Compra es de tipo float), debes poner (Compra>=1000) en lugar de (Compra>=1001)!
podrias ahorrarte condiciones en los ifs que pones
por ejemplo
Código:
```
if (Compra<500)
   {
   }
else
   {
   if ((Compra>=500)&&(Compra<=1000))
   :
   }
```
Nota que la parte en rojo ya no tiene porque ir, ya que si no cumplio la condicion del primer if es porque Compra es >= 500, asi que tu codigo quedaria algo como asi:
Código:
```
if (Compra<500)
   {
   }
else
   {
   //aqui ya sabemos que compra es >=500
   if (Compra<=1000)
   :
   }
```

Poner codigo con tanto if y else confunde (como en tu caso), podrias tener una variable auxiliar que te indique el tipo de descuento y ya luego usar un switch para ejecutar el codigo correspondiente al descuento, o sea, algo como asi:

Código:

int tipodescuento;

if(condicion1)
   tipodescuento=0;
else
   {
   if (condicion2)
      tipodescuento=1;
    else
       {
       if (condicion2)
          tipodescuento=2;
        else
       :
       :
       }
   }

y luego el switch:

Código:

switch(tipodescuento){
case 0://codigo para el descuento 0
            break;
case 1://codigo para el descuento 1
            break;
        :
    }

pienso que asi quedaria mas claro el codigo.

Si solo quieres quitar ese error, pues solo busca donde te equivocaste al poner las llaves {} en tu codigo, te sugiero que indentes el codigo para mas claridad.

Saludos