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

2017/11/29(水) 19:23:58.05

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

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

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part133
http://mevius.5ch.net/test/read.cgi/tech/1510056685/
■関連スレ
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: checked:vvvvv:1000:512:----: EXT was configured

2018/02/03(土) 00:58:38.87

>>850
いや、今のルーチンはコメントの削除からやってるのよ

2018/02/03(土) 01:03:27.72

コメントも文字列も先に始まったほうが勝つからどうしょうもない

2018/02/03(土) 01:06:21.86

質問しといて聞かないパターン＝成長しない初心者パターン

2018/02/03(土) 01:12:38.71

ただ悩みを聞いて欲しかったんじゃないですか
女の相談事みたいなw
まあ、どうせ人も減ってるしそれもありでしょう

2018/02/03(土) 01:24:32.23

>>853
いや、アドバイス通り
頭から読んでいくしかないっぽいっすね
クレームが来てるところがまさにそこなので誤魔化しが効かない

2018/02/03(土) 08:34:38.62

>>854
お姉さんが聞いてあげるわ。
今夜、川口駅リリア側の出入り口で待ってる

2018/02/03(土) 09:02:44.63

swich() { case xxxx: ~
xxxxに変数をつかいたいのだが、switchに変わるものってなに？

2018/02/03(土) 09:33:50.59

2018/02/03(土) 09:53:42.04

>>857
xxxが数値か文字列なら変数だろうと使えたはずです。オブジェクトの場合、同値の判定が基本型のように単純じゃないから無理。

2018/02/03(土) 10:50:39.85

switchi( (button)sender) ){
case Button1: ~
case Button2:~
case Button3:~
}
こんな感じのことがしたい。

～はMethod()でいいので、
void switch_tb()= {
{Button1, Method1}
{Button2, Method2}
{Button3, Method3}
}
switch_suru((button)sender);
こんなことできんか？　

2018/02/03(土) 10:52:21.97

C#はメソッドの中のローカルメソッドってできへんの？

2018/02/03(土) 11:00:14.66

>>860
素直にButtonのNameプロパティ使え

2018/02/03(土) 11:01:41.64

>>861
C#7で出来るようになったよ

2018/02/03(土) 11:41:19.78

糞化が止まらないのか。

2018/02/03(土) 12:15:40.34

メソッド内メソッドは無くてもいい機能
メンバー封じ込めの方法はいくらでもあるし

ないほうがエラー見つけやすいと個人的には思う

2018/02/03(土) 12:24:29.31

いい方法をみーつけ

2018/02/03(土) 12:24:59.79

バグだろwなにちょっと誤魔化そうとしてんのw

2018/02/03(土) 12:29:16.06

>>864
言語の変化についていけないやつの糞化な

2018/02/03(土) 12:30:20.78

>>860
Buttonと関数のDictionaryでできそう
試してないけどこんな感じでメソッド取ってくれば…

Dictionary<Button, Func<T, TResult>>

2018/02/03(土) 12:31:12.90

メソッド内メソッドはjavaでずっと使ってたけど
クラスを小さくしていくと使う場面は限られていく

2018/02/03(土) 12:33:37.55

初心者っぽくていいじゃん
「この機能が存在する意味が分からないんですけど、何の役に立つんですか？」
って定番の質問じゃん

2018/02/03(土) 12:40:45.58

使ってる奴も糞ばっかになったな。新しい機能一つ覚えただけでマウンティングしだすアホとか。
なぜその機能が糞かが理解できない脳みそ。取り巻きからして残念な言語になってしまった。

**愛知人** · 2018/02/03(土) 12:41:41.35

あらら、怒っちゃった（笑）

2018/02/03(土) 12:43:15.80

>>860
上は>>862に追加して
if (sender is Button) switch ((sender as Button).Name)
{
case "button1":
case "button2":
}
isの判定とasのキャストはControlとかToolStripMenuItemでも状況に合わせて

2018/02/03(土) 12:45:57.31

今後も言語仕様はどんどん拡張されるよ
他の言語の有名なのどんどん取り入れてってカオスになっていく

個人的には範囲指定が欲しい
1..5か1:5みたいなの

**愛知人** · 2018/02/03(土) 12:50:20.68

Enumerable.Rangeじゃだめなん？

2018/02/03(土) 12:51:56.63

>>874
せめてnameof使って書けよ

2018/02/03(土) 12:56:57.86

foreach(var i in Enumerable.Range(1,5))

foreach(var i in 1..5)

どっちがいい？

2018/02/03(土) 12:58:29.57

圧倒的に上

2018/02/03(土) 13:02:13.44

>>860
ButtonのTagにやりたいこと突っ込んどけ

2018/02/03(土) 13:06:02.52

いい方法
buttun1.Click=>{Method1();};

2018/02/03(土) 14:17:08.45

どう書いたらいいか分からないのが糞言語の証左。

2018/02/03(土) 15:26:42.67

証左、難しい言葉をしってるね。

>>881
エラー 2 '=>' は無効です。
ってなるが、、、

2018/02/03(土) 15:30:37.45

>>874
こういう実装を参考にして欲しくないな
コントロールのプロパティを自由に修正できなくなるじゃん

2018/02/03(土) 15:33:25.80

var aa = ((Button)sender).Equals(button1) ? func1() :
((Button)sender).Equals(button2) ? func2() :
func2();
これどう？

2018/02/03(土) 15:34:12.79

>>884
ButtonのName変更ってどんな実装に必要なんだよ・・・

2018/02/03(土) 15:39:26.04

＞ButtonのName変更ってどんな実装に必要なんだよ・・・

最初はBUtton1だろ。出来上がって綺麗に整理するときにbtnSetParamとか分かりやすく書き換える。

2018/02/03(土) 15:40:26.14

>>885
それならEqualsじゃなくてsender==button1?でいいよ。キャストもいらない

2018/02/03(土) 15:40:45.84

Button.Nameをつかうくらいなら、Tagに書き込んだ方がまし。

2018/02/03(土) 15:41:39.80

>888
あ、そうなん？

2018/02/03(土) 15:43:31.84

実装後にスペルミスに気付いた、他の画面との兼ね合いで名称を統一することになった、仮の名前をちゃんとした名前に置き換えた、とか色々あるだろ
リフレクションでデータ取得する方針のせいでガチガチに名前が決められている画面を作らされたときの面倒さったらなかったわ

2018/02/03(土) 15:43:52.16

キャストしないとインテリセンスがでないから、キャストしたほうがいいよ。

2018/02/03(土) 16:57:11.29

こんなの考えたが、
public class CaseAction
{
　　public CaseAction(object @object, Action action)
　　{
　　　　Object = @object; Action = action;
　　}
　　public Action Action { get; private set; }
　　public object Object { get; private set; }
}

public static class Extensions
{
　　public static void Switch(this object o, params CaseAction[] caseActions)
　　{
　　　　caseActions.FirstOrDefault(x => x.Object == o)?.Action();
　　}
}

やっぱり使いづらそうだなw

ところで書いててちょっと思ったけど、==演算子の動作って隠蔽（new）と同じように変数の型に
依存するのか、それともオーバーライドと同じようにオブジェクトの型に依存ずるのか、どっちだっけ？

2018/02/03(土) 20:30:42.15

ふと思ったが、ふつうにそれぞれのボタンのイベントに実装したらダメなの

2018/02/03(土) 20:34:51.83

>>894
ほとんど共通でボタン毎にちょっとだけ処理が違うとかじゃね

2018/02/03(土) 21:11:27.96

グループボックスにボタンを貼り付けて、それをコピーして貼り付けると、イベントが同じところになるだろ。
だからエディット都合だな。
イベントで分けるのが一番いい方法かもしれない。ｗ

2018/02/03(土) 21:13:35.51

再表示するために tabPage1.Controls.Add(grp);で追加したgrpだけをクリアするにはどうしたらいいの？
tabPage1.Controls.Clear();　やるとボタンが消えてしまう。

2018/02/03(土) 21:27:16.93

Controlsプロパティの型はControl.ControlCollectionで
そのクラスのドキュメントを見ましょう。
Remove(Control)というのがあったのでそれで削除できそうです。

2018/02/03(土) 21:35:57.81

Removeを使うのはわかるんだが、要するになにがしたいのかというと
Linqを使ってControlsにAddしたGroupBoxだけを選択して、クリアしたいということ。
その書き方がわからない。ということ。

2018/02/03(土) 21:39:38.95

再表示とかクリアとか俺様な用語法されても何言ってるのか分からないよw

2018/02/03(土) 21:46:23.78

tabPage1に貼り付けたコントロール群をLinqを使って選択的に消去したいということ。これでもわからん？

2018/02/03(土) 21:49:30.86

>>901
言葉が不自由なんですね
わかります

2018/02/03(土) 21:52:32.07

煽って悪かったけど
その書き方じゃ誰にもつたわらないよ

意味がわからないけどgrpを消したいならgrpをどこかに記録しておけばいい
終わり

はい次の方～

2018/02/03(土) 21:54:28.34

解決した。ｗ

2018/02/03(土) 21:55:50.09

>>904
よかったね
言葉遣いや表現力だけ見ると小学生かなって思うけど

2018/02/03(土) 21:57:07.86

>903
Keyになる言葉の意味がわからないと解らないと思うよ。それが解る人にはわかる。

2018/02/03(土) 21:59:39.27

なんでこいつこんなに偉そうなの？

2018/02/03(土) 22:00:08.72

小学生だから

2018/02/03(土) 22:08:58.48

var toRemove = tabPage1.Controls.Cast<Control>().FirstOrDefault( .... );
if (toRemove != null)
　　tabPage1.Controls.Remove(toRemove);

これを1行にしたいってことだったのかな？
拡張メソッド使ってControl側に自分を包含するコンテナのControlCollectionを
引数に撮るメソッドを定義しないとできない気がする

何でもいいけど、冗長でもいいから分かるように書いて欲しいよね

2018/02/03(土) 22:13:41.72

そんな書き方しかでけん？

2018/02/03(土) 22:16:15.65

>>897
これでスタートして答えをもらおうとして失敗

次に　>>899　要するにと言ってるけど何も要してないどころか条件を追加してる

そして　これ　>>901　への流れ

トンチンカン
最初のレスでなぜ伝わると思ったのかが分からない

2018/02/03(土) 22:23:57.61

無理矢理だなあ
Action<Control> removeMe = x => x?.Parent?.Controls.Remove(x);
removeMe(
tabPage1.Controls.Cast<Control>().FirstOrDefault( .... )
);

**愛恥人** · 2018/02/03(土) 22:34:23.85

最初のレスだけで理解できる俺は天才なのか

2018/02/03(土) 22:38:53.52

むしろエスパー

2018/02/03(土) 22:43:30.44

そして明後日の方向のレスに食いついてる

2018/02/03(土) 22:54:26.09

private void button3_Click(object sender, EventArgs e)
{
var grps = tabPage1.Controls.Cast<Control>().Where(c => c is GroupBox );
foreach (var grpB in grps)
{
tabPage1.Controls.Remove(grpB);
}
}
これで一回で消えないのは何故だろう？　grpsには全部のグループボックスが
入っている。しかしforeachでところどころ飛ばしてしまう。

2018/02/03(土) 23:19:59.77

あっ、謎がとけた。けっこう落とし穴だな。

2018/02/03(土) 23:32:01.66

ケツから消していくとうまくいくんだろきっと。
まえからでもこうやるとうまく行く。
　　　　private　void　button3_Click(object　sender,　EventArgs　e)
　　　　　　　　　{
　　　　　　　　　　　　　var　grps　=　tabPage1.Controls.Cast<Control>().Where(c　=>　c　is　GroupBox　);
　　　　　　　　　　　　　List<GroupBox>　gg　=　new　List<GroupBox>();
　　　　　　　　　　　　　foreach　(var　grp　in　grps)
　　　　　　　　　　　　　{
　　　　　　　　　　　　　　　　　gg.Add(grp);
　　　　　　　　　　　　　}
　　　　　　　　　　　　　foreach　(var　grpB　in　gg)
　　　　　　　　　　　　　{
　　　　　　　　　　　　　　　　　tabPage1.Controls.Remove(grpB);
　　　　　　　　　　　　　}
　　　　　　　　　　}

2018/02/03(土) 23:33:46.33

さて、原理は分ったが、これを1行で書くにはどうするのか？

**愛恥人** · 2018/02/03(土) 23:40:30.32

逆にグループボックス以外を抽出してからパネルの中身全消去からの追加とかはどうかな(>_<)

2018/02/03(土) 23:40:55.06

>>919
改行全部消せばいいじゃん

2018/02/03(土) 23:43:21.17

>>918
>>916のLinqにReverse()追加すれば

2018/02/03(土) 23:43:41.06

>>919
なんで1行にこだわってるの？
どれだけ読みにくいソースにできるか研究でもしてんの？

2018/02/03(土) 23:49:22.30

＞９２０　それ、俺もちょっと考えた。
＞９２１　それは最終手段にとっておいたのに。

Reverse（）ってあるんか？　それだ！！

2018/02/04(日) 00:42:43.56

もう気が済んだかもしれんけど、無理に何でもインラインで書こうとして
可読性落とすなんて愚の骨頂で、普通にメソッドに外だし（別に拡張メソッドじゃなくてもいいけど）
した方がいいと思うけどね

public static class Extensions
{
　　public static void RemoveChildren(this Control control, Func<Control, bool> predicate)
　　{
　　　　var controls = control.Controls;
　　　　controls.Cast<Control>().Where(predicate).ToList().ForEach(x => controls.Remove(x));
　　}
}

2018/02/04(日) 05:54:59.74

https://qiita.com/taigaaa2/items/7f05ad950a57033892ae
この方法で値を保存しているのですが
cドライブに保存されている様でプログラムを他のPCに移すなどすると値が保持されていません
どうすれば他のPCでも記憶した値を使える様になるでしょうか

2018/02/04(日) 06:23:40.03

ファイルをコピーする

2018/02/04(日) 06:38:53.27

真面目に答えるならサーバーやクラウドストレージのSettingsを読み書きするSettingsProviderを自作することだが
初心者にはひと月ふた月かかるだろうしはっきり言って時間の無駄

2018/02/04(日) 08:14:14.99

真面目な答の方が愚策という変な人、というかおそらくバカ

2018/02/04(日) 08:55:21.88

>>926
ユーザー共通になっちゃうけど、スコープをユーザーからアプリケーションに変更すればexeと同じディレクトリに保存するよ

**愛恥人** · 2018/02/04(日) 10:35:42.99

>>762でユーザーにしたら動いた原因はなに？

2018/02/04(日) 12:47:08.55

例えば.net 4.7で作られたプログラムを4.5しか入ってないPCで動かすとどうなりますか？

2018/02/04(日) 13:40:36.37

うごかない

2018/02/04(日) 13:41:03.95

てか4.5とかサポート切れだから論外だし

2018/02/04(日) 13:48:00.41

＞ユーザー共通になっちゃうけど、スコープをユーザーからアプリケーションに変更すればexeと同じディレクトリに保存するよ

アプリケーションじゃ動かんかったな。動くこともあるんだろうな。意味がわらかんし、マニュアル読んでも書いてないし
それにビルドで設定値が消えるから、、、最初から打ち直し。ｗ
こんな意味の解らん機能はつかわない方がいい。JSON.NET使った方が遥かに楽。というのが俺の感想だ。

2018/02/04(日) 14:04:22.94

>>932
開発環境が4.7でも対象フレームワークを4.5に設定してビルドすれば、4.5が入ってるPCでも動く。
ただし、4.7でないと動かない機能は使えなくなる。

2018/02/04(日) 14:13:13.77

public static void RemoveChildren(this Control control, Func<Control, bool> predicate)
　　{
　　　　var controls = control.Controls;
　　　　controls.Cast<Control>().Where(predicate).ToList().ForEach(x => controls.Remove(x));
　　}
これは非常にスマートだという感じだが意味わからんなー。
predicate: 対象となるコントロールの場合はTrueを返す関数？
ToList() :predicateで判定したコントロールのListを作る。
ForEach：ToListからひとつづつコントロールをとりだしてｘにセット、Removeする。
という意味だろうか？

Funcって初めて見た。いま調べてみるとよく使う標準デリゲートらしい。であれば
　controls.Cast<Control>().Where(ラムダ式).ToList().ForEach(x => controls.Remove(x));
で一行でできる筈だな。
つまり
　controls.Cast<Control>().Where(x=>x is GroupBox).ToList().ForEach(x => controls.Remove(x));
で実験してみると
tabPage1.Controls.Cast<Control>().Where(c => c is GroupBox).ToList().ForEach(x => tabPage1.Controls.Remove(x));
わーいうまく行ったぞ。

2018/02/04(日) 14:22:22.77

日記はちらしのうらにでｍ…

2018/02/04(日) 14:22:36.96

>>936
>.net4.7で作られた
をどう解釈するかだな

2018/02/04(日) 14:24:43.87

Castを無くすために
tabPage1.Controls.OfType<GroupBox>().ToList().ForEach(x => tabPage1.Controls.Remove(x));
これでも動くのに、

tabPage1.Controls.OfType<GroupBox>().ForEach(x => tabPage1.Controls.Remove(x));　
これは動かないのはなぜ？

2018/02/04(日) 14:26:44.56

実装してないものが使えるわけないだろ
魔法じゃないから

2018/02/04(日) 14:30:00.97

そもそも列挙してる最中のものを削ると列挙に失敗する可能性が高い

最初から何番目の要素を取り出す
次はそれプラス1の要素としてるのに
1番目から消すと何番目というのがずれるだろ？

一番目を取り出すそれを消す
○×▼
↓
×▼

二番目を取り出すそれを消す
×▼
↓
×
三番目は取り出せない

2018/02/04(日) 14:31:09.88

ずっと一番目を消すと全部消える

2018/02/04(日) 14:34:48.14

ToLisｔ()で控えを作る

できたList 　○×▼ をみて対象を消していく
こちらは消し終わってList内容はそのまま

2018/02/04(日) 14:35:11.95

>>941
いやまあそうなんだが、疑問というのは
var grps = tabPage1.Controls.OfType<GroupBox>();

varじゃわからないのだが、この型は何なの？　Listじゃないよね。

2018/02/04(日) 14:36:12.24

>>945
IDE使えばでるだろ
カーソル当てろ

2018/02/04(日) 14:36:23.56

>>945
メモ帳で開発してんの？

2018/02/04(日) 14:37:00.62

可能性が高いと書いたのは列挙の方法がちがうかもしれないから　

2018/02/04(日) 14:37:31.37

>>945
Enumerable.OfType<TResult> メソッド
https://msdn.microsoft.com/ja-jp/library/bb360913(v=vs.110).aspx

2018/02/04(日) 14:38:10.32

ここは個別指導スレじゃないだろ
いい加減にわきまえろ

質問じゃなくて誰かに教えてもらいたいなら金を払って個人指導受けろ