Sprawdzanie czy zmienna jest pusta (null) czy przechowuje tekst

Question

Witam!

   Używając $.ajax z jQuery łączę się do skryptu php i otrzymuję pakiet pewnych danych który w JavaScript mogę przedstawić tak ( w tym pakiecie na 100% zmienne val_1, val_2 przechowują wartości typu string) :

var od_serwera = { type = "xxx", val_1 = "yyy", val_2 = null }

I pytanie moje brzmi czy nie jest błędem ani żadnym innym kwiatkiem sprawdzenie czy któraś z tych zmiennych val_1 lub val_2 coś przechowuje umieszczając ją w if'ie if(val_1) ? Skoro val_1 i val_2 jest albo null albo przechowuje tekst to przechowywanie tekstu przez tą zmienną jest jednoznaczne z prawdziwym warunkiem ? if("xxx") <- prawda if(null) <- fałsz ?.

$.ajax({
         type     : "POST",
         url      : "skrypt.php",
         data     : post_data,
         dataType : "json",
      
         success: function( data ) {

                         if( data.type == "xxx"  ) {
                               
                              if( data.val_1 ) { // data.val_1 !== null
                                      
                                          $('#el_1').text( data.val_1 );
                               } else {
                                          $('#el_1').text("");
                              }
                         
                              if( data.val_2 ) { // data.val_2 !== null
                                      
                                          $('#el_2').text( data.val_2 );
                               } else {
                                          $('#el_2').text("");
                              }

                          }
         }
});

 

Answer 1

A nie wpadłeś na pomysł, żeby to może hmm... no nie wiem...

Sprawdzić?

P.S. Tak null da false, a stringi da true. Pamiętaj tylko, że jeśli string będzie pusty, też zwróci false.

Comments

Wpadłem na taki pomysł i sprawdziłem i tak jak jest w kawałku kodu który wstawiłem to działa tylko czy tak się robi ?

---

No nie wiem, ja to bym proponował sprawdzić to za pomącą algorytmu radzieckiego uczonego Pietriejeva Boolinova, z wykorzystaniem architektury, w której repozytoria są abstrakcją persystencji dla agregatów. Wtedy będziesz miał pewność.

(Tak, tak się robi. Czemu miałoby się tak nie robić, skoro jest proste i działa?)

---

@mtk3d A Ty patrzyłeś w ogóle na kod OP? Bo nikt z jakimkolwiek doświadczeniem tak kodu nie pisze.
 

Czemu miałoby się tak nie robić, skoro jest proste i działa?

Bo to nie jedyne kryteria jakie powinien spełniać kod.

Answer 2

Nie rób tam zadnych ifów, jeśli do $().text() przekażesz pusty string, null lub undefined to efekt będzie taki sam.