1
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

どうでもいいコード(もしかしたら勉強になるかも)

Posted at

このHTMLコードは、無駄に効率が悪い、または意図的に使われている不合理なコードの例です。各部分を詳しく説明します。

1. イベントリスナーの追加

document.getElementById('box').onclick = function() { alert('クリックしたよ'); };
document.getElementById('box2').onclick = function() { alert('ここもクリックしたよ'); };
  • 説明:
    • これは onclick イベントをそれぞれ boxbox2 の要素に直接バインドしています。この方法は、addEventListener を使った方法に比べて柔軟性がありません(複数のリスナーを追加できない、イベントの削除が困難など)。
    • 理想的には、addEventListener を使うべきです。例えば、同じ要素に複数のイベントを簡単に追加でき、後で削除も可能です。

2. 色をランダムに1秒ごとに変更する

setInterval(function() {
    var colors = ["#ff0000", "#00ff00", "#0000ff", "#ffff00", "#ff00ff"];
    document.body.style.backgroundColor = colors[Math.floor(Math.random() * colors.length)];
}, 1000);
  • 説明:
    • 1秒ごとにランダムな色を選んで、body の背景色を変更します。
    • 無駄に背景色が頻繁に変わるだけで、ユーザー体験を損ねる可能性が高いです。通常、こうした変更はユーザーの操作に関連した場面で使うべきです。

3. 無意味な再帰的関数(スパムアラート)

function spamAlert() {
    alert('クリックし続けるよ!');
    setTimeout(spamAlert, 1000); // 終わらない再帰
}
spamAlert();
  • 説明:
    • このコードは spamAlert 関数を呼び出すと、1秒ごとに alert を表示します。しかもその後 setTimeout によって再帰的に自分自身を呼び出し続けます。
    • これは スパム的なアラート を発生させ、ブラウザを一時的に操作できなくする可能性があります。ユーザーにとって非常に不快であり、悪意のあるコードの一例です。実際には使用すべきではありません。

4. その他のスタイル(CSS)

body {
    background-color: #ff0000;
    color: white;
    font-family: Arial, sans-serif;
}
#box {
    width: 100px;
    height: 100px;
    background-color: yellow;
    margin: 50px;
}
#box2 {
    background-color: blue;
    height: 200px;
    width: 200px;
}
.weird {
    color: black;
    background-color: green;
    border-radius: 50%;
    position: absolute;
    left: 50%;
    top: 50%;
    margin-top: -25px;
    margin-left: -25px;
    z-index: 99;
}
  • 説明:
    • body は赤い背景色、白い文字色、Arial フォントが指定されています。これ自体は特に問題はありませんが、ユーザーにとって視覚的にあまり快適ではないかもしれません。
    • #box#box2 はそれぞれ異なるサイズ、色、配置が指定されていますが、どちらも目立つ色使いとレイアウトです。
    • .weird クラスは円形で中央に配置される要素を定義していますが、z-index: 99 を使って他の要素の上に表示されることになります。この要素が画面上で干渉する可能性があるため、あまり実用的ではありません。

まとめ

  • 効率が悪いコード:

    • イベントリスナーの付け方が非効率(onclick 直接設定)。
    • 無駄な繰り返しのイベントやアラートの表示(無駄に色を変えたり、再帰的にアラートを表示)。
  • 悪影響を及ぼす可能性のあるコード:

    • スパム的なアラート表示 (alert の再帰的呼び出し)。これにより、ユーザー体験を著しく損なう可能性があるため、実際には避けるべきです。

このようなコードは意図的に不合理に書かれており、通常は避けるべきパターンです。

1
0
1

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
1
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?