質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

Q&A

解決済

1回答

3924閲覧

javascriptで処理が上書きされてしまう

KensukeTsutsumi

総合スコア15

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

0グッド

1クリップ

投稿2015/11/28 02:58

下記ソースで処理が上書きされてしまいます。
処理が上書きされずに保持するにはどのように実装すればよいか、おわかりでしたらご教授のほどよろしくお願い致します。

for(var i in result.feed.entries){
if(!CTL.checkImg(result.feed.entries[i].content)){
$.ajax({
beforeSend : function(xhr) {
xhr.overrideMimeType("text/plain; charset=UTF-8");
},
dataType:"jsonp",
url: 'http://ajax.googleapis.com/ajax/services/search/images?v=0.1&q=' + result.feed.entries[i].title,
success: function(data) {
//ここの処理が上書きされて、最後の1行しか実行されません。
$("#img" + i).attr("src",data.responseData.results[0].url);
},
error:function(e){
console.log(e);
}
});
}
}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

WataruTerada

2015/11/28 03:27 編集

`$("#img" + i).attr("src",data.responseData.results[0].url);` の部分ですが、実際には複数行あるということでしょうか? 問題の核の部分ですので、1行しかない現状ですと、最後の1行しか実行されない問題を想定しづらいものがありますね。 あ、理解しました。for の最後という意味ですね。
guest

回答1

0

ベストアンサー

i がグローバル変数になっており、$.ajax({ は平行して同時に実行されるためにこのようなことが起こります。つまり、 success を実行するときには既に、for は回り終わってるので、参照する i は最後の値なのです。
jQuery を使っているなら、for の代わりに、

javascript

1$.each(result.feed.entries, function(i, entry) { 2 3});

のようにすると、i がローカル変数になりますので、平行で動いても大丈夫になりますよ。

投稿2015/11/28 03:33

編集2015/11/28 03:35
WataruTerada

総合スコア70

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

KensukeTsutsumi

2015/11/28 04:38

みごとに出来ました!ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問