気に入った記事をブックマーク
- 気に入った記事を保存できます
保存した記事の一覧は、はてなブックマークで確認・編集ができます
- 記事を読んだ感想やメモを書き残せます
- 非公開でブックマークすることもできます
www.m3tech.blogエントリーの編集
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント48件
- 注目コメント
- 新着コメント
オーナーコメントを固定しています
turanukimaru
複数の Enum の組み合わせから特定の Class を Map し、その Class に処理を書けばいい。継承や override はアンチパターンと言われて久しいけどそれは正しく使えていないだけだ。条件分岐を一切書かないのが真の OOP プログラマ
rck10
逆にenum使わずにclassでデザインしたら綺麗にできるの？そうでないならenumの辛みではない。 / 基本的には属性値として使うべきで、属性の保有者として使うとロジック過多で痛い目をみるイメージがある。
canadie
これをEnumで実装するのは開放閉鎖の原則（OCP）違反。ちょうどいい例があったから読んでみて https://gntk.dev/post/20210916-learn-solid-principle-ocp/
オーナーコメントを固定しています
vermeer-1977
1. 分岐条件をEnumに持たせる については、EnumSetをつかえば、上記の条件分岐毎ののtrue,falseを準備するよりは扱いやすくなるかな？と思った次第 参考URL https://qiita.com/suke_masa/items/bd242617a2b9bb773cbd [上級編]EnumSetの利用
yarumato
“種別が3種類以上の可能性があるならEnumを使うことになる。将来的に増えること、これがEnumの つらみの根源です。既存のif文が意図しない方に倒れる。データ分析のことまで考慮した対策：Enum要素は増やさない”
YaSuYuKi
Switch式が使えるということは14以上、サポートを考えると現実的には17以上だろうから、Sealed class/interfaceを使えば良いのでは。switchでもパターンマッチングで書けば通らないパターンがあるとコンパイルエラーになる
tsutaya0225
既出表現だが、elseとdefaultは必ずエラー処理とすれば、単独if以外は100%カバレッジできて防御的プログラミングとなり得る訳で、論点がまるで明後日の方向を向いてる。 そのため"だけ"にenumをclassにするとかもあり得ない。
daishi_n
enumをクラスの中に納めて見えないようにする局所化には必要だよね。プロパティでenum値を返すのではなく、enumに基づいた属性情報を返す方が適切かな。この手のだと認可系は今でもどう実装すべきか悩ましい
dorapon2000
“switch式とはswitch文をベースにしつついくつかの点で強化された複数分岐構文なのですが、中でもswitch文と違って網羅性チェックをしてくれるという大きな特徴があります。”
cbkf
仕事で書くコードなら普通 default で処理漏れ拾って「バグだよー」って例外投げると思うのだけど。そしたらテストで検出できる。/「カバレッジ100％はあり得ない」って、ここはカバー必須でしょ
daichirata
分析用の SQL への反映が辛いという話は非常によくわかるが、だからといって dbt とかではなく enum 側を増やさないで対応するのは本末転倒というか分岐の位置がズレただけな気も。言語の話はおまけっぽい。
yuutetu
goだとゼロ値を用意しておいて、ゼロ値をswitch文のdefaultで処理する。 TypeScriptだと(ユニオン型だが)全てswitch文で処理してdefault内ではNever型になっていることをチェックしつつassertしとおくと良い感じ。
ene0kcal
enumを使ってる以上、全てのenumのswitchやif文を検索して改修対象か調査すべきでそれを忌避したいならOOPで各要素をclassでまさに"分類"として定義し多態性を使利用したシステム設計をすべきでは？／既にブコメに同じ指摘が
ghostbass
ステートパターンとストラテジーパターン組み合わせたとてそういう変更要件への対応が楽になるわけでもなく…結局はドメイン次第になるのでは。const int だと多分もっとつらいことになる
PrivateIntMain
モデルのEnumがモデル外で直接参照できるようになっているのが間違いじゃなかろうか。モデル外で知りたいならモデルを介して聞くしかできないのが望ましい。まあその状態を保つのはめんどくさいけど。
Flume
Enumの辛さではないような.../intよりマシって意見多いみたいで確かにそれが正しいと思うけど、個人的にはintの方が好き。俺が苦手なだけだがEnumってなんとなく分かりづらいのと属性毎にクラス作るのが嫌
canadie
これをEnumで実装するのは開放閉鎖の原則（OCP）違反。ちょうどいい例があったから読んでみて https://gntk.dev/post/20210916-learn-solid-principle-ocp/
xlc
Pascalで育ちAdaの仕事もしたので元々は型にうるさい考え方だったけど、PerlやJavaScriptと長く付き合ううちに列挙型はクソと思うようになりました。拡張性がないよね。DOMのイベントにも列挙型は使われないでしょ。
dekaino
分岐条件なら、enumの型はintとして分岐条件の配列リスト作ればいいんじゃないのかなあ? if (condArray[enum_AA]) xxxx とかする。condArray[]の定義なり初期化するところを1か所にまとめておけばよし。
queeuq
switchもdafaultがないことを指摘すべきだし、大抵のよく使われる規約はそうなっている。カバレッジ100%にはならないとか書いてるがそれ今回実装が必要な正常系処理なのに漏れるの？テスト方針を見直すべきでは？
rck10
逆にenum使わずにclassでデザインしたら綺麗にできるの？そうでないならenumの辛みではない。 / 基本的には属性値として使うべきで、属性の保有者として使うとロジック過多で痛い目をみるイメージがある。
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
いまの話題をアプリでチェック！
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
関連記事
Enumとてもつらい、でも負けない - エムスリーテックブログ
列挙型、JavaでいうならEnum型、使っていますか。使わないわけにいきませんよね。 でも、Enumを使ってい... 列挙型、JavaでいうならEnum型、使っていますか。使わないわけにいきませんよね。 でも、Enumを使っていたせいで辛い目にあったことありませんか。ないですか。それならきっともうすぐに辛い目にあうと思います。 Enumはすべてのプログラマに等しく辛みを与えてくれるからです。そんな辛みについて、ちょっと一緒に直視してみましょう。 エムスリーエンジニアリンググループ、Unit1（製薬企業向けプラットフォームチーム）三浦(@yuba@reax.work) [記事一覧 ]がお送りいたします、エムスリー Advent Calendar 2023の6日目です。 アプリケーションプログラミング上の辛み 1. 既存のif文が偶発的に意図しない方に倒れる 2. switch文に至っては「どちらでもない」で処理不発に アプリケーションプログラミング上の対策 1. 分岐条件をEnumに持たせる 2. swi
ブックマークしたユーザー
すべてのユーザーの
詳細を表示します
詳細を表示します
www.m3tech.blog
www.m3tech.blog
www.m3tech.blog
www.m3tech.blog
blog.tenjuu.net
anond.hatelabo.jp
togetter.com
anond.hatelabo.jp
pyopyopyo.hatenablog.com
www.bloomberg.co.jp
blog.father.gedow.net
qiita.com/birdwatcher
forest.watch.impress.co.jp
blog.adobe.com
av.watch.impress.co.jp
dailyportalz.jp
www.ele-king.net
reinmeer-aomori.jp
2023/12/06 リンク