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

2018/02/04(日) 22:02:28.59

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

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

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part134
http://mevius.5ch.net/test/read.cgi/tech/1511951038/
■関連スレ
C#, C♯, C＃相談室 Part95
http://mevius.5ch.net/test/read.cgi/tech/1508180530/
C#, C♯, C＃相談室 Part93
https://mevius.5ch.net/test/read.cgi/tech/1492818720/

■コードを貼る場合は↓を使いましょう。
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: default:vvvvv:1000:512:----: EXT was configured

2018/02/06(火) 18:04:54.61

>>116
お前は一年間Linqを使わないで普通にコード書け
Linq使うと上達から遠ざかる

2018/02/06(火) 18:09:27.95

馬鹿な初心者の例

エラー内容を読まない
質問するときも上手くいかないなどあいまいな言葉しか使わない

2018/02/06(火) 18:29:23.79

>>116
自己解決！！

2018/02/06(火) 19:15:12.35

みなさんありがとうございます
全部参考にさせて頂きます
anyの使い方を知りませんでした
初心者的には>>111が一番わかりやすかったです
私の書いたコードを流用してくれているからだと思いますが
他の例の様にエレガントなコードを書けるように精進します

2018/02/06(火) 20:28:32.37

private　void　test3()
{
　　　　List<Label>　dev=Enumerable.Range(0,　10).Select(d=>new　Label()　{　Name="Label"　+　d.ToString()　}).ToList();
　　　　List<string>　strdt=Enumerable.Range(0,　10).Select(d=>"--"　+　d.ToString()　+　"--"　).ToList();

　　　　dev.Zip(strdt,　(x,　y)=> x.Text=y);
　　　　Debug.Print("--------------------------------");
　　　　foreach　(var　a　in　dev)
　　　　{
　　　　　　　　Debug.Print(a.Name　+　":"　+　a.Text);
　　　　}
}
これが動かない。同じ単純なアレイどおしなら問題なく動くと思うが、この場合は
色々やってみたがわからない。ZIPの後にForeachをかけるという
意見があったが、どうやって？　マージした後にForeachかけるにはベクトルのコレクションを
返す必要がある。その方法が分からない。

**デフォルトの名無しさん** (ｱﾒ MMfb-xw+F) · 2018/02/06(火) 20:32:12.84

以下のコードを実行すると
array[index]へのアクセスで
out of range exceptionがでます
何故index==numItemsという状態が起こるのでしょうか？

for (var i = 0; i < numItems; i++)
{
task[i] = new Task(() =>
{
var index = i;
foreach ()
{
array[index]にアクセス
}
});
task[i].Start();
while((int)task[i].Status < (int)TaskStatus.Running)
{
Console.WriteLine(i+"\t"+task[i].Status);
}
Console.WriteLine(i + "\t" + task[i].Status);
}

2018/02/06(火) 20:47:55.13

>>121
何がしたいの？
動かないって言われてもエスパーじゃないんで...

2018/02/06(火) 20:50:18.07

っていうか、
＞dev.Zip(strdt,　(x,　y)=> x.Text=y);
ここのラムダ式の戻り値はyになるんだけど、これは意図してるのか勘違いなのか...

2018/02/06(火) 20:57:00.40

>動かないって言われてもエスパーじゃないんで...

int count = 0;
dev.ForEach(x => x.Text = strdt[count++]);

やりたいのはこれと同じこと。これは動作する。

2018/02/06(火) 21:07:22.14

＞＞dev.Zip(strdt,　(x,　y)=> x.Text=y);
＞ここのラムダ式の戻り値はyになるんだけど、これは意図してるのか勘違いなのか...

ここの意図はdevがLabelのコレクションだからｘはそのラベルの一個を取り出す。
ｙはsrtdtがstringコレクションだからｙはその一つの要素
x.Text=y
はラベルのテキストにストリングｙを代入すること。
ラムダ式の戻り値はｙになるんか？　でもラベルにｙをセットするのが目的だから戻り値は関係ないはず。

var retval = dev.Zip(strdt, (x, y) => x.Text = y);
Debug.Print("--------------------------------" + retval);
foreach (var a in dev)
{
Debug.Print(a.Name + ":" + a.Text);
}
戻り値をretvalに入れてプリントしたらこんなになってる。
--------------------------------System.Linq.Enumerable+<ZipIterator>d__61`3[System.Windows.Forms.Label,System.String,System.String]
Label0:
Label1:
Label2:
Label3:
Label4:
Label5:
Label6:
Label7:
Label8:
Label9:

2018/02/06(火) 21:10:54.60

--------------------------------
Label0:--0--
Label1:--1--
Label2:--2--
Label3:--3--
Label4:--4--
Label5:--5--
Label6:--6--
Label7:--7--
Label8:--8--
Label9:--9--
これが期待値

2018/02/06(火) 21:16:50.87

>>125
だったら最初からこれでいいのでは？知らんけど
List<Label> dev=Enumerable.Range(0, 10).Select(d=>new Label() {Name="Label" + d,Text = String.Format("--{0}--", d)}).ToList();

質問する前にもうちょっと自分で頭使って考えた方がいい気がするw

2018/02/06(火) 21:19:36.18

隔離スレが必要なのでは？

2018/02/06(火) 21:20:12.19

なぜか専用ブラウザから書き込めなかったのでChromeから書き込んだけど、
これだけでワッチョイって変わるんだね知らんかった

2018/02/06(火) 21:24:21.16

ここ数日の流れでわかったと思うけど
自前のアプリが完成するまでずっとこの人は何かコード書くたびに質問してくると思うけど
それでいいのかな？

2018/02/06(火) 21:26:28.75

質問じゃなく承認欲求のための報告
連続してどうでもいいことまで書き込んだり質問する
今までも割といたタイプ

2018/02/06(火) 21:33:07.27

>>128
今は補完文字列が使えるんだったね...
最近コード書いてないからもう忘れてた

恥人 · 2018/02/06(火) 21:36:56.00

>>115
ネタじゃないです(>_<)
ダメな点を教えてください(>_<)

2018/02/06(火) 21:41:08.26

>>131
ウザいよね
目的の処理自体は作成できるくせに
聞いてくるなんてキチガイだよね

2018/02/06(火) 21:44:25.92

＞＞１２８
ZIPの理解が目的なんでな。ｗ

2018/02/06(火) 21:52:36.71

>>134
横からだけど
いくつか気になるところがあるけど

とりあえず最後の二つの操作をよ～く見て動作を考えてみたらいいのでは？

var maxDataList = flaggedList.Where(data => data.Value == maxValue);
var hasMaxValue = maxDataList.Any(data => data.Key == key);

**愛恥人** · 2018/02/06(火) 21:53:01.64

>>121
自分が言ったのはこういうことです(>_<)
匿名クラスを使ってます
var marged = dev.Zip(strdt, (x, y) => {Label=x, Text=y});
marged.ForEach(item => item.Label.Text = item.Text);

2018/02/06(火) 21:54:49.64

>>136
理解が目的なら挙動をテストするコード書けばいい
どこかの記事をコピーして実行
そして徐々に変えていって仕組みを理解したらいい

**愛恥人** · 2018/02/06(火) 21:58:11.36

>>137
意図した動作にならないという意味でしょうか、それともひとまとめにできるという意味でしょうか(>_<)
後者の場合は、一行づつで簡単な意味しか持たないほうが良いと思いすべて分離したのですがまとめまくったほうが良いのでしょうか(>_<)

2018/02/06(火) 22:12:46.39

>>134
ネタにしか思えんけどw

コメントも余分な一時変数も可読性に寄与しないどころか悪くしかしてないと思う。
好みの問題じゃないと思うけどな
こっちの方が読みやすいでしょ
public bool hasMax(list<Data_hoge> list, int key)
{
　　var flaggedList = list.Where(data => data.Flag);
　　var maxValue = flaggedList.Max(data => data.Value);
　　return flaggedList.Any(x => x.Key == key && x.Value == maxValue);
}

**愛恥人** · 2018/02/06(火) 22:13:47.78

>>141
ありがとうございます(>_<)
今後の基準にします(>_<)

2018/02/06(火) 22:14:50.74

>>136
その割にZIPの本来の使い方してないじゃんw
意味が分からないよ

2018/02/06(火) 22:18:48.00

もともと条件がふわっとしてるからまあ適当でいいよ
条件をつめて（keyがユニークなのかなど…）行くと普通こういうコードは書かないと思う

2018/02/06(火) 22:21:05.28

>>141
元ネタからそうだけど、メソッドはパスカル記法(先頭大文字)で。

**愛恥人** · 2018/02/06(火) 22:21:37.74

>>144
一行づつで一つの意味しか持たないレベルに分解するのはやりすぎということでしょうか(>_<)
ありがとうございました。

**愛恥人** · 2018/02/06(火) 22:22:53.83

>>145
確かにそうするべきでした(>_<)
ありがとうございます。

2018/02/06(火) 22:24:39.50

>>136
ああついでに言っとくと、あなたはたぶん

(1) 列挙を行うオブジェクトを返すこと
(2) 列挙を行うこと

この二つを混同してるんだろうね。
LINQは前者であって後者じゃないのよ。

2018/02/06(火) 22:26:05.95

匿名関数なんてパインソのリスト内包表記とかでイメージつかまないとムズいよ？
それになぜそんなにリンクにこだわるのか
いまの時代にカッコつけてスーツに肩パッドいれてる人みたい
要するにカッコつけかたミスってるよ

2018/02/06(火) 22:26:42.07

>>146
深入りしなくてもいいけど
そのコードともとの2行のコードは結果は同じだけど
ロジックによる動きを追っていくと違いが判ると思う

2018/02/06(火) 22:27:51.04

＞＞１３８
いい線行きました。！！！
　　var marged = dev.Zip(strdt, (x, y) => new {lbl = x, txt = y} );
　　foreach (var aa in marged ){
　　　　aa.lbl.Text = aa.txt;
　　}
これで動作してますが、Foreachはmargedにメソッドがでてこない。

2018/02/06(火) 22:32:31.84

みんなよく構ってるな

2018/02/06(火) 22:36:21.97

>>138
できました。解決。ありがとう。

**愛恥人** · 2018/02/06(火) 22:40:30.16

>>149
LINQは手続き型（？）よりも簡単で良いと思っていたのですが勘違いだったのですね(>_<)
参考にします(>_<)

>>150
元の２行のコードというものが分からないです(>_<)
すみません。

>>151
列挙型にForEachがないみたいですね(>_<)
ごめんなさい(>_<)

2018/02/06(火) 22:56:56.00

＞この二つを混同してるんだろうね。
＞LINQは前者であって後者じゃないのよ。

折角のアドバイスにきつい言い方と思うかもしれないが、抽象的な説明よりも具体的に説明をすると
初心者には分かりやすい。あなたにも初心者の部下が沢山いるとおもうがそういうことを心掛けた方がいいと思う。
一番具体的なのは正しいコードで示すこと。
二番目はキーとなる概念が入った間違ったコード。（実際には正しいコードを空で書くのは、ベテランでも難しいからね）
３，４，がなくて５が説教。ｗ

2018/02/06(火) 23:11:07.29

＞それになぜそんなにリンクにこだわるのか

リンクに拘ってるのはもちろん初心者に最も理解しやすいからで、Zipにとくに拘ったのは
二つのシーケンスを同時に回せないと、一方でForeachを使いながら、同時にカウンターを
回すという不細工なことになるから。

2018/02/06(火) 23:36:29.47

>>155
なんか楽しい人だなw

2018/02/06(火) 23:38:39.75

なんだそのゴミレス...

2018/02/06(火) 23:38:57.77

偉そうだな
わからないのが偉いのかな？
わからないのは自分の責任だと思うが

小学生の頃も学校の先生に文句言ってたんだろ

2018/02/06(火) 23:46:58.21

回答者に対して具体的にコードを書かないといけないと言い出す初心者
モンスター質問者

自分で手を動かして試して考えられないならコードを書く意味なんてない
永遠に誰かに書いてもらえよ
ここでいくらコードを聞いても無駄だな

2018/02/06(火) 23:47:34.86

必死やな。

2018/02/07(水) 01:20:10.99

わからないことを知っている

2018/02/07(水) 01:30:26.34

ごちゃごちゃ言わずにスルースキルを鍛えろよ
最近はLINQコード書いたレスは全部NGブチ込んでる
こいつの質問超つまんないし
だいたい自分で作成できる処理について他人に質問するなんて気持ち悪いよ

2018/02/07(水) 01:41:27.68

オッサンか。

2018/02/07(水) 01:43:34.31

いいえjkです

2018/02/07(水) 05:11:16.76

早く人間になるか死ね

**122** (ｱﾒ MMfb-xw+F) · 2018/02/07(水) 10:29:05.60

>>122 ですが自己解決しました。
Parallel.Forにしたら問題なく動きました。

2018/02/07(水) 12:29:01.11

>>167
.net4止まりの環境か？
なぜasync/await使わないんだ？

2018/02/07(水) 12:37:35.45

>>168
asyncとParallelは役割が違う
Parallelはデータ並列の同期処理

2018/02/07(水) 12:43:07.80

>>169
よくコード見てみ

2018/02/07(水) 12:50:17.72

>>170
TaskやめてParallel.Forにするなら意図通りに動作するし別におかしな処理ではないだろ
awaitしたら逐次処理になっちゃうぞ

2018/02/07(水) 12:53:02.46

難を付けるとしたら、arrayの各要素に対する処理を並列化したいんだったらParallel.ForEachの方がいいな

2018/02/07(水) 17:10:10.88

SVGを使いたくてNuGetからSVG2.3というのをインストールしたんですが
これってマニュアルみたいなものはないんですか？
とりあえずググってみて簡単な長方形とテキストは表示できたんですが
どうやって覚えたらいいんでしょうか

2018/02/07(水) 18:05:16.53

SVGが使いたいなら普通にSVGを勉強したらいいんじゃないの？

ここでHTMLの書き方について質問されてるのと変わりない

2018/02/07(水) 18:11:47.28

一般論として使い方のわからないライブラリがあるならNuGetに書いてあるオフィシャルサイトに
飛んでそこから調べる

最終更新が3年前とかだったら捨てる

2018/02/07(水) 19:06:14.35

Visual Studio Community 2017 を使って
特定のフォルダ「c:\hoge」以下のサブフォルダを列挙する
UWPアプリを作ろうとしています。

IEnumerable<string> subFolders = Directory.EnumerateDirectories(@"c:\hoge", "*", SearchOption.AllDirectories);

foreach (string subFolder in subFolders) {
MessageDialog md = new MessageDialog("");
md.Content = subFolder;
md.ShowAsync();
}

とすると
System.UnauthorizedAccessException:
'Access to the path 'c:\hoge' is denied.'
とのエラーとなります。

しかし、対象フォルダをデバッグビルドされるフォルダ
「C:\foo\bin\x64\Debug\AppX」にすると思った挙動となります。

デバッグビルドされる以外のフォルダにアクセスできるようにするには
どうすればいいでしょうか？
ちなみに Visual Studio を管理者権限で実行しても変わりませんでした。
以上、お知恵をお貸しください。よろしくお願いします。

2018/02/07(水) 19:17:14.76

>>176
UWPはそういうもん

セキュリティ上ユーザーに許可されない場所は基本的にアクセスできない

2018/02/07(水) 19:19:24.20

忘れたけどユーザーにフォルダピッカーで選ばせないとアクセスできない

アクセスしたらその情報を保存する方法があって次回からそこは普通にアクセスできるようになる
めんどくさいからUWPやPCLは触りたくない

2018/02/07(水) 19:25:31.81

>>169
いやいや、taskつかっとるやん！

2018/02/07(水) 19:25:52.82

初心者はUWPを避けたほうがいいと思う
UWｊP自体使われていないからこんな質問が出てくる
これ以外にもめんどくさい仕様がてんこ盛り

ある程度勉強が終わった後改めてUWPやったほうがいいよ

2018/02/07(水) 19:29:40.16

初心者が上手にアプリの状態を保存したりしつつ画面遷移できるとは思えない

2018/02/07(水) 19:43:44.01

>>177,178,180,181
レスありがとうございます!
素直にフォームアプリで挑戦します。

**122** (ｱﾒ MMfb-xw+F) · 2018/02/07(水) 20:06:28.97

>>172
後出しですいませんが
並列化したかった箇所は外部の特殊な装置へのアクセスで並列化した数だけドライバのインスタンス持ってます。
arrayとしましたが、最終的にはconcurrentDictonaryを使い装置からのデータをまとめています。

async await使ったことないのですが、
また調べてみます。ところでそもそもの質問ですが、
>>122のコードでtask内からループカウンタを参照した時にnumItems以上になるのでしょうか？

2018/02/07(水) 20:14:13.39

>>174
スレチだったんですね
ありがとうございました

2018/02/07(水) 20:34:01.56

UWPの件をひっくり返してみてみると
その他のプラットフォームのアプリは好きなところを読出し放題なんだよ

恥人 · 2018/02/07(水) 20:49:50.30

スレチ

2018/02/07(水) 21:17:02.65

>>183
whileのタスクの終了待ち？が機能してない
だからタスクより先にi++が行われる可能性があるのでエラーの現象がでてる
IsCompleteってプロパティがあるからそれを使うといいよ

2018/02/07(水) 21:23:15.78

そもそも>>122は別スレッドで実行しているだけで非同期実行になってないね
実行するとこんな感じ
https://i.imgur.com/mxuCicn.jpg

2018/02/07(水) 21:31:07.99

既存のXMLを編集したい
編集したタグ以外の要素はテキスト表現を変えたくない

どうしたらいいかな？
コンペア取った時に編集した箇所以外も差分に含まれてしまい困ってる
1回DOMを経由してる以上仕方がないのかもしれないけどなんとかしたい

2018/02/07(水) 21:41:31.84

>>189
もっと詳しく

**122** (ﾜｯﾁｮｲ 97b3-xw+F) · 2018/02/07(水) 21:42:13.93

>>188
検証ありがとうございます。
各taskがパラレルに実行されてるから期待どおりです。
IsCompleteにすると当然Serialに実行されるのでやりたい事じゃないです

whileは終了まちではなくdebug用途で足しました。

2018/02/07(水) 21:42:16.51

それは簡単だよ。編集した箇所いがいは差分にでないコンペアをすればいいだけだ。

2018/02/07(水) 22:16:31.84

>>191
そうなのか
それならインデックスの宣言をタスクの外に持ってくだけで希望通りの動作になったのに
パラレルforだと、forのあと重たい処理やってからwaitみたいなことできないから気を付けてな

**122** (ﾜｯﾁｮｲ 97b3-xw+F) · 2018/02/07(水) 22:36:14.66

>>193
あ、確かにtaskの外に置くべきでした。
元はindex = iもなしで直接iを参照してたのでってよく考えると危ないことしてました

2018/02/07(水) 23:03:46.15

>>189
手で編集すればいいじゃん

2018/02/08(木) 07:40:14.42

いきなり質問失礼します
スクレイピングで取得した要素を
while (true)
{
var B = driver.FindElementByid(''id1").Text;
var S = driver.FindElementByid(''id2").Text;
Console.WritLine("要素Bは{0} 要素Sは{1}", B, S);
}
でコンソールに表示しています
BとSには常時変化する小数点のある数字が入ります。
この後、

if (B > S)

を実行したいのですが、BもSも文字列として認識されている為出来ません。

数字として認識させる為にはどうすればいいですか？

2018/02/08(木) 07:50:14.20

「c# 文字列数値変換」で検索！

2018/02/08(木) 08:55:18.85

もしかしてバージョン比較だったりしてね。

2018/02/08(木) 10:13:31.71

>>197
変換出来ませんと出て行き詰まってます

>>198
単にどちらが大きいか見て
コンソールに大きい方を表示させたいだけです

2018/02/08(木) 10:19:57.85

実行すると変換できませんと表示されるコードを貼れ

2018/02/08(木) 10:34:17.49

MessageBox.Show("変換できません");

2018/02/08(木) 10:47:13.30

int.parse

2018/02/08(木) 10:49:23.48

>>196
というか、こんなコードでスクレイピングとかdos攻撃を疑うレベル。
まじて迷惑だからやめて

2018/02/08(木) 11:11:17.51

if ((true == double.TryParse(B, out double valueB)) &&
(true == double.TryParse(S, out double valueS)))
{
// valueB と valueSを比較
}

2018/02/08(木) 12:20:41.72

>>203
ワイはお前の知性を疑っとるで
余計な事言うとすぐばれんねんバカって

2018/02/08(木) 12:40:08.26

>>200
それをする事でどうなりますか？

>>203
始めて1週間なので、
そう思われる理由が分からないので教えてください

>>204
ありがとうございます
試してみます！

2018/02/08(木) 14:11:43.01

>>206
どういうコードを書いたせいで動かないのかが分かるんだろ
そうでなければお前は何を解決したくて質問したの？

2018/02/08(木) 15:08:58.18

無駄なメモリの確保を防ぎつつ、フィールドのスコープを限定する方法を教えてください

現在、このようなメソッドを用意して秒間300回程度と頻繁に呼び出しています
void A(){
byte[] data = new byte[1024];
//dataに書き込む処理省略
}
しかし、私の行う用途にでは、呼び出しのたびにdataの配列分確保するのは無駄で、前回の使い回しで十分なんです
dataをメソッドの外で宣言しメモリの確保もすればいいのかもしれませんが、スコープが広がってしまいます(すでに書いてしまったコードがdataという名前のフィールドだらけで収拾がつかなくなっています)
何か手軽に無駄なメモリ確保を防ぐ方法は無いのでしょうか？
よろしくお願いします

2018/02/08(木) 15:19:58.53

static

2018/02/08(木) 15:34:43.45

>>208
VBにはインスタンスと同じ寿命でスコープだけメソッドの変数があるけどC#にはありまへん。
意味があると思えんけど、ラムダ式のキャプチャを利用して、byte[]ではなくFunc(byte[]>のフィールドを持つとか。

まあ、そういう馬鹿馬鹿しいこだわりを捨てて、素直にbyte[]のフィールドを持つのが一番でしょう。

恥人 · 2018/02/08(木) 15:45:33.68

>>196
常時更新ってC#のコードでDOM操作するってこと？

2018/02/08(木) 16:39:22.84

>>208
どうせ初心者のうちに書いたコードなんて後で全部捨てて書き直すことになるんだから余計なこと気にしなくてよろしい

2018/02/08(木) 16:39:44.59

この間のwin10のアップデートでwebbrowserの挙動に何か変化有りませんでしたか？

2018/02/08(木) 17:48:49.85

>>207
変換出来ませんと表示させるコードを書けば、解決策が見つかると言う意味で解釈したらいいですか？

変換出来ませんと表示させるコードを書く理由が分からないので教えてもらえると助かります。

恥人 · 2018/02/08(木) 17:50:05.38

ワロタ

2018/02/08(木) 17:53:07.66

>>211
これがDOM操作と言うのか私には分かりませんが、
ウェブサイトに表示されている、リアルタイムで変化している数字を読み取って比較して表示するという操作がしたいって事です