JSのお勉強 公開コードレビュー その1
初心者hinatamiがJavaScriptを勉強するにあたり、コードレビューをしていただけることになりました。なぜかTwitter上で。その公開コードレビューのまとめです。
-
- いいね！0
edom18@XR / MESON CTO
@edo_m18
@hinatami @kyo_ago これ、DOMツリーを捜査して見つけようとしているんだと思うけどchildrenは孫要素までは取ってこないから、実際ループは1回しか処理されず、innerHTMLでHTMLをテキスト化しているからマッチして成功してる気がする。
2015-02-14 22:54:14
edom18@XR / MESON CTO
@edo_m18
@hinatami @kyo_ago なので「document.getElementById('fiona-publisher-signup-link').innerHTML.match」ってやっても動くと思う。
2015-02-14 22:54:54
ひなたみ
@hinatami
@edo_m18 @kyo_ago そうですｗ その通りですｗ なんか色々と試してみた名残が残っててよくわからないことしてましたねｗ （笑い事じゃない…うぅっ…）
2015-02-14 22:57:58
edom18@XR / MESON CTO
@edo_m18
@hinatami @kyo_ago ちなみにこういう部分で、意図を明確にする意味でコメントとかあるといいね。久々に読むときも「なんでこう書いたんだっけ？」ってならなくて済むしｗ
2015-02-14 22:55:57
kyo_ago
@kyo_ago
@hinatami @edo_m18 では次のお題です「平日の昼間にamazonを見れなくなる」ユーザスクリプトを書いて下さい。ベースはこのコードを使って下さい（jQueryが使えるようになっています）期限はお任せします。ファイトー gist.github.com/kyo-ago/fd6250…
2015-02-14 23:21:05