Warum soll hier ein "]" hin?

Justus · 7. April 2019

Hallo,

ich schreibe mir gerade mit JavaScript / jQuery ein Script um einsen un nullen im Hintergrund laufen zu lassen (bewegt wird alles mit css keyframes).

Dabei habe ich erst einmal kein Problem, aber ich bekomme immerwieder eine Fehlermeldung:

"SyntaxError: missing ] after element list 1:21 note: [ opened at line 1, column 13 1:13"

Bei 1:13 ist aber garnichts und 1:21 ist mitten in einem function().

Hier ist der code:

JavaScript

$(document).ready(function() {
  var column1 = [];
  var column2 = [];
  var column3 = [];
  var column4 = [];
  var column5 = [];
  var counter = 0;
  console.log("gestartet");
  const columnpositions = [2, 8, 22, 38, 57];
  const possibilitys = [0, 1, ""];
  function createElement(column) {
    console.log("new Element");
    counter ++;
    var character = possibilitys[Math.round(Math.random() * 2)];
    $("#Background").append("<p id='" + counter + "'>" + character + "</p>");
    var paragraph = $("#" + counter).css({"left": columnpositions[column] + "rem"});
    eval("column" + column + ".push(" + paragraph + ")");
  }
  function createRow() {
    for (var i = 1; i < 6; i++) {
      createElement(i);
    }
    window.setTimeout(createRow, 1000);
  }
  createRow();
});

Alles anzeigen

Vielen Dank im Vorraus.

Justus

basti1012 · 7. April 2019

da sind meiner meinung nach ein Paar sachen nicht ok

erstmal das

Code

$("#" + counter)

eine ID darf keine Zahl sein. Ein Wort mit einer Zahl ist ok ,aber nicht nur nee Zahl.

Also mach zb aus das

Code

$("#" + counter)   ein   $("#bla" + counter)

Dann sagt man auch das eval böse ist und man nicht nehmen soll.

Das dein Code nicht geht liegt meiner meinung auch an den zusammen gebauten Code im EVAL. Muss iich gleich zuhause mal testen fals wer anderes nicht schneller ist .

Könnte mich aber auch wieder mit allem irren

Sempervivum · 7. April 2019

Ja, hier zeigen sich die Tücken von eval und der Debugger bringt es an den Tag: Der String, der übergeben wird, sieht so aus:

column1.push([object Object])

weil Du einen String zusammen baust und Javascript wandelt das Objekt paragraph in diese Stringdarstellung um.

Lösung:

Code

window["column" + column].push(paragraph);

Dabei nutzt man aus, dass eine Variable ein Element des window-Objektes ist, das mit dem Variablennamen angesprochen werden kann.

Ich würde es jedoch vorziehen, auf diesen Trick zu verzichten und das Ganze in einem zweidimensionalen Array abzulegen:

Code

        var columns = [[], [], [], [], []];
        var counter = 0;
        console.log("gestartet");
        const columnpositions = [2, 8, 22, 38, 57];
        const possibilitys = [0, 1, ""];
        function createElement(column) {
            console.log("new Element");
            counter++;
            var character = possibilitys[Math.round(Math.random() * 2)];
            $("#Background").append("<p id='" + counter + "'>" + character + "</p>");
            var paragraph = $("#" + counter).css({ "left": columnpositions[column] + "rem" });
            columns[column].push(paragraph);
        }
        function createRow() {
            for (var i = 0; i < 5; i++) {
                createElement(i);
            }
            window.setTimeout(createRow, 1000);
        }
        createRow();

Alles anzeigen

Warum soll hier ein "]" hin?

Jetzt mitmachen!

Teilen