はてなブックマーク

賛成反対かと言われるとどちらともいえないです。 理由としては結構表面的ですが、こんな感じです。 賛成理由：可読性がif-elseより良いから。 完全に主観ですが、最初だけifで、残りがelse ifだと、スペースを入れないと条件式のインデントが揃わなくて見づらいです。 反対理由：汎用的な書き方でないから。 今回のサンプルではJavaScriptですが、Cとか古いバージョンのJavaではこんな書き方はできません。 JavaScriptだとif-elseとswitchの速度差ってどうなのかなと思いましたが、このサイトにこんなコメントがありました。 大量の条件分岐を是としないなら、速度差がないというのも仕方ないとは思いますが。 ・[JavaScript] とっても長いswitch/case文 http://unkode-mania.net/view/5018ddaa7e19b40f5a00000

#define NUM_PORT 8 #define NUM_CHANNEL 8 unsigned char data[NUM_PORT][NUM_CHANNEL]; #define BATTERY_VOLTAGE 0][0 #define MAINPOWER_VOLTAGE 0][1 // (略) #define MOTOR_CURRENT 7][6 #define MOTOR_TEMPERATURE 7][7 #define LOW_BATTERY 0x30 void output_log(void) { unsigned char i,j; set_new_page_log(); for(i=0;i<NUM_PORT;i++) { for(j=0;j<NUM_CHANNEL;j++)add_log(data[i][j]); } } void check_battery_voltage

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

nullをスローするので、StackTraceが出るときにヌルポ。 コンパイラも通る。 Findbugsで検知。 private Hoge doHoge(List<String> foo) { for (String str : foo) { if (...) { ... } } throw null; // ← なんだコレ } 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

"とりあえず"で書いた 2010 年 4 月。 これくらいは許容範囲だと思った 翌 5 月。 気になってるんだけど直すスキルが無い 7 月。 何かを諦めた 12 月。 そして時は流れ... 2011 年、暮れ。 誰かが冗談で書き添えた else 節。 // switch-case にしたいが、関数で値を取得しないといけないため、 // やむを得ず if-else を使うことにする。(2010/12/15 課長承認) if(flag == get_flag_2010_04()) { write_data(2010, 4); } else if(flag == get_flag_2010_05()) { write_data(2010, 5); } else if(flag == get_flag_2010_07()) { write_data(2010, 7); } else if(flag

例外って何だろう。SQLってなんだろう。エラーとは。そして帰ってこない関数。 先人の知恵をすべて無に返す実装。 // DBに接続 $pdo = new PDO($dsn,$user,$password); $id = $_POST['id']; $data = getHogeData($pdo,$id); $smarty->assign('data',$data); $smarty->display(); function getHogeData(&$pdo,$id){ try { // データの取得 $sql = "SELECT * FROM hoge"; $stmt = $pdo->query($sql); while($data = $stmt->fetch()){ if($data['id'] == $id){ return $data; } } } catch(exception

javascriptが悪いのかhtmlが悪いのか・・・仕様の問題な気もします。 iframeとかheadとかでもおかしな事になるので是非試してみてください。 <form name="getElementById"></form> 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

山田さん(仮名)専用機らしいことがクラス名からも推察できる。 再発明された車輪が大盛りになっている。 すでに辞めた人の似たような専用機もついでに発掘される。 /** * My Army Knife * 20XX/XX/XX * @author X. Yamada(仮) */ public class YamadaUtil { public static double max (double a, double b) { … } public static String[] yamadaSplit (String str, String delimiter, boolean isRegExp, boolean escape, boolean ignoreXxx, boolean ignoreYyy, boolean isDebug) { … //TODO … } public static

[Java] 以前のコードが残されていて、何がなんだかもうっ！っていうコード 投稿者からのアピールポイント /* --- 2007.06.21 故障-ST-001 start --- */ // public void uncode (int status_code) { // int status = 0; // List unkodes = null; // if (status_code == 1) { // unkodes = fetch_unkode(); /* #### 2006.05.11 仕様追加-ST-012 k.sato start #### */ // // } else { // } else if (status_code == 2) { // unkodes = search_unkode(); // } else { // unkodes = create_u

class Hoge { /* * インストラクター */ function Hoge() { } } 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

class hoge { public: hoge() { memset(this,0,sizeof(*this)); } virtual void foo(){} }; 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

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

[Java] 内部的にunicodeだから日本語つかえるけど・・・いくら英語苦手だからって・・・ 投稿者からのアピールポイント これを書いた先輩に「英語で書きましょうよ！」と言ったら、「頭が固い」と一蹴された。そういうことじゃない。 ちなみに先輩はこれをコードアシストの無いただのテキストエディタで、日本語入力を巧みにon/offしてコーディングしているから、ある意味すごい。 class 会員 { private int 会員番号; private String 名前; public int get会員番号 () { return this.会員番号; } public String get名前() { return this.名前; } public void 入会する() { .... } public boolean 会員状態をチェックする() { .... } .... } 使い方ヒ

/*======================================================================*/ /*　関数名称 : */ short _strsyori6 /* */ /*　機能概要 : 文字列処理６ */ /*　入 力 値 : */ ( /* */ /* : */ int mode /* 処理モード */ /* : */ , unsigned char * moji_in /* 入力文字列 */ /* : */ , unsigned char * moji_out1 /* 出力文字列１ */ /* : */ , unsigned char * moji_out2 /* 出力文字列２ */ /* : */ ) /* */ /*　戻 り 値 : 受け取った文字数 */ /*=================================

java.util.Dateには確かに非推奨ですが YYYY/MM/DD の文字列を取れるコンストラクタがあるようです。 が、それ文字列ですか？ 01 と書くと8進表記になるのも無駄に ポイント。 Date kijyun = new Date(1980/01/01); 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

もはや人間が読むものではない。 その昔、「連番やめましょうよ」と提案したところ、「カンニングペーパーをモニタに貼っとけば、何のクラスかすぐわかるでしょ？」と言われて、早く脱出することを決意したことがあります。 package com.renban.erq053.czp008; /** * ZWQI001 クラス */ public class ZWQI001 { /** * m_F001 */ private String m_F001 = ""; /** * コンストラクタ */ public ZWQI001() { } /** * m_F001 を返却します。 */ public String get_m_F001() { return m_F001; } ..... 使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter

この操作をするためにはTwitter認証が必要です。 Twitter認証をすれば、コード投稿をしたりコメントしたりsmellボタンを使えるようになります。