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

1 名前：デフォルトの名無しさん mailto:sage [2020/05/27(水) 20:28:41.27 ID:Pggemlyq0.net] !extend:checked:vvvvv:1000:512 次スレを立てる時は↑を2行冒頭に書くこと（1行分は消えて表示されない為） 「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。 他のスレッドでは書き込めないような低レベルな質問、質問者自身なんだか意味がよく分からない質問、 ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。 内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。 なお、テンプレが読めない回答者、議論をしたいだけの人は邪魔なので後述のC#相談室に移動して下さい。 C#に関係の無い話題や荒らしの相手や罵倒レスや酔っぱらいレスはやめてください >>980を踏んだ人は新スレを建てて下さい。>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。 ■前スレ ふらっと C#,C♯,C＃（初心者用） Part147 mevius.5ch.net/test/read.cgi/tech/1582100741/ ■関連スレ C#, C♯, C＃相談室 Part94 mevius.5ch.net/test/read.cgi/tech/1553075856/ ■コードを貼る場合は↓を使いましょう。 ideone.com/ https://dotnetfiddle.net/ ■情報源 https://docs.microsoft.com/ja-jp/dotnet/standard/class-libraries https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/index https://docs.microsoft.com/en-us/dotnet/standard/class-libraries referencesource.microsoft.com/ ・Insider.NET > .NET TIPS - ＠IT https://www.atmarkit.co.jp/ait/subtop/features/dotnet/dotnettips_index.html ・DOBON.NET　.NET Tips https://dobon.net/vb/dotnet/index.html VIPQ2_EXTDAT: checked:vvvvv:1000:512:: EXT was configured 724 名前：デフォルトの名無しさん mailto:sage [2020/11/28(土) 20:02:20.04 ID:M3lV8yHm0.net] Sectionの配列を得たいなら>>714にLinqのini.Keys.ToArray() 725 名前：デフォルトの名無しさん [2020/11/28(土) 20:33:03.95 ID:pMGwV9fR0.net] >>714,715 .NET 5 が出たのを機に c# を始めたところなのでよくわかっていません。 勉強がてら自作の Delphi7(Lazarus) の自作ライブラリと同じ物の作成に挑戦中です。 ini ファイルの読み込みもネットからダウンロードしたものをベースに 改造して動きを勉強しているところです。 ですので List になっている理由もまだ理解できていませんし、Dictionary も 今日初めて見た状態です。一応 List を外してみると foreach (var dic1 in ini) { if (dic1.ContainsKey(section)) { foreach (var dic2 in dic1[section]) { if (dic2.ContainsKey(key)) { return dic2[key]; } } } } のような場面で ContainsKey がエラーとなりました。 とりあえず Delphi の TMemIniFile と同じ程度までできました。 726 名前：デフォルトの名無しさん mailto:sage [2020/11/28(土) 21:13:18.17 ID:M3lV8yHm0.net] >>716 Listを外してforeachする場合はこう変える必要がある foreach (var dic1 in ini) { if (dic1.Key == section) { foreach (var dic2 in dic1.Value) { if (dic2.Key == key) { return dic2.Value; } } } } もっと簡単な方法は>>714に書かれてる通り if (ini.TryGetValue(section, out var keys) && keys.TryGetValue(key, out var value)) return value; 727 名前：デフォルトの名無しさん mailto:sage [2020/11/28(土) 21:20:41.11 ID:M3lV8yHm0.net] pascalのiniなんてもう遥か昔に使ったきりだからあんまり覚えてないがそれっぽいの作るとしたらこんな感じか Dictionary<string, Dictionary<string, string>> ini = Dictionary<string, Dictionary<string, string>>(); string ReadString(string section, string key) { if (ini.TryGetValue(section, out var keys) && keys.TryGetValue(key, out var value)) return value; else return null; } これだとnullチェックが必要になるから、Tryメソッドの方が便利だったりする bool TryReadString(string section, string key, out string value) { value = null; return ini.TryGetValue(section, out var keys) && keys.TryGetValue(key, out value); } 728 名前：デフォルトの名無しさん mailto:sage [2020/11/28(土) 21:27:31.90 ID:M3lV8yHm0.net] そういえばiniは無かった場合の規定値を第3引数に入れておけばいいだけか それならTryメソッドいらんね 729 名前：デフォルトの名無しさん mailto:sage [2020/11/28(土) 23:40:53.47 ID:xlHMBUaz0.net] sectionの一覧が欲しいって言ってるのに、valueを取り出す説明を延々してる奴何なの？ 730 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 00:02:00.65 ID:agHPEKHb0.net] string GetValue(Dictionary<string, Dictionary<string, string >> ini, string section, string key){ if(! ini.ContainsKey(section)) return null; if(! ini[section].ContainsKey(key)) return null; return ini[section][key]; } c#に限った事ではないが、何をしたいのか分かりやすいコードを目指そうな。 731 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 00:10:23.95 ID:eEpZgXkU0.net] だから質問者が欲しいのはvalueじゃなくてsection一覧だっての 732 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 06:31:38.45 ID:kNuJF9Ar0.net] datagridviewにユーザーコントロールを埋め込みたいんですけどどうすればいいですか？ windowsformアプリです ていうかdatagridviewじゃなくてもユーザーコントロールをリストで表示できれば何でもいいです どなたか教えて下さい 733 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 08:16:40.50 ID:+pk7xniq0.net] >>723 TableLayoutPanel とか FlowLayoutPanel はどう？ 734 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 08:16:40.50 ID:+pk7xniq0.net] >>723 TableLayoutPanel とか FlowLayoutPanel はどう？ 735 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 09:35:53.03 ID:yrxBBFH+0.net] >>720 それは一番上に書いたんだが 736 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 09:40:01.71 ID:yrxBBFH+0.net] >>721 section取り出す説明は昨日一番最初のレスで書いたからもう脱線して話すけどなんでここまでTryGet使わない奴って多いんだろうな どうせ後々インデクサアクセスするならTryGetValueの方がよっぽど分かりやすいと思うんだが 737 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 12:22:24.94 ID:kNuJF9Ar0.net] >>724 ありがとうございます 試してみます 738 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 17:19:06.82 ID:b9LXd7l70.net] >>723 DataGridViewCellとDataGridViewColumnを継承して自作 739 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 17:32:18.08 ID:MpNpLm4W0.net] >>721>>727 ConstainsKey → []は、まずContainsKeyの内部でFindEntry走らせて存在確認し、存在したらインデクサアクセスして再度内部でFindEntry走らせて取り出すという2度手間してるIQ低いコード TryGetValueは存在確認時にFindEntryしたついでに値も取り出すというスマートな関数 そもそも前者で済むならTryGetValueは存在意義ないから、上記の仕組みを想像できずとも「なんでTryGetValueってあるんだろう？」と疑問に思わなければいけない ま、C#は多様性受け入れてる言語だから個人開発なら好きに書いたらいいが 740 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 18:14:07.88 ID:o/YPVzQP0.net] そういう値をローカル変数に代入するのを無駄と言い張って勝手にワンライナーに書き直すような地雷プログラマの振りをするのはやめろよ……初心者が本気にしたらどうするんだよ 741 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 18:23:09.50 ID:MpNpLm4W0.net] >>731 今の話でローカル変数やワンライナーが出てくる余地あるか？ 742 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 18:52:20.22 ID:o/YPVzQP0.net] >>732 明確にガード節で判定が切り分けられているのに、 「TryGetValueなら判定と取得を同時に行える」と言い出すのはまさしくワンライナー君の物言いだぞ ifをあえて二つ用意してContainsKeyを使ってる時点で 「ははーん、論理重視のコーディングスタイルか」と察してやるのが有能なプログラマってもんだ 743 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 18:58:47.11 ID:MpNpLm4W0.net] >>733 ？？？ 2連続で全く不必要に同じ処理を挟むような奴がとても論理重視とは思えないんだが、ContainsKey + []とTryGetValueじゃ2倍以上パフォーマンス変わってくるって知ってる？ さっきも言ったけどContainsKeyでも内部的にFindEntryが走り、インデクサアクセスでも再度FindEntryが走るという無駄がある 再度言うがMSがなんでTryGetValueというメソッドを公式でわざわざ用意したかって疑問に思わない？ 簡潔さ全振りならそもそもLinq使えばいい話 744 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 19:01:28.82 ID:tRrXGfded.net] メソッドの戻り値を一回判定するだけなら 一行で書かないほうが変じゃね？ それをわざわざ一旦保存とか逆になんの意図があるのかと たまにはデバッグやらログで出したいからと言うのはあるがそれは2回利用に当たるから入れて当然やしな 745 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 19:41:11.64 ID:o/YPVzQP0.net] >>734 久々にふふってなった まるで昨日の俺を見ているようだ……お前にもいつか悟れる日が来ると良いな 746 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:16:26.39 ID:EVjNfLc/0.net] 判定と取得を行いたいんじゃなくて取得を行いたい なのに判定しないとプログラム上まずいことになるので判定機能付きの取得メソッドを使う 読む上でわざわざ判定と取得を別に書いてるならそこにあえての理由があるんだろうな、と推測しないといけない 今回のケースならあえてのケースなど無いので可読性を下げる要因でしか無い 747 名前：デフォルトの名無しさん mailto:hage [2020/11/29(日) 20:17:12.32 ID:3Ka2HmX40.net] >>736 きみ中学生？ 男？女？どこ住み？ 748 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:28:09.84 ID:HJnVnweF0.net] >>736 それは廓然大悟かね？ 749 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:31:48.44 ID:yrxBBFH+0.net] >>737 これのどっちが可読性良いかなんて個人の感想レベルだろ 実際俺は100%下のほうで書いてるけど一度もコードレビューで指摘されたことないしな そして>>734が言うように実行効率は下の方が2倍近く良いので下を選択すべき理由はある 一方で上を選択すべき理由は見当たらない if(! ini.ContainsKey(section)) return null; if(! ini[section].ContainsKey(key)) return null; return ini[section][key]; if (!ini.TryGetValue(section, out var keys)) return null; if (!keys.TryGetValue(key, out var value)) return null; return value; 750 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:43:57.67 ID:yrxBBFH+0.net] 一応言っておくけどここでは713や720に合わせて書いてるだけで実際には1行で書いたりこんなに複数retは使わない returnは一つにしろというレビューは受けたことあるんでな 751 名前：デフォルトの名無しさん mailto:hage [2020/11/29(日) 20:51:31.28 ID:3Ka2HmX40.net] >>740 普通に英語として下の方が直感的に分かる 上だと今ですら非直感的な上に変数名によっては更に分かりづらくなる可能性をはらんでる 752 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:54:18.24 ID:YUoXGIHl0.net] >>720 TryGetValue()知ってるオレカッケー って言うおこちゃまだろw 753 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 20:56:55.50 ID:EVjNfLc/0.net] 無理に一行にするな、が通るならreturnを1つにしなくていいのでは？ 数十行あるメソッドのいろんな箇所でreturnしてると気持ち悪いと思うが、こういうタイプなら複数returnで良い 可読性が個人レベルなのはそのとおり 文字の並びが似通っている、ではなくどういう機能を呼んでいるか？という意味 判定付き取得メソッドが標準で用意され、しかもメジャーな内容でパフォーマンスも良い それでもあえてそれを呼ばないのは特殊な事情があるのでは？と読み手の思考を止めることにつながるという意味 754 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:06:40.39 ID:o/YPVzQP0.net] >>740 上のコードを採用する理由は何の値を戻しているか即座に分かるところ、と俺は判断した return ini[section][key]; この一文で読解に要する数秒を削る為だけに2倍程度のパフォーマンスを許容するのは普通にありだよ 755 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:23:27.82 ID:EVjNfLc/0.net] メソッド名見りゃそれはわかんじゃん return文にそういう責務を与える意味は？ どうしても必要というならコメントでも書いとけば良い 756 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:36:16.22 ID:yrxBBFH+0.net] >>744 returnについては可読性に関わる納得できた理由はちゃんとあるんだが自分やチームが使わないというだけで別にうちのコーディング規約を議題に上げるつもりはない（ツッコんでくる人いそうだったから一応書いただけ） まぁもう好きにしたらええんやない C#メインで使っていきたいなら言語として用意されてるものは出来る限り利用すべきと思うけど そこまで学習コスト払いたくないってことなら一つの方法を出来る限り使いまわすというのも利に適ってるしな 757 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:49:34.68 ID:EVjNfLc/0.net] チーム内でのローカルルールがどうなってるかはどうでも良いんですよ そのチームで合意が取れてるならなんの問題もない ローカルルールを隠した状態で突っ込まれてローカルルールなんです！って言われても知るかよ、という感想しか出ない ローカルなものでないなら、基本的には標準に準拠すべきで標準がわざわざtryメソッドを回避している書き方をしていない限り公の場でもそれに準じたほうが良いのでは？というだけ 758 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:53:07.17 ID:yrxBBFH+0.net] >>748 いやだからTryGetValueに関してはそれが標準だろっていうのが一貫した俺の意見であってローカルルールなのは>>741で書いた事 実際にギブハブなんかで数多くのプロジェクト参考にしてきたけどLinqで手っ取り早く書く事は多くてもTryGetValueを使わずにContainsとインデクサとかいう非効率な方法採用してる人なんて見た記憶ないしな 759 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 21:58:49.11 ID:EVjNfLc/0.net] >>749 なんで突っかかってくるかわからんけど747の前半は複数retの話、後半はtryの話 わざわざローカルの話である複数retのことなんざ話題に上げんで良いよ、try使うのが標準なんだからそれに習ったほうがよくね、 後半は君と同意見なんだよ 760 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:01:07.98 ID:yrxBBFH+0.net] >>750 捕捉程度のノリで書いた事に異様に突っ込んでくるなと思ってビックリしたんでね それも議題に上げたいのであれば>>740でそういう書き方してるから 761 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:11:12.12 ID:EVjNfLc/0.net] >>751 前提として標準に準拠したほうが良いよね、という主張がある 標準では複数retなんて当たり前に使ってる それをなぜか否定しているので突っ込んだらローカルルールだ！と言われる そういうローカルルールがあるのは構わんが、それならそう明記しなきゃ標準に準拠しようという主張とズレてブレブレなだけでは？ まさかレビューを受けたことがある、がうちにはそういうコーディング規約があるんです、の意味だったの？ 762 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:16:38.50 ID:yrxBBFH+0.net] >>752 だから本気で否定したいなら739でもそうやって書いてるって言ってるやん 740で書いたのは「沢山return書いてる時点で可読性低い」みたいなツッコミが入るかなーと予測して"一応"と前置きして書いたと言っただろ そこを議論するつもりはないと書いたのに異様に拘るんだな 763 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:22:25.03 ID:kxXRvhGP0.net] 複数リターン禁止ｗｗｗ 764 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:22:55.87 ID:o/YPVzQP0.net] >>749 正直俺も見たことないが、>>721が書いてるんだからいるところにはいるんだろ 肝心なのはそいつが何を思ってそういうスタイルにしたのか察して評価してやることで、 >>730みたいに重箱の隅突いて喜ぶのはただの地雷コーダーだからやめるべきだよ 例えば誰かが「ギブハブ」とか言ってても、わざわざ「ギブハブってなんだよバカかよ」と噛み付く必要はないだろ？ GitHubなのは分かるし話は通じてるんだからこいつの中ではこうなのかと納得して「ほーん、ギブハブがどうした？」と聞き返すのが有能なんだよ 765 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:26:49.20 ID:EVjNfLc/0.net] >>753 意味がわからんけど補足がなけりゃツッコミなんてなかったんだよｗ こだわってるのはそっちでしょｗこっちはそうですか、という感想しか無いよって言ってるやんｗ 766 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:38:58.86 ID:5TTB8+bC0.net] チャゲアスASUKAのギフハブに監視されてるネタに掛けてGitHubをギフハブと呼ぶのは5chの標準語だと俺も思ってたがそこまで浸透してないのか 767 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:42:09.42 ID:T9vV3uABa.net] なんとも不思議な世界があるんだなあ・・・・ TryGetValueを使うのが今どきの常識的なコーディングだと思ってた自分としては 老害（あるいは悪意があって不適切なコーディングを押し付けようとしてる人）が世迷言をほざいてるだけにしか見えないや 768 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 22:53:33.81 ID:yrxBBFH+0.net] >>755 その理屈で行くとコードレビューなんて出来なくなるな （これ明らかに意味のない文だけど何かスピリチュアル的な深い意味があるのだろうか…?配慮して指摘すべきではないか…?）なんて一々考察しなきゃいけないのかよ 769 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:00:13.09 ID:YUoXGIHl0.net] 了解 770 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:03:55.81 ID:YUoXGIHl0.net] >>754 MISRAにもあるし高信頼性を要求される場面ではそれなりにありがちな話だよ 771 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:14:22.90 ID:iy3zNsHi0.net] >>759 レビュー受けてるような新人はそんなこと考えなくていいと思うよ 多次元辞書でアクセスそのまま返す利点が分からないならまだお勉強段階なんだから、そういうやり方もあると素直に学んでおけば良い とりあえずチューニングはベテランがもっと後でやる仕事だから、パフォーマンスがーとか言い出すのは十年早い 772 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:30:20.42 ID:yrxBBFH+0.net] そう主張するのであれば具体的な利点を述べろよ TryGetValue派は俺以外にもメリット述べてる奴いるけどContainsKey派は「そのほうが見やすい」程度の個人の感想以上は出てきてない 勿論それもメリットではあるがTryGetValueの方が見やすいと言う人もいるのでそこでは差は付かない 773 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:51:14.64 ID:kxXRvhGP0.net] >>761 ねーよ 774 名前：デフォルトの名無しさん mailto:sage [2020/11/29(日) 23:59:15.56 ID:AAs5LLYH0.net] 宗教論争オモロイなw 775 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:00:33.52 ID:n5NKB6GR0.net] >>763 しょうがないな こういうケースは極端化すれば明確になる dic.TryGetValue(a, out var dic2) dic2.TryGetValue(b, out var dic3) dic3.TryGetValue(c, out var dic4) dic4.TryGetValue(d, out var dic5) dic5.TryGetValue(e, out var value) vs var value = dic[a][b][c][d][e] とりあえず値の取得を比較した場合こうなる 非常に馬鹿げたコードだが後者の方が察しやすくバグりにくいのは分かるはず これは分岐を差し込んでも同じこと こんなコードは現実にあり得ないから無効と言い張るなら、所詮はその程度のレベルなんだから黙るべき 776 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:20:59.97 ID:9x2/PpIx0.net] TryGetValue vs ContainsKey+[]の争いでしょ？ なら下はちゃんと一つ一つにContainsKey挟むべき そして後者の方がバグりにくい理由が全く分からない。まず何のバグを対象としているのか明確に 777 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:25:39.57 ID:G1P0/YpK0.net] レビューを受けるのが新人とか言ってる時点でまともじゃないでしょ いつの時代のエンジニア思想よｗ 778 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:26:29.87 ID:E5czHbG2a.net] なんというかb7e4-nc77みたいなのがレビュアーやったら地獄だな 自分が新人だったらたまったもんじゃない 779 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:36:41.15 ID:EEdVS6ls0.net] >>767 お前さんプログラマ向いてないから転職した方が良いよ、マジで ちゃんと理屈で教えられたことを受け付けないとか先が無さすぎる 780 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:39:46.85 ID:qXuQLmTVa.net] どうせ過疎スレだからスレ消費するのは問題ないと思うけど あまりに下らな過ぎる論争で萎えるねw よく読んでないので素っ頓狂なこと言ってるかもしれないけど、 iniファイルぐらいなら馬鹿正直にDictionaryをネストなんかしなくても section+keyをキーにして平坦化しても良いような気もする 781 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 00:45:42.05 ID:9x2/PpIx0.net] >>770 答えられないんだね 782 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 01:29:55.64 ID:a85534SW0.net] ini parserの中身を見たらContainsKey + インデクサだった https://github.com/rickyah/ini-parser/blob/development/src/IniParser/Model/SectionCollection.cs#L62-L77 自分なら使う側のコードがシンプルになるようにini[section][key]かini[section]?[key]の形で sectionかkeyがnullならnullを返すようにDictionaryを少し拡張するかな 783 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 05:30:56.15 ID:lgcZd+iw0.net] >>764 お前がそういう世界を知らんだけ 784 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 08:23:44.56 ID:gN60+pZv0.net] 相変わらず質問者無視で議論してるのウケる テンプレ読めない人多すぎでしょｗ 785 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 08:57:23.41 ID:8S8hZPx60.net] >>771 そもそもini直接操作するならWin32APIにセクション名とキー名と無かった場合の規定値を突っ込むだけで良いからDictionaryすらも何もいらない pascalにはMemIniみたいな基本はファイルには書き込まないクラスがあるからそれを実装しようと思ったらDictionaryがベストなんじゃない >>775 section一覧を返す方法は一昨日の一番最初のレスでしたって昨日言ったはずだが無視してるのはお前の方だろ 786 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 09:04:32.51 ID:8S8hZPx60.net] >>773 そのプロジェクト見た感じTryGetValueを一度も使ってないから恐らくJavaか何か出身で存在知らないんだろうな ContainsKeyは割と色んな言語にあるからこっちの方が認知度高いのは間違いない なので昨日も言ったが一々C#に学習コスト払いたくないって人もいるだろうからそういう場合は好きにしたらいい 787 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 10:07:26.68 ID:3MSQwEh40.net] >>776 テンプレ読めよ 「議論」するなと言っているんだが 788 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 10:12:11.16 ID:I8/NWgIp0.net] テンプレ読めないって事はレスも読めないんだから言っても無駄 粛々とNG 789 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 10:29:32.82 ID:lgcZd+iw0.net] >>766-767 まあここまで極端な例なら try { var value = dic[a][b][c][d][e]; } catch(){ } ってやる事を検討してもいいかも 790 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 10:45:42.20 ID:0LXXOsdO0.net] >>777 お、ようやく自分の間違いを認めたか まあContainsKeyの方が可読性高いのは当たり前なんだが 762 名前:デフォルトの名無しさん (ﾜｯﾁｮｲ 9f24-GtYN) [sage] :2020/11/29(日) 23:30:20.42 ID:yrxBBFH+0 そう主張するのであれば具体的な利点を述べろよ TryGetValue派は俺以外にもメリット述べてる奴いるけどContainsKey派は「そのほうが見やすい」程度の個人の感想以上は出てきてない 勿論それもメリットではあるがTryGetValueの方が見やすいと言う人もいるのでそこでは差は付かない 791 名前：デフォルトの名無しさん [2020/11/30(月) 11:16:43.36 ID:o8riI3080.net] >>781 可読性と認知度は全然違う意味だと思うけど、っていうのは置いといて >>733とか>>766とかさすがに釣りでしょって思わせるレスがちらほらあるから、喧嘩両成敗はちょっと可哀想な流れ 792 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 11:55:30.78 ID:tpJ2df0N0.net] >>774 ちょっとはググれよ 793 名前：デフォルトの名無しさん [2020/11/30(月) 11:58:06.51 ID:UK/ABAdxa.net] 認知度と可読性が別物か 職場で宇宙人って呼ばれてそう 794 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:11:19.73 ID:zbjAWIZjd.net] >>780 これでええやん 例外が重い？知るかよ 795 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:13:44.19 ID:lgcZd+iw0.net] >>783 > ちょっとはググれよ ブーメラン過ぎる、ってかググっても悪魔の証明はできないぞw 796 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:15:09.00 ID:0LXXOsdO0.net] TryGetValueを使うことに拘りすぎててアスペ臭しか感じない 好きにしろと言いながら必死で反論してんのが特にな 多様性を許容するっていうのは>>755みたいなスタンスなんだよ ここまで言われて「何言ってんのか分かんない」と騒ぎ続けるのは感情で理解を拒んでるだけの地雷君 「そういうの嫌われるからやめよ？」って指摘されてるのに「なんで？　俺が正しいじゃん？」って返すのは馬鹿を通り越した何かだわ 797 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:26:00.27 ID:tnWbc17e0.net] こういうレスちんぽバトルやめねぇ？ 798 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:51:21.50 ID:tpJ2df0N0.net] >>786 MISRAって名前まで分かってるのに？ 799 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 12:52:22.17 ID:PPubPKASd.net] いまだに複数リターン禁止なんて言ってる時代錯誤なやつがいるのかよ… 800 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 14:08:46.39 ID:lgcZd+iw0.net] >>789 ？？？ ないことの証明だよ？ 悪魔の証明って知らんの？ 801 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 14:09:59.63 ID:lgcZd+iw0.net] >>790 だからお前等の知らない世界があるって話 まあC++とかC#だと例外とか使えるからだいぶ緩和されるけど 802 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 14:11:44.42 ID:tpJ2df0N0.net] >>791 まずはあることを示してから 803 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 14:31:16.69 ID:zbjAWIZjd.net] >>792 アセンブラ以外でFalseは即リターンなんてベストプラクティスやん if elseとか基本ダサいんだよね 804 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 15:11:50.10 ID:lgcZd+iw0.net] >>793 それこそ misra multiple return とかでググればいくらでも出てくるだろw https://www.viva64.com/en/w/v2506/ >>794 だからそう言うのが非推奨な現場があるというだけの事 個人的には関数の頭でのエラーチェックでエラーだったら即リターンとかよくやってる 805 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 15:27:06.61 ID:tpJ2df0N0.net] >>795 それのどこがC#なんだ？ 806 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 16:36:59.20 ID:r8rQ87Qr0.net] 関数の出口が、複数ある奴か。 危険だから、MISRA-C の禁止事項 一方、Linux では関数の冒頭で、処理しない場合は、即リターンするとかやってる。 関数の出口に、goto で例外処理をまとめたりとか MISRA-Cを守らない場合は、逸脱手続きが必要。 トヨタなどは、MISRA-Cで、4/100 しか点数を取れなかったとか聞いたw MISRA-C研究会にも、トヨタのベテランが加入してるのに 807 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 16:46:43.31 ID:dO4rlTbV0.net] 俺社環だけど、これ指摘したらそれは例外処理や自動メモリ管理がないCであって、C#ではそこまで気にする必要ないんじゃね？で終わったな 808 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 16:53:06.40 ID:r8rQ87Qr0.net] 辞書のキーが存 809 名前：在しないのは、例外じゃない。 最初から想定内の事で、予想がつくから 例外はファイルが存在しないとか Go では例外を嫌って、複数の戻り値で、エラー値を返す [] [ここ壊れてます] 810 名前：796 mailto:sage [2020/11/30(月) 16:56:08.66 ID:r8rQ87Qr0.net] 漏れは、戻り値を変数に入れておいて、return は1つだけにしてる if 条件 result = 1 else result = 2 end return result 811 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 17:07:26.72 ID:pPofn0IB0.net] うっかりミスってreturnし忘れてもIDEが警告してくれるから、分岐それぞれでreturnするほうが好き 812 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 17:08:29.73 ID:8S8hZPx60.net] 複数returnとかワンライナーとかはそれこそ正解の無い事だから各々好きにしてくれよ TryGetValueとContainskey+インデクサは正解があると思ってるからこっちはつい反論し続けてしまったが 813 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 17:11:06.92 ID:DAJE5R+zd.net] なぜこのスレはテンプレを見る限り質問スレなのにスレタイにそれを示唆する文言が入っていないのですか？ 814 名前：デフォルトの名無しさん [2020/11/30(月) 17:15:26.09 ID:3PLmCTmV0.net] ヌルチェックやりはじめてから一刻も早くreturnすることを考えて書くようになった 階層が深くならないので見やすいし、上の部分だけである条件にあてはまった処理が完結してるので読みやすい 815 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 17:24:24.88 ID:tpJ2df0N0.net] >>798 ね、Cですら"推奨"なのに 816 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 18:17:44.47 ID:a85534SW0.net] >>797 >危険だから、MISRA-C の禁止事項 Mandatory, Required, Advisoryの3段階のうちAdvisoryなので禁止事項ではないね 817 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 18:30:53.38 ID:n+7nNLssM.net] >>801 さすがにMISRAとかが言ってるのはそう言う低レベルな話じゃなくて途中脱出でのリソース開放漏れとかを心配してる なのでC#なら例外とかusingとかを適切に使えばそれほど問題にならない クラス継承させた時のIDisposeのコーディングが面倒なのがたまにキズだけど 818 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 18:52:25.38 ID:+JMSCADPD.net] 議論は然るべきスレでどうぞ ふらっと C#,C♯,C＃（議論用） [無断転載禁止]©2ch.net https://mevius.5ch.net/test/read.cgi/tech/1469538912/ 819 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 19:48:14.13 ID:xZxi3SwM0.net] >>806 またRuby君がいい加減な知識で嘘を垂れ流そうとしてたわけか 820 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 21:11:19.50 ID:X4ikUZ5gM.net] いい加減なのは知識じゃなくて遺伝子じゃねえのか 欠陥品なんだから 821 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 21:11:29.21 ID:a85534SW0.net] 少し調べてみたけどMISRAがsingle returnを推奨する根拠に上げてるIEC 61508には ”subprograms should have a single entry and a single exit only;”と書かれているだけ そこに参考文献としてヨードン/コンスタンティンの「ソフトウェアの構造化設計法」を上げてるが ヨードンやダイクストラが提唱したsingle exitは時代的にも意味が違うのでルール自体の根拠が薄い https://softwareengineering.stackexchange.com/questions/118703/ 1993年出版のCode Completeでも今回のようなケースはmultiple returnが推奨されてるくらいなので single returnはリソースクリーンアップが必要だけど 他の方法がどうしても取れないような特殊なケースでしか意味がないと思って良さそう 822 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 21:15:38.40 ID:tnWbc17e0.net] どうでもよくねｗ 823 名前：796 mailto:sage [2020/11/30(月) 21:56:17.24 ID:r8rQ87Qr0.net] Cプログラミングの落とし穴 C Traps and Pitfalls, Andrew Koenig, 1989 ここから、Lint, MISRA-C などが出来た 824 名前：デフォルトの名無しさん mailto:sage [2020/11/30(月) 22:27:10.33 ID:n5+DCEBK0.net] c#スレなんですがそれは

---

---

[ 続きを読む ] / [ 携帯版 ]

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef