Warum funktioniert dieser JavaScript-Code nicht?

Question

Ich habe für die Schleife, die das Array mit Argumenten durchläuft. Wenn das nächste Argument "?", "&" oder "||" ist, sollte es kein Komma hinzufügen, es wird jedoch immer hinzugefügt. Ich konnte nicht verstehen, warum, hier ist der Code:

var args = ["arg1","arg2","?","arg3"]; 
 
var query = ""; 
 
for (var i = 0; i < args.length; i++) { 
 

 
\t \t switch (args[i]) { 
 

 
\t \t \t case "?": 
 
\t \t \t \t query += " where "; 
 
\t \t \t \t break; 
 

 
\t \t \t case "&": 
 
\t \t \t \t query += " and "; 
 
\t \t \t \t break; 
 

 
\t \t \t case "||": 
 
\t \t \t \t query += " or "; 
 
\t \t \t \t break; 
 

 
\t \t \t default: 
 
\t \t \t \t if (args[i+1] != "?"); 
 
\t \t \t \t { 
 
\t \t \t \t \t query += args[i] + ", "; 
 
\t \t \t \t \t break; 
 
\t \t \t \t } 
 
\t \t \t \t query += args[i] + " "; 
 
\t \t \t \t break; 
 

 
\t \t } 
 

 
\t } 
 
document.write(query);

Wenn ich schreibe dies (dies wird gesplittet durch "" und Array args gesendet):

arg1 arg2 ? arg3 

Es druckt es so:

arg1, arg2, where arg3, // while it should be arg1, arg2 where arg3, 

Dank für das Helfen von Leuten, Problem wurde durch ein externes Skript verursacht. Und ja, I entfernt Semikolon;)

Answer 1

Ihre if Aussage gebrochen:

 if (args[i+1] != "?"); // <---- remove that semicolon 
     { 
      query += args[i] + ", "; 
      break; 
     } 

Sie eine verirrte Semikolon haben. Es ist kein Syntaxfehler, aber es bedeutet, dass if nichts tut. Der Code, der das Komma immer enthält, wird ausgeführt, und das switch wird vor dem Code nicht das Komma hinzufügen beendet.

Answer 2

Sie haben ein Semikolon zwischen wenn und dem Block:

if (args[i+1] != "?"); 

Sollte

if (args[i+1] != "?") 

Answer 3

kann es sein, ganz andere Möglichkeiten, dieses Problem zu lösen, die Ihren Code einfacher machen würde, ohne zu verlängern Vertiefung Bäume von if oder switch

Ein kurzes Beispiel,

// define some dictionaries 
let logicDict = Object.assign(Object.create(null), { 
    '?': 'where', 
    '&': 'and', 
    '||': 'or' 
}); 

// define some flags 
let noComma = false; 

// reduce your array 
['arg1', 'arg2', '?', 'arg3'].reduceRight((str, e) => { 
    if (e in logicDict) { 
     noComma = true; 
     return logicDict[e] + ' ' + str; 
    } 
    if (!noComma) e += ','; 
    noComma = false; 
    return e + ' ' + str; 
}, '').slice(0, -1); 
// "arg1, arg2 where arg3," 

Answer 4

Vielen Dank für Semikolon Jungs, Problem wurde von einem externen Skript verursacht.