繰り返しを20回で止めたい

Question

見て頂いてありがとうございます。
javascriptでfacebookの自分の投稿を取得し、サイトに表示させているのですが、
直近の20件のみを表示させたいと思っています。

現在以下のようなソースコードにしているのですが、これでは20件以上取得してしまいます。

現在のソースコード

success: function(json){
 json = json.data;
 var num = json.length;
 var Feed = "<ul>\n";
 for(var i=0; i<num; i++){
 var baseid = json[i].id;
 var id =  baseid.substr( 16 ) ;
 var img = 'https://www.facebook.com/plugins/post.php?href=https%3A%2F%2Fwww.facebook.com%2Fvelleglantz%2Fposts%2F' + id + '&width=200';
 var size = 'width="300" height="400" style="border:none;overflow:hidden;height=100%;width=100% scrolling="no" frameborder="0" allowTransparency="true"'

  Feed += "<iframe src="+img+" "+size+"></iframe>" ;
 }

 Feed += "</ul>\n";
 $("#topics").append(Feed);
 }

そこで20件で止めるため、以下の様なソースコードにしたところ、
1件も取得されず何も表示されなくなりました。
どこがおかしいのか何度見ても分かりません。
簡単な質問で申し訳ないのですが、よろしくお願いします。

success: function(json){
 json = json.data;
 var Feed = "<ul>\n";
 for(var i=0; i<20; i++){
 var baseid = json[i].id;
 var id =  baseid.substr( 16 ) ;
 var img = 'https://www.facebook.com/plugins/post.php?href=https%3A%2F%2Fwww.facebook.com%2Fvelleglantz%2Fposts%2F' + id + '&width=200';
 var size = 'width="300" height="400" style="border:none;overflow:hidden;height=100%;width=100% scrolling="no" frameborder="0" allowTransparency="true"'

  Feed += "<iframe src="+img+" "+size+"></iframe>" ;
 }

 Feed += "</ul>\n";
 $("#topics").append(Feed);
 }

Comments

[shi_ue]

他のところにエラーが出てませんか？開発者ツールのコンソールを見てみてください。

Answer 1

修正後のソースで取得できたjsonのLengthが20より小さいということはないでしょうか？
forループの中で、iがLengthより大きい場合はbreakが必要かと思います。

Comments

[Takuma_Tanaka]

仰るとおりLengthが20より小さかったのでbreakを入れたところ、正常に動作しました。
ありがとうございました。