ふらっと 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/02(金) 22:39:44.98

間違い

/*は*/を読み捨てしたらダメだなｗ

2018/02/02(金) 22:40:01.25

パーサジェネレータ使えよ

2018/02/02(金) 22:48:59.30

>>826
それがわからない
文字列リテラルなのかコメントなのか判断できない
ダブルコーテーションから判定するとそれがコメントだったとき駄目
コメントから判定するとそれが文字列リテラルだったとき駄目
っていうパラドックス

2018/02/02(金) 22:52:11.72

>>829
それは変なやり方してるからだろ
おおかた正規表現などで判定してるんだろう

コードを頭から読んでできないのは
文法がおかしいかパースのやり方がおかしいからだ

2018/02/02(金) 22:56:57.54

急がないならどれかの言語のコードでもじっくり読んで勉強したらいい
大体の言語はオープンソースなので好きなのをやればいい
ただしc++の知識ぐらいはいる

急ぐならテストパターンを十分に用意して手あたり次第やればいい

2018/02/02(金) 23:08:58.70

>>830
やけっぱちで
解析のコメント削除時に「//」か「/*」見つけたら同じ行の前側みて
文字列リテラル中かどうかみる処理入れて乗り切ってみた

2018/02/02(金) 23:10:39.49

>同じ行の前側みて
うん、かなり無茶苦茶だな
先頭から順に読んでいって、「今は文字列リテラル中」とか「今はコメント中」とかの状態を保持しておけよ

2018/02/02(金) 23:15:27.28

独自言語の解析してるんだろ？

他の部分が出来てるなら
普通に頭から読んでって　"が出たらリテラルフラグオンにして
//が出たら行末まで読み捨て
/*だったら次の*/がでるまで捨てるだけ

何も難しくない

2018/02/02(金) 23:17:24.91

文字が抜けた

"が出たらリテラルフラグオンにして仕様通りリテラルを最後まで読み切るだけ

2018/02/02(金) 23:18:02.29

独自言語が逐次実行される前提で話してるな

2018/02/02(金) 23:18:46.57

もしかして、1文字ずつじゃなく1行ずつ読み込んでるんじゃないだろうな

2018/02/02(金) 23:22:20.25

>>836
逐次実行じゃなくても、頭から順に読むのは同じだよ
読んだ物を構文木に積んでいって、評価・実行は最後にやれば良いだけ

2018/02/02(金) 23:24:38.81

>>829
難しく考えすぎでしょう。
>>826が言ってるのはたぶん単純なステートマシン。

enum State
{
　　トークン探索中,
　　文字列検出中,
　　インラインコメント検出中、
　　....
}

stateを表すenumを定義して、stateの値に応じて動作を変えるだけ。
文字列リテラルの始め「"」やコメントの始まりを検出するのはstate = State.トークン探索中の時だけ。
素朴すぎるやり方だと思うが、少なくともこれでそんなパラドックスには陥らない

2018/02/02(金) 23:24:41.10

>>832
本当にめちゃくちゃだな
前方見ても意味ない
確定するためには本当に最初までさかのぼることになる

2018/02/02(金) 23:24:55.99

>>834-835で答えが出てるのでこれ以上はスレの無駄遣い

2018/02/02(金) 23:27:13.14

>>838
横ぐるまを押したか

2018/02/03(土) 00:00:03.20

>>840
大丈夫だよ
文字列リテラルって2行続かないじゃん確か

2018/02/03(土) 00:06:47.29

>>843
そんなのはルール次第

2018/02/03(土) 00:15:01.77

例えばC#なら@""で改行入れられるし

**愛知人!id:ignore** (ﾜｯﾁｮｲ 9775-jUAt) · 2018/02/03(土) 00:17:09.98

そのときだけ行またげばいいだけだし

2018/02/03(土) 00:34:25.55

>>845
ぎゃああああ
出来やがった

既存コードが解析の初っ端でコメント削除からやってるから
弄りたくなかったけど
いよいよ駄目だな

2018/02/03(土) 00:38:09.24

知らんかったんかいｗ

コメントは削除するんでなく、「読み捨てる」んだ

2018/02/03(土) 00:41:40.01

>>848
いや、似非c#だからそんなことできんだろと

2018/02/03(土) 00:53:02.56

ヒアドキュメントを除いたとしてもしソースの先頭の/*をどうやってみつける？
状態を持たないならソースの頭までさかのぼるしかない

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でも記憶した値を使える様になるでしょうか