Chiether Chiether 2012-11-24 11:52:38

[Ruby] [Rails] 鉄壁の構え このエントリーをはてなブックマークに追加

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

おまじない教……という単語が思い浮かんだが。 それ以上考えるのを、やめた。

正直。 @unko.merge() のためだけに、transactionしているのを見た時は、目を疑った。

class Hoge

  def func
    transaction do
      func1
      func2
    end
  end

  def func1
    transaction do
      # findするだけのメソッドだったり
      # アクセサ操作するだけのメソッドだったり
    end
  end

  def func2
    transaction do
      # 別のモデルを操作している何かだったりもする。
      # 当然そっちもtransactionの嵐
    end
  end

  # 他、あらゆるメソッド内で全体をtransactionしている
end

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

コメント

まだコメントがありません。最初にコメントを残しませんか?

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

Twitter認証

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

[Java] is禁止令

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

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »

[Java] 内部的にunicodeだから日本語つかえ...

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

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...

鑑賞する »

[PHP] つ、釣り!?

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

ツボって思い出し笑いが止まらなかったので...

class Hoge {
    /*
      * インストラクター
...

鑑賞する »