ふらっと C#,C♯,C＃（初心者用） Part134

**デフォルトの名無しさん** · 2017/11/28(火) 21:30:48.55

「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。

他のスレッドでは書き込めないような低レベルな質問、
質問者自身なんだか意味がよく分からない質問、
ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。

内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。

なお、テンプレが読めない回答者は邪魔なので後述のC#相談室に移動して下さい。
C#に関係の無い話題や荒らしの相手や罵倒レスはやめてください

>>980を踏んだ人は新スレを建てて下さい。
>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part133
http://mevius.5ch.net/test/read.cgi/tech/1510056685/

■関連スレ
C#, C♯, C＃相談室 Part95
http://mevius.5ch.net/test/read.cgi/tech/1508180530/

■コードを貼る場合は↓を使いましょう。
http://ideone.com/
https://dotnetfiddle.net/

■情報源https://msdn.microsoft.com/ja-jp/library/gg145045.aspx
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/index
https://msdn.microsoft.com/en-us/library/gg145045.aspx
http://referencesource.microsoft.com/
-
VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured

**デフォルトの名無しさん** · 2017/12/19(火) 14:57:42.73

>>761
NG
ハナクソに顔が付いてる可能性を除外できてない

**デフォルトの名無しさん** · 2017/12/19(火) 14:57:48.67

>>762
おじいちゃん口臭いですよ＾＾

**デフォルトの名無しさん** · 2017/12/19(火) 15:02:05.90

それケツだ

**デフォルトの名無しさん** · 2017/12/19(火) 15:58:21.40

ASP.NET MVCのControllerの処理で、そこだけでちょちょっと利用する
関数などが沢山増えて来てコードが数千行になって来たのだが、
そう言う関数はどこに置けばいいの？
MVPクラスの人の回答をお願いします。

**デフォルトの名無しさん** · 2017/12/19(火) 16:10:19.74

partialだな（by一般兵）

**デフォルトの名無しさん** · 2017/12/19(火) 16:37:05.61

partialとかブッコイて一般兵気取りかよｗ
見習いもいいとこだぜ。入隊前じゃねぇの？

**デフォルトの名無しさん** · 2017/12/19(火) 16:41:36.67

>>766
初心者スレにMVPクラスの人はいない
regionとTODOで整理したらいいだけじゃね

**デフォルトの名無しさん** · 2017/12/19(火) 16:53:03.26

Controllerに何でも詰め込むからだろ
Modelにやらせればええのだ

**デフォルトの名無しさん** · 2017/12/19(火) 17:05:55.38

>>768
代案を出せよMVP

**デフォルトの名無しさん** · 2017/12/19(火) 17:48:31.52

俺が呼ばれた気がした

**デフォルトの名無しさん** · 2017/12/19(火) 17:52:51.22

昔は相談室とこのスレにあの人が住んでたんだけどね
今もいるんだろうか？

**デフォルトの名無しさん** · 2017/12/19(火) 18:37:30.91

>>766
決まった答えはない
責務に応じて分割するだけ

**デフォルトの名無しさん** · 2017/12/19(火) 18:40:59.36

「決まった答えはない」とか言うの決まった答えしかできない奴よくいるよねw

**デフォルトの名無しさん** · 2017/12/19(火) 18:53:48.71

>>775
日本語ｗｗｗ

**デフォルトの名無しさん** · 2017/12/19(火) 21:56:50.83

費用合計クラスの者ですが、以下のようなクラス設計をしました(実装は一応英語です)
public abstract class 費用
{
public abstract int 金額();
}

public class 支払家賃 : 費用
{
public override int 金額()
{
return 100;
}
}

class 水道光熱費 : 費用
{
public override int 金額()
{
return 200;
}
}

費目が増えても多態で対応できるのでオブジェクト指向しているなと思ったのですが
費用合計を出すのに都度インスタンスを作るのがなんだか野暮ったく
一般的なやり方をしているのか気になり投稿しました
このような書き方は、何か問題があるでしょうか？

**デフォルトの名無しさん** · 2017/12/19(火) 22:01:25.18

データベース使いなさい

**752** · 2017/12/19(火) 22:02:23.97

ちなみに、費用クラスで各費目を管理し
費用合計や各費目の値を返すメソッドを作ろうと思ったのですが
抽象クラスに「費目」という名前を使ってしまったので
テンパっています

OOは命名が難しいなと

**デフォルトの名無しさん** · 2017/12/19(火) 22:05:19.18

ありがちな形にこだわるタイプか
継承しなきゃならないほど金額()が込みいっているのかは置いておいて「動けばいいんだよ」

**デフォルトの名無しさん** · 2017/12/19(火) 22:06:13.10

全体見てないからなんとも言えんけど
その粒度でクラス作るとクラス図ってどうなるの？
2000項目あったら2000クラス作るの？
って心配になった

あとは動けばいいよ

**デフォルトの名無しさん** · 2017/12/19(火) 22:12:15.71

>>779
だから悪いけど結局何が聞きたいのか伝わってないと思うよw

public static class 費用
{
　　public const int 支払家賃 = 100;
　　public const int 水道光熱費 = 200;
　　public const int 総費用 = 支払家賃 + 水道光熱費 ;
}

なんかこれで問題ないようにも聞こえるけどこれじゃダメなの？

**デフォルトの名無しさん** · 2017/12/19(火) 22:12:45.55

>>777
才能無し、問題外
以上

**デフォルトの名無しさん** · 2017/12/19(火) 22:13:51.01

>>783
クッサｗ
死ねよハゲ＾＾

**デフォルトの名無しさん** · 2017/12/19(火) 22:15:56.11

>>784
センスの欠片もない
キミにプログラム開発は無理

**デフォルトの名無しさん** · 2017/12/19(火) 22:17:09.67

>>785
口臭いです＾＾
歯磨きサボらないで下さい・・・

**デフォルトの名無しさん** · 2017/12/19(火) 22:20:34.71

>>786
ここまで酷いのは見たことがない
誰かが適切なアドバイスをしたとしてもキミは斜め45度下へぶっとぶだろう
上ではなく下だwww

**デフォルトの名無しさん** · 2017/12/19(火) 22:21:50.93

どうやったらこんなつまんないレスを返せるんだよ...
重症やな w

**デフォルトの名無しさん** · 2017/12/19(火) 22:24:47.20

>>787
顔赤くなってますよ＾＾
もっとクールなキャラだったような気がします・・・

**デフォルトの名無しさん** · 2017/12/19(火) 22:29:22.36

MVPクラスの人は居ますか？

**デフォルトの名無しさん** · 2017/12/19(火) 22:30:42.82

>>790
>>777をエントリーしといた

**デフォルトの名無しさん** · 2017/12/19(火) 22:30:56.03

>>789
余りにも寒いコードを見たものでなw

**デフォルトの名無しさん** · 2017/12/19(火) 22:32:05.29

VIP患者なら大勢いるねw

**デフォルトの名無しさん** · 2017/12/19(火) 22:35:08.57

>>792
誰と勘違いしているのでしょう・・・
IDはちゃんとチェックしましょうね＾＾

**デフォルトの名無しさん** · 2017/12/19(火) 22:38:19.96

>>794
>>777 の同類だろwww

**デフォルトの名無しさん** · 2017/12/19(火) 22:41:32.93

>>795
おやおや・・・
とても都合の良い解釈ですね＾＾
右も左も分からない初心者に「センス無し、問題外」なんて冷たい言葉をかけるのはやめましょうね・・・

**デフォルトの名無しさん** · 2017/12/19(火) 22:50:03.39

まあ、自分が駆け出しの頃に上司にそう言われたことの腹いせにここで鬱憤晴らしてるんだろうなぁ....

**デフォルトの名無しさん** · 2017/12/19(火) 23:01:26.33

別にどう組んでも動くし
正解はないのかもな

ただ、この言語で表現できる中規模のまとまりをそんなプロパティレベルのもんに使われると粒度の把握がし難いなと

結局設計の良し悪しなんてソイツの気分だからね

**デフォルトの名無しさん** · 2017/12/19(火) 23:48:04.54

例が例だから粒度的におかしいけどこんなこともできますよ的な…
Addで費用つっこんで使える
Removeとか全部略

public class 費用リスト
{
long sum = 0;
public long 合計 => sum;
List<費用> list = new List<費用>();
public ReadOnlyCollection<費用> 品目 { get;}

public 費用リスト()
{
品目 = new ReadOnlyCollection<費用>(list);
}

public void Add(費用 hiyo)
{
if (hiyo == null) { return; }
sum += hiyo.金額;
list.Add(hiyo);
}
}

**デフォルトの名無しさん** · 2017/12/20(水) 00:00:42.82

わざわざ費目をクラスで分けている理由のことじゃないの？
>>799だけなら別に必要ないかと。

**デフォルトの名無しさん** · 2017/12/20(水) 05:02:03.39

>>777
例として固定値を返していると仮定
金額が費用の状態によって複雑になるなら有な考え方だと思う

**デフォルトの名無しさん** · 2017/12/20(水) 05:27:10.69

ねーな
業務上3000とか4000とか値扱ってる奴のが多いだろ
こういう組み方見るだけでうげっとする
じゃあ、構造を把握するためにクラス図でも見てみよーかって
資料開いた瞬間閉じるわ

**デフォルトの名無しさん** · 2017/12/20(水) 06:24:16.03

プログラムセンスのあるヤツなら最初からやらない間違いで
残念ながら矯正不能なんだわ

**752** · 2017/12/20(水) 07:08:03.81

費目が8個なのでサブクラスに各費目クラスを作ったのですが。。。

サブクラスは何個くらいが適当なのでしょうか
10個超えるとアウトでしょうか

**752** · 2017/12/20(水) 07:15:48.78

ちなみに費目をクラスで分けた理由は、費目ごとにちょっと込み入った算出処理をするので、それぞれを閉じ込めておきたいと思いました

でも、費用が最小の粒度の方が一般的な雰囲気ですね

急遽プログラムをやれと言われた人間なのでセンスとか言われると厳しいです

**デフォルトの名無しさん** · 2017/12/20(水) 07:35:47.88

>>805
ならサブクラスでいいよ
ここの連中は単純な家計簿みたいなツールを前提に話してる
そうじゃなく対象が複雑なら話は変わる

**デフォルトの名無しさん** · 2017/12/20(水) 07:42:10.14

>>806
変わるかなぁ？
日○とか三○でもそんなの見たことないよ
ちなみにそこだと項目数が10万とか超えちゃってるかんね
それ全部クラス作ったらｗ

**752** · 2017/12/20(水) 07:53:51.41

>>806
クラス分けしても悪くはないですか
ありがとうございます

ただ、費用の抽象クラスを費用クラスにしてメソッドを各費目算出とすると、メソッドに「費用合計算出」を作っても違和感が無いかなとも思い

プログラムは難しいとしみじみ思いました

**デフォルトの名無しさん** · 2017/12/20(水) 07:59:44.59

>>807
対象としてるドメインが違うだろ
呆れるわ

**デフォルトの名無しさん** · 2017/12/20(水) 08:14:04.00

業務アプリの設計なんかDFD描いてトランザクションスクリプトでいいよ
開発者全員が相当慣れてない限り、まともなオブジェクト指向でスケールさせるのは現実的に無理

**デフォルトの名無しさん** · 2017/12/20(水) 08:17:09.42

費目の数が10万！？
ないわー w

**デフォルトの名無しさん** · 2017/12/20(水) 08:20:29.69

>>808
> ただ、費用の抽象クラスを費用クラスにしてメソッドを各費目算出とすると、メソッドに「費用合計算出」を作っても違和感が無いかなとも思い
個々の費目で合計なんて求めようがないと思うが...
合計の算出は費目を束ねるクラスでやるべき話じゃね？

**デフォルトの名無しさん** · 2017/12/20(水) 08:23:01.94

それ以前に、費用の集計なんかSQLでいいよ

**デフォルトの名無しさん** · 2017/12/20(水) 08:44:12.56

>>811
余裕っしょ
設定数100個の部品が100個あっただけでもう一万よ
でかい機械で部品数たった100個とかありえないから10万は確実に超える
もっとあるだろうなぁ

**デフォルトの名無しさん** · 2017/12/20(水) 08:44:55.30

>>805
にわかプログラマーならオブジェクト指向に拘るより
もっと単純な仕組みでやったほうが良いね
目的がオブジェクト指向になって簡単なものを複雑にしてる気がする

**デフォルトの名無しさん** · 2017/12/20(水) 09:24:55.44

急遽プログラムやれと言われてこれだけオブジェクト指向理解したならすごいな

**デフォルトの名無しさん** · 2017/12/20(水) 09:30:51.88

費目毎のクラス分けは目眩がするけど

**デフォルトの名無しさん** · 2017/12/20(水) 09:30:52.50

>>804
ケースバイケース

**デフォルトの名無しさん** · 2017/12/20(水) 09:33:53.14

>>808
非常にセンスが良い
自信持って良い
注意点はおかしな現場(こっちの方が多数派だったりするがw)へ行った時に絶望しないことかな

**デフォルトの名無しさん** · 2017/12/20(水) 10:12:49.94

授業でC言語ちょっとやっただけでオブジェクト思考はちっとも理解してないから>>777がそんなに悪いのかわからない
費用ってクラスを使って家賃、光熱費のインスタンスを作るようにしたらいいのか?

**デフォルトの名無しさん** · 2017/12/20(水) 10:28:45.17

>>799
sum += hiyo.金額;
エラー CS0019 演算子 '+=' を 'long' と 'メソッドグループ' 型のオペランドに適用することはできません
と出るのですが、正解を教えて下さい。

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

元の奴が金額がメソッドになってるけど
それはプロパティになってる

というか金額をメソッドにしちゃだめ
元のコードがダメ

**デフォルトの名無しさん** · 2017/12/20(水) 10:36:11.97

>>822
このエラーが出ている行を
エラーが出ないように修正する方法を教えて下さい。

**デフォルトの名無しさん** · 2017/12/20(水) 10:38:08.67

>>777の　金額をメソッドからプロパティに書き直す

**デフォルトの名無しさん** · 2017/12/20(水) 10:44:45.87

どーせ()忘れ

**デフォルトの名無しさん** · 2017/12/20(水) 10:47:18.91

>>824
>>825
文章で説明して頂いても理解出来ないので
完全版のコードを掲載して頂けると有難いです。

**デフォルトの名無しさん** · 2017/12/20(水) 10:50:33.77

プロパティが自分で書けないならここからコード書いてても無駄だよ

早く勉強しろよ

**デフォルトの名無しさん** · 2017/12/20(水) 10:53:01.07

>>826
hiyo.金額()

**デフォルトの名無しさん** · 2017/12/20(水) 11:01:57.32

>>828
ありがとうございました。

**デフォルトの名無しさん** · 2017/12/20(水) 11:04:38.98

煽りでも何でもなく
プロパティが自分で書けないならここからコード書いてても無駄だよ

**デフォルトの名無しさん** · 2017/12/20(水) 11:11:07.65

あ、そすか。

**デフォルトの名無しさん** · 2017/12/20(水) 11:19:04.73

>>822
金額をメソッドにしちゃダメな理由を３行で

**デフォルトの名無しさん** · 2017/12/20(水) 11:44:22.53

まあ普通に考えればメソッドなら金額計算みたいに動詞的にしろよとは思うな
日本語メンバ名だと曖昧だけど

**デフォルトの名無しさん** · 2017/12/20(水) 11:48:55.09

まだやっていたんだ、簡単なことを難しくするバカのお手本

**デフォルトの名無しさん** · 2017/12/20(水) 11:50:57.09

そうですか＾＾

**デフォルトの名無しさん** · 2017/12/20(水) 11:55:45.07

>>834
同意する

**デフォルトの名無しさん** · 2017/12/20(水) 11:57:47.91

初心者に説教するアホ多過ぎ
弱いものイジメはｲｸﾅｲ(´･ω･`)

**デフォルトの名無しさん** · 2017/12/20(水) 12:04:02.26

>>752の人はたぶんネタなんだろうね。
結局何が聞きたいのか明確にしろって言っても答えないし
ネタのつもりなら何が面白いのかさっぱり分からんが

こんな曖昧な質問に曖昧なまま答える奴も頭どうかしてると思うけど

**デフォルトの名無しさん** · 2017/12/20(水) 12:05:42.91

マジでネタと思ってる奴って頭どうにかしてるよね

**752** · 2017/12/20(水) 12:15:20.79

>>838
費用のようなものを費目単位でクラス化したのですが、一般的にはどうなのでしょうかと質問したつもりですが、意図が伝わらなかったみたいですみません

結論としては、費用をクラスとし、費目メソッドを数分作る方が一般的なのかなという結論に辿り着きました

色々と参考になりました

**752** · 2017/12/20(水) 12:20:38.93

>>838
モデリングの対象次第で答えが変わるもの、答えようが無いということなのでしょうね

最近やっと覚えた抽象クラスを使いたかっただけかもと言われたらそうかも知れません

反省しています

**752** · 2017/12/20(水) 12:41:55.46

最後にすみません。ある条件で作業時間×レートが費用になるので、費用クラスと作業時間クラスを費用算出メソッドを持つインターフェースで継承させようと思うのですが、インターフェース名が「費用算出」はセンスが無いでしょうか?

インターフェースを使えばコードが仕様を語ってくれるのではと期待しているのですが基本情報資格も持っていない自分にはこれで良いのか解らず

実務経験豊富な諸兄にお尋ねしたい次第です

**デフォルトの名無しさん** · 2017/12/20(水) 12:45:22.87

レートクラスは無いのん?

**デフォルトの名無しさん** · 2017/12/20(水) 12:49:20.00

>>814
お前は費目の意味わかってないだろ...
部品一個一個に異なる費目がつくんじゃねーぞ w
クラスとインスタンスの違いも理解してないと見た

**デフォルトの名無しさん** · 2017/12/20(水) 12:51:34.80

>>837
てか、説教してるつもりの奴の方がレベル低いし w

**デフォルトの名無しさん** · 2017/12/20(水) 12:53:14.86

>>842
俺は算出って用語をあまり見ないから少し違和感あるけどまあそれで違和感ないならいいんじゃね

**デフォルトの名無しさん** · 2017/12/20(水) 13:49:49.11

>>844
違うよ
こんな細かい項目クラスにしちゃって全体の粒度はどうなるの？って話

**デフォルトの名無しさん** · 2017/12/20(水) 13:51:39.57

やっぱり設計のときにクラス図書かせるべきだな
あまりにも小さい項目いちいちクラスで構えるつもりならそれも全部
クラス図として書いてもらおう

**デフォルトの名無しさん** · 2017/12/20(水) 17:45:30.12

>>847
だから費目が10万とかはあり得んだろって話
うちの会社はそこそこの規模で結構色々手掛けてるけどそれでも経理関係の費目コードは50ぐらいしかない
当然大分類、中分類、小分類と階層化されてるからそれに合わせて継承させればいい
まあうちの会社の情シスはそこまでやってなくて費目コードを見て分岐してそうだけど w

**デフォルトの名無しさん** · 2017/12/20(水) 17:53:32.28

>>849
じゃそれでもいいけど
それだけで50個クラスできるんでしょ？
クラス図どうなるの？クラス50個作るん？
って言いたいわけよ

結局設計ってさ
わかりやすいか
わかりにくいか
だけなんだよね

上下のわからんクラスって粒度を
そんなちっぽけなもんに使うと
全体が見えなくなっちゃうんだよ

費目は少なくてってそれお前の環境がそうってだけの偶然でしょ？
この方針はオススメできないね俺は

まあ、どう組んでも動けばよいのであくまでオススメの域

**デフォルトの名無しさん** · 2017/12/20(水) 17:57:31.91

50個のクラスができるものをクラス使わずにごちゃ混ぜにしたら余計にわけわからんだろ
こういう話題で数が増えたらどうするの？って反論するやつが必ず現れるけどそれ墓穴掘ってるだけだぞ

**752** · 2017/12/20(水) 18:12:27.89

>>843
レートクラスは必要かも知れないですね
まだまだ考慮不足でした

**デフォルトの名無しさん** · 2017/12/20(水) 18:25:24.19

データが構造を持ってしまうと、別の使い方をする時に手間が増えるだけなんだよ
データはデータでクラスにする必要なんかまったく無い

**デフォルトの名無しさん** · 2017/12/20(水) 18:31:13.93

>>850
> クラス図どうなるの？クラス50個作るん？
必要なら作るだけだろ

> 上下のわからんクラスって粒度を
> そんなちっぽけなもんに使うと
> 全体が見えなくなっちゃうんだよ
それお前さんの管理能力の問題...

> 費目は少なくてってそれお前の環境がそうってだけの偶然でしょ？
世の中の会計ソフトとか見てみな
もしくは自分の会社の経理担当者に聞けばわかると思うよ

> この方針はオススメできないね俺は
別にお前もそうしろなんて言ってないから勝手にやってなよ

**デフォルトの名無しさん** · 2017/12/20(水) 18:47:00.16

>>853
別の使い方する時点で分析設計ミス

**デフォルトの名無しさん** · 2017/12/20(水) 18:56:33.13

>>854
残念ながら、世の中の会計ソフトなんかほぼ100%トランザクションスクリプトだよ

**デフォルトの名無しさん** · 2017/12/20(水) 19:01:31.53

>>851
好きなようにすればいいよ
ただ、お前のクラス図に50個のクラスが並ぶだけ

**デフォルトの名無しさん** · 2017/12/20(水) 19:08:20.84

>>856
なんの話をしてるんだよ w

**デフォルトの名無しさん** · 2017/12/20(水) 19:13:04.16

費目、時間単価、時間がそれぞれ独立したクラスか
めまいを禁じ得ない

**デフォルトの名無しさん** · 2017/12/20(水) 19:35:42.09

それらが同じクラスに混在する方が目眩するわ

**デフォルトの名無しさん** · 2017/12/20(水) 19:59:30.76

で、最終顧客がのぞむ要求に答えられるアプリケーションなのかと

**デフォルトの名無しさん** · 2017/12/20(水) 20:00:24.04

全体を把握しないと作業に取り掛かれないなら
既にクラス設計として破綻してるとしか

**デフォルトの名無しさん** · 2017/12/20(水) 20:45:21.17

結局要件があいまいだから何とも言えんけど普通に考えれば費目ごとに派生するのはないね。

せっかくC#にはデリゲートがあるんだから、費目で違う処理はコンストラクタで
デリゲート食わせて対応できるし、その上でプロトタイプパターン適用した方が分かりやすいと思うよ。

もちろん費目ごとの固有の処理が費目固有のプロパティに依存するような場合は
継承を使うしかないけどね