Блог

Главная - Вопросы по программированию - Может ли этот код быть преобразован во что-то более короткое?

Может ли этот код быть преобразован во что-то более короткое?

#javascript #refactoring

#javascript #рефакторинг

Вопрос:

У меня есть следующий код

 let objProvince = paramsData.find(element => element.name === "province")
      if (objProvince) {
        let error = false;
        if (objProvince.value === "VI" amp;amp; value.substring(0, 2) !== "01") error = true
        if (objProvince.value === "AB" amp;amp; value.substring(0, 2) !== "02") error = true
        if (objProvince.value === "A" amp;amp; value.substring(0, 2) !== "03") error = true
        if (objProvince.value === "AL" amp;amp; value.substring(0, 2) !== "04") error = true
        if (objProvince.value === "AV" amp;amp; value.substring(0, 2) !== "05") error = true
        if (objProvince.value === "BA" amp;amp; value.substring(0, 2) !== "06") error = true
        if (objProvince.value === "PM" amp;amp; value.substring(0, 2) !== "07") error = true
        if (objProvince.value === "B" amp;amp; value.substring(0, 2) !== "08") error = true
        if (objProvince.value === "BU" amp;amp; value.substring(0, 2) !== "09") error = true
        if (objProvince.value === "CC" amp;amp; value.substring(0, 2) !== "10") error = true
        if (objProvince.value === "CA" amp;amp; value.substring(0, 2) !== "11") error = true
        if (objProvince.value === "CS" amp;amp; value.substring(0, 2) !== "12") error = true
        if (objProvince.value === "CR" amp;amp; value.substring(0, 2) !== "13") error = true
        if (objProvince.value === "CO" amp;amp; value.substring(0, 2) !== "14") error = true
        if (objProvince.value === "C" amp;amp; value.substring(0, 2) !== "15") error = true
        if (objProvince.value === "CU" amp;amp; value.substring(0, 2) !== "16") error = true
        if (objProvince.value === "GI" amp;amp; value.substring(0, 2) !== "17") error = true
        if (objProvince.value === "GR" amp;amp; value.substring(0, 2) !== "18") error = true
        if (objProvince.value === "GU" amp;amp; value.substring(0, 2) !== "19") error = true
        if (objProvince.value === "SS" amp;amp; value.substring(0, 2) !== "20") error = true
        if (objProvince.value === "H" amp;amp; value.substring(0, 2) !== "21") error = true
        if (objProvince.value === "HU" amp;amp; value.substring(0, 2) !== "22") error = true
        if (objProvince.value === "J" amp;amp; value.substring(0, 2) !== "23") error = true
        if (objProvince.value === "LE" amp;amp; value.substring(0, 2) !== "24") error = true
        if (objProvince.value === "L" amp;amp; value.substring(0, 2) !== "25") error = true
        if (objProvince.value === "LO" amp;amp; value.substring(0, 2) !== "26") error = true
        if (objProvince.value === "LU" amp;amp; value.substring(0, 2) !== "27") error = true
        if (objProvince.value === "M" amp;amp; value.substring(0, 2) !== "28") error = true
        if (objProvince.value === "MA" amp;amp; value.substring(0, 2) !== "29") error = true
        if (objProvince.value === "MU" amp;amp; value.substring(0, 2) !== "30") error = true
        if (objProvince.value === "MA" amp;amp; value.substring(0, 2) !== "31") error = true
        if (objProvince.value === "OR" amp;amp; value.substring(0, 2) !== "32") error = true
        if (objProvince.value === "O" amp;amp; value.substring(0, 2) !== "33") error = true
        if (objProvince.value === "P" amp;amp; value.substring(0, 2) !== "34") error = true
        if (objProvince.value === "GC" amp;amp; value.substring(0, 2) !== "35") error = true
        if (objProvince.value === "PO" amp;amp; value.substring(0, 2) !== "36") error = true
        if (objProvince.value === "SA" amp;amp; value.substring(0, 2) !== "37") error = true
        if (objProvince.value === "TF" amp;amp; value.substring(0, 2) !== "38") error = true
        if (objProvince.value === "S" amp;amp; value.substring(0, 2) !== "39") error = true
        if (objProvince.value === "SG" amp;amp; value.substring(0, 2) !== "40") error = true
        if (objProvince.value === "SE" amp;amp; value.substring(0, 2) !== "41") error = true
        if (objProvince.value === "SO" amp;amp; value.substring(0, 2) !== "42") error = true
        if (objProvince.value === "T" amp;amp; value.substring(0, 2) !== "43") error = true
        if (objProvince.value === "TE" amp;amp; value.substring(0, 2) !== "44") error = true
        if (objProvince.value === "TO" amp;amp; value.substring(0, 2) !== "45") error = true
        if (objProvince.value === "V" amp;amp; value.substring(0, 2) !== "46") error = true
        if (objProvince.value === "VA" amp;amp; value.substring(0, 2) !== "47") error = true
        if (objProvince.value === "BI" amp;amp; value.substring(0, 2) !== "48") error = true
        if (objProvince.value === "ZA" amp;amp; value.substring(0, 2) !== "49") error = true
        if (objProvince.value === "Z" amp;amp; value.substring(0, 2) !== "50") error = true
        if (objProvince.value === "CE" amp;amp; value.substring(0, 2) !== "51") error = true
        if (objProvince.value === "ML" amp;amp; value.substring(0, 2) !== "52") error = true

        if (error) {............

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

Я думал о том, чтобы сделать что-то вроде:

 const provinces = ["VI" , "AB", "A", "AL", "AV", "BA", "PM" , "B" ,"BU", "CC", "CA", "CS", "CR", "CO", "C", "CU", "GI", "GR", "GU", "SS", "H", 
"HU", "J","LE","L","LO","LU","M","MA","MU","MA","OR","O","P","GC","PO","SA","TF","S","SG","SE","SO","T","TE","TO","V","VA","BI","ZA","Z","CE","ML"]


function proviceCheck (_provinceValue) {
    let indexPosition = array.indexOf(_provinceValue)   1;
    let formatedIndex = ""
    
    if(indexPosition < 10){
        formatedIndex = ('0' indexPosition).slice(-2);
    }else{
        formatedIndex = indexPosition.toString()
    }
    
    if(provinces.includes(_provinceValue) amp;amp; provinceValue.substring(0,2) !== formatedIndex return true;
    
    return false;
} 

if(objProvince){
    let error = false;
    error = provinceCheck(objProvince.value)
    
    if(error) {......
}

Проблема с этим заключается в том, что индексы массива 28 и 30 повторяются, поэтому я не могу использовать indexOf идею, которая у меня была.

Есть ли какая-нибудь другая идея по рефакторингу этого? Может быть, с каким-нибудь функциональным кодом. Но я не знаю другого решения.

Обратите внимание, что мое решение, приведенное выше, нарушено, потому что массив имеет повторяющееся значение индекса 28 == 30

Комментарии:

1. Не могли бы вы пояснить, что вы подразумеваете под «значением индекса 28 == 30»?

2. Поскольку существует один проблемный случай, вы можете сначала просто перейти к нему. Это работает для вас? Если вы ожидаете большего, вы можете использовать структуру данных, в которой значение provinceValue связано со списком неправильных значений.

3. @phtrivier если вы проверяете массив, индекс 28 равен «MA», а индекс 30 тоже равен «MA»

Ответ №1:

Я бы пошел другим путем — извлек индекс, а затем проверил _provinceValue , соответствует ли он:

 function proviceCheck (_provinceValue) {
    const index = Number(provinceValue.substring(0,2)) - 1;

    return provinces[index] === _provinceValue;
}

Ответ №2:

Что вы можете сделать, чтобы избежать ошибки индексации, так это сгенерировать число рядом со всеми названиями провинций, перебрав все провинции в массиве provinces , я бы сделал это следующим образом:

 const provinces = ["VI" , "AB", "A", "AL", "AV", "BA", "PM"].map((a, b) => {'name': a, 'index': b}]);

теперь вы могли бы сделать provinces[index].name , чтобы получить имя и provinces[index].index
получить его индекс

Ответ №3:

Вы можете упростить логику, имея словарь, и использовать такой словарь для проверки того, что возвращаемое значение объекта является известным ключом, и его значение правильное.

 const provinces = {
  "VI": "01",
  "AB": "02",
  "A": "03",
  "AL": "04",
  "AV": "05",
  "BA": "06",
  "PM": "07",
  "B": "08",
  "BU": "09",
  "CC": "10",
  "CA": "11",
  // the rest ...
};

objProvince = paramsData.find(element => element.name === "province");
let error = false;
if (objProvince amp;amp; provinces.hasOwnProperty(objProvince.value))
  error = value.substring(0, 2) !== provinces[objProvince.value];
Метки: