「ん?」と思ったのは以下5点です。ある側面では「動けばよい」とも言えるものですから、あまり深くは気にしないでくださいね。
- nowをcprの引数でもらおうとしているけど、呼び出し側で与えていないこと。
- 関数の内部でも複数回nowを宣言していて、使いまわしているところ。
- console.logを呼び出して、その戻り値を返しているところ。
- cprの呼び出して戻り値を利用していないところ
- インデントの付け方。
そこで、cpr関数が「コピーライト文字列を返す」ものだとして、以下のように変更してみました。
これが正解ってわけじゃないです。
function cpr() {
var now = new Date();
var year = now.getFullYear();
if(year != 2015) {
return "(c) 2015 - " + year + " / なまえ";
}
return "(c) 2015 - / なまえ";
}
console.log(cpr());
以下、おせっかいコードです。
WEBページのコピーライトを更新するなら、ページが表示された時にこの関数を呼び出して、戻り値をページ内に表示しないといけないですね。そのためのHTMLも書いてみました。
lang
1<html>
2<head>
3<script type="text/javascript">
4function cpr() {
5 var now = new Date();
6 var year = now.getFullYear();
7 if(year != 2015) {
8 return "(c) 2015 - " + year + " / なまえ";
9 }
10 return "(c) 2015 - / なまえ";
11}
12function pageOnLoad() {
13 document.getElementById('copyright_area').innerHTML = cpr();
14}
15</script>
16</head>
17<body onload="pageOnLoad()">
18
19ウェブのコンテンツ(略)
20
21<span id="copyright_area"></span>
22</body>
23</html>
上のコードは動作の確認していませんのでちゃんと動かないかもしれません。
他にも、いろんな作法やマナーみたいなものがありますけど、とりあえず無視して書いてみましたので、熟練者さんの突っ込みはご容赦。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2015/02/14 12:12