takezoux2 2012-08-14 06:29:06

[Java] ついやっちゃうんだ このエントリーをはてなブックマークに追加

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

結構やりがち。初心者だと確実にやる。むしろ今でもたまにやっちゃう。さらに、たまに正常に動いちゃう。

String value = ...  
    
if( value == "OK"){  
  // do anything  
}

コメント(10)

#1 karno 2012-08-14 06:44:05  

あるあるすぎて辛い

#2 akaibohshi 2012-08-14 06:47:35  

ついやりそうになる

#3 math_neko 2012-08-14 07:16:06  

初心者は確実にやらかす

#4 tehe819 2012-08-14 08:46:04  

完全初心者やね。

#5 k5n6 2012-08-14 13:16:56  

動かない時があるから見て欲しいといわれて、商品として稼働しているコードを貰ったんだけど、文字列チェックが殆どそうなってたよ。逆に動くときあったの?と思いたい。

#7 InternalServerE 2012-08-14 15:38:09  

VMの実装依存だが、Java VMは同じ値の文字リテラルを同じStringのインスタンスに紐付ける実装が多いので、「偶然」動く場合も多い。SUNのVMが確かそうだったかと。 当然NGっちゃNGですし、ユーザからの任意の入力文字列を == で比較してたら破綻するけどね。。

#8 oe_roel 2012-08-14 15:45:03  

String#intern()を呼べば、まだいける!

#9 k5n6 2012-08-15 03:52:57  

繰り返し比較するところで、String#internと==で高速化したことはあったけど、 ==でサラッと比較しちゃう人はinternの存在自体知らないと思う。

#10 Ukai_H 2012-08-27 05:26:09  

value.equals("OK")もいやだよね。


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

[JavaScript] とっても長いswitch/case文

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

配列変数を全く理解できていなかった。

var rnd = Math.floor(Math.random() * 128...

鑑賞する »

[その他] HOW TO ABC..

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

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »

[Java] フルHD推奨。

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

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »