Bezdepbonusu7Программирование: статьи и ответы

Вопрос

TypeError: undefined не является объектом (оценка "XYZ") - JavaScript

Автор вопроса: AlphaLeviathanОтветов: 2

надеюсь, у всех пока хороший день.

У меня странная проблема в JavaScript. Задача скрипта - упорядочивать элементы div каждый раз, когда кто-то посещает страницу. Для этого я генерирую и массив со случайными числами, убедившись, что нет удвоенных. Затем я даю каждому div его номер и сортирую их перед тем, как вернуть на страницу. Прямо сейчас я застрял в части, где я сортирую div в массиве.

Я получаю TypeError, которая, кажется, не имеет смысла, так как я уже использовал "typeof", чтобы увидеть, не определены ли выходные данные, но это сплошные числа. Вот код:

HTML:

<div class="padding"></div>
<div class="MosaikBilder">1</div>
<div class="MosaikBilder">2</div>
<div class="MosaikBilder">3</div>
<div class="MosaikBilder">4</div>
<div class="MosaikBilder">5</div>
<div class="MosaikBilder">6</div>
<div style="clear: both"></div>
<div class="padding"></div>

CSS:

title {
    display: none;
}

.padding {
    width: 100%;
    height: 200px;
    background-color: red;
    margin-top: 20px;
    margin-bottom: 20px;
}

.MosaikBilder {
    height: 50px;
    width: 40px;
    background-color: blue;
    margin: 10px;
    float: left;
    color: white;
}

а теперь оценка, JS:

var BilderListe = ["","", "", "", "", "", "", "", "", "","","","","",""];
//generating an array until there are no doubled values
do{
    var funktionsfaehig = true;
    Zahlenzuweisung();
    Dopplungspruefung();
}while(funktionsfaehig == false);
console.log(BilderListe);
Zuordnung();

//generating a random list of numbers
function Zahlenzuweisung(){
    for(var i=0 ; i<BilderListe.length; i++){
        BilderListe[i] = Math.round(Math.random()*1000);
        if(BilderListe[i] > 1000){
            BilderListe[i] = 1000;
        }
        //alert(BilderListe[i]);
    }
    console.log("Zahlenzuweisung " + BilderListe);
}

//sorting the list so it can be checked more easily, .sort did always miss one number for some reason
function Dopplungspruefung(){
    var Tauschen
    var PruefungsListe;

    PruefungsListe = BilderListe.slice();
    console.log("Dopplungspruefung ist " + BilderListe)
    console.log(PruefungsListe);
    do{
        var getauscht = false;
        for(i=0; i<PruefungsListe.length; i++){
            if(PruefungsListe[i] > PruefungsListe[i+1]){
                Tauschen = PruefungsListe[i];
                PruefungsListe[i] = PruefungsListe[i+1];
                PruefungsListe[i+1] = Tauschen;

                getauscht = true;
            }
        }
    }while(getauscht == true);
    console.log(PruefungsListe);
    console.log("Nach der Prüfung ist " + BilderListe);
    DerPruefer(PruefungsListe);
}

//checking the sorted list for doubled numbers
//if there are any, new numbers will be generated
function DerPruefer(PruefungsListe){
    console.log("Der Pruefer wurde gestartet");
    for(i=0; i < PruefungsListe.length; i++){
        if(PruefungsListe[i] == PruefungsListe[i+1]){
            console.log("Etwas war gleich");
            funktionsfaehig = false;
            break;
        }
    }
}

//all divs needed get gathered in one array
//then every element gets a random generated value
function Zuordnung(){
    var AlleBilder = document.getElementsByClassName("MosaikBilder");
    for(i=0; i<AlleBilder.length; i++){
        AlleBilder[i].value = BilderListe[i]
    }
    var Check = AlleBilder[0].value;
    console.log(Check);
    //Bubblesort the Pictures
    BilderSortierer(AlleBilder);
}

//Now i want to sort the elements by their value
//But they TypeError occurs, claiming that the 2nd one
//Has the value 'undefined (log says its a number)
function BilderSortierer(AlleBilder){
    var i = 0
    console.log(AlleBilder[i]);
    console.log(AlleBilder[i].value);
    console.log(typeof(AlleBilder[i].value));
    console.log(AlleBilder[i+1]);
    console.log(AlleBilder[i+1].value);
    console.log(typeof(AlleBilder[i+1].value));
    var Wechsler
    var BilderCheck = true;
    do{
        for(i=0; i<AlleBilder.length; i++){
            if(AlleBilder[i].value > AlleBilder[i+1].value){
                Wechsler = AlleBilder[i];
                AlleBilder[i] = AlleBilder[i+1];
                AlleBilder[i+1] = Wechsler;
                BilderCheck = false;
            }
        }
    }while(BilderCheck == true);
}

это журнал консоли, так что вам не нужно пробовать это самостоятельно:

[Log] Zahlenzuweisung       142,595,442,790,668,978,652,721,154,280,819,239,425,65,270  (Bilderordner.html, line 64)
[Log] Dopplungspruefung ist 142,595,442,790,668,978,652,721,154,280,819,239,425,65,270 (Bilderordner.html, line 72)
[Log] Array (15) (Bilderordner.html, line 73)
[Log] Array (15) (Bilderordner.html, line 86)
[Log] Nach der Prüfung ist 142,595,442,790,668,978,652,721,154,280,819,239,425,65,270 (Bilderordner.html, line 87)
[Log] Der Pruefer wurde gestartet (Bilderordner.html, line 92)
[Log] Array (15) (Bilderordner.html, line 53)
[Log] 142 (Bilderordner.html, line 108)
[Log] <div class="MosaikBilder">1</div> (Bilderordner.html, line 115)

[Log] 142 (Bilderordner.html, line 116)
[Log] number (Bilderordner.html, line 117)
[Log] <div class="MosaikBilder">2</div> (Bilderordner.html, line 118)

[Log] 595 (Bilderordner.html, line 119)
[Log] number (Bilderordner.html, line 120)
[Error] TypeError: undefined is not an object (evaluating 'AlleBilder[i+1].value')

Может быть у кого-то есть идеи, почему это происходит и как это исправить. Я не менял никаких имен или журналов на случай, если они являются частью проблемы. Но, как я уже говорил ранее, оба значения являются числами typeof, так почему же второе не работает в цикле if?

Кроме того, если есть какой-либо другой способ сделать это проще, не стесняйтесь исправлять меня, я относительно новичок и благодарен за любую помощь.

С уважением и заранее большое спасибо

  • Когда i равно AlleBilder.length - 1, тогда AlleBilder[i+1] будет undefined. Pointy 13.08.2018

javascripthtmlcsstypeerror

Ответы

Ответ 1 · рейтинг 3 · epascarello

Довольно просто

AlleBilder[i+1] не определено, когда вы находитесь на последнем индексе.

Так что вам нужно будет изменить свой цикл

for(i=0; i<AlleBilder.length-1; i++){

Ответ 2 · рейтинг 3 · TLP

Похоже, ваша последняя итерация будет искать элемент в вашем массиве, который не существует (т.е. за пределами)

Попробуйте изменить

for(i=0; i<AlleBilder.length; i++)

to

for(i=0; i<AlleBilder.length-1; i++)

Перевод вопроса и ответов сообщества Stack Overflow, лицензия CC BY-SA. Оригинал на Stack Overflow.

Похожие материалы