テンプレートをDRYにするのは慎重にやったほうがいいですよねというお話 - 猫型の蓄音機は 1 分間に 45 回にゃあと鳴く

社内でレビューおじさん業してて書いた内容ですけど守秘する必要ない情報なんでちょっと内容書き換えてオープンアンドシェアーします。 本文 見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品（些細な変更されることが多い部品）なので、「同じような部品だから共通化しちゃおう」ってやると失敗することが多いです。 特に気をつけるべきなのは、たとえばコンテンツをランキング形式でテーブルで表示する画面と、新着から順にテーブルで表示する画面があって、このふたつのテーブル部分は一緒だからパーシャルにしちゃおう、みたいなやつです。 見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品、というのがここで効いてきて、「新着とランキングは基本的に同じ表示なんですけど、ランキングのほうではランクがアップしたかダウンしたかのアイコンを表示してほしいんですよね〜」とか言われたり、「今見てるページ

[lalala360]

分散したものを後から共通化するより、共通のものをバラすほうが楽だと思うので、迷ったら共通化してる。

[te2u]

UIに限らず「同じような部品だから」と「同じ部品だから」は異なるもので、この見極めが難しい。特に、挙動は同じだが仕様が異なる場合、挙動だけ見て同じようだと判断して共通化すると、後で痛い目に遭う。

[tdakak]

"見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品"

[raimon49]

＞見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品（些細な変更されることが多い部品） / 知見。ここが結局テストの書きづらさにも繋がってる。

[masa8aurum]

自分も同じようなことを思っていた。 / ビュー層に適した抽象化はどのようなものなのだろう。

[fumikony]

むずい

[otchy210]

こと HTML においては、見た目の問題だけだったら css で片付くことがほとんどなので、この例くらいなら共通化しても if 要らないし、メリットの方が多いと思う。

[woshidan]

属性が同じだけど違うエンティティをDBの同じテーブルに突っ込んでしまってやっぱりまずかったので分離した的な思い出(激しく自省)

[rryu]

今は見た目は同じだけど意味が違うので別のテンプレにしたら結局見た目は別にならず毎回テンプレを2枚修正する羽目になったという悲しい思い出が…

[taketyan]

理想的には UI が正しくコンポーネント化されていれば、共通の A から専用の B への差し替えが簡単にできると思から共通化を迷う必要はないのでは

[Dai_Kamijo]

"見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品" / “テンプレートをDRYにするのは慎重にやったほうがいいですよねというお話 - 猫型の蓄音機は 1 分間に 45 回にゃあと鳴く” — tadaaki (@tdakak) F

[izumin5210]

「柔らかい部品」 いい言葉だ “見た目とかUIというのはソフトウェアの中でめちゃめちゃ柔らかい部品”

[tbpg]

(ฅ ･ิ ω ･ิ )ฅ { 俺、DRY。ｷﾘｯ。とかやってると痛い目を見るパターンですね }

[side_tana]

大事だ