как я могу сделать этот код более оптимизированным

как я могу оптимизировать этот код? Я не хочу, чтобы был аргумент case, есть ли способ, которым я могу улучшить этот код?как я могу сделать этот код более оптимизированным

protected void ddlFilterResultBy_SelectedIndexChanged(object sender, EventArgs e) 
{ 
    string selVal = ddlFilterResultBy.SelectedValue.ToString().ToLower(); 

    switch (selVal) 
    { 
     case "date": 
      pnlDate.Visible = true; 
      pnlSubject.Visible = false; 
      pnlofficer.Visible = false; 
      pnlCIA.Visible = false; 
      pnlMedia.Visible = false; 
      pnlStatus.Visible = false;      
      break; 

     case "subject": 
      pnlDate.Visible = false; 
      pnlSubject.Visible = true; 
      pnlofficer.Visible = false; 
      pnlCIA.Visible = false; 
      pnlMedia.Visible = false; 
      pnlStatus.Visible = false; 
      break; 

     case "officer": 
      pnlDate.Visible = false; 
      pnlSubject.Visible = false; 
      pnlofficer.Visible = true; 
      pnlCIA.Visible = false; 
      pnlMedia.Visible = false; 
      pnlStatus.Visible = false; 
      break; 

     case "status": 
      pnlDate.Visible = false; 
      pnlSubject.Visible = false; 
      pnlofficer.Visible = false; 
      pnlCIA.Visible = false; 
      pnlMedia.Visible = false; 
      pnlStatus.Visible = true; 
      break; 

     default: 
      pnlDate.Visible = false; 
      pnlSubject.Visible = false; 
      pnlofficer.Visible = false; 
      pnlCIA.Visible = false; 
      pnlMedia.Visible = false; 
      pnlStatus.Visible = false; 
      break; 
    } 
}

источник

2010-08-03 Nick Kahn

Простой. Вы только когда-либо делает один элемент видимым в зависимости от варианта корпуса, так что просто установить видимость следующим образом:

pnlDate.Visible = (selVal == "date"); 
pnlSubject.Visible = (selVal == "subject"); 
pnlofficer.Visible = (selVal == "officer"); 
pnlCIA.Visible = false; 
pnlMedia.Visible = false; 
pnlStatus.Visible = (selVal == "status");

Это лучше, чем установка все, чтобы visible = false; и только показывая нужный вам пункт, поскольку это содержит его все в 6 строк кода для фактической настройки видимости.

источник

2010-08-03 14:03:06 GenericTypeTea

Возможно, я бы добавил скобки, чтобы сделать его более читаемым. –

+1: простой и понятный – RedFilter

@ray dey - Fairy noof. Готово. – GenericTypeTea

protected void ddlFilterResultBy_SelectedIndexChanged(object sender, EventArgs e) 
      { 
       string selVal = ddlFilterResultBy.SelectedValue.ToString().ToLower(); 
         pnlDate.Visible = false; 
         pnlSubject.Visible = false; 
         pnlofficer.Visible = false; 
         pnlCIA.Visible = false; 
         pnlMedia.Visible = false; 
         pnlStatus.Visible = false; 
       switch (selVal) 
       { 
        case "date": 
         pnlDate.Visible = true;      
         break; 

        case "subject": 
         pnlSubject.Visible = true; 
         break; 

        case "officer": 
         pnlofficer.Visible = true; 
         break; 
        case "status": 
         pnlStatus.Visible = true; 
         break; 
       } 

      }

источник

2010-08-03 14:03:28 azram19

Другой способ:

// set everything to false 
Dictionary<string, type> d = new Dictionary<string, type>() 
{ 
    {"date", pnlDate}, 
    {"subject", plnSubject}, 
    {"officer", plnOfficer}, 
    {"status", plnStatus} 
}; 

d[selVal].Visible = true;

источник

2010-08-03 14:07:52 NullUserException

+1. Очень приятно .. – Steven

+1 Хорошая мысль ... –

Что такое downvote? Я знаю, что код не делает * точно * то же самое, что и оригинальное; это просто показать эту идею. – NullUserException

Вы можете сделать это:

protected void ddlFilterResultBy_SelectedIndexChanged(object sender, EventArgs e) { 
    string selVal = ddlFilterResultBy.SelectedValue.ToString().ToLower(); 

    pnlDate.Visible = (selVal == "date"); 
    pnlSubject.Visible = (selVal == "subject"); 
    pnlofficer.Visible = (selVal == "officer"); 
    pnlCIA.Visible = (selVal == "cia"); 
    pnlMedia.Visible = (selVal == "media"); 
    pnlStatus.Visible = (selVal == "status"); 
}

Или это один, в то время как менее читаемым, было бы более точным:

protected void ddlFilterResultBy_SelectedIndexChanged(object sender, EventArgs e) { 
    string selVal = ddlFilterResultBy.SelectedValue.ToString(); 

    pnlDate.Visible = String.Equals(selVal, "date", StringComparison.OrdinalIgnoreCase); 
    pnlSubject.Visible = String.Equals(selVal, "subject", StringComparison.OrdinalIgnoreCase); 
    pnlofficer.Visible = String.Equals(selVal, "officer", StringComparison.OrdinalIgnoreCase); 
    pnlCIA.Visible = String.Equals(selVal, "cia", StringComparison.OrdinalIgnoreCase); 
    pnlMedia.Visible = String.Equals(selVal, "media", StringComparison.OrdinalIgnoreCase); 
    pnlStatus.Visible = String.Equals(selVal, "status", StringComparison.OrdinalIgnoreCase); 
}

источник

2010-08-03 14:08:10 mattmc3

+1 спасибо ....... –

Там это разница между оптимизацией и улучшением reada Билити. Так что, я думаю, вы смотрите на улучшение читаемости, поскольку оптимизация на самом деле здесь не нужна. Здесь нет никакого алгоритма, который вы можете настроить, чтобы сделать это быстрее в течение значительного времени.

Ответ 1 и 2 будет мой выбор улучшения читаемости

источник

2010-08-03 14:22:10 Ash

Я думаю, что вам нужно TabControl здесь ..

Просто скрыть вкладки, если вам не нравится, и те, и индекс страницы с помощью код.

Преимущество в том, что вы сможете редактировать и просматривать графический интерфейс в режиме разработки.

Это будет намного проще в обслуживании.

источник

2010-08-03 14:32:26

Я использую jquery с div, но у меня есть проблемы, вы можете увидеть вопрос, который я разместил здесь http://stackoverflow.com/questions/3367166/jquery-div-disappears-on- postback, и я отбрасываю идею использования jquery/div –

как я могу сделать этот код более оптимизированным

ответ

Смежные вопросы