宣伝

プログラマ向けの謎解きゲームを作りました

http://kokuyouwind.com/a

宣伝

• Githubで公開中
  • http://kokuyouwind.com/b
  • 「プログラマ 脱出ゲーム」でググると多分出てくる

http://kokuyouwind.com/a

RubyKaigi2015

http://kokuyouwind.com/a

http://kokuyouwind.com/a

http://kokuyouwind.com/a

私が人生で見た

最悪のRubyコード

http://kokuyouwind.com/a

http://kokuyouwind.com/a

聞きに行った

http://kokuyouwind.com/a

感想

• 実際、割と酷かった
• ​RubyKaigiのセッションページに動画がある
  • ​教訓もあり、良い発表だった

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC

    /view - MVCのV

    /template - Smartyテンプレート

http://kokuyouwind.com/a

controller

• MVCのC…ではない！
• 大体のメイン処理を書く場所
• 変数名が大体ひどい
  • 後述する
• 例外駆動プログラミングという技法がフル活用される
  • 後述する
• とりあえずひどい
  • 後述する

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC メイン処理を書く場所

    /view - MVCのV

    /template - Smartyテンプレート

http://kokuyouwind.com/a

view

• MVCのV…ではない！
• 謎
  • そもそもテンプレートは別にある
  • controllerから受け取った値を表示用に加工する？
    • 大体は受け流すだけ
    • たまにネストのレベルを変えてて悶絶する
• できることはControllerとほぼ一緒
  • たまにすごい処理が書いてあってビビる

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC メイン処理を書く場所

    /view - MVCのV 謎

    /template - Smartyテンプレート

http://kokuyouwind.com/a

model

• MVCのM…ではない！
• 配列を上手く扱うためのクラス
  • ドメインモデルのデータはすべて配列で扱う
  • (PHPの配列は連想配列)
• DBとの入出力も扱う
  • PDOでクエリを直接投げる
  • 独自のクエリビルダーが4種類くらいある
• どこからでも$this->getModel('Name')で取れる

http://kokuyouwind.com/a

model

<?php

class UserModel extends HogeFWModel
{
  public function get($id, $ignore_delete = false) {
    $table = $this->get_table_name($id); // ソフトシャーディング
    $query = "SELECT * FROM $table WHERE id = ?";
    if (!$ignore_delete) {
      // deletedが未来のことがあるので、IS NULLだけではだめ
      $query .= "AND (deleted IS NULL OR deleted <= '$(data('Y-m-d h:i:s'))'");
    }
    $records = $this->db->execute($query, $id);
    if ($records === false) {
      // 厳密にfalseならSQL接続エラー
      throw new SQLException('SQL接続エラーです');
    } elseif (!$records) {
      // クエリは成功したが中身が空
      throw new NotFoundException('ユーザーが見つかりませんでした');
    }
    $user = $records[0];
    // 他に必要な情報を補完する
    return $this->appendInfo($user);
  }
}

http://kokuyouwind.com/a

model

// こんな感じで使う
// callはメソッドの呼び出し結果をキャッシュする関数
$this->cache->call('User', 'get', $id);

// callには配列形式で渡しても良い
$um = $this-> getModel('User');
$this->cache->call([$um, 'get'], $id);

// 返ってくるのは配列(というかハッシュ)
['id' => '758',
 'name' => '黒曜',
 'registered' => '2016-04-16 15:50:00', // 大抵MySQL形式
 'last_login' => 1460789400, // たまにUnixTimeが入っている
 'attributes' => [
   'belongs' => [
     'Misoca' => ['id' => '331'] // どんどんネストする
   ],
   'is_penalty' => 'none', // なぜか真偽値ではなく文字列
 ],
 'append_info' => [
    // 3段くらいネストしたなんらかの情報
 ]
]

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM 配列の加工とDB処理

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC メイン処理を書く場所

    /view - MVCのV 謎

    /template - Smartyテンプレート

http://kokuyouwind.com/a

logic

• 共有ロジック…ではないものがほとんど
• 大体はcontrollerの中身を分割したもの
  • バージョン移行の際に、controllerの処理を
    ​まるまるコピーしlogicクラスができたりした
• controllerでできることはlogicでもmodelでもできる
  • たまにlogicやmodelから直接viewに渡す値を
    設定していてビビる
• どこからでも$this->getLogic('Name')で取れる

http://kokuyouwind.com/a

2文字変数

http://kokuyouwind.com/a

変数名が酷い

<?php

class ContentController extends HogeFWController
{
  public function execute() {
    // 大体2文字変数
    $cm = $this->getModel('Content');
    $nl = $this->getLogic('Normalize');
    $cont = $this->cache->call([$cm, 'get'], $this->getAttribute('id'));
    $cont = $nl->norm($cont);
    // 間に300行くらい挟まる
    if($has_body) {
      $cont = $nl->normBody($cont);
    }
    // 間に500行くらい
    if ($need_next) {
      // さっきの$nlを上書きする
      $nl = $this->getLogic('NextContent');
      $content['next'] = $nl->calc($content);
    }
    // ...まだ続きます

http://kokuyouwind.com/a

変数名が酷い

    // ...続き

    // 間に300行くらい
    // さっきの分岐を通ってると死ぬバグ
    $content['append'] = $nl->normAppend($content);
    //間に200行くらい
    if ($need_extra) {
      // 明らかに$nlじゃないけど$nlを上書きする
      $nl = $this->getLogic('Extra');
      $content['extra'] = $nl->compute($content);
    }
    // 間に300行くらい
    // viewにcontentを受け渡す(Model, Logicからも可能)
    $this->setAttribute('content', $content);
    return;
  }
}

http://kokuyouwind.com/a

例外駆動プログラミング

http://kokuyouwind.com/a

例外駆動プログラミング

<?php

class ContentController extends HogeFWController
{
  public function execute() {
    try {
      $cm = $this->getModel('Content');
      $content = $this->cache->call($cm, 'get', $this->getAttribute('id'));
      if(!this->user) {
        throw new NotLoginException();
      }
      // ログイン時、コンテンツありの処理が1000行くらい続く
    } catch (NotLoginException $e) {
      // 非ログイン時の処理
      if ($cm->isAllowGuest($content)) {
        // 非ログイン時、コンテンツ閲覧可の処理が800行くらい続く
        // うち600行くらいはログイン時処理のコピペ
      } else {
        // view/content_error_viewを描画する
        return 'content_error';
      }
    }
  }
}

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC

    /view - MVCのV

    /template - Smartyテンプレート

http://kokuyouwind.com/a

public

• webからのエントリポイント
  • apacheでDirectoryとして指定される
• Controllerをkickするphpファイルが配置される
  • URL routesの数だけファイルが置かれる
  • Router? なにそれおいしいの?
• 画像/js/cssなどの静的ファイルもここに配置される

http://kokuyouwind.com/a

構成

/core

    /model - MVCのM

    /logic - 共有ロジック  

/for_xxx

    /public - web公開ディレクトリ

    /controller - MVCのC メイン処理を書く場所

    /view - MVCのV 謎

    /template - Smartyテンプレート

http://kokuyouwind.com/a

template

• Smartyのテンプレートを置く場所
• ネストが非常に深い
  • 3重ifの中身が数十行とかはざら
  • ロジック分離？ なにそれ美味し(ry
• 多言語対応の影響でさらにカオスになった
  • 日本語以外では出さない、みたいな分岐が
    そこら中にある

http://kokuyouwind.com/a