Java入門・初心者質問スレ Part.5©2ch.net

**デフォルトの名無しさん転載ダメ©2ch.net** · 2017/09/29(金) 17:53:25.24

Java初心者のためのスレッドです。

※前スレ
Java入門・初心者質問スレ Part.4
http://mevius.2ch.net/test/read.cgi/tech/1498828456/

**デフォルトの名無しさん** · 2017/10/29(日) 13:18:28.37

>>339
いや、h2*.jar が入っているディレクトリごと指定しないと普通はダメなのでは？
カレントディレクトリにあるならそれでも良いのかも知れんが。

**337** · 2017/10/29(日) 15:33:37.93

コンパイルは通るのですが、実行できませんでした…。
ClassNotFoundExceptionとSQLExceptionが発生してしまいます。
どなたかヒントを下さい。
ソースコードは以下です。
https://ideone.com/4xrmgz

**337** · 2017/10/29(日) 15:35:28.67

>>340
すいません、今回はコマンドプロンプトで書かせて下さい。

**デフォルトの名無しさん** · 2017/10/29(日) 15:57:54.47

>>342
javacで取り込むわけではないので
javaで実行時にもCLASSPATHを通して見つかるようにしてやる必要がある

**デフォルトの名無しさん** · 2017/10/29(日) 16:23:24.35

>>342
コンパイル時と同じ classpath にしてもダメなの？

**337** · 2017/10/29(日) 16:26:53.49

>>344
はい。それは分かります。javacと同じクラスパスをjava実行時にも付けています。

実は根本的に、h2*.jarファイルがどこにあるのかよく分かってない気もするんですが、
H2 DatabaseのホームページからDL出来るh2-setup-2017-06-10.exeファイルの
中にあるんですよね？

**337** · 2017/10/29(日) 16:32:10.42

>>345
はい。
なのでjarファイルの位置指定がそもそも間違ってるんじゃないかと思い始めて
いるんですが…。.jarという拡張子のファイルが見当たらないんです。

**デフォルトの名無しさん** · 2017/10/29(日) 17:11:13.08

>>346
知らないけど
それは自己解凍ファイルなのかな
検索して実際のjarファイルを探してみればいい

コンパイル時はインタフェースさえわかればいいから
その実装のclassには通ってない可能性がある

ちゃんと何してるかから理解した方がいいと思うぞ

**337** · 2017/10/29(日) 20:16:42.57

>>348
あれからいろいろ調べて、Cドライブを検索した結果、h2*.jarファイルが
見つかりました！やはり.exeファイル内には無かったようです。

クラスパスを更新したら実行(javaコマンド)も無事成功しました。
回答して頂いた方々、ありがとうございました。

**デフォルトの名無しさん** · 2017/10/29(日) 22:13:14.03

コンストラクタで引数チェックをし、例外を投げることは設計上良くないのでしょうか？例えば
class hoge {
private final int i;

hoge(int i) {
if(i >= 0)
this.i = i;
else
throw new IllegalArgumentException
}
}

フィールドiに負の数を持たせたくない場合このようにするのが自然かなと考えたのですがいかがでしょうか？

**デフォルトの名無しさん** · 2017/10/29(日) 23:14:29.92

>>350
クソだと思う
値をチェックする責務は専用のクラスに任せて
それ以外のクラスに持ち込むべきじゃない

**デフォルトの名無しさん** · 2017/10/30(月) 00:07:29.30

>>350
別にいいと思うぞ

作ってからIsValidで確認するか
そもそも作らせないようにするかの違い
どっちのやり方も良し悪しがある

**デフォルトの名無しさん** · 2017/10/30(月) 00:34:24.41

空のオブジェクト作って後でsetter使って入れるのも微妙だしコンストラクタで値チェックとかしていいと思うぞ
俺ならPattern.compileメソッドみたいに自分自身を返すstaticメソッドで価チェックする

**デフォルトの名無しさん** · 2017/10/30(月) 00:44:57.12

>>350
有り得ない状態で存在できるより良い

**デフォルトの名無しさん** · 2017/10/30(月) 00:45:02.55

>>351
値チェック専用のクラスを作る場合どのように作るべきでしょうか
値チェッククラスでOKだった数値(正の数)をどのようにhogeのコンストラクタに渡すのかいまいち正しいイメージがつかめません
値チェッククラスのインスタンスをhogeの内部に持たせる(フィールドとコンストラクタの引数を変更する)のが良いのでしょうか？
とんちんかんな質問かもしれませんがよければお答えいただけると嬉しいです
>>352
なるほど
isValidメソッドであとからチェックするというのは思い付きませんでした

**デフォルトの名無しさん** · 2017/10/30(月) 01:04:31.28

>>350
＞コンストラクタで引数チェックをし、例外を投げる
問題ない
バグでおかしくなるのが一番の問題

**デフォルトの名無しさん** · 2017/10/30(月) 01:12:17.89

Hibernate Validatorみたいなバリデーションフレームワークは一般的に作ってから確認する方式

そもそも作らせないようにするのは”always valid”と呼ばれてる
“always valid model”とかで検索すれば参考になるサイトが出てくる

長所短所を理解して使い分ける

**デフォルトの名無しさん** · 2017/10/30(月) 01:16:09.26

俺だったらPositiveNumberみたいなジェネリッククラスを作るか、@positiveみたいなアノテーション作る

**デフォルトの名無しさん** · 2017/10/30(月) 01:51:11.96

>>353
ありがとう
>空のオブジェクト作って後でsetter使
>って入れるのも微妙だし
私もそう思ったのですがコンストラクタで例外を投げるのは良くないというような意見をどこかで目にしたので聞いてみました
>>357
ふむふむ
個人的にはそもそも作らせないって方が好みな感じがします
調べてみます情報ありがとう

**デフォルトの名無しさん** · 2017/10/30(月) 02:19:42.12

コンストラクタとセッタに同じことを示す文が複製されるのもちょっと良くないな

**デフォルトの名無しさん** · 2017/10/30(月) 03:04:30.08

>>351
クソ？しかし API にも new した時に例外出すやつ沢山あるわけだが、そういうのは全部クソ？

**デフォルトの名無しさん** · 2017/10/30(月) 07:16:59.94

>>360
コンストラクタで最初に値をセットして後から変更できないようにsetterは持たせないというつもりでした
(フィールドをfinalにしているのはそのためです)

**デフォルトの名無しさん** · 2017/10/30(月) 07:22:42.37

>>355
オブジェクト作る前にValidatorクラスでチェックするだけ

**デフォルトの名無しさん** · 2017/10/30(月) 07:24:54.59

>>361
全部クソ。
オブジェクトの責務をわかっていない。

**デフォルトの名無しさん** · 2017/10/30(月) 08:14:47.50

インスタンス変数（やクラス変数）は
そのオブジェクトに付属するデータだから
コンストラクタなどでデータチェックの責務を
オブジェクト自身に持たせるのはおかしくない

ただしチェックが複雑化したときに
責務を委譲するのもアリ

**デフォルトの名無しさん** · 2017/10/30(月) 08:43:10.77

>>365
そのただしが問題
検証はシステムの関心ごとだから
集約するべきでオブジェクト毎に持つものじゃない
コンストラクタはオブジェクトを初期化するという
特別な処理であってここでデータチェックが必要になるのは
責務の分離ができてないシグナルみたいなもの
staticメソッドでtryCreateという名前なら100歩譲ってオーケーだ

**デフォルトの名無しさん** · 2017/10/30(月) 08:54:32.49

負の値を持たせたくないのは業務の要件なんですよ！！

**デフォルトの名無しさん** · 2017/10/30(月) 08:55:34.55

>>366
＞検証はシステムの関心ごとだから
＞集約するべきでオブジェクト毎に持つものじゃない

責務を分離するのは重要だがそのオブジェクトの
規模が大きくなって複雑化するにつれて分けていくもの

たとえば生成の責務をファクトリに分離してもいいが
何でもかんでも全部ファクトリにする必要はない

だからコンストラクタでチェックしてもいいし
バリデータみたいのに分離してもいい

**デフォルトの名無しさん** · 2017/10/30(月) 08:56:23.29

思わず激昂してしまいましたがとにかく小生が言いたいのはコンストラクタで例外投げるのおかしいですよということなんです

**デフォルトの名無しさん** · 2017/10/30(月) 09:00:29.54

>>368
どっちでもいいなんていうそんな優柔不断で玉虫色な考えに真実があるわけない、あなたは思考を放棄しているに等しい、ここにはこういう値しか保持したくないんです、立派な業務要件でしょうが！！

じゃあその要件をまとめましょうよ、システムの関心ごととして切り出しましょうよ

**デフォルトの名無しさん** · 2017/10/30(月) 09:01:41.86

>>367
業務要件で例外処理を考えてるってなかなかおもしろいね

**デフォルトの名無しさん** · 2017/10/30(月) 09:04:21.57

>>371
どういうこと？

**デフォルトの名無しさん** · 2017/10/30(月) 09:06:06.45

もっと具体的にちゃんと説明して
小生良い事言ってるからそれちゃんと説明して

**デフォルトの名無しさん** · 2017/10/30(月) 09:12:06.74

>>370
いやそれは極端でしょ
OOPの要素はどれも使い分けじゃないの？

複雑化したら切り出すが抽象化にはキリがないので
IF文を全部ポリモーフィズムにするわけでもないし
最初から全部デザパタにするわけでもない

**デフォルトの名無しさん** · 2017/10/30(月) 09:17:39.73

>>374
ifの話じゃないだろ
なんでもかんでも一緒にするな
極端なのはそなたの方

こなたが言いたいのはとにかくコンストラクタでチェックするのおかしくないですかってこと

**デフォルトの名無しさん** · 2017/10/30(月) 09:21:09.70

>>375
コンストラクタでチェックするのはおかしくない
JavaのAPIでやってるくらいなんだから
むしろやるなって方が押しつけ

**デフォルトの名無しさん** · 2017/10/30(月) 09:22:44.13

コンストラクタでチェックして例外投げるのは
ようわからんけどオブジェクト作ってみるわー
でけへんかったわーゲヘヘーと言ってるようなもの
文明人のやることじゃない

アサートを使うべき
アサートは本番では動かさないからね
検証は専用のオブジェクトでやりましょう

**デフォルトの名無しさん** · 2017/10/30(月) 09:25:45.46

>>376
アホかお前は、お前はアホか
JavaのAPIははっきり申し上げてクソでござる
クソがやってるから大丈夫なんだとはならんだろうが
どこがクソかはお前に任せるよ

**デフォルトの名無しさん** · 2017/10/30(月) 09:46:48.33

アサートでもいいが例外投げてもいい

つーかコンストラクタで例外投げるのが大問題なら
Javaの言語仕様でできないようにするだろ……

普通に考えて2ちゃんの一個人より
Java（API）のやり方の方がいいわ

**デフォルトの名無しさん** · 2017/10/30(月) 10:01:28.68

>>379
ファイナライザでは例外投げないようにしましょう
というのがベストプラクティスだけど言語仕様ではできるでしょうに

言語仕様ってその程度のものですよ

コンストラクタでも例外投げるのやめましょうよ

2chネラーだから信用しないっていうのは権威主義的というか誰が言ってるかより中身で判断していただきたい、小生絶対良い事言ってるから！！

**デフォルトの名無しさん** · 2017/10/30(月) 10:21:49.81

どっちがキチガイかは一目瞭然だな
ありがとう

**デフォルトの名無しさん** · 2017/10/30(月) 10:37:40.45

それでええんか？
議論って考えを広げるためにあるんやで
キチガイとレッテルを張っておしまいでほんとにええんか？
小生が可哀想だと思わない？罪悪感を感じない？お前のお母さんもお前には優しい人に育ってほしいと思ってたのじゃないか？

**デフォルトの名無しさん** · 2017/10/30(月) 10:38:44.40

ありがとうの後にはございますを付けろ、それが礼儀だ

**デフォルトの名無しさん** · 2017/10/30(月) 10:39:13.84

礼儀正しく優しい人になって欲しい

**デフォルトの名無しさん** · 2017/10/30(月) 10:46:46.93

コンストラクタでデータチェックしない人になって欲しい

**デフォルトの名無しさん** · 2017/10/30(月) 12:29:49.28

>>377
何が悪いか分からん。
オブジェクトの生成失敗も業務メソッドの実行失敗も、中断すべき状況になったら例外飛ばして何が悪いのか。

コンストラクタで例外飛ばすなって話はC++ならメモリの回収がややこしくなるんで一利あるけど、JavaやC#なら問題ない。

**デフォルトの名無しさん** · 2017/10/30(月) 12:30:44.96

>>364
オブジェクトの責務とは？

**デフォルトの名無しさん** · 2017/10/30(月) 12:38:10.73

遡及IDは嘘つかないな
ソノトキ何をしたかよりもソノアト何をしたかが一目瞭然である
便利でよろしい

**デフォルトの名無しさん** · 2017/10/30(月) 12:38:19.13

>>367
なんというか、それそのものは Java 自体の問題のようにも思えるね。
つまり、unsigned が型として用意されてないからとか、PASCALみたいに数値の範囲を指定した新たな型を作れないからとかね。
まあそういう数値型のクラスを自作してしまえば良いだけかも知れないが。

**デフォルトの名無しさん** · 2017/10/30(月) 12:47:31.30

>>385
そんなにそれを求めるなら最初からそれができない新言語作れば？
大丈夫。君にも作れるよ。実装までできなくても言語仕様考えてネットで晒してフォーラム作って議論してまとめ直してから作ってくれる人募集すればその内なんとかなるだろう。全てを君が作る必要はない。

**デフォルトの名無しさん** · 2017/10/30(月) 12:53:44.32

Google Apps Scriptで
http://www.pre-practice.net/2017/10/line-bot_22.html?spref=tw
ここを参考にLINEBOTを作っていて、
文章中の特定の文字列に反応して返答するようにしたのですが、

else if (strpos(input_text, "寒い") !== false ) {
reply_text = "もう冬だね";
　　}

これを挿入しても反応してくれません。
何がいけないのでしょうか教えてください。
あと質問場所が適切でなければどこで聞けばいいか教えてください
お願いします。

**デフォルトの名無しさん** · 2017/10/30(月) 13:41:37.04

たまに生成されたクラスファイルが分割されることがあるんですがどういう条件で起きるんですかね？
ファイル容量の問題かと思いきや大きいものでも分割されてなかったり…

**デフォルトの名無しさん** · 2017/10/30(月) 13:52:19.55

>>392
１ファイルに複数のクラスを書いてるんじゃない？

**391** · 2017/10/30(月) 14:17:58.64

>>391
これがPHPというのがわかりました
スレ汚し失礼致しました。

**デフォルトの名無しさん** · 2017/10/30(月) 15:15:46.44

>>392
インナークラスの悪寒…
$付いてない？

**デフォルトの名無しさん** · 2017/10/30(月) 15:50:40.06

>>394,395
色々調べてみたらどうもシリアライズ可能なクラス？というのが原因みたいでした…
お手数おかけしました

**デフォルトの名無しさん** · 2017/10/30(月) 18:18:23.77

>>362
final見逃してただけだった
それなら尚更コンストラクタだね

**デフォルトの名無しさん** · 2017/10/31(火) 08:25:04.95

>>359
範囲外の入力値エラーだろ

private、つまりクラス内では、戻り値でエラーを示すが、
public、つまりクラス外では、例外でエラーを示す

なぜなら、クラス内をコーディングしているのは、普通は同じ会社だから、
その会社内で規定したエラー値を使っても、見落としが少ない

でも、クラス外の場合は、クラスをコーディングした会社と、異なる会社が使うから、
そのクラス内で規定したエラー値を使っては、いけない。
なぜなら、他社の人は、クラスのドキュメントを読まずに、クラスを使うから

つまり、クラス外部の人に対しては、絶対にエラーが起こるように、例外を使う

クラス内をコーディングする人と、そのクラスを使う人は、
異なる会社を想定している。
他社の場合は、クラスのドキュメントを読まないことを想定すべき

「スッキリJava」にも書いてある。
クラスを使う側と、フレームワークなど、他社が使うクラスを作る側の、
コーディングの違い。
他社を想定している場合は、全く違う

この辺は、品質管理をやってないと知らないはず

**デフォルトの名無しさん** · 2017/10/31(火) 08:35:40.63

>>398
どうしてもスッキリの宣伝をしなきゃ気が済まないようだな
その一節がなけりゃいいのに

**398** · 2017/10/31(火) 09:49:23.34

「スッキリJava」では、クラスの責務を、明確に分けている

前半は、クラスを使う側の知識で、
上級編はフレームワークなど、他社が使うクラスを作る側の知識

フレームワークを作る側では、検査例外を使って、コンパイルエラーにしないと、
ドキュメントを読まずに実行した客が、実行時エラーを起こして、
電話をかけてくるから、商売にならない

本を読んで、実行時エラーを起こした客が、出版社に電話をかけてくるだろ。
それと同じで、商売にならない

だから例外にして、その例外をキャッチしない人は、実行できないようにする

サポート契約をしない、無料相談の客をはじかないといけない。
無料で見積もりさせる客も

**デフォルトの名無しさん** · 2017/10/31(火) 12:09:41.25

経営の話か

**デフォルトの名無しさん** · 2017/10/31(火) 21:22:09.98

javaでWebアプリ作ってるのですがグラフ表示をしたいです
jfreechartとかchart.jsとか出てくるんですがおすすめとかってありますか？

**デフォルトの名無しさん** · 2017/10/31(火) 21:22:56.96

質問スレと間違えましたすみません。

**350です** · 2017/11/03(金) 17:28:51.99

>>380
しばらく2ちゃんを見れない状況だったので亀レスとなってしまいましたが、コンストラクタで例外投げるのが良くない理由をできれば聞きたいです

**デフォルトの名無しさん** · 2017/11/03(金) 17:36:15.65

>>404
責務の分離ができてない証拠だから

**デフォルトの名無しさん** · 2017/11/03(金) 17:39:40.36

コンストラクタっていうのはオブジェクトを初期化する特別な処理を書くところ
渡された値をそのまま何の加工もせずにフィールドにお納めするのが義務なんです

**デフォルトの名無しさん** · 2017/11/03(金) 17:42:31.79

どうしても例外を投げたいならtryCreateというスタティックメソッドを作って
例外投げますよ作れないかもしれないですよということをメソッド名で
陽に示すべきだ、この処理は検証も含んでいますよと示すべきだ
なぜならば検証も含むからだ、オブジェクトを作るメソッドなのにだ

**デフォルトの名無しさん** · 2017/11/03(金) 18:39:37.30

FileStreamもコンストラクタで例外を投げるが
あの設計も大間違いだから

コンストラクタでリソース確保しようとするのが完全に間違ってる
closeの処理でリソースを解放するならばリソースを確保するのはopenだ
openの責務をFileStreamは持つべきでそれはオブジェクトを初期化する
コンストラクタとは役割が根本的に100憶パーセント異なるわけ

**デフォルトの名無しさん** · 2017/11/03(金) 19:04:39.65

>>406
何もしなかったら意味ないだろ……
直通のゲッターセッターと同じただのお飾り

**デフォルトの名無しさん** · 2017/11/03(金) 19:18:22.44

>>409
フィールドを初期化するという大事な処理がある
セッターは代入だから、初期化じゃないから

**デフォルトの名無しさん** · 2017/11/03(金) 19:40:27.12

UI -> アプリ(バリデータ) -> ドメイン -> DAO
レイヤーはこのように分けてドメインから先にバリデーションのコードを持ち込んではいけない
オブジェクトをクリーンに保つためのベストプラクティス

**デフォルトの名無しさん** · 2017/11/03(金) 19:47:48.30

バグがないように引数をチェックするという目的でやるなら
アサートを使うべきだ
ただしアサートを使うのは開発時のみで本番コードには含めない
本番コードで引数チェックが必要になるようならそれは
業務要件として抽出するべきだ

**350です** · 2017/11/03(金) 20:10:50.22

なるほど
405~410まではなんとなくわかりました
411, 412は私の経験or勉強不足でいまいち理解できませんでした
でも、解説してくれてありがとう

**デフォルトの名無しさん** · 2017/11/03(金) 20:17:00.78

>>413
ありがとうの後にはございますだ
それが礼儀だ

**デフォルトの名無しさん** · 2017/11/03(金) 20:19:43.27

なるほど。わかった。スレが過疎って来たらコンストラクタで例外の話を出せば書き込みで溢れてスレが活性化するのだな。
時折反論を書きありがとうの後にございますを付けなければなお良いと。

**デフォルトの名無しさん** · 2017/11/03(金) 20:21:29.16

僕が活性化しちゃうおおお

**デフォルトの名無しさん** · 2017/11/03(金) 21:17:51.19

またキチガイか

**デフォルトの名無しさん** · 2017/11/03(金) 21:35:59.49

コンストラクタに不正なもの渡された場合に例外を出すことが
なぜいけないのかについては1ミリも説明されてなくて草

**デフォルトの名無しさん** · 2017/11/03(金) 21:41:52.50

>>418
責務の分離ができてない証拠だから
100ミリくらい説明してますけど
10メートルですよ

**デフォルトの名無しさん** · 2017/11/03(金) 21:42:46.79

違うわ1,000ミリの間違いね

**デフォルトの名無しさん** · 2017/11/03(金) 21:52:44.52

むしろ初期化の異常で例外を投げるのは
コンストラクタの責務だろ

なぜならコンストラクタの引数で渡されたデータから
インスタンス変数（やクラス変数）を初期化するわけだから
情報エキスパートの原則からして
データの処理はデータを持ってるオブジェクトが行うのが原則

ただ処理を全部抱え込むと肥大化するから
他に委譲することは構わない
newの代わりにファクトリを使ったりするのと同じ
しかしそれは「newを使うな」ではない

「外部に委譲してもいい」のであって
「コンストラクタで例外を投げてはいけない」ではない

**デフォルトの名無しさん** · 2017/11/03(金) 22:00:57.69

>>419
責務を理解できてない典型だね
君がそういう規約で開発するのは勝手だけど
それをベストプラクティスだからとか言って
他人に押し付けるのはよくないよ

**デフォルトの名無しさん** · 2017/11/03(金) 22:01:56.36

>>421
ベストプラクティスってそういうものだよ
こういう実装もできるけどこっちの方が好ましいよねっていうものだから
そういう意味で言ってるわけよ
たとえばModel View ControllerのアーキテクチャがあるけどViewに
Modelのコードも全部書きまくっちゃってもいいわけよ
でも分離するのが好ましいだろ、そういう意味で言ってるわけよ
その真意をくみ取って欲しいわけよ
コンストラクタでデータチェックヤラナイのが好ましいわけよ
え？ヤラナイの？なんていうふしだらな誘いに乗らないで欲しいわけよ

**デフォルトの名無しさん** · 2017/11/03(金) 22:02:38.01

>>422
理解できてないことを1ミリも説明してないじゃん
話にならないよ

**デフォルトの名無しさん** · 2017/11/03(金) 22:03:47.06

自分の考える責務の分離はこうっす
自分こう思うっすとはきはきと説明できないかね
若さが感じられない

**デフォルトの名無しさん** · 2017/11/03(金) 22:42:06.17

説明を求められたら言葉を失ってしまうみなさん

**デフォルトの名無しさん** · 2017/11/03(金) 22:47:53.53

どんなに外部でチェックしてもインスタンス生成に失敗することはあるわけで、
そういうときに失敗を通知する手段は例外しかないわけで、
コンストラクタで例外を投げてはいけないとする方が問題だ。

**デフォルトの名無しさん** · 2017/11/03(金) 22:49:18.72

モデルに存在するべきコードがコントローラに存在するものをファット・コントローラと言うけれども
バリデータに存在するべきコードがコンストラクタに存在するものをファット・コンストラクタと呼んで
クラスの生活習慣病に位置付けるのはどう？
徐々にシステムを蝕んでいきます
コーディングの習慣を見直しましょう

**デフォルトの名無しさん** · 2017/11/03(金) 22:52:55.66

>>427
どんなに外部でチェックしてもインスタンス生成に失敗するときは例外投げて良い
そんなのケースバイケースだろ、柔軟に対応しろよ

**デフォルトの名無しさん** · 2017/11/03(金) 22:53:28.36

ベストプラクティスっていうのは柔軟っていう意味もありまぁす！

**デフォルトの名無しさん** · 2017/11/04(土) 00:31:14.87

入力値チェックは、MVC のビューでやるけど、

ドラクエ10 の記事によると、
通信データの500バイトに、1バイトは間違っているから、
通信データは、絶対に信用しちゃいけない。
必ずサーバー側でも、チェックすべき

途中のルーターが、データが間違っていても、データを破棄せず、
間違ったデータで、パリティを再計算して送ってくるから、
正常なデータのように見える

**デフォルトの名無しさん** · 2017/11/04(土) 01:00:49.14

サーバーが受け取ったデータが外部入力なんだから
それをチェックするのは当たり前
ユーザー入力と同じ

境界の外から入ってきたデータを
どのレイヤーでチェックしてエラーハンドリングするかということと
モデルレイヤーのコンストラクタが事前条件に反した不正値を受け取った場合に
例外を投げるかどうかは関係はしてるが種類の違う問題
という話をしてる

**デフォルトの名無しさん** · 2017/11/04(土) 02:59:59.82

できれば不正値はコンパイル時にエラーになって欲しいものではあるな。
不正な値を引き渡せるようにプログラミングできること自体がなくなった方が良い。
I/O絡みのエラーは実行時じゃないとわからないから別としてね。
(指定したファイルがなくて読み出しオープンでエラーとかは仕方がない)

**デフォルトの名無しさん** · 2017/11/04(土) 08:21:09.10

javaってできることが多すぎて勉強しんどい

**デフォルトの名無しさん** · 2017/11/04(土) 08:35:59.26

>>431
へー、チェックサムも独自に仕込むのかな

**デフォルトの名無しさん** · 2017/11/04(土) 08:37:10.99

>>432
事前条件ってまさにアサートの領分じゃん

**デフォルトの名無しさん** · 2017/11/04(土) 08:49:44.67

>>434
C++より楽

**デフォルトの名無しさん** · 2017/11/04(土) 08:55:38.30

モデルの引数を間違えてるっていうのはバグなのでー
開発時に分かればよいことなのでー
アサート使うべきだよねー

コンストラクタをぶくぶくと太らせる理由にはならぬよ
メタボリッククラスから脱却してスマートクラスにしよう

**デフォルトの名無しさん** · 2017/11/04(土) 13:21:03.20

>>436
Javaの場合はアサートって言うとビルトインのassertをさすからね
事前条件なら自動的にアサート使えとはならないよ

それともコンストラクタでは
ExceptionではなくErrorを投げるべきって言いたいのかな？

**デフォルトの名無しさん** · 2017/11/04(土) 13:36:25.85

>>439
何言ってんだおめー
assert使うべきで
例外を投げるべきじゃない

**デフォルトの名無しさん** · 2017/11/04(土) 15:21:29.77

javaなら、コンストラクタで例外スローでいいでしょ。