blackwing_n 2012-08-17 05:05:58

[JavaScript] よくある失敗、よくみる失敗。 このエントリーをはてなブックマークに追加

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

この気持ち、分かってもらえるはず.....さぁ....

var a=true;var b=[1,2,3];var c="12";var z=0; //varは,でつなげると綺麗
if(a===true){ //if(a)でいいだろ
 for(var i=0;i<array.length;i++){
  //どうみてもサイトからのコピーですarrayじゃなくてbに変えないと
  //初心者よくやるやつ
  var d=document.getElementByTagName("div")[i].innerHTML+c;
  //↑Element「s」じゃないとダメ divの内容は数値とする
  if(d.indexOf("12")!==-1){console.log("12含まれてる")}
  //dはNumになってるからindexOf使えない toString()必要
  var h=new Date().getHours();
  var m=new Date().getMinutes(); //Dateオブジェクト作れよ
  //toString反省してみる
  if(h.toString().indexOf("1")!=-1){coonsole.log("1時")} //11・12時でもヒットする
 z++; //先頭に全角スペース 気づかないと悩む
  document.write(i.toString()); //ページ真っ白になって涙目
 }
 return z;13:59 2012/08/17   //←メモ帳でF5押しちゃって日付と時間が入る
 //地味によくあるバグの原因だったり
 document.getElementById("post").submit(); //return 文の後は実行されない
 //ついついやっちゃう
}

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

コメント(16)

#1 Fushihara 2012-08-17 05:11:03  

document.querySelectorってほんっっっっっとうに便利だと思うんだ

#2 blackwing_n 2012-08-17 05:19:22  

Fushihara jQueryのアレに近いよね でもIE6対応してないからなぁ 未だにWin2000軍すらいるというのに。 正直、新しい規格とか出来てもそれが脚光を浴びるのは実装されてから何年後になることやら。 ほぼ完全に普及しないと使えないから困る。。特にIE

#4 diaqt 2012-08-22 05:23:37  

2行目と3行目がェって感じ

#5 htsign 2012-08-22 13:01:43  

「メモ帳でF5押しちゃって日付と時間が入る」で吹いた

#6 BlackerPanda 2012-08-27 08:33:01  

var直せよwwてか、a===trueってこまめすぎ書いたこともみたこともなかったwwif(a)だろ

#7 osaru3j 2012-09-24 08:40:13  

「先頭に全角スペース」←眠い時に、たまにやってしまうw しかし、秀丸の表示で解るようににしてあるから、救われているw

#9 ogatama 2012-10-12 09:06:56  

3行目、これよく見るけど個人的には一番気持ち悪い。 for文の条件の中にarray.length書いちゃうとループ中に都度配列の長さ数えるから変数に入れとけってじっちゃが言ってた。

#10 math_neko 2012-10-12 09:11:27  

このウンコードを JavaScript 初心者がファックファック言いながら書いてるところを想像しただけでご飯三杯余裕

#11 InternalServerE 2012-10-12 11:41:59  

それは、低いな。。

#12 math_neko 2012-10-13 09:45:09  

> for文の条件の中にarray.length書いちゃうとループ中に都度配列の長さ数える

そう言えば JavaScript の配列って可変長なんだっけか。それだと確かにループの条件式として使うのは怖いなぁ。

#13 InternalServerE 2012-10-13 15:13:59  

なんか話が噛み合ってないね。ogatamaさんは性能視点で言ってるんでないの。
array.lengthを変数に持たせる云々の話題と、可変長配列を駆動表としてループする云々の話じゃ、観点が全然違ってる。

#14 hardtimes777 2012-10-14 00:14:43  

fgetsでググって最初に見つかるサイト。 http://hitorilife.com/fgets.php こういうのみると、ogatamaさんのじっちゃと同じこと言いたくなるなぁ。

#16 math_neko 2012-10-31 01:07:33  

> ogatamaさんは性能視点で言ってるんでないの

あ、そういう意味でしたか。

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

Twitter認証

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

[JavaScript] <noscript>

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

3年ぐらい前に見かけたコード

<!-- 以下 body タグ内 -->

<noscript>
    ...

鑑賞する »

[PHP] 初心者の頃に書いたプログラム

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

/home/username/www/ ...

<?php
ini_set("display_errors", "off");...

鑑賞する »

[C#] コーディング時最適化

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

最適化の効いた良いコードだと思います。 ...

static void Main() {
    Console.WriteL...

鑑賞する »