ふらっと 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/18(月) 23:28:28.84

>>734
ワシの環境だとこれで成功したんだがどこがやりたいことと違うんやろう
https://ideone.com/4MqfUE

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

>>735
その2 JsonConverterを使ってデシリアライズの10行目p.nameでnameが見えずエラーになります…

>>736
デバックを行うと
foreach (var r in Test.status
{
Console.WriteLine(r.name);
}
のrの部分がnullになり結果が何も表示されません
cvrl + spaceでr.name自体は出るのですが…

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

>>737
変数名Testを違うのに変えてもダメですか？

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

>>738
deserializeに変えてみても駄目ですね
とりあえず今日は寝て明日先輩にも聞いてみます

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

c#にレクトトラッカーってないですか？

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

非同期な複数のスレッドから、staticクラスの中のstreamwriterでcsvにログを書いてます
タイミングにより、ファイル競合？ロック？するようで、一つのスレッドからしかログが出力されません
ロック解除を待機するみたいなことはどうやるのでしょうか？

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

>>741
俺、そういうログ一生懸命作ろうとしたらスレッド毎にログ出してよって言われたよ
結局待ちが発生すると
ログの発生タイミングっていうか順序が全くの嘘っぱちになっちゃうじゃん
そんなゴミいらねーって言われたぜ
上司がいるならちょっと仕様確認オススメ

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

Queue使えばよくね
ログ出力メソッドはエンキューするだけ
ログ出力はワーカスレッドでのんびりやればいい

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

素直にLoggingライブラリ使う
メンテナンスする人の気持ちも考えなきゃ

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

いつも思うけど、やっぱAutoRrsetとかのイベント付きキューなんてないの？

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

>>745
BlockingQueueのコンストラクタにConcurrentQueueを渡す

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

BlockingCollectionだな
ちなみにコンストラクタに基になるコレクション渡さなかったら既定でConcurrentQueueが使われる

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

>>741
SeriLogの実装みてみ

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

やってみた感じぶっちゃけ難しい
片方のスレッドが超忙しいとき待ちが発生すると他のスレッドが全く動かなくなっちゃう

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

もう書いてる人がいるけど、前後関係が重要なら文字通りバッファーを設けるだけだよね
ライブラリのQueueは使いづらいからたぶん自作することになるけど、そんな難しくない

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

Nlogじゃないの？

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

C#特有というわけでは無いですが、例えば抽象クラス「費用」を作った後、費用合計を求めたい場合、費用の各サブクラスから値を取得し合算するのと「費用合計」クラスを作って、そこだけに問い合わせるの、どちらが良い作りでしょうか

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

>>746
>>747
へーそうかい!?
調査してみるよ。

ありがとうね

**741** · 2017/12/19(火) 12:37:26.95

順序は気にしません
2つのスレッドそれぞれから1回/秒ずつ出力されるだけです

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

>>752
サンプルコードを見せてよ

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

>>752
何を聞きたいのかよく分からないw
たぶん俺だけじゃないと思う

とりあえず「費用合計」クラスは普通に考えればない

合計を求める費用を集約しているオブジェクトがあるなら、そのオブジェクトに
合計させる（LINQのSum使ってもいいし自分でメソッドかプロパティを書いてもいい）のが普通

そんなものなくても配列にでもぶち込んでLINQ使うだけじゃないの？

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

いやなかなかいい質問だと思うよ
初心者でそこに気付けるのはいい視点だと思う
素質がある

中級者でもそこに全然思いが至らない人がいて困る
素質のない人だ

Sumを呼ぶとその時点で計算が行われるのことが困る場合がある
10個スレッドがあってそれぞれでSUM呼ばれて値が一致しないとか
そのつどDB読んじゃって負荷がかかりすぎるとか

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

>>752
費用クラスが合って費目毎にインスタンスを作るとか
費用合計クラスとか
何かを大きく勘違いしている

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

才能なし最下位確定じゃ

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

費用合計の仕様次第じゃん

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

>>759
おまえ顔にハナクソ付いてんぞ

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

ウンコ確定

**デフォルトの名無しさん** · 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

そうですか＾＾