2

Quisiera saber una alternativa para evitar el uso execesivo de IF dentro de este código

if (paq.Lunes)
    this.lu.Attributes.Add("class", "select");
if (paq.Martes)
    this.ma.Attributes.Add("class", "select");
if (paq.Miercoles)
    this.mi.Attributes.Add("class", "select");
if (paq.Jueves)
    this.jue.Attributes.Add("class", "select");
if (paq.Viernes)
    this.vie.Attributes.Add("class", "select");
if (paq.Sabado)
    this.sa.Attributes.Add("class", "select");
if (paq.Domingo)
    this.dom.Attributes.Add("class", "select");

if (!paq.aplicaTransporte)
    this.transporte.Visible = false;
if (!paq.aplicaEntrada)
    this.entrada.Visible = false;
if (!paq.aplicaGuia)
    this.guia.Visible = false;
if (!paq.aplicaComida)
    this.comida.Visible = false;
if (!paq.aplicaSeguro)
    this.seguro.Visible = false;
if (!paq.aplicaTransporteMaritimo)
    this.maritimo.Visible = false;
if (!paq.aplicaBebidasAlcoholicas)
    this.alcohol.Visible = false;

Esta es la clase

public class PaqueteTour
{
public Boolean Lunes { get; set; }
    public Boolean Martes { get; set; }
    public Boolean Miercoles { get; set; }
    public Boolean Jueves { get; set; }
    public Boolean Viernes { get; set; }
    public Boolean Sabado { get; set; }
    public Boolean Domingo { get; set; }
{
        //
        // TODO: Agregar aquí la lógica del constructor
        //
        precio = new preciotourlistado();
        promocion = new PromoHotel();
    }
}

Estoy trabajando en .NET C#, he pensado en usar funciones como el .map o alguno de esos pero no encuentro la forma de implementarlo, pues esta información viene de una API.

3
  • Si Lunes, Martes, etc es un enum, puedes usar un switch.
    – Luis
    el 8 oct. 2018 a las 2:04
  • Pregunta: De que tipo son las propiedades: this.lun, this.mar , etc. Y en el segundo grupo de ifs que pasa cuando si se cumpel la condición, el valor de la propiedad a actualizar debería ser true ? el 8 oct. 2018 a las 5:40
  • las propiedades this.lu etc, son elementos del html el 8 oct. 2018 a las 15:29

2 respuestas 2

5

Una alternativa a los if sucesivos es el condicional switch.

Por lo que veo en tu código paq hace referencia a un enum, dado el caso, podrías hacer lo siguiente utilizando switch

var paq = TuEnum.Lunes;
switch (paq)
    {          
        case TuEnum.Lunes:
        //Aquí iría la lógica en caso de ser lunes
        break;

        case TuEnum.Martes:
        //Aquí en caso de ser martes
        break;

        default:
        //Por último aquí iría la lógica si ninguno de los otros casos se cumplió.
        break;
    }

De todos modos, te recomiendo que leas la documentación de Microsoft, que no es muy larga, usa varios ejemplos muy fáciles de entender.


En cuanto a la segunda parte de tu código, donde evaluas, por ejemplo:

if (!paq.aplicaTransporte)
this.transporte.Visible = false;

Estas utilizando un if completamente innecesario, sería lo mismo escribir this.transporte.Visible = paq.aplicaTransporte;


EDIT: Para usar enum en lugar de Boolean deberías hacer lo siguiente:

Modificar la definición de tu clase por algo como lo siguiente

public class PaqueteTour
{
     public Dias DiaDeLaSemana;
     public enum Dias { Lunes,Martes,Miercoles,Jueves,Viernes,Sabado,Domingo };
}

Luego para setearlo, lo harías de la siguiente manera: DiaDeLaSemana = Dias.Martes por ejemplo.

Por último, al momento que chequear su valor, podrías hacerlo con el switch como explique más arriba, o también con un if, que sería de la siguiente manera

if(DiaDeLaSemana == Dias.Martes)
{
    //Lógica si es martes
}
8
  • Los atributos Lunes, Martes, Miercoles, etc, son atributos por separados, acabo de aactuliza el post para mostrar la estructura de mi clase el 8 oct. 2018 a las 14:48
  • @ErnestoEmmanuelYahLopez Hay algo que te impida definir un enum? me parece muchísimo más lógico el 8 oct. 2018 a las 15:06
  • No de hecho me parece una buena idea tu respuesta, solo queria saber si aun con esa estrctura de la clase yo puedo crear un enum? el 8 oct. 2018 a las 15:09
  • Cuando inicio un enum, todo el codigo que esta debado de, me marca como error, ya cheque la sintaxis pero aun me da el mismo error el 8 oct. 2018 a las 15:22
  • 1
    Ahora si ya entendí mejor gracias hermano el 8 oct. 2018 a las 15:57
0

Podrias usar bitwise para guardar multi seleccion en un solo campo.

No se si conoces como funciona este concepto, pero basicamente se basa en guardar los valores 1,2,4,8,16 etc

Enum, Flags and bitwise operators

Create Enumeration as Bit Flags

entonces podrias tener un enum para la semana como ser

enum SemanaEnum{
   Lunes = 1,
   Marte = 2,
   Miercoles = 4,
   Jueves = 8,
   Viernes = 16,
   Sabado = 32,
   Domingo = 128,
}

entonces usando el OR logico puede unir varios de estos valores en un solo campo, no necesitas una propiedad dia de la semana

SemanaEnum result = SemanaEnum.Martes | SemanaEnum.Jueves;

La idea es que despues cuando evalues puedas determinar si esta esa seleccion o no

if ((paq.Semana & SemanaEnum.Marte) == SemanaEnum.Marte){
   //codigo
}

por supuesto viendolo asi no evita escribir lo if, pero si un Dictionary como ser

Dictionary<SemanaEnum, object> lista = new Dictionary<SemanaEnum, object>(){
   {SemanaEnum.Lunes, this.lu},
   {SemanaEnum.Marte, this.ma},
    //resto
}

foreach(var item in lista){
  if ((paq.Semana & item.Key) == item.Key){
      item.Value.Attributes.Add("class", "select");
   }
}

Asi evitas los if y haces tu codigo ams dinamico

Nota: Puse el value del Dictionary como object porque no sabia de que tipo se trata, pero lo correcto es que pongas el tipo que corresponda

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.