Skip to main content
adicionou 146 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

As duas principais condições são quase idênticas, só uma subexpressão é diferente e o que executa quando entra em cada um é idêntico, então realmente não faz sentido ter duas condições. O código tem um padrão idêntico exceto por um detalhe, então fazer ficar só uma vez é fácil e apenas muda a subexpressão, então ele entra sendo uma subexpressão sendo verdadeira OU a outra que está no outro if sendo verdadeira também. Coloquei entre parênteses porque ela precisa ser feita antes do resto.

Também tirei a comparação com true porque ele já espera um true não faz sentido fazer essa comparação, o que já reduz um pouco o tamanho do código.

Dá para reduzir um pouco mais mas acho que não compensa, começa ficar amontoado e confuso.

Não garanto que faz o que deseja, só fiz a redução do código sem contexto.

function checkFieldInsert(normalInitialLevel, masterInitialLevel){
    for (let i = 0; i <= checkfields.length; i++){
        if ((checkfields[i] == currentLevel || checkfields[i] == desiredLevel) && levelNormal.checked && parseInt(checkfields[i].value) >= 400 || levelMaster.checked && parseInt(checkfields[i].value) < 400){
            if (levelMaster.checked){
                alert(`Level ${checkfields[i].value} is not master level`)
                break
            } else if (levelNormal.checked){
                alert(`Level ${checkfields[i].value} is not normal level`)
                break
            }
        }
    }   
}

Coloquei no GitHub para referência futura.

As duas principais condições são quase idênticas, só uma subexpressão é diferente e o que executa quando entra em cada um é idêntico, então realmente não faz sentido ter duas condições. O código tem um padrão idêntico exceto por um detalhe, então fazer ficar só uma vez é fácil e apenas muda a subexpressão, então ele entra sendo uma subexpressão sendo verdadeira OU a outra que está no outro if sendo verdadeira também. Coloquei entre parênteses porque ela precisa ser feita antes do resto.

Também tirei a comparação com true porque ele já espera um true não faz sentido fazer essa comparação, o que já reduz um pouco o tamanho do código.

Dá para reduzir um pouco mais mas acho que não compensa, começa ficar amontoado e confuso.

Não garanto que faz o que deseja, só fiz a redução do código sem contexto.

function checkFieldInsert(normalInitialLevel, masterInitialLevel){
    for (let i = 0; i <= checkfields.length; i++){
        if ((checkfields[i] == currentLevel || checkfields[i] == desiredLevel) && levelNormal.checked && parseInt(checkfields[i].value) >= 400 || levelMaster.checked && parseInt(checkfields[i].value) < 400){
            if (levelMaster.checked){
                alert(`Level ${checkfields[i].value} is not master level`)
                break
            } else if (levelNormal.checked){
                alert(`Level ${checkfields[i].value} is not normal level`)
                break
            }
        }
    }   
}

As duas principais condições são quase idênticas, só uma subexpressão é diferente e o que executa quando entra em cada um é idêntico, então realmente não faz sentido ter duas condições. O código tem um padrão idêntico exceto por um detalhe, então fazer ficar só uma vez é fácil e apenas muda a subexpressão, então ele entra sendo uma subexpressão sendo verdadeira OU a outra que está no outro if sendo verdadeira também. Coloquei entre parênteses porque ela precisa ser feita antes do resto.

Também tirei a comparação com true porque ele já espera um true não faz sentido fazer essa comparação, o que já reduz um pouco o tamanho do código.

Dá para reduzir um pouco mais mas acho que não compensa, começa ficar amontoado e confuso.

Não garanto que faz o que deseja, só fiz a redução do código sem contexto.

function checkFieldInsert(normalInitialLevel, masterInitialLevel){
    for (let i = 0; i <= checkfields.length; i++){
        if ((checkfields[i] == currentLevel || checkfields[i] == desiredLevel) && levelNormal.checked && parseInt(checkfields[i].value) >= 400 || levelMaster.checked && parseInt(checkfields[i].value) < 400){
            if (levelMaster.checked){
                alert(`Level ${checkfields[i].value} is not master level`)
                break
            } else if (levelNormal.checked){
                alert(`Level ${checkfields[i].value} is not normal level`)
                break
            }
        }
    }   
}

Coloquei no GitHub para referência futura.

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

As duas principais condições são quase idênticas, só uma subexpressão é diferente e o que executa quando entra em cada um é idêntico, então realmente não faz sentido ter duas condições. O código tem um padrão idêntico exceto por um detalhe, então fazer ficar só uma vez é fácil e apenas muda a subexpressão, então ele entra sendo uma subexpressão sendo verdadeira OU a outra que está no outro if sendo verdadeira também. Coloquei entre parênteses porque ela precisa ser feita antes do resto.

Também tirei a comparação com true porque ele já espera um true não faz sentido fazer essa comparação, o que já reduz um pouco o tamanho do código.

Dá para reduzir um pouco mais mas acho que não compensa, começa ficar amontoado e confuso.

Não garanto que faz o que deseja, só fiz a redução do código sem contexto.

function checkFieldInsert(normalInitialLevel, masterInitialLevel){
    for (let i = 0; i <= checkfields.length; i++){
        if ((checkfields[i] == currentLevel || checkfields[i] == desiredLevel) && levelNormal.checked && parseInt(checkfields[i].value) >= 400 || levelMaster.checked && parseInt(checkfields[i].value) < 400){
            if (levelMaster.checked){
                alert(`Level ${checkfields[i].value} is not master level`)
                break
            } else if (levelNormal.checked){
                alert(`Level ${checkfields[i].value} is not normal level`)
                break
            }
        }
    }   
}