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

2020/12/16(水) 11:29:35.35

!extend:checked:vvvvv:1000:512
次スレを立てる時は↑を2行冒頭に書くこと（1行分は消えて表示されない為）

「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。
他のスレッドでは書き込めないような低レベルな質問、質問者自身なんだか意味がよく分からない質問、
ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。
内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。
なお、テンプレが読めない回答者、議論をしたいだけの人は邪魔なので後述のC#相談室に移動して下さい。
C#に関係の無い話題や荒らしの相手や罵倒レスや酔っぱらいレスはやめてください
>>980を踏んだ人は新スレを建てて下さい。>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part148
https://mevius.5ch.net/test/read.cgi/tech/1590578921/
■関連スレ
C#, C♯, C＃相談室 Part94
https://mevius.5ch.net/test/read.cgi/tech/1553075856/
■コードを貼る場合は↓を使いましょう。
http://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/
https://docs.microsoft.com/en-us/dotnet/standard/class-libraries
http://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

2021/03/21(日) 22:58:09.33

>>975
いいよ別に
後はリアルで味わったらいいよ
みんな暖かく接してくれるよ

2021/03/21(日) 23:16:39.83

>>967です
たくさんのレスがついていてビックリしています
書いていて思ったのは>>977さんがいうように各メソッドの間に改行を入れればまだ読みやすいと思ったのと、
Orderのプロパティが多いようだとどんどん読みづらくなっていくな、と思いました

foreachで書いたのはこんな感じですが、よりいい方法があれば知りたいです
var allItem = new List<Order>(order1);
foreach (var o in order2)
{
var duplicate = allItem.FirstOrDefault(x => x.ItemCode == o.ItemCode);

if (duplicate is null)
allItem.Add(o);
else
duplicate.Quantity += o.Quantity;
}

2021/03/21(日) 23:21:28.52

>>979
最初にvar allItem = new List<Order>(order1);をしていますが、
duplicate.Quantity += o.Quantity;をするとorder1の内容も更新されてしまいますね
DeepCopyなどで複製するべきでした

2021/03/22(月) 00:13:51.82

>>971
質問者も簡潔と完結を間違えてるんじゃね？ってダブルの冗談さ

2021/03/22(月) 00:23:14.95

C# Seleniumでブラウザを閉じるときに

Quit();
Close();

の動作が最近のChromeのアップデートでかなり遅くなっています。
私だけでしょうか？

2021/03/22(月) 01:09:40.25

>>967のコードは、ドットで改行したらいいぐらいでなんも悪くないだろう

2021/03/22(月) 02:12:24.15

ちゃんとしたテストコード書いてあるなら967のでかまわん

2021/03/22(月) 02:17:21.58

ログもちゃんと出力してね
指摘する人間に粒度が依存するけど
少なくとも俺はこんな一括でやられたらどこがバグってるのかわからない

2021/03/22(月) 02:21:02.72

まあ、もっとも指摘する人間の粒度に依存する部分に対応できない処理を書いちゃうあたり優秀とは程遠いというのが俺の感想だがね

2021/03/22(月) 03:06:05.30

この程度のコードが理解出来ないで文句垂れてるとか恥ずかしくないのかな
足を引っ張りあって低い方に合わせて仲良くってのが日本のITで
生産性の低さに表れてるのだろうな

2021/03/22(月) 07:03:59.88

これが読みにくかったら、簡単なSQL文すら書けない&読めない現場
20年前の現場ではSQLでとってきたレコードをループしながら集計しているコードがゴロゴロあった

2021/03/22(月) 07:27:03.21

C#3.0リリース直後でLINQがまだ物珍しかった頃なら>>972の指摘もまあ分かるけど
C#の最新バージョンなんてもう9.0だぜ?
LINQだって10年くらい前の時点で十分スタンダードになってた技術なのに
いまだに真顔で>>972みたいな指摘する人がいるのか・・・・

>>985
>少なくとも俺はこんな一括でやられたらどこがバグってるのかわからない
え、この程度で分からないの？
「優秀とは程遠い」理解力しか持ち合わせてないのはあなたのほうじゃ
あなたの口ぶりベテランの人なのかと思ってたんだけどただの能力低いロートルだったりする？

2021/03/22(月) 08:20:54.66

うちにもLINQ読めないからやめろ！って言うおっさんがいたな
最初年長者の言うことだからと素直に聞いていたけど
だんだん腹立ってきてボロカスに言ったら何も言わなくなった

2021/03/22(月) 09:33:10.92

>>989
あっそう
じゃ途中経過のログ出してよ

2021/03/22(月) 09:36:15.54

君のプログラムは
たったこれだけ、
ホントこれっぽちの、
普通のプログラマなら本当に誰でも可能な、
ことすらできないゴミカスなので

対応できませーん!（笑）(´∀｀*)ｳﾌﾌ

2021/03/22(月) 09:44:59.10

>>989
>>972の指摘は技術的な無知から真顔でやってるんじゃないよ。

和を乱すなら撤退しろ、ってメッセージなんだから、技術論で正しいって言っても無駄なんよ。

これが正しいってコンセンサスを先に取れば誰も何も言わない。
実際に工数が下がることを証明して、パフォーマンスが落ちていないことを証明して、コーディングルールにLinqの使い方を明示して、やってはいけない使い方なんかも書いて、それで通すもの。
俺は使いたかったからそうして通したから間違いない。

引っ張り合いではなくて、皆のレベルを上げる必要があるの。
一人が踊り狂っても、そいつが辞めたときに誰も面倒が見れない物が残るだけ。
おっさんになってからつくづく思うけど、お前らほんと自己中だな。

2021/03/22(月) 10:00:26.34

>>993
え？なんか長いけど延々と誰でもできることを自分ができない正当性を長文使って説明してるの？( ´,_ゝ｀)ﾌﾟｯ

控えめに言って

クソダッセｗ

2021/03/22(月) 10:02:49.77

と思ったら内容違うじゃん
はじめの一文で立場を明確にしろよ

2021/03/22(月) 10:18:56.38

>>991
途中経過をログ出力したければ普通にできるけど・・・

2021/03/22(月) 10:37:03.10

>>995
クソダッセｗ

2021/03/22(月) 10:39:26.64

>>996
ほう、やってみろ

2021/03/22(月) 10:41:32.69

>>994
ちゃんと読まないとかそういうところもだぞ。

2021/03/22(月) 10:54:40.38

>>1000なら俺は今年株で大儲けする

**1001** · Over 1000

このスレッドは１０００を超えました。
新しいスレッドを立ててください。
life time: 95日 23時間 25分 5秒