Chiether 2012-11-24 02:52:38

[Ruby] [Rails] 鉄壁の構え

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

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

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

  1. class Hoge

  2.  

  3. def func

  4. transaction do

  5. func1

  6. func2

  7. end

  8. end

  9.  

  10. def func1

  11. transaction do

  12. # findするだけのメソッドだったり

  13. # アクセサ操作するだけのメソッドだったり

  14. end

  15. end

  16.  

  17. def func2

  18. transaction do

  19. # 別のモデルを操作している何かだったりもする。

  20. # 当然そっちもtransactionの嵐

  21. end

  22. end

  23.  

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

  25. end

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

コメント

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

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

Twitter認証

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

* [Java] is禁止令

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

// Mod yamada Start

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

鑑賞する »

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

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

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

鑑賞する »

* [PHP] つ、釣り!?

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

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

鑑賞する »