Cześć.
Mam prośbę o ocenę kodu i ewentualnie wskazówkę czy dało by się to jakoś zoptymalizować ?
function validationLogic() { /*TODO here is a logic to change a right size of etykiet*/
const listDone = document.getElementById("select");
const number = document.getElementById("volume").value; /*TODO dobrze by było się tego pozbyć przesyłać redundancja*/
const select = document.querySelector('select[name="select"]').value;
if(select == "ml" || select == "cm"){
if(number <= 3000){
listDone.selectedIndex = 1;
}if(number >= 3000 ){
listDone.selectedIndex = 2;
}if(number >= 50000){
listDone.selectedIndex = 3;
}if(number >= 500000){
listDone.selectedIndex = 4;
}
}if(select == "l" || select == "dm"){
if(number <= 3){
listDone.selectedIndex = 1;
}if(number >= 3){
listDone.selectedIndex = 2;
}if(number >= 50){
listDone.selectedIndex = 3;
}if(number >= 500){
listDone.selectedIndex = 4;
}
}if(select == "m"){
if(number <= 0.003){
listDone.selectedIndex = 1;
}if(number >= 0.003){
listDone.selectedIndex = 2;
}if(number >= 0.05){
listDone.selectedIndex = 3;
}if(number >= 0.5){
listDone.selectedIndex = 4;
}
}
}