nyoro2pu 2016-06-07 11:49:14

[JavaScript] 強烈な異臭のダブル攻撃… + 中身もごみ このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

これで動いてるのがすごい。

var index = 0;
for (index++; index < value.length; index++) {
	== 色々臭いコード
	if (臭い) {
		break;
	}
}


for (index++; index < value.length; index++) {
	== 色々臭いコード
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(6)

#1 kwakita 2017-05-21 01:29:37  

11行目は実行されない?

#3 deigotter 2017-06-10 12:14:11  

index++;が最初についてるのがもっともウンコ

#4 exliDevelop 2017-10-27 08:44:57  

初期値いくつよ……? あとforをなぜbreakしたがるのか。

#6 InternalServerE 2017-10-28 09:07:25  

deigotterさんのコメントが一番しっくりくるかな。
データが悪いのか、コードがバグってるのかは不明だけれど、0番目がスキップされてるのは違和感あるね。

それ以外のコードは、まぁ汚いっちゃ汚いが、何がしたいのか察することはできる。

  • value内の要素が「breakに引っかかるまで」と「それ以降」に大別される。
  • 何個目の要素でbreakが発生するのかをループ前に知る手段はない。

ということなら、この「forループ2回」のコードパターンは、ある意味仕方ないっちゃ仕方ない気はする。

ちなみに、11行目のコードは実行される可能性ありますよ、kwakitaさん

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[PHP] 見た目はコメント、中身は未定義

このエントリーをはてなブックマークに追加

シンタックスハイライトに騙されてしまいま...

<!-- 未定義なのでコメントアウト
<?php undefined_func...

鑑賞する »

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[PHP] ログイン処理

このエントリーをはてなブックマークに追加

もう突っ込みどころが多すぎで、卒倒したソ...

<?php


/****************************...

鑑賞する »