Код рефакторинга для удаления if else в отчетах - PullRequest
0 голосов
/ 11 сентября 2011

Мы унаследовали систему, которая генерирует много отчетов.Есть комбинированные списки для выбора различных опций.После выбора различных параметров в выпадающих списках (по меньшей мере 5 комбинированных списков на форму) отчеты создаются путем вызова различных хранимых процедур.

    if cmbCategory.text = "Category A" and cmbProdType = "Kids" 
    then 
        <code for calling stored proc **SPA** and populating controls on winform> 
    else if cmbCategory.text = "Category A" and cmbProdType = "Mens" 
    then 
        <code for calling stored proc **SPB** and populating controls on winform> 
    else if cmbCategory.text = "Category B" and cmbProdType = "Shoes" 
    then 
        <code for calling stored proc **SPC** and populating controls on winform> 
    else if cmbCategory.text = "Category C" and cmbProdType = "Hosiery" 
    then 
        <code for calling stored proc **SPD** and populating controls on winform> `

.... и так далее.Как изменить код, чтобы сделать его более читабельным?

1 Ответ

0 голосов
/ 11 сентября 2011

Посмотрите мой код ниже, как вы думаете, теперь он более читабелен?

Select Case cmbCategory.Text
    Case "Category A"

        If cmbProdType = "Kids" Then
            '<code for calling stored proc **SPA** and populating controls on winform>
        ElseIf cmbProdType = "Mens" Then
            '<code for calling stored proc **SPB** and populating controls on winform>
        End If

    Case "Category B"


    Case "Category C"

    Case Else

End Select
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...