2016-04-18 17 views
0

Ich möchte einfach meine Top 500 Nachrichten in mein DOM laden/anzeigen. Ich bin mir nicht sicher, warum es nicht auf meinemJquery Newseinträge in dom laden

angezeigt Und wenn es andere bessere Möglichkeit gibt, dies anzuzeigen? oder um es zu codieren? Vielleicht kann ich lernen.

var LoadNews = $("#hackernewsrss"); 
 

 

 
LoadNews.load(function(event) { 
 
    parseTopStories('https://hacker-news.firebaseio.com/v0/topstories', '#hackernewsrss'); 
 
}); 
 

 

 
function parseTopStories(url, container) { 
 
    var hackernewsAPI = "https://hacker-news.firebaseio.com/v0/topstories.json"; 
 
    $.getJSON(hackernewsAPI, function(json) { 
 
    var requests = []; 
 
    for (var i = 0; i < 10; i++) { 
 
     requests.push($.getJSON('https://hacker-news.firebaseio.com/v0/item/' + json[i] + '.json')); 
 
    } 
 
    $.when.apply($, requests) 
 
     .done(function() { 
 
     var results = [] 
 
      .slice.call(arguments); 
 
     var list = results.map(function(arr) { 
 
      var thetemplate = '<li><a href="' + arr[0].url + '">' + arr[0].title + '</a></li>'; 
 
      return thetemplate; 
 
     }); 
 
     $(container).html('<ol>' + list.join('') + '</ol>'); 
 
     console.log(container); //logs #hackernewsrss 
 
     }); 
 
    }); 
 
};
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 
<div id="hackernewsrss"></div>

Kann mir jemand bitte helfen?

+0

Haben Sie Ihre Konsole auf Fehler überprüft, die dort gemeldet wurden? –

+0

@ Jeremy Nein, ich werde jetzt versuchen – user3699998

+0

Überprüfen der Fehler Konsole ist * immer * der erste Schritt des Debuggens. –

Antwort

0

Wenn Sie versuchen:

$(function(){ 
     parseTopStories('https://hacker-news.firebaseio.com/v0/topstories', '#hackernewsrss'); 
}); 

alles gut geht.

Ich bin mir nicht sicher, warum Sie Load-Ereignis an das div angehängt haben. Es wird normalerweise verwendet, um auszulösen, wenn das Bild oder das Fenster geladen wird: JQuery reference

+0

Dank wirklich dummer Fehler – user3699998

+0

Keine Sorge, wir tun alle diese. – NotJustin

+0

Außerdem gibt https://hacker-news.firebaseio.com/v0/topstories.json jetzt 446 Werte zurück, nicht 500, und es funktioniert nicht mit 500. Sie möchten vielleicht zählen, wie viele es zurückgibt und diese Zahl setzen in deiner Schleife statt 10 oder 500. – Walk