X



C#, C♯, C#相談室 Part93©2ch.net
■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん 転載ダメ©2ch.net (ワッチョイ 1e06-m8Mb)
垢版 |
2017/04/22(土) 08:52:00.93ID:iVvswOrb0
■Visual Studio 2017 Community(無償の統合開発環境)等はこちら
http://www.visualstudio.com/downloads/

■コードを貼る場合はこちら
http://ideone.com/

■前スレ
C#, C♯, C#相談室 Part92
http://echo.2ch.net/test/read.cgi/tech/1485589613/

■次スレは>>970が建てる事。
建てられない場合は他を指定する事。
VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured
0568デフォルトの名無しさん (ワッチョイ 1f8a-DJU/)
垢版 |
2018/06/25(月) 11:21:29.65ID:pxATIqrb0
Debug.Failのサンプルコードがこんなのなんですが
プログラムのバグでもない限りは絶対に訪れないような場所には
片っ端からDebug.Failを撒いておいたほうがいいんですか?
それともそれ専用のthrowすべき例外なりがあるんでしょうか?
Assert.Fail?
何も書かないのは気持ち悪いので指針を教えてください

switch (option) {
case Option.First:
result = 1.0;
break;

// Insert additional cases.

default:
Debug.Fail("Unknown Option " + option);
result = 1.0;
break;
}
0569デフォルトの名無しさん (ブーイモ MM0f-STG2)
垢版 |
2018/06/25(月) 12:12:52.11ID:DuFsc7dIM
>>568
そういうのはフロー解析による不要なコンパイルエラーを回避するために throw する
例外の種類は状況やポリシーに応じて InvalidOperationException, NotSupportedException, ArgumentException, NotImplementedException あたりを使うのが普通
0571デフォルトの名無しさん (アウアウエー Sa7f-3IMT)
垢版 |
2018/06/25(月) 12:33:59.90ID:G9vuY89la
そうかな
>>569が挙げてくれてるので十分でしょ。
少なくともメッセージは自由に設定できるし、本当に必要なら派生もできる。

そもそも例外をそんなに細かく分類する必要が本当にあるのかちょっと疑問
0572デフォルトの名無しさん (アウアウウー Sae3-STG2)
垢版 |
2018/06/25(月) 12:40:32.20ID:M/oLTPjda
常識的にキャッチされない種類の例外なら何でもいいわな
InvalidOperationExceptionだけはキャッチされる場合があるから微妙だけど、慣習的にはわりと多数派な気がする
個人的には常に NotImplementedException にしたい
0575デフォルトの名無しさん (ワッチョイ 7fe9-5Tpg)
垢版 |
2018/06/25(月) 13:00:29.11ID:5ya7SPIm0
細かいけどコンパイルエラー回避のためにthrowする、ってのは違くない?
現実問題そんな感じになってしまうことはあるけど理想はコンパイラの都合に合わせたコードなんて無いほうがいいんだから
0576568 (ワッチョイ 9f8a-DJU/)
垢版 |
2018/06/25(月) 13:23:21.50ID:vUyRGEqk0
>>569
>InvalidOperationException, NotSupportedException, ArgumentException, NotImplementedException
これらの例外は自クラスには非はなく、呼んだ側の使い方が間違っているといった感じの例外に見えますが
そんなにこだわらなくていいんでしょうか

public class MyClass
{
public class BugException: System.SystemException{}
public void buggedmethod() { ... throw new BugException(); }
}

これで良ければ1行で定義できて1番明快に思えますが
検索してもこんなことをやってる人は誰もいないっぽい…
0577デフォルトの名無しさん (ブーイモ MM0f-STG2)
垢版 |
2018/06/25(月) 14:49:07.56ID:o35Q9ijAM
>>574
>>568のケースに限れば、defaultに来るのはenumにメンバを追加したのにcaseを追加していない「変更漏れ」であると考えることもできるから、
NotImplementedException でも意味的におかしくはない
NotSupportedExceptionは(継承の都合で)メンバ自体が使えないことを意味するからちょっと違う
0579デフォルトの名無しさん (アウアウエー Sa7f-3IMT)
垢版 |
2018/06/25(月) 18:51:31.43ID:uZsljbpsa
>>577
なるほどそうかも

関係ないけど、いつも思うけどあるインターフェイスを継承してるのに
そのインターフェイスのメソッド使ったらNotSupportedExceptionが飛んでくるって設計は
なんか不条理を感じるのは俺だけかなあw

インターフェイスって契約じゃなかったのかよw
0583デフォルトの名無しさん (ワッチョイ 7f96-qn6i)
垢版 |
2018/06/25(月) 21:43:12.62ID:xJKhLBIA0
Debugクラスのメソッドはリリース版のとき消えます、素通りします
Debug.Fail() は起こりえない状況であり、かつ、素通りしても問題ない箇所、
またはリカバリ処理を行っている箇所にしか使用できません

基本は例外か Trace.Fail() を使用するべきです

Debug.Fail() を使用して問題ないのは具体的には下記の条件に当てはまるときぐらいだと思います

・『絶対』に起こらないことを目視や単体テストで確認した上で、後続のコードでリカバリ処理を行っている箇所で、
開発によってコードが壊れてないかチェックするためにコードに挿入する
(絶対に発生しないので単体テストを行う場合は #if で慎重にコードを切り替えてエラー状態を再現することになります)

つまり、リカバリ処理してるなら Debug.Fail() は入れなくても問題ないはずです
0584568 (ワッチョイ 618a-VCRa)
垢版 |
2018/06/28(木) 16:32:42.98ID:mhaIhaYV0
遅くなりましたが例外とログの使い方大変参考になりました。
ありがとうございます。
0585デフォルトの名無しさん (ワッチョイ 45fa-2e90)
垢版 |
2018/07/04(水) 22:15:33.33ID:gFgZc5FG0
81F
0589デフォルトの名無しさん (ササクッテロル Spbd-p+0f)
垢版 |
2018/07/12(木) 21:32:30.73ID:wWmeulvip
単なる構造体にインスタンスもくそもないからな。
0591デフォルトの名無しさん (ブーイモ MMc9-hVvx)
垢版 |
2018/07/14(土) 10:59:20.35ID:21Pqhes0M
型のデフォルト値でね?
0592デフォルトの名無しさん (ワッチョイ a5b3-8eTX)
垢版 |
2018/07/14(土) 11:24:43.76ID:td9rD4BU0
Visual Studio 2017
合計 90 日間が過ぎました。

無料で使いたいんですが、どうすれば良いでしょうか?
0596デフォルトの名無しさん (ワッチョイ a5b3-8eTX)
垢版 |
2018/07/14(土) 12:06:05.24ID:td9rD4BU0
Community版に移行する事にしました。
ありがとうございます
0599デフォルトの名無しさん (ワッチョイ 6db5-p+0f)
垢版 |
2018/07/15(日) 11:14:40.73ID:GMJtRfj30
サブスクリプションで全シリーズ揃えてる俺に死角無し。
0601デフォルトの名無しさん (ササクッテロル Spc7-Bhrm)
垢版 |
2018/08/02(木) 11:31:31.36ID:ijDqx/PBp
Entity Framework Coreについて質問です。

以下のように1対多の入れ子(left join)になったデータを取ってくる場合で、一番最下層のテーブルに取得条件を追加したいです。
どのようにしてwhereメソッドを書けばよいでしょうか。
.Include(x => x.Departments)
.ThenInclude(x => x.Groups)
.ThenInclude(x => x.SubGroups)
.ThenInclude(x => x.Employees)
.ThenInclude(x => x.Skills)

例えば以下のような感じです。
Skill.ExpireDate >= DateTime.Today

それぞれのプロパティはコレクションのため以下のようには書くことができません
Where(x => x.Departments.Groups.SubGroups.Employees.Skills.ExpireDate >= DateTime.Today)

ThenIncludeで指定しているプロパティがすべて列挙でなければ以下のように書けると思うのですが・・・
Where(x => x.Department.Group.SubGroup.Employee.Skill.ExpireDate >= DateTime.Today)

それぞれの階層のクラスは全プロパティ取得することを想定しています。
Entity Framework Coreで実現可能でしょうか。
0603デフォルトの名無しさん (ササクッテロル Spc7-Bhrm)
垢版 |
2018/08/02(木) 12:30:22.50ID:ijDqx/PBp
>>602
辿らない場合、スキルクラス単位で条件絞り込みができるのでしょうか?
これができればとても楽なのですが、、、
0605デフォルトの名無しさん (ササクッテロル Spc7-Bhrm)
垢版 |
2018/08/02(木) 15:11:34.29ID:ijDqx/PBp
いえ、辿りたいわけではなくて、辿らないとできないと勝手に思ってただけです
辿らなくて良いなら辿らないでやれればベストです。
ですがその方法がわからないので教えてもらえませんでしょうか。
0606デフォルトの名無しさん (ワッチョイ 8f9f-LI4B)
垢版 |
2018/08/02(木) 21:52:06.55ID:nmSSZypj0
var q = _context.Hoges.FromSql($@"
select H.* from Hoges H where exists (
select null from Departments D where H.DeptId = D.DeptId and exists (
select null from Groups G where D.DeptId = G.DeptId and exists (
select null from SubGroups SG where G.GrpId = SG.GrpId and exists (
select null from Employees E where SG.SGrpId = E.SGrpId and exists (
select null from Skills SK where E.EmpId = SK.EmpId and SK.ExpireDate >= {DateTime.Today}
)))))
");

var hoges = q.Include(x => x.Departments)
.ThenInclude(x => x.Groups)
.ThenInclude(x => x.SubGroups)
.ThenInclude(x => x.Employees)
.ThenInclude(x => x.Skills)
.ToList();
0610デフォルトの名無しさん (ワッチョイ ff58-0Uuo)
垢版 |
2018/08/07(火) 20:55:48.43ID:7nA6P5yg0
Linqでの書き方で質問なのですが
同じサイズのdataAとdataBのデータがあるとして
データが違う箇所のIndexを取り出すとしたらどう書けば良いでしょうか?
今は↓のようなコードになっています。

List<int> dataA = new List<int>() { 1, 2, 3, 4, 5, 6 };
List<int> dataB = new List<int>() { 1, 2, 4, 3, 5, 6 };
bool resultAB = dataA.SequenceEqual( dataB);

if(resultAB = false)
{
foreach(var A in dataA)
{

比較処理
}
}


結果
2
3
0611デフォルトの名無しさん (ワッチョイ b3e3-10BI)
垢版 |
2018/08/07(火) 21:22:20.14ID:UKw4EMmv0
>>610
ない頭絞って考えてみた。
Enumerable.Range(0, dataA.Count()).Where(p => dataA[p] != dataB[p]);
やら
dataA.Select((v, i) => new { value = v, index = i }).Where(p => p.value != dataB[p.index]).Select(p => p.index);

どっちも汚い。もっと賢いのおしえろください
0614611 (ワッチョイ b3e3-10BI)
垢版 |
2018/08/07(火) 21:35:47.39ID:UKw4EMmv0
無理矢理LINQつかってみたけど>>613に一票
0616デフォルトの名無しさん (ワッチョイ ff58-0Uuo)
垢版 |
2018/08/07(火) 21:41:47.05ID:7nA6P5yg0
dataAとdataBが1000件以上あるので、まず単純に変更してるのがあるか?で
SequenceEqualを使ってみた流れでLINQを調べていた流れで知りたかったです。
forで書くのがわかりやすいとは思うけど、LINQでもっとうまく書けるかなと思って

>>611参考になりました。ありがとうございます!
0618デフォルトの名無しさん (アウアウカー Sa07-kkiZ)
垢版 |
2018/08/07(火) 23:11:20.30ID:EHUuk9/6a
zip使えばいいんでね?
0619デフォルトの名無しさん (ワッチョイ 8f9f-I2NA)
垢版 |
2018/08/07(火) 23:26:49.96ID:WUtnzCvr0
Linqのほうがわかりやすいし、ソースがランダムアクセスできるとも限らんだろ

シーケンスを束ねてインデックス振って比較して違うやつだけ取り出してインデックスを取り出す
Linqなら思ったことをそのまま書けばいい

IEnumerable<int> Hoge<T>(IEnumerable<T> a, IEnumerable<T> b) {
var comp = Comparer<T>.Default;
return a.Zip(b, (x, y) => (x, y)) // シーケンスを束ねて
.Select((e, i) => (i, e.x, e.y)) // インデックス振って
.Where(e => comp.Compare(e.x, e.y) != 0) // 比較して違うやつだけ取り出して
.Select(e => e.i); // インデックスを取り出す
}

ループじゃぱっとみ何やってるかわかんねえよ
0621デフォルトの名無しさん (ワッチョイ cfe8-Xflc)
垢版 |
2018/08/07(火) 23:35:51.16ID:vAj7NHSO0
LINQってEntityFrameworkでしか用途なくねえ?
0624デフォルトの名無しさん (アウアウエー Sadf-Eypl)
垢版 |
2018/08/08(水) 00:39:26.55ID:cXv1v5r0a
>>619
うーん、普通にこっちのほうが分かりやすいと思うけどw
別にLINQ否定派じゃないよ
IEnumerable<int> GetIndices()
{
  List<int> dataA = new List<int>() { 1, 2, 3, 4, 5, 6 };
  List<int> dataB = new List<int>() { 1, 2, 4, 3, 5, 6 };
  for (int i = 0; i < dataA.Count; i++)
    if (dataA[i] != dataB[i]) yield return i;
}
0625デフォルトの名無しさん (ワッチョイ 338a-Xflc)
垢版 |
2018/08/08(水) 03:30:51.92ID:x4iNladl0
Zipしたものをforeachで回すのが一番自然
IEnumerable<int> Hoge(IEnumerable<int> A, IEnumerable<int> B)
{
  var i = 0;
  foreach (var eq in A.Zip(B, (a, b) => a == b))
  {
    if (!eq) yield return i;
    i++;
  }
}

自分はやらんけどLinqならこう
dataA.Zip(dataB, (a, b) => a == b).Select((eq, i) => eq ? -1 : i).Where(i => i >= 0);
0627デフォルトの名無しさん (スップ Sd1f-Bh6x)
垢版 |
2018/08/08(水) 13:38:09.59ID:CtU8HUVGd
インデックスが必要な時、わざわざselect挟んでインデックス貼るの馬鹿らしい感じがして好きになれないんだけど仕方ないよね?
そんなにインデックスを参照することが多いなら生成時に一緒に貼っとけって話かもしれんが…
そもそもlinqマスターはインデックスを必要としない設計にいつもなるの?
自分もインデックスが無くて済むように考えるけど結果的に必要になる箇所が細部で出てくるんだよね
0628デフォルトの名無しさん (スプッッ Sd1f-hmJy)
垢版 |
2018/08/08(水) 14:08:00.53ID:csO2M6exd
どうしても必要な時はselect((value, index) => new {V = value, I = index})でインデックス引っ張ってるわ
それでもインデックス欲しい条件1行で書けるのが楽だからLinq便利派だけども
0629デフォルトの名無しさん (スップ Sd1f-Bh6x)
垢版 |
2018/08/08(水) 14:25:20.52ID:CtU8HUVGd
>>628
俺もそれ、ちょくちょくするんだけどforで回したほうがよくね?って思っちゃうことがあるんだよね
もちろん処理によりけりと言ってしまえばそれまでなんだけど
比較するようなもんじゃないかもしれんがjavascriptなんかだと大抵インデックス取れるから、そっち触ったあとにC#に戻ると処理考えるときにインデックスありきで考えがちになっちゃう気がしてる
0630デフォルトの名無しさん (スップ Sd1f-Bh6x)
垢版 |
2018/08/08(水) 14:32:55.42ID:CtU8HUVGd
先に出てた課題でもzipしたあと、そのままlinqで処理を完結させるか、zip後にforで回すか
個人的にはzipがどちらにせよ入るなら最後までlinqの方が1つの処理を1つの手法で解決してる感じがして好き
だけどインデックスが処理に必要だからfor使いたいって気持ちもわかる
0632デフォルトの名無しさん (ワッチョイ 338a-Xflc)
垢版 |
2018/08/08(水) 15:07:53.10ID:x4iNladl0
こういう拡張メソッド作っておくのはたまにやる
public static IEnumerable<int> WhereIndex<T>(this IEnumerable<T> self, Func<T, bool> predicate)
{
  var i = 0;
  foreach(var s in self)
  {
    if(predicate(s)) yield return i;
    i++;
  }
}

dataA.Zip(dataB, (a, b) => a == b).WhereIndex(eq => !eq);

拡張メソッド嫌いな人もいるだろうけど一番読みやすい
わざわざSelectでインデックスつけてる冗長さもないし、添字アクセスもないし、使う時はすっきり意味がわかりやすい
0633デフォルトの名無しさん (ブーイモ MMa7-A/YO)
垢版 |
2018/08/08(水) 16:44:49.34ID:Gn4Y43YUM
>>632
正直クソ分かりにくい
俺がレビューしたらリジェクトするわ
WhereIndexというメソッド名を見たら大抵の人はインデックスをフィルタ条件にして要素の値を返すメソッドと誤解する
そもそもメソッド名以前に、LINQって普通はキーやインデックスではなく要素の値を操作していくものっていう暗黙的了解があるから、
キーやインデックスの方を返すような操作については特に念入りにそれがわかるような明示的な表記になるように工夫したほうがいい
0639デフォルトの名無しさん (ワッチョイ 338a-Xflc)
垢版 |
2018/08/08(水) 18:27:19.45ID:x4iNladl0
>>633
>WhereIndexというメソッド名を見たら大抵の人はインデックスをフィルタ条件にして要素の値を返すメソッドと誤解する

そんなのはWhereでできるし

>LINQって普通はキーやインデックスではなく要素の値を操作していくものっていう暗黙的了解があるから、
>キーやインデックスの方を返すような操作については

それもSelectでできるし
だいたい要素の値を操作していくものっていう暗黙的了解があるならWhereIndexも要素の値が対象と思うんじゃない?
なんか矛盾してね?
0644デフォルトの名無しさん (ワッチョイ de81-8ulZ)
垢版 |
2018/08/11(土) 00:11:00.13ID:p6HuaRWz0
>>643
それってインデックスつけることが一つの目的じゃない?
インデックスってかソートかな?
いろんなところでいろんな処理されるコレクションがあって、その処理の殆どにインデックスが不要だけどごく一部にインデックスがほしいとき、わざわざ1行追加するのスッキリしないなぁって感じ

たった1行なんだから書きゃいいんすよ?
それはわかってるけどなんか気持ち悪いなぁって気がするだけ
0645デフォルトの名無しさん (ワッチョイ 4e9f-+CD1)
垢版 |
2018/08/11(土) 12:41:19.45ID:YyI+aygL0
ほとんど必要が無いインデックスだけど
ごく稀に必要になるから必要になるところでだけインデックスを追加するんだろ
ほとんど使わないインデックスのためにずっとランダムアクセス性を維持するとか無駄じゃん
0647デフォルトの名無しさん (ワッチョイ de81-8ulZ)
垢版 |
2018/08/11(土) 14:23:04.22ID:p6HuaRWz0
>>645
いや、linqでインデックスを付与しないけどランダムアクセスすることはあるよ?
前提変えたらそら意味通らんよ
コレクションをlinq以外で処理しちゃ駄目ルールでもあるの?
0648デフォルトの名無しさん (ワッチョイ 4e9f-+CD1)
垢版 |
2018/08/11(土) 14:36:01.31ID:YyI+aygL0
>>647
なにいってんだ?
IEnumerable<T>にインデックスなんてない
ないからSelectで付与するんでしょ

そんなルールはないけどほとんどほとんどLinqで済むでしょ?
Linqでサクサク処理してるのに中に突然ループが紛れ込んできたらキモいって話をしてる

var tmp = list.Where(...).OrderBy(...).ToArray();

for (int i = 0; ...) {
...
}

var result = tmp.Where(...).Select(...);
0649デフォルトの名無しさん (ワッチョイ de81-8ulZ)
垢版 |
2018/08/11(土) 14:48:08.19ID:p6HuaRWz0
>>648
え?ずっとそういう話をしてたのになんで突然
>ほとんど使わないインデックスのためにずっとランダムアクセス性を維持するとか無駄じゃん
って書いたの?

インデックスのないlinq処理でそれが必要になったときわざわざselectで付与するのなんか気持ち悪いねーって話よ?
0650デフォルトの名無しさん (ワッチョイ 4e9f-+CD1)
垢版 |
2018/08/11(土) 14:59:36.83ID:YyI+aygL0
>>649
気持ち悪いのはお前の感覚ではという話だろ
俺からすればToArrayやToListをかましてメモリとCPUを無駄にしたり余計なメソッドを追加するほうが嫌
メソッド化に関しては再利用性が高いかうまく抽象化されていて可読性が高まるというなら別だが
真にランダムアクセス性が必要なアルゴリズムならば変換せざるを得ないがそもそも今回の問題には必要がない
0651デフォルトの名無しさん (ワッチョイ de81-8ulZ)
垢版 |
2018/08/11(土) 15:12:53.25ID:p6HuaRWz0
>>650
うん、そうだよ
個人的に気持ち悪いなーってだけ
はじめからそういうふうにしか話ししてないつもりだけど

ランダムアクセスが不要だからあえてランダムアクセスできないコレクションを選択するの?
ランダムアクセスは不要だけどあったって使わないだけだからどっちでもいい、ってなるのが普通じゃないの?
パフォーマンス要求がシビアでランダムアクセスできるコレクションを選択することで、その要求から外れてしまう、とかいうならわかるけど
0652デフォルトの名無しさん (ワッチョイ 4e9f-+CD1)
垢版 |
2018/08/11(土) 15:28:37.80ID:YyI+aygL0
>>651
ランダムアクセスが不要なアルゴリズムなら「当然」IEnumerable<T>(ランダムアクセスできないコレクション)を選択するだろう
まったくもって「あえて」と言う理由が分からん
もしもSelectやWhereの引数が配列だったら普通のプログラマなら「あえて配列にしたのはなぜか」と疑問を持つだろう
なんか逆なんだよね思考パターンが
0654デフォルトの名無しさん (ササクッテロラ Sp03-bBK3)
垢版 |
2018/08/12(日) 00:56:37.14ID:UOoEXFjMp
コレクションって言葉がダメだと思うぞ?
そもそもIEnumerableはシーケンシャルな列挙の機能しか提供しない
根本的に理解が間違ってる
0660デフォルトの名無しさん (ワッチョイ b759-EyNT)
垢版 |
2018/09/13(木) 22:10:23.24ID:IOzhaO6j0
doxygenみたいなドキュメント生成ツールを使うとxaml部分がまともに解析出来ません。
bindingしているviewとviewmodelの関係なんかが分かるドキュメントにしたいんですが何か良い方法ないでしょうか?
0661デフォルトの名無しさん (ワッチョイ 57d2-NCQU)
垢版 |
2018/09/14(金) 22:17:10.47ID:BClpbb6X0
Func<string> func = () => { return "text"; };
var res = func();

これを一文で書くことはできますか。

ふと気になっただけではあるけれど、自分で解決できなくて気になって仕方がない。
基本的なことが分かってないような気がする。
0664デフォルトの名無しさん (ワッチョイ ffe8-okpm)
垢版 |
2018/09/15(土) 00:21:48.95ID:elLmwu/20
本当はなにをしたかったんだろう
例題が悪い気がする
0665661 (ワッチョイ 57d2-NCQU)
垢版 |
2018/09/15(土) 00:41:48.54ID:LMkhUY/p0
ああ、そうか。w

パラメータに対してちょっと複雑な変換を掛けるのだけど、汎用性がないから専用関数作るのもなんだし、
だったらラムダ式で書けばいいんじゃね?→ ラムダ式は Func 型?だから戻り値とれねーじゃん ってところから進めなった。
他にやり方はあるので出来なくてもいいのだけど、気にはなって。
0667デフォルトの名無しさん (ワッチョイ ff81-meyQ)
垢版 |
2018/09/15(土) 05:59:42.97ID:crViL/AR0
>>665
参照渡すとか?
■ このスレッドは過去ログ倉庫に格納されています

ニューススポーツなんでも実況