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

2017/11/07(火) 21:11:25.20

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

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

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

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

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

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

■関連スレ
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/11/14(火) 21:11:28.16

>>329
どこがみんなにとって難しいと思うから322で難しいとか書いたの？

2017/11/14(火) 21:13:21.13

>>334
これだけ意見相違があるから

2017/11/14(火) 21:16:24.25

よくわからんが難しくないだろ。

2017/11/14(火) 21:17:36.48

例外を変なところでキャッチしちゃう人が多い

メソッドAの中でメソッドBを呼んでまたメソッドCを呼んだとする

メソッドC中で例外が起こってもメソッドC内でキャッチしちゃって
メソッドCの戻り値をnullでリターンして
メソッドBで戻り値のnullチェックしてnullだったらまた戻り値をnullでリターンして
またメソッドAでチェックしてみたいな馬鹿な使い方をしてる

メソッドAでキャッチしろって思う

2017/11/14(火) 21:18:54.31

>>337
それだけだよね。
なんも難しくない

2017/11/14(火) 21:22:31.53

>>333
んーそれ余計に面倒にならない？

2017/11/14(火) 21:23:42.17

例外を目の前でキャッチするくらいなら他の制御構文でヤレ

2017/11/14(火) 21:33:26.00

>>337
ホントそれ。
至る所にtry/catchがある。
try/finallyは多用するがtry/cathなんて滅多に使う事はない。

例外処理は至る所に例外処理コードを書かないために存在する。

2017/11/14(火) 21:54:38.57

>>341
だから何度も書いてやってんじゃん
仕分けが絶望的に面倒臭ぇから握り潰しちまうって

だってさよく考えろよ
ドキュメント化されてないメソッドが出す例外って全部でいくつかどうやって調査する？

2017/11/14(火) 22:03:00.14

仕分けめんどくさいのは同意するけど、握りつぶすのはどうなんだ
仕分けした結果握りつぶして問題ないって判断したものならいいかもしれんが

極論として、全部握りつぶす or 全部スルーして上位にキャッチさせる
どっち選ぶ？

2017/11/14(火) 22:03:51.57

>>342
↑
例外処理を誤解している典型例です。
皆さん絶対に真似しないように。

2017/11/14(火) 22:08:15.89

>>337
>例外を変なところでキャッチしちゃう人が多い

かと思えば、例外が必要なところでtry/catch { } で握りぶしちゃう人も多い
これも追加ね。

無駄なtry/cathと握りつぶし
例外が分かってないヤツの典型例

2017/11/14(火) 22:09:59.38

例外の仕分けとは何なのか

2017/11/14(火) 22:11:36.04

分かったからママに教えてやれ
お前が可愛いなら誉めてくれるだろう

2017/11/14(火) 22:12:26.72

>>346
だから例外で帰られちゃ困るのよ
ちゃんと仕様で動作が決まってるのにもかかわらずそのメソッド的には例外なの

2017/11/14(火) 22:17:45.36

例外を拾ってエラーチェックをしようとしているのが根本的な間違いだな。

2017/11/14(火) 22:28:42.81

>>349
バッカ、飛んでっちゃうだろ

2017/11/14(火) 22:33:03.34

仕分けとは
握りつぶしとは

2017/11/14(火) 22:41:27.61

>>351
アスペにもほどがあるだろ

2017/11/14(火) 22:56:41.83

初心者だから経験が浅いので分からん

2017/11/14(火) 22:58:32.66

あからさまに変な人は相手にしないほうがいいよ
俺も言ってる意味がわからない

2017/11/14(火) 23:00:13.12

>>348
戻り値だったらいいの？例外となんか違うか？

2017/11/14(火) 23:02:10.19

>>355
お前、なんの例外がくるのか全部把握してる前提で話してるだろ？

2017/11/14(火) 23:06:39.11

把握できてる（かつ握っても問題ない）例外だけ握ればいいんじゃないの？
なんで全部握りつぶすの？

2017/11/14(火) 23:18:43.22

>>357
どこで例外発生するかわかんないから

2017/11/14(火) 23:25:37.86

>>358
何処で発生するか、しかも例外の内容も分からない例外を仕分けして何をしたいんだ?

2017/11/14(火) 23:33:00.11

>>356
お前なんの戻り値がくるのか全部把握してる前提で話してるだろ？

2017/11/14(火) 23:43:11.80

>>359
だから失敗でアプリ落としたくねーんだよ
アスペか
んで全ての例外に対応したいがメソッドの中身を読むも広大過ぎてなんの例外がくるのかわからない

ときたら握り潰すしかないよね

2017/11/14(火) 23:43:49.21

>>360
それは成功以外は失敗じゃない？

2017/11/14(火) 23:48:21.45

>>361
それはダメなやり方だよ

例外をキャッチしきれなければアプリを落とすべきなんだよ
それが基本なんだよ

特殊な場合は別だけど普通はそう教わる

2017/11/14(火) 23:48:53.64

>>361
だったら、例外をcatchして例外メッセージをMessageBox.Show()しとけば良いだけだろ。

2017/11/14(火) 23:49:57.17

>>362
だったら例外も同じじゃない？

2017/11/14(火) 23:53:24.48

こんな深夜に要件定義されていない議論するとか正気か？

2017/11/14(火) 23:58:02.52

>>365
だから落ちんなよって

2017/11/14(火) 23:58:11.24

>>363
>例外をキャッチしきれなければアプリを落とすべきなんだよ

どんな例外もキャッチしなきゃダメだろ。
キャッチして「駄目だコリャ」の場合はログ吐いて落とすしかない。

2017/11/14(火) 23:58:46.20

Windowsの場合、ハンドルされない例外で落としておけば、ユーザーには後日解決策が
提示される場合があると表示されて、パートナーには統計情報が連絡される。

あ (ｽﾌﾟﾌﾟ Sd9f-oWMR) · 2017/11/14(火) 23:59:42.34

動いてそうだけど動いてないかもしれない、エラーが中で出てるけど見えないプログラムなんて、

ブレーキランプはつくけどホントにブレーキがかかるかわからない車みたい。
いっそ走らないほうがマシ。

2017/11/15(水) 00:01:02.73

>>367-368
どうせネタのつもりで言ってるんだろうけど面白くないから

2017/11/15(水) 00:02:05.46

>>369
MS以外に例外情報を送信することができるのか?

2017/11/15(水) 00:03:28.48

>>372
統計情報だけもらえる。

2017/11/15(水) 00:03:40.40

>>368
キャッチして何になるんだよ？

アプリは予測してない状況になってるんだぞ
予測できてないから復旧もできないし
誤動作してシステム食いつぶすこともあるんだから生かしておく意味はありません

そういうことしてDBに特定のデータだけ一晩中書き続けた例を知ってる

2017/11/15(水) 00:03:57.83

>>367
むしろ例外の仕組みがない言語の方がプログラマーに制御できないとこで勝手に落ちるんじゃないの？

2017/11/15(水) 00:04:36.82

>>369
ダイアログ出てる間はプロセス掴んだままになるから、
素直にプロセス落として自動的に再起動させたい場合には邪魔になるやつね。

2017/11/15(水) 00:04:47.32

>>373
なにそれ?

2017/11/15(水) 00:05:46.61

ロガークラスなんかだと
中で例外全部握り潰すけど
catch {}

2017/11/15(水) 00:06:23.35

>>374
だから、ログ吐いて落とすって書いてあるだろ。
殆どの場合、例外を拾ったら落とすしかない。

2017/11/15(水) 00:10:28.88

ダメだこいつら
話になんねー
あるべき論はいいからさっさとアプリ動かせよ

握りつぶして動くなら動くでいいだろ
メソッドの中身なんて3万行だか4万行だか読んでらんねんだよバカが
その全てでテキトーに例外発生させてんだから握りつぶしてナンボだろこんなの

2017/11/15(水) 00:11:54.55

Office系は印刷中キャンセルボタン押しただけで
COMException投げてきたりするから
とりあえず握り潰すようにしている

2017/11/15(水) 00:12:29.08

DBはコミットしなければいいと思う。

2017/11/15(水) 00:20:37.66

>>380
じゃ、メッソッドの呼び出しもと全部にcatch {}入れとけよ。

2017/11/15(水) 00:26:18.34

例外で落としたくないプログラムってのはたしかに存在するけど
基本そんなプログラムをC#で書くのが間違ってると思うわ

例外処理のっていうときに、C#での（あるいは.NETでの）例外処理は、って言うようにしようぜ
つかまあこの話もう終わりで良いけど

2017/11/15(水) 00:26:49.27

ただまあ例外なんて滅多に出ないからね。

2017/11/15(水) 00:27:26.82

C++スレじゃないのか。

2017/11/15(水) 00:27:39.78

>>337
そのメソッドAが戻り値あり立った場合は？さらに外？それともヌル返す？

2017/11/15(水) 00:28:57.41

長々と無意味なレスバが続いてたけど、ひとつだけ皆が同意できそうなのは
例外がどういう用途で使われるべきかの一般的なコンセンサスに従う人は
ID:D89+JmTA0 = ID:pJdoypJN0 と一緒に仕事したくない、という点だけだな

2017/11/15(水) 00:30:13.38

異常は戻り値で知らせるなって言いたいんじゃないの。
まあそりゃそうだよねって言う。

2017/11/15(水) 00:57:36.26

ここc#スレで合ってるよね…

2017/11/15(水) 01:09:16.87

勘違いする人がいると見てられないので一応書いとくけど、
例外の目的の一つはフェイルセーフ、つまりプログラマの想定外のことが起こった時に
「安全側に倒す」 = 「プログラムを強制終了させる」ことなので、
「プログラマの想定外のことが起こってもプログラムの実行が止まらないようにしたい」
などという馬鹿なことを考えてはいけない。

2017/11/15(水) 01:35:57.65

>>391
はぁ？
他の誰かのライブラリを使ったときに
使用者の想定内がライブラリ作成者の想定外のときなんて簡単に発生すんだろーがバーカ
テメーはちんちんでも弄ってろ

2017/11/15(水) 02:16:30.38

ここは間を取ってContractしよう

2017/11/15(水) 03:59:39.11

>>331
誰もみてないだろうけど多分解決したっぽいので質問取り下げます
ほんとlinqって便利だな

2017/11/15(水) 06:34:16.49

>>392
具体的には何のライブラリ?

2017/11/15(水) 08:15:52.85

>>395
キチガイに触んなすな

2017/11/15(水) 08:45:58.03

>>394
LINQでどうやるの？

2017/11/15(水) 09:21:08.88

>>395
会社の
お前のレス
ガイジ過ぎるだろ
ちょっと考えてからレスしたら

2017/11/15(水) 09:39:52.81

ガイジコラボ

2017/11/15(水) 10:00:24.72

>>399
お前、単発だけど
連結するとガイジなんだろ？ｗ

2017/11/15(水) 10:01:55.68

>>397
やりたいのは文字列じゃないけどこんな感じのイメージ
パフォーマンスはしらない

var list = new List<string>() { "○", "×", "×", "×" , "○", "△", "△", "○", "△" };
Func<string, bool> isFirst = (item) => item == "○";

int groupingId = 0;
var result = list.Select(item =>{
if (isFirst(item)) { groupingId++; }
return new { Id = groupingId, Value = item };
})
.GroupBy(pair => pair.Id, pair => pair.Value);

// { "○", "×", "×", "×"}{"○", "△", "△"}{"○", "△" }
List<List<string>> hoge = result.Select(group => group.ToList()).ToList();

2017/11/15(水) 11:14:21.87

LINQと関数型プログラミングって何が違うの？

2017/11/15(水) 11:42:46.04

>>402
ガイジ乙

2017/11/15(水) 11:51:23.71

>>401
畳み込みですやってみた。

var list = new List<string>() { "○", "×", "×", "×", "○", "△", "△", "○", "△" };

var isFirst = new Func<string, bool>((item) => item == "○");

var result = list.Aggregate(new List<List<string>>(), (prevList, item) =>
{
if (isFirst(item))
prevList.Add(new List<string>() { item });
else
prevList.Last().Add(item);
return prevList;
});

// resultからhogeにクローン
var hoge = (List<List<string>>)result.Select(group => (List<string>)group.Select(item => item));

2017/11/15(水) 12:03:19.06

こっちの方がいいか。

var list = new List<string>() { "○", "×", "×", "×", "○", "△", "△", "○", "△" };

var isFirst = new Func<string, bool>((item) => item == "○");

var hoge = list.Aggregate(new List<List<string>>(), (prevList, item) =>
{
if (isFirst(item)) { prevList.Add(new List<string>() };
prevList.Last().Add(item);
return prevList;
});

2017/11/15(水) 12:05:06.75

if (isFirst(item)) { prevList.Add(new List<string>() };
括弧ミスった

2017/11/15(水) 12:05:11.35

Func<T,bool> を受け取るSplit関数定義した方が後で楽じゃね？

2017/11/15(水) 12:07:15.90

確かにその方がスッキリするな。
いずれにしろsplitの中身を定義しないといけないけど

2017/11/15(水) 12:29:09.51

>>405
Agreegate余り使ったことなかったけど便利だな
こっちの方が綺麗だ

if (isFirst(item) ||prevList.Count>0)
書いてなかったけど、リストの先頭が〇とは限らないのでこんな感じで

2017/11/15(水) 12:35:31.31

Aggregateは宣言的でないので関数型に慣れたらむしろ読み辛く感じるようになる
考え方が実質的に手続き型だから、手続き型に慣れてる人には馴染みやすいので濫用されてしまいやすい

2017/11/15(水) 12:43:50.52

関数型の典型的な関数みたいにバッ、バッって感じじゃなくてズィーンて感じだからか。
>>401のやり方だと使ってる関数はバッバだけ感じかもしれないけど外部変数使ってるから内外に絡み合ってるのがあれだけどどうすればいいんだろう

2017/11/15(水) 12:46:35.76

Select/Aggregateなんて変な書き方せず素直にmap/reduceでいいのにな
SQLっぽくというコンセプトで台無しになってる感じ

2017/11/15(水) 12:48:32.27

ズィーン→ズインッ→ズィッ→バッ
こう見れば畳み込みも関数型に馴染む

2017/11/15(水) 12:49:52.25

>>412
まあこの場合ではSQLを扱ってるわけじゃないしな

2017/11/15(水) 12:54:47.35

関数にした方が見やすいじゃん
無駄に不具合出して喜ぶ低能過ぎる

2017/11/15(水) 13:11:40.68

確かに目新しさにおおーってなったけど、これただのforeachでいいな
やっぱシンプルにループさせるのが最強か

var result = new List<List<string>>();

list.ForEach(item => {
if (isFirst(item) || list.Count <= 0){
result.Add(new List<string>());
}
result.Last().Add(item);
});

2017/11/15(水) 13:14:41.39

var sep = list.Select((x, i) => new {x, i})
.Where(xi => xi.x == '○')
.Select(xi => new[] {'○'}.Concat(
　list.Skip(xi.i + 1).TakeWhile(x => x != '○')));
こうかな
関数作った方がいいのは同意する

2017/11/15(水) 14:15:45.67

上のお題、結果がIEnumerable<IEnumerable<string>>でいいなら、

var list = new List<string>() { "○", "×", "×", "×", "○", "△", "△", "○", "△" };
var index = 0;
var hoge = list.GroupBy(x => x == "○" ? ++index : index);

ではだめ？

2017/11/15(水) 17:26:29.53

>>416
それ意味的にはaggregateと全く変わらない
初期値が外にあるか引数のところにあるかしか違いない

2017/11/15(水) 17:30:36.24

>>417
それaggregateより読みやすいと思うか？

2017/11/15(水) 17:40:07.85

>>418
その型じゃダメでしょ。

2017/11/15(水) 18:02:02.20

>>421
過去ログろくに読まずに>>418書いたけど、質問者もGrouBy使ってるじゃん>>401

2017/11/15(水) 18:04:09.97

>>422
変換しろよ
その分だけコード長くなるぞ

2017/11/15(水) 18:09:17.28

>>423
何に変換するの？
元々の質問には最終的に欲しい型の指定はない
単に列挙できれば良いならIEnumerable<IEnumerable<string>>として扱えれば十分

2017/11/15(水) 18:38:01.15

いつも思うんだが、この手のやつをLINQでやりたがるやつって何が目的なんだろ
自分ですらすらと書けるならLINQでいいけど、そうじゃないなら勉強以上の意味があるとは思えん

個人的にはシンプルさを競うなら>>418がいいと思う
戻りの型が気に食わないなら
.Select(t => t.ToList<string>()).ToList<List<string>>();
あたり追加すればいいだろ

2017/11/15(水) 18:40:53.79

こんなのやっててきったねーコード書く癖がついたら嫌だなｗ

2017/11/15(水) 18:46:37.59

お前らが思ってる程きれいでもきたなくもないから安心しろ

2017/11/15(水) 18:49:57.02

おい！待望の独習C#新版がでるみたいだぞ！

2017/11/15(水) 18:56:00.52

>>425
変数や関数を定義するのにいちいち名前をつけるのが面倒だからよく使う
無名だらけ

2017/11/15(水) 18:56:47.00

関数を定義するときには場所にも悩むしな

2017/11/15(水) 19:00:28.04

>>418
Selectなくても大丈夫だったか
hogeのGetEnumeratorが呼ばれる度にindexが増えていくのが気になるけど

戻り値はIEnumerableでもIGroupingどっちでもいいけどkeyは隠すかも
もし必要になったら以下のメソッドを作って、foreachと速度比較してって採用するかどうかって感じかな
色々と参考になった

public static IEnumerable<IEnumerable<string> Split<T>(IEnumerable<T> source, Func<T, bool> predicate){
var index = 0;
return source.GroupBy(x => predicate(x) ? ++index : index);
}

2017/11/15(水) 19:05:07.59

俺嫌われすぎだろ
アスペルガー症候群だから仕方ないのか

2017/11/15(水) 19:06:53.89

スリップが****-7wKgのは全部ワシじゃ