絶対に笑ってはいけないRailsコード集 - Qiita

株式会社LITALICO のklrutsaです。 『LITALICO Advent Calendar 2016』13日目の記事です。 はじめに 私が遭遇した、Railsアンチパターン集です。 笑えるよりも、笑えないコードのほうが多いですが、よろしくお願いします。 前回の、負債を抱えすぎたRailsアプリのリファクタリング - Qiitaでは、複雑な状態遷移への対応方法を書きましたが、その他の負債をどうしたかみたいなことについて書いてみます。 一般的に書いてはいけない、とまではいえないかもしれないですが、 個人的には書かないほうが良いと思っているコード集です。 default_scope class Article < ActiveRecord::Base default_scope { where(status: 'publish') } end 要点 プログラマの認識している動作と実際の

[pochi-mk]

Railsに限らず「絶対に笑ってはいけないコード」ネタでアドベントカレンダー作ったら面白そう。

[tofu-kun]

逆に遭遇してみたい気持ちあるけど、遭遇したら気持ち死ぬ

[shifumin]

そうだねって感じで読み進めていたけど、一番最後で死んだ。

[razokulover]

Rails関係ないけど、 個人的に今年一番笑ってはいけないコードは if true # ゴメンな ってやつ。過去の俺が現在の俺に 書いたっぽいけどなんかその時疲れてたんだろな。

[nntsugu]

タイトルの毒気が心地よいですね

[masayoshinym]

Python版が見たい。

[akatakun]

プログラマの認識と実際の動作がズレる，Default Scopeをモデルの特定の条件を表した集合を定義する際には役に立つ

[teracy_junk]

『僕はrm -rfが一番笑えました。』あっ…（察し）

[hamichamp]

インスタンス変数はやっちゃうかも。

[tkrd]

解決策: default_scopeは使わない "articlesを参照するとdefault_scopeが有効になってしまいます"

[hyottokoaloha]

良記事

[Sixeight]

積極的にrmしていきたい