ウンコード・マニア
test79824007 2013-01-11 04:30:22

[PHP] 配列をコメント代わりに使用し、「5,7,6,8,10」 このエントリーをはてなブックマークに追加

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

インデントが配列にひっぱられて最後に戻すとこもさすがです。

    /**
     *  ○○費を計算します。
     *
     */
    private function _compute_hoge($hoge1_type,$hoge2_type,$hoge3_type){
      
    

      /*
      ○○費の価格一覧
        ※価格一覧表をテキストで表現したコメント

      */

      //○○の名称（配列１番目）
        $hoge1_list_data = array(
          1 => '○○',
          2 => '○○',
          3 => '○○',
          4 => '○○',
          5 => '○○',
          6 => '○○',
          7 => '○○',
          8 => '○○',
          9 => '○○',
          10 => '○○'
          );

      //○○の名称（配列２番目）
        $hoge2_list_data = array(
          1 => '○○',
          2 => '○○',
          3 => '○○',
          4 => '○○'
          );

      //○○の名称（配列３番目）
        $hoge3_list_data = array(
          1 => '○○',
          2 => '○○'
          );

        //※価格は10000に統一して記述

        //○○
          $k_list_data[1][1][1] = "10000";//○○
          $k_list_data[1][2][1] = "10000";//○○
          $k_list_data[1][3][1] = "10000";//○○
          $k_list_data[1][4][1] = "10000";//○○
        
        //○○
          $k_list_data[1][1][2] = "10000";//○○
          $k_list_data[1][2][2] = "10000";//○○
          $k_list_data[1][3][2] = "10000";//○○
          $k_list_data[1][4][2] = "10000";//○○

        //※上記のパターンで配列1番目の順に4まで繰り返し

        //※以降、省略のため先頭のみ記述
        //○○
          $k_list_data[5][1][1] = "10000";//○○

        //○○
          $k_list_data[7][1][1] = "10000";//○○
          
        //○○
          $k_list_data[6][1][1] = "10000";//○○

        //○○
          //$k_list_data[8][1][1] = "10000";//○○

        //○○
          $k_list_data[8][1][1] = "10000";//○○

        //○○
          //$k_list_data[10][1][1] = "10000";//○○
    
      //○○費を返します。
      return $k_list_data[$hoge1_type][$hoge2_type][$hoge3_type];
    }

コメント(5)

#1 m_shige1979 2013-01-12 10:25:24  

無駄にロジックが単純なのに長そうですね

#2 Chiether 2013-01-12 10:56:31  

た、たぶん将来的には、名称もreturnするんだよ。きっと。　たぶん。　おそらく。　maybe...。 ちなみに、『将来のために』は、排除すべしと手元の本に……。

#3 Miraranran 2013-01-13 00:24:05  

逆に将来じゃなくて過去使ってたけど、処理で使わなくなった定義部分だけ残されてるとか？

#4 token7 2013-01-13 00:33:47  

使う宛の無い変数が生きてるのは気持ち悪いけど、うんこーどと言う程ではないんじゃない？やめた方が良い程度。

#5 Verna_Velna 2013-01-15 01:53:15  

関数が呼ばれる度にデータ配列を再生成してることについては何か突っ込んだ方が良いのだろうか。

« PHP一覧に戻る

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

[HTML] 日本語のサイトにlang="en"

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

今見ているサイト。

<!DOCTYPE html>
<html lang="en">
  <he...

鑑賞する »

[C++] constとは一体…

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

constの意味あるの…？

class Hoge
{
private:
　　// メンバ変数は数百ある...

鑑賞する »

[C] ( 0][0 )/

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

とある組み込み系のコード。元気に動いては...

#define NUM_PORT 8
#define NUM_CHANNEL ...

鑑賞する »