こんにちはゲストさん。会員登録(無料)して質問・回答してみよう!

解決済みの質問

javascript src書き換え

javascript 初心者なのですが、質問させてください。

クリックした画像を3.jpgに書き換えるというコードを書いたのですが、うまく動きません。
どこが違間違っているのでしょうか。

/-------------------------------------------------------------/
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>神経衰弱</title>
<style type="text/css">

ul {
width: 300px;
list-style:none;
margin: 0;
padding: 0;
}

li {
float:left;
margin: 0;
padding; 0:
}

img {
vertical-align: bottom;
}

#all {
width: 300px;
margin: 0px auto 0px;
margin-top: 50px;
}

</style>
</head>
<body>

<div id="all">
<ul>
<li><img src="none.jpg" alt="3" width="100px" height="100px" id="c_1" onclick="conce(c_1)" /></li>
<li><img src="none.jpg" alt="1" width="100px" height="100px" id="c_2" onclick="conce(c_2)" /></li>
<li><img src="none.jpg" alt="4" width="100px" height="100px" id="c_3" onclick="conce(c_3)" /></li>
<li><img src="none.jpg" alt="4" width="100px" height="100px" id="c_4" onclick="conce(c_4)" /></li>
<li><img src="none_2.jpg" alt="" width="100px" height="100px" id="c_5"/></li>
<li><img src="none.jpg" alt="2" width="100px" height="100px" id="c_6" onclick="conce(c_5)" /></li>
<li><img src="none.jpg" alt="1" width="100px" height="100px" id="c_7" onclick="conce(c_6)" /></li>
<li><img src="none.jpg" alt="2" width="100px" height="100px" id="c_8" onclick="conce(c_7)" /></li>
<li><img src="none.jpg" alt="3" width="100px" height="100px" id="c_9" onclick="conce(c_8)" /></li>
</ul>
</div>

<script type="text/javascript">

function conce(aaa){
document.getElementById("aaa").src = "3.jpg";
}



</script>
</body>
</html>
/-----------------------------------------------------------------------/

よろしくお願いします。

投稿日時 - 2013-10-28 14:35:20

QNo.8324281

すぐに回答ほしいです

質問者が選んだベストアンサー

引用符が付け間違えしています

誤: <img src="none.jpg" id="c_1" onclick="conce(c_1)" />
正: <img src="none.jpg" id="c_1" onclick="conce('c_1')" />

誤: document.getElementById("aaa").src = "3.jpg";
正: document.getElementById(aaa).src = "3.jpg";

本質ではありませんが、一緒にこれも直しましょう

誤: <img width="100px" height="100px"
正: <img width="100" height="100"

投稿日時 - 2013-10-28 15:03:16

お礼

お返事ありがとう御座います。

なるほど、引用符を間違えていたのですね。
関数の引数とidを同じにするとまずいのかなと少し考えていたのですが、このやりかたでも動くようで安心しました。

htmlの間違いも指摘いただきありがとう御座います。
ピクセル指定の場合、単位は要らないのですね。
勉強になりました。

ありがとう御座いました。

投稿日時 - 2013-10-28 15:23:47

このQ&Aは役に立ちましたか?

1人が「このQ&Aが役に立った」と投票しています

回答(2)

ANo.1

検証していませんが、これでどうでしょうか?


onclick="conce(c_1)" → onclick="conce(this)"

document.getElementById("aaa").src → aaa.src

投稿日時 - 2013-10-28 14:56:26

お礼

お返事ありがとう御座います。

こちらのやり方を試したところ、無事動きました。
ありがとう御座いました。
thisを使うやり方もあるのですね。
勉強になりました。

投稿日時 - 2013-10-28 15:23:35