Javascript - несколько сложных операторов if

Question

Я борюсь с множеством сложных операторов в javaScript и задаюсь вопросом, может ли кто-нибудь указать мне в правильном направлении.

function findFlights() 
    { 
    var yourDestination = readTheDestination(); 
    var yourAirline = readTheAirline(); 
    var yourFare = readTheFare(); 


    if (yourDestination == 'Choose your destination') 
     { 
     displayMessage('<B>Please choose a destination city from the Destination menu and then click Find flights again.</B>'); 
     } 
    else 
     { 
     var destinationTime, destinationOperator, destinationFare; 
     var message = '<B>You asked about flights to ' + yourDestination + '</B><BR>' + ""; 

     for (var i=0; i < flightTimes.length; i++)      //flight destinations 
     { 
     if // statement            // IF flight: 
      ((flightDestinations[i] == yourDestination &&    // destination = selected destination & 
      yourAirline == 'Any airline' &&        // airline = any airline & 
      yourFare == 'Any price'))         // fare <= chosen fare 
      ||               // OR 
      (flightDestinations[i] == yourDestination &&    // destination = selected destination & 
      yourAirline == flightOperators[i] &&      // airline = chosen airline & 
      yourFare <= flightFares[i]))        // fare <= chosen fare 

      { 
      destinationTime = flightTimes[i]; 
      destinationOperator = flightOperators[i]; 
      destinationFare = flightFares[i]; 



      message += destinationTime + ' ' + destinationOperator + '. £' + destinationFare + '<BR>'; 
      displayMessage(message); 

      } 
     } 
     else if (flightDestinations[i] == yourDestination && 
       flightOperators[i] != yourAirline && 
       flightFares[i] != yourFare) 
      { 
      displayMessage('There are no flights to ' + yourDestination + ' with ' + yourAirline + '. Please select Any Airline and try again.'); 
      } 
     } 

Это то, что у меня есть, и это делает меня серым.

Answer 1

Я не уверен, в чем вопрос, но вы должны использовать более последовательный отступ, а сложные if заявления определенно станут яснее.

Мои правила:

• вкладки используют для отступа ваших строк кода по размаху (например, +1 вкладка в то время как в фигурные скобки, if заявления и т.д.)
• не использовать вкладки для чего-то еще ; использование пространство для выравнивания, несколько пробелов, при необходимости (так, использование пространства для выстилает свои условия в if заявлении, если заявление охватывает несколько строк)

Answer 2

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

if (yourDestination == 'Choose your destination') { 
    displayMessage('<B>Please choose a destination city from the Destination menu and then click Find flights again.</B>'); 
    return; 
} 

Это поможет вам удалить else и его кронштейны {...}.

Вы можете изменить цикл, чтобы уменьшить размер кода:

if(flightDestinations[i] != yourDestination) continue; 

Так что, вам не нужно писать это состояние неоднократно.

Answer 3

Вы несовпадающие свою скобку здесь:

((flightDestinations[i] == yourDestination &&    // destination = selected destination & 
yourAirline == 'Any airline' &&        // airline = any airline & 
yourFare == 'Any price'))         // fare <= chosen fare 
||               // OR 
(flightDestinations[i] == yourDestination &&    // destination = selected destination & 
yourAirline == flightOperators[i] &&      // airline = chosen airline & 
yourFare <= flightFares[i]))        // fare <= chosen fare 

Изменения yourFare == 'Any price')) в yourFare == 'Any price').

Answer 4

Refactor сложный код для работы

Если у вас есть комплекс, если заявления попытаться обернуть их в функции. Так

(flightDestinations[i] == yourDestination && 
yourAirline == 'Any airline' && 
yourFare == 'Any price') 

может стать

function YourDestinationIsTheSameForAnyAirlineOrPrice(flightDestination, yourDestination, yourAirline, yourFare){ 
    return flightDestination == yourDestination && 
     yourAirline == 'Any airline' && 
     yourFare == 'Any price'; 
} 

// And called from if 
if (YourDestinationIsTheSameForAnyAirlineOrPrice(flightDestinations[i], yourDestination, yourAirline, yourFare)) {} 

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

Используйте объект над нескольких массивами

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

var flight = { 
    destination = "London", 
    operator = "BA", 
    time = "18:00 UTC", 
    fare = "£239829" 
} 

Это должно сделать код более читаемым, чем использование нескольких массивов.Пример:

destinationTime = flightTimes[i]; 
destinationOperator = flightOperators[i]; 
destinationFare = flightFares[i]; 

message += destinationTime + ' ' + destinationOperator + '. £' + destinationFare + '<BR>'; 

// Using an object 
message += flight.Time + ' ' + flight.Operator + '. £' + flight.Fare + '<br />'; 

Возвращение рано

Наконец, я бы вырваться из функции как можно скорее. Поэтому использование:

if (yourDestination == 'Choose your destination') { 
    displayMessage('<B>Please choose a destination city from the Destination menu and then click Find flights again.</B>'); 
    return; 
} 

вместо того, если ... еще. Я лично считаю это более читаемым, поэтому не стесняйтесь игнорировать это.