jQuery игнорирует else if statement

Question

Я работаю над веб-приложением, в котором пользователю предлагается добавить «приложение», введя идентификатор StudentID и JobID. Я использую jQuery, чтобы сообщить пользователю, когда учащийся, которого они ввели, не существует, задание не существует, приложение уже существует в базе данных или когда приложение было успешно добавлено. Я проверяю все на бэкэнд с помощью PHP и отправляю сообщение в интерфейс, чтобы отобразить это для пользователя.

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

jQuery полностью игнорирует утверждения «else if». Если первое условие истинно, оно успешно выполняет свое тело, однако, если первое не соответствует действительности, оно переходит прямо в инструкцию else, игнорируя инструкции else if, даже если они истинны.

Вот код:

var myData = {"tablename": tablename, "StudentId": StudentId, "JobId": JobId}; 
      $.ajax({ 
       url: "insert.php", 
       type: "POST", 
       data: myData, 
       success: function (data, status, xhr) { 
        console.log(data); 
        if (data =="success") { 
         $('#message').fadeIn().delay(3000).fadeOut(); 
         $('#StudentId').val(''); 
         $('#JobId').val(''); 
        } else if(data=="fail") { 

         $('#message5').fadeIn().delay(3000).fadeOut(); 

        } else if (data =="student"){ 

         $('#message2').fadeIn().delay(3000).fadeOut(); 
        } 
         else if (data == "job"){ 

         $('#message3').fadeIn().delay(3000).fadeOut(); 
        } 
         else { 

         $('#message4').fadeIn().delay(3000).fadeOut(); 
        } 
       } 

      }); 

Answer 1

У вас ошибка в вашем if-блоке, последний else имеет состояние. Это должно появиться как ошибка в консоли.

Либо изменение else (data == "studentjob") к

else 

или

else if (data == "studentjob") 

Answer 2

Почему бы вам не использовать switch заявление? Я рекомендую вам использовать его. Оператор switch используется для выполнения различных действий на основе разных условий.

Теперь ваш код должен работать. Вот решение:

switch(data){ 
     case "success": 
      $('#message').fadeIn().delay(3000).fadeOut(); 
      $('#StudentId').val(''); 
      $('#JobId').val(''); 
      break; 
     case "fail": 
      $('#message5').fadeIn().delay(3000).fadeOut(); 
      break; 
     case "student": 
      $('#message2').fadeIn().delay(3000).fadeOut(); 
      break; 
     case "job": 
      $('#message3').fadeIn().delay(3000).fadeOut(); 
      break; 
     case "studentjob": 
      $('#message4').fadeIn().delay(3000).fadeOut(); 
      break;     
} 

Answer 3

Подобно тому, как пример способа проверки, я создал plunker для вас

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

E.g. от plunkr:

HTML

<div> 
    <button onclick="testSuccess()">Success</button> 
    <button onclick="testFail()">Fail</button> 
    <button onclick="testStudent()">Student</button> 
    <button onclick="testJob()">Job</button> 
    <button onclick="testStudentJob()">Student Job</button> 
    <button onclick="testUnexpected()">Unexpected</button> 
</div> 

JavaScript

var testSuccess = function() { 
    successFn("success"); 
} 

var testFail = function() { 
    successFn("fail"); 
} 

var testStudent = function() { 
    successFn("student"); 
} 

var testJob = function() { 
    successFn("job"); 
} 

var testStudentJob = function() { 
    successFn("studentjob"); 
} 

var testUnexpected = function() { 
    successFn("somethingSilly"); 
} 

реорганизовать HTTP вызов:

var myData = {"tablename": tablename, "StudentId": StudentId, "JobId": JobId}; 
      $.ajax({ 
       url: "insert.php", 
       type: "POST", 
       data: myData, 
       success: successFn 
      }); 

И немного изменить ваш если-то еще (очень рекомендую идти с A Предложение lexandru Михая выписок переключателя здесь)

var successFn = function (data, status, xhr) { 
       console.log(data); 
       if (data =="success") { 
        $('#message').fadeIn().delay(3000).fadeOut(); 
        $('#StudentId').val(''); 
        $('#JobId').val(''); 
       } 
       else if(data=="fail") { 

        $('#message5').fadeIn().delay(3000).fadeOut(); 

       } 
       else if (data =="student"){ 

        $('#message2').fadeIn().delay(3000).fadeOut(); 
       } 
       else if (data == "job"){ 

        $('#message3').fadeIn().delay(3000).fadeOut(); 
       } 
       else if (data == "studentjob") { 

        $('#message4').fadeIn().delay(3000).fadeOut(); 
       } 
       else { 
        alert('Unexpected response'); 
       } 
      }; 

https://plnkr.co/edit/KZrWMrcybko6P32yFopz