Mam następujący kodpoprawić kod warunku
var oDataEn = aData[0][aProperties[0].split('/')[0]];
if (!(oDataEn != null && oDataEn.results && oDataEn.results.length > 0)) {
...
else
...
ten pracuje OK z wyjątkiem gdy
aData[0] = 'undefined'
moje pytanie brzmi, czy istnieje lepiej sposób napisać to zamiast po prostu dodawanie przed
Nie chcę mieć dwóch, jeśli to możliwe ...
Rozpocznij trójskładnikowy - sądzę Adata [0] Może być falsy (null nieokreślone 0 lub ""):
var oDataEn = aData[0]?aData[0][aProperties[0].split('/')[0]]:null;
if (oDataEn && oDataEn.results && oDataEn.results.length > 0) {
if(aData[0] === undefined){}
to najlepszy sposób, jaki do tej pory wiem.
jak na swój komentarz
if(typeof aData[0] != 'undefined' /*&& other if statements*/){}
Więc to nie będzie przetwarzać jeśli jest niezdefiniowany.
Dzięki, ale mam zamiar napisać to lepiej dla całego kontekstu, nie tylko jeśli dodałem na końcu pytania:) (mam na myśli spojrzenie na całe oświadczenie z pierwszym i drugim, jeśli stwierdzenie ...) –
Nie chcę mieć dwóch, jeśli to możliwe ... –
@RaynD Sprawdź moją zaktualizowaną odpowiedź –
Stosując operatora trójskładnikowego u można to zrobić. Operator trójskładnikowy zredukuje tylko linię kodów.
var oDataEn = aData[0] === undefined ? '' : aData[0][aProperties[0].split('/')[0]];
Czy chcesz pominąć, jeśli aData [0] jest niezdefiniowana? –
Myślę, że masz literówkę w 'aData [0] = 'undfiend'' powinno być" niezdefiniowane " – kabirbaidhya
@mplungjan - Dzięki, ale nie udało się wcześniej z aData [0], która jest niezobowiązująca ... –