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

**デフォルトの名無しさん** · 2017/11/28(火) 21:30:48.55

「どんなにくだらない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/

■コードを貼る場合は↓を使いましょう。
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

**デフォルトの名無しさん** · 2017/11/28(火) 21:31:16.08

(*ﾟ∀ﾟ)ｖ2get!

**デフォルトの名無しさん** · 2017/11/28(火) 21:33:19.34

いちおつ

**デフォルトの名無しさん** · 2017/11/29(水) 12:38:24.28

調子にのって式形式の略記しまくったら=>まみれになったでござる

**デフォルトの名無しさん** · 2017/11/29(水) 12:56:15.86

using dispose は便利な機能だが実装の闇が深くて台無し。魔剣ダイナシ。

**デフォルトの名無しさん** · 2017/11/29(水) 12:57:31.62

とVB6おじさんが申しております

**デフォルトの名無しさん** · 2017/11/29(水) 12:58:28.27

>>5
で

**デフォルトの名無しさん** · 2017/11/29(水) 13:06:25.32

SQL鯖へのConnectionもUsingの中に入れちゃっていいのかな？

昔は最初にConnection開いて、ずっと開きっぱなしの中で処理して、最後にCloseすることが多かったけど
不安定な接続(スリープで回線断)やミラーリングサーバーを考慮すると、いちいちOpen/Closeしたほうがいい。

だけど、usingの中に入れることで、いちいちDisposeされたら
接続キャッシュが機能せずに体感速度が落ちる、って風になる？

**デフォルトの名無しさん** · 2017/11/29(水) 13:44:13.05

接続管理はコネクションプーリングに任せてプログラム上は最短でOpen-Close

**デフォルトの名無しさん** · 2017/11/29(水) 13:46:22.65

>>8
SQL鯖でusing多用はいかん
try/finally使ってopen/closeしとけ
速度はopenしっぱなしとほぼ変わらん

**デフォルトの名無しさん** · 2017/11/29(水) 13:57:37.88

>>8
それ別の人がアクセスしても大丈夫なの？

**デフォルトの名無しさん** · 2017/11/29(水) 14:00:58.53

>>11
キミ以外なら

**デフォルトの名無しさん** · 2017/11/29(水) 15:09:14.71

>>10
それdisposeと一緒やん。
ってか、disposeじゃん。

なにいってんの？

**デフォルトの名無しさん** · 2017/11/29(水) 15:12:27.07

>>13
disposeじゃねぇ。usingだわ。まちげーた

**デフォルトの名無しさん** · 2017/11/29(水) 15:37:12.11

>>13
disposeしてないよ

何言ってんの?

**デフォルトの名無しさん** · 2017/11/29(水) 15:59:59.48

>>15
やってることが、using使ったdisposeと等価なんだけど。

usingはtry/finalyの糖衣構文
(sql鯖のコネクションについては)closeとdisposeは等価

何か違いがあんの？

**デフォルトの名無しさん** · 2017/11/29(水) 16:18:48.57

var SqlConn = new System.Data.SqlClient.SqlConnection(xxxx);

という感じの宣言は try/finaly の外でやっておいて
open/close は try/finaly の中でやる

って意味で書いたと思ってたけど、そうだよね？

**デフォルトの名無しさん** · 2017/11/29(水) 16:32:53.30

>>16
msdnのSqlConnectionメソッドの
closeとdisposeの説明をちゃんと嫁

**デフォルトの名無しさん** · 2017/11/29(水) 16:35:00.95

インターフェースおじさんの話は聞かなくていい

**デフォルトの名無しさん** · 2017/11/29(水) 16:42:07.66

>>19
おじさん?
アフォな専門学校の生徒かと思ったわw

放置プレイしま～す

**デフォルトの名無しさん** · 2017/11/29(水) 16:58:19.20

SqlConnectionメソッドってのはちょっと分からなかったけど
SqlConnection.Closeにはこう書かれてるな
> SqlConnection は、適用範囲外では閉じられません。
> そのため、Close または Dispose を呼び出して、明示的に接続を閉じる必要があります。
> Close と Dispose は、機能的に同じです。

DisposeはComponent.Disposeに飛ばされたから
そもそもSqlConnection.Disposeのページが存在しない

**デフォルトの名無しさん** · 2017/11/29(水) 17:08:15.43

>>18
msdnに機能的に等価って書いてあるんだけど、なに読めば良いの？

>>17
try/finalyの外で宣言した変数を使い回すってこと？

usingを抜けなければ、disposeは走らないから、一度開いた同一の接続を使い回すのであればusingの中で、接続(open)を複数回するのであれば、個別の変数を切り直した方が良いと思うよ？
前提が違ってたらごめんね。

**デフォルトの名無しさん** · 2017/11/29(水) 17:14:21.18

アプリは生きててDBだけリセットかけたらもう一回接続しないと駄目？
なんかアクセス毎に開いて閉じて入れておかないと面倒な感じじゃね？

**デフォルトの名無しさん** · 2017/11/29(水) 17:26:22.28

最近はMicrosoftDocsに飛ばされるね、MSDNもあるけど英語がメイン
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/keywords/using-statement

ついこないだIDisposable実装したけど、これまたクイックアクションで親切すぎるコメントつきテンプレが流し込まれる

**デフォルトの名無しさん** · 2017/11/29(水) 17:58:34.09

iPhoneのキーボードのこれhttps://i.imgur.com/PStMLdz.jpgはプログラム書くときのダブルクォーテーションとは違うんだな。ideoneでなぜかエラーが出たので気づいた。
https://i.imgur.com/wyCNyNM.jpgこれだと使える。

英字キーボードのやつ “””””””””””
ABCのほう """""""""""

**デフォルトの名無しさん** · 2017/11/29(水) 18:05:53.37

長押ししたらこんなん出てきたわ
https://i.imgur.com/TpvDvkf.jpg

**デフォルトの名無しさん** · 2017/11/29(水) 18:05:59.40

>>20
間違いは認めてから逃げようね

**デフォルトの名無しさん** · 2017/11/29(水) 18:10:42.78

>>25
C#と関係無いけど、この業界にはよくある事
U+301CとU+FF5Eの違いとかな

**デフォルトの名無しさん** · 2017/11/29(水) 18:27:36.17

ソフトウェアキーボードでコード書くとか苦行過ぎるw

**デフォルトの名無しさん** · 2017/11/29(水) 18:28:48.92

ideoneでちょっとした動作を試したりするときだけだな

**デフォルトの名無しさん** · 2017/11/29(水) 19:16:09.42

明日屁湧いてるなw

**デフォルトの名無しさん** · 2017/11/29(水) 20:50:56.63

C#ってmalloc・calloc・realloc的なかんすえはありますか？
全部自動？

**デフォルトの名無しさん** · 2017/11/29(水) 20:53:11.64

>>32
基本的には出来ないし自動だけどunsafeコンテクスト内なら可能
http://ufcpp.net/study/csharp/sp_unsafe.html

**デフォルトの名無しさん** · 2017/11/29(水) 21:01:58.77

>>33
バッファの確保できるんだ
ありがとう

**デフォルトの名無しさん** · 2017/11/29(水) 21:17:10.66

いやmalloc的なのは普通の配列のnewかMarshal.AllocHGlobalだろ

**デフォルトの名無しさん** · 2017/11/29(水) 21:41:03.14

ただいま勉強中なのですが、↓のようなコードの設計、ネーミングその他は適切でしょうか？
身近に現役プログラマーの方がいないためいろいろと聞けないのが現状です。
なんでも指摘してくださると助かります。

public static class DirectoryUtils{
public static string ErrorMessage { get; private set; } = null;
public static bool DeleteFiles(string directoryPath) {
if (!Directory.Exists(directoryPath)){
ErrorMessage = "ディレクトリが存在しません";
return false;
}
try {
foreach (var file in Directory.GetFiles(directoryPath)){
File.Delete(file);
}
}catch (Exception exc){
ErrorMessage = exc.Message;
return false;
}
return true;
}
}
使う側
var success = DirectoryUtils.DeleteFiles(directoryPath);
if (!success){
MessageBox.Show(DirectoryUtils.ErrorMessage);
return;
}

//総変換数を求める
var totalNumber = GetNumberOfTotalConverting()
//Converting,Conversion,All,Total,Number,Count等々、悩んでいます

**デフォルトの名無しさん** · 2017/11/29(水) 21:57:07.61

自分だったら
public class DeleteFilesResult
{
public string ErrorMessage { get; set; }
public bool Success { get; set; }
}
みたいなクラス作ってそれを結果として返すかな

**デフォルトの名無しさん** · 2017/11/29(水) 21:59:38.11

マジでぇ？？

**デフォルトの名無しさん** · 2017/11/29(水) 22:03:52.52

>>36
エラーならエラー文字列返して、正常なら空文字かnull返すのが素直じゃないかな？

**デフォルトの名無しさん** · 2017/11/29(水) 22:10:55.88

successならsucceededとかじゃないの
もしくは構造体なりで
if (result == RESULT.OK)
とか

**デフォルトの名無しさん** · 2017/11/29(水) 22:14:46.71

さっそくでたな

ｔｒｙ　catchはDeleteFilesメソッドの中じゃなくて
外で使うべき

**デフォルトの名無しさん** · 2017/11/29(水) 22:31:52.96

>>36
静的メンバーにエラー情報を持たせるのは時代錯誤で論外。
エラー情報は普通に例外を投げようよ。

単に失敗成功が分かってエラーメッセージが取得できればよいだけなら、
エラーメッセージはoutな引数で戻せばいい

あと、一つのファイルの削除に失敗した時点で処理中断する仕様でいいの？
失敗しても継続する方が自然なような気もするけど...

DeleteFilesより明示的にDeleteAllFiilesの方がいいんじゃないか？
GetNumberOfTotalConvertingは長すぎ
それを含むクラスに適切な名前がついてればそんな冗長な名前にする必要ないでしょ

**デフォルトの名無しさん** · 2017/11/29(水) 22:32:46.18

>>41
以前暴れていた
CsvHelper例外で落ちるな糞野郎くんだろう

**デフォルトの名無しさん** · 2017/11/29(水) 22:33:52.93

DeleteFiles内部でtry catch受けていいのは
そのファイルをスキップして残りを消す場合など

**デフォルトの名無しさん** · 2017/11/29(水) 22:34:19.89

>>43
人違いだわｗ

**デフォルトの名無しさん** · 2017/11/29(水) 22:38:45.34

例外が飛んでそれをもとに制御が変わるなら
適切なtry catchの場所を考えるべき

**デフォルトの名無しさん** · 2017/11/29(水) 22:42:39.35

あと、例外についてはとにかく何でもキャッチしたがる人と、
それに対するアンチテーゼで「例外は絶対キャッチするな」と主張する人がいるけど、
どっちも何か盛大に勘違いしてる極論だから眉唾で聞いた方がいいと思う

**デフォルトの名無しさん** · 2017/11/29(水) 22:45:33.13

そうだね

例外利用するならFile.Deleteのページを見て
どんな例外をスローするか調べたほうがいい

権限無くて削除できないとかあるから

**デフォルトの名無しさん** · 2017/11/29(水) 22:48:25.13

・ω・Exception

**デフォルトの名無しさん** · 2017/11/29(水) 22:54:19.77

頻繁な例外を想定するならロギングで出力した方がいいよ
エラーを毎回表示なんて非現実的だし無視してOK連打したら意味ないでしょ
それよりタイムスタンプつけてテキストファイルに保存した方が後から照会できる

問い合わせの多くは、何もしてないのに突然おかしくなった、というやつばかり
そういう時はログフォルダをzipで圧縮して添付してもらおうというわけ

**デフォルトの名無しさん** · 2017/11/29(水) 22:57:51.57

状況が許すならDirectory.Delete　(string path　,　bool recursive)を使いたい

**デフォルトの名無しさん** · 2017/11/29(水) 23:02:36.65

>>45
人違いなのに丁寧にレスありがとww

**デフォルトの名無しさん** · 2017/11/29(水) 23:02:52.52

>>36
同じく静的メンバにエラー状態持たせるのは論外だけど
どうしてもやりたいなら成功時にそれをクリアしないとダメ

bool返すよりoptionalみたいなのを返すのがモダン

**デフォルトの名無しさん** · 2017/11/29(水) 23:03:14.96

そのフォルダが相手のミスでリードオンリーにされた場合は例外情報が全部死ぬけどな

**デフォルトの名無しさん** · 2017/11/29(水) 23:12:59.73

>>36です
こんなにレスがつくとは思いませんでした。みなさんありがとうございます。
①結果を格納するクラスを作成し、それを返し、参照する
②エラー文字列を返す
③例外を投げるかoutな引数で戻す
どれもそういった発想が無かったので勉強になります。
ちなみにoutを使う場合はtupleを使って2つ返すのもありですか？

>静的メンバーにエラー情報を持たせるのは時代錯誤で論外。
こういう風に指摘されるともっともっと勉強しようと励みになります。ありがとうございます。

＞あと、一つのファイルの削除に失敗した時点で処理中断する仕様でいいの？
＞失敗しても継続する方が自然なような気もするけど...
たしかに処理の中断をするまでもないですね

例外処理についてはメソッドを呼ぶたびにtry catchを書くのが冗長という理由だけで中にいれてました。
制御が変わるかなど考慮すべきでした

optionalや＞もしくは構造体なりでif (result == RESULT.OK)　
などまだまだ理解できない部分が多いので調べてきます

**デフォルトの名無しさん** · 2017/11/29(水) 23:21:16.55

>>50
予期せぬ例外は
https://qiita.com/exliko/items/42715a0c9fd7519eb6d9
このような感じでログファイルに出力するようにしていたのですが、
たしかにエラー毎回表示も微妙なので、もっとログをとっていくようにしていきます

**デフォルトの名無しさん** · 2017/11/29(水) 23:30:30.87

>>47 >>48
例外が起きた際、個々の例外によって制御を変える場合は個別に書いて、
MessageやStackTraceをロギングして終了するだけの場合はExceptionですべて捕捉する感じですか？
当初はIOException,UnauthorizedAccessExceptionなど書いていたのですが、
結局Messageしか使っていなかったので全ての例外を捕捉するようにしていました。

**デフォルトの名無しさん** · 2017/11/29(水) 23:36:03.91

Directory.GetFilesはファイルを全部探してから配列でそれを返すので色々と無駄
列挙するだけならDirectory.EnumerateFiles

**デフォルトの名無しさん** · 2017/11/29(水) 23:48:45.63

>>58
直しました。ありがとうございます。

**デフォルトの名無しさん** · 2017/11/30(木) 00:47:19.32

教えてもらったことにお礼を言うのはいいとして変える必要があったのだろうか

**デフォルトの名無しさん** · 2017/11/30(木) 00:53:26.33

>>55
Tupleでいいけどちゃんと名前付けてね
もしvs使ってるならコード分析かけてみれば？
静的メンバーの公開とかは警告で教えてくれたはず

**デフォルトの名無しさん** · 2017/11/30(木) 02:34:13.97

>>57
基本的に例外を避けたい理由がなければ例外を使えば良い

ロギングしてアプリを終了するならExceptionでcatch
復帰可能のは個別に対処（ディレクトリが見つからない場合は再入力を求めるなど）

**デフォルトの名無しさん** · 2017/11/30(木) 08:10:34.13

>>55
outはもはや要らない子
名前付Tuple使っとけ

**デフォルトの名無しさん** · 2017/11/30(木) 09:21:54.56

Tupleってエラーの場合何を返すの？

**デフォルトの名無しさん** · 2017/11/30(木) 09:22:06.99

トプル

**デフォルトの名無しさん** · 2017/11/30(木) 09:40:45.12

bool ok,int unkor,int unkog,int unkob

var unkc = GetUnkoColor();
if(unkc.ok){
成功
}else{
失敗
}

みたいな？

**デフォルトの名無しさん** · 2017/11/30(木) 09:41:02.83

Exception

**デフォルトの名無しさん** · 2017/11/30(木) 09:47:18.95

また、Exceptionが不遇な扱いを受けている
哀れすぎて何も言えねえ

**デフォルトの名無しさん** · 2017/11/30(木) 09:48:16.44

また、大して考えず産廃を作った予感
言語開発者暇なのか？
Getしようとしたけど要素がないときもあるじゃん
全部例外で飛ばされるとかマジやめて

**デフォルトの名無しさん** · 2017/11/30(木) 09:59:17.09

釈尊はnullは「無」であると説かれた
ﾅﾑｰ

**デフォルトの名無しさん** · 2017/11/30(木) 11:21:23.77

例外が邪魔とかoutイラナイわめいてるのは大抵Dictonaryでしょ
連想配列は銀の弾丸ではない、期待しすぎ

**デフォルトの名無しさん** · 2017/11/30(木) 11:29:17.22

>>63
>outはもはや要らない子
これは嘘

**デフォルトの名無しさん** · 2017/11/30(木) 11:31:00.67

>>66
タプル知らないの？

**デフォルトの名無しさん** · 2017/11/30(木) 11:53:07.37

>>73
nullっぽく使えるのある？

**デフォルトの名無しさん** · 2017/11/30(木) 12:25:17.12

今はout引数で変数宣言できるようになったからかなりスマートになった
むしろタプルの必要性が疑問なくらい
TryGetXならnull問題もないし

**デフォルトの名無しさん** · 2017/11/30(木) 12:26:38.17

何が返されるか分かりにくいじゃん
タプルで返して分割代入すればいい

**デフォルトの名無しさん** · 2017/11/30(木) 12:29:50.89

タプルだったらわかりやすくなる例が思いつかんが

**デフォルトの名無しさん** · 2017/11/30(木) 12:34:03.03

そもそもタプルいらん

**デフォルトの名無しさん** · 2017/11/30(木) 12:47:20.27

>>78
訳：新しいことを覚える気力がない

**デフォルトの名無しさん** · 2017/11/30(木) 12:51:45.93

outでもクラスでも構造体でもタプルでも好きなもの使えばいいよ
>>78
いらんというお前のレスがいらん
チラシに書いとけ

**デフォルトの名無しさん** · 2017/11/30(木) 12:52:07.78

分散どうやって出すの？

**デフォルトの名無しさん** · 2017/11/30(木) 12:54:30.58

>>81
http://whoopsidaisies.hatenablog.com/entry/2013/11/30/175613

**デフォルトの名無しさん** · 2017/11/30(木) 12:56:35.10

MS的にはXOrDefaultじゃないの

**デフォルトの名無しさん** · 2017/11/30(木) 13:04:29.34

>>79
.NETライブラリでも使われてないしな

**デフォルトの名無しさん** · 2017/11/30(木) 13:15:30.72

outって呼び出す時に一々変数を定義しなきゃいけないのが面倒
Tupleは戻り値が分かりづらいって欠点がある
どちらも一長一短だから好きな方を使えばいい

.NETライブラリが使ってないから使わないって思考停止すぎんだろ

**デフォルトの名無しさん** · 2017/11/30(木) 13:19:26.22

公認じゃない感あるし.NETの関数とマッチしないからな

**デフォルトの名無しさん** · 2017/11/30(木) 13:21:37.72

クラス作ろうぜ

**デフォルトの名無しさん** · 2017/11/30(木) 13:26:10.35

>>81
自分で書いてみた。
https://ideone.com/0fyIQN

**デフォルトの名無しさん** · 2017/11/30(木) 13:26:15.27

それが一番修正とか楽だしな
クラスは正義

**デフォルトの名無しさん** · 2017/11/30(木) 13:26:30.06

そうそう、クラスでやれってことな。

**デフォルトの名無しさん** · 2017/11/30(木) 13:35:54.67

Average関数もあったんだな
https://ideone.com/0fyIQN

**デフォルトの名無しさん** · 2017/11/30(木) 13:39:45.69

>>88と>>91は連動してて同じになっとるわ。

**デフォルトの名無しさん** · 2017/11/30(木) 13:42:28.63

c#が気に入らなきゃMatzみたいにオレオレ言語作れよ

**デフォルトの名無しさん** · 2017/11/30(木) 14:01:20.98

時間帯によってgpuの色味をコントロールするソフトを作りたいです
具体的にはf.luxみたいなソフトを作りたいのですが、かんたんですか？C#でいいんです？
f.luxはガンマ？が高目に設定されるようで、余計眩しく感じたりします

**デフォルトの名無しさん** · 2017/11/30(木) 14:06:23.80

Windows10のナイトモードでいいよ。

**デフォルトの名無しさん** · 2017/11/30(木) 14:08:43.36

時間帯によって徐々に変わるようにしたいのですが

**デフォルトの名無しさん** · 2017/11/30(木) 14:18:40.16

グラフィックドライバの設定を変えるプログラム書けばいいんじゃね？知らんけど。
https://software.intel.com/en-us/forums/developing-games-and-graphics-on-intel/topic/607388

**デフォルトの名無しさん** · 2017/11/30(木) 18:39:49.21

>>94
たまに似たような人がいるけど仕様段階で先が見えないくらいならあきらめた方がいい
このスレは設計のアドバイスをする役割ではないから

**デフォルトの名無しさん** · 2017/11/30(木) 18:42:54.90

実装のアドバイスもできないけどね