ふらっと 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/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

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

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

画面関係ってOSのAPIレベルで標準化されてるのかな。
Win10だとそれっぽい機能がついてるから存在するのかもしれんが、それ以前のOSには存在しない気が

そもそも、昔よくあったOOPの責務の仕分け問題じゃないけど、画面の調整機能って
本来はモニター側が担当すべき機能でPC側でやるのはいかにも筋悪だよねw

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

>>99
お前はな

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

スキルの無いやつほど難しい事をやりたがる法則

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

まあ、でも、一人も試みるやついないってのも寂しい話だ

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

>>100
アドベガンマが昔からあるのでアプリケーションで変えることはできる
スレ違いだし興味も無いのでこれ以上調べる気にもならない
>>103
やりたい人はやればいい
要件整理からはじめてガチ初心者にずっと教えるのが覚悟できれば

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

>>85
c#7の新機能に出力変数宣言(out-var)がある。
これはこれで変数宣言を見落としそうだけど。

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

>>102
賢者は困難を知り愚者は不可能を知らず

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

>>106
愚者は厚顔無恥あり正しい事を指摘するとキレる

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

クッサ

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

値渡しと参照渡しの違いって、エクセルの貼り付けの「値のみ貼り付け」と「すべて貼り付け」みたいなもんですか？

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

「図として貼り付け」と「リンクした図として貼り付け」の方が近いかな

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

>>110
なるほど。そこまで外れてなかったようで安心しました。

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

>>109
変数：割り当てられたメモリー上の特定のアドレス範囲
値：変数の中身（変数に割り当てられたメモリーの範囲のバイト列）
参照：変数に割り当てられたメモリーの範囲の開始アドレス

まあ、値がwebページのコンテンツなら参照はそのURIみたいなもの

**デフォルトの名無しさん** · 2017/12/01(金) 00:04:00.62

>>112
値をコピーする際にアドレスをよりどころにしているのが参照渡しということですね。
なんとなくイメージはわかるのですがなかなか。

**デフォルトの名無しさん** · 2017/12/01(金) 00:20:18.07

>>113
何も難しくないよw

2chで言えば参照渡しは人（メソッド）に対してレスのアンカーを示すだけで
中身は教えない。

値渡しはレスの内容のコピペ

参照渡し：
>>113

値渡し：
値をコピーする際にアドレスをよりどころにしているのが参照渡しということですね。
なんとなくイメージはわかるのですがなかなか。

**デフォルトの名無しさん** · 2017/12/01(金) 09:09:10.01

C#の次期バージョンではin参照渡しとかreadonly ref structとか参照渡し周りが超絶複雑になるよ

**デフォルトの名無しさん** · 2017/12/01(金) 09:15:31.37

.NETライブラリがそれらを採用しないなら使わん

**デフォルトの名無しさん** · 2017/12/01(金) 09:53:17.68

>>116
ライブラリってのが何を指してるのかしらんが、CLRはパフォーマンス向上のために採用するだろ

**デフォルトの名無しさん** · 2017/12/01(金) 09:54:03.08

なんの思想もなくただ思い付いた機能追加してんな
次世代の課題はどう考えてもコードの自動生成だからな
もうc#にそれを受け切るキャパはないと結論出してくだらん機能追加してんのかな？

この設計を実現するためのこの機能ってのがないよな

**デフォルトの名無しさん** · 2017/12/01(金) 09:57:33.53

馬鹿は要らん心配しなくていいぞｗ

**デフォルトの名無しさん** · 2017/12/01(金) 10:17:38.98

>>119
ごもっともだがmsの言語拡張にいったい何パーセントのc#プログラマーがついて行けてるんだろ?

**デフォルトの名無しさん** · 2017/12/01(金) 10:18:45.36

無駄にごちゃごちゃするだけ
要らんやろ

**デフォルトの名無しさん** · 2017/12/01(金) 10:34:20.88

>>120
基盤のためだから、馬鹿な一般プログラマーはついていけなくていい
パフォーマンス向上だ間接的な利益を受けるから

**デフォルトの名無しさん** · 2017/12/01(金) 10:37:55.11

値型参照関連の新機能はパフォーマンス向上のために追加されるから
使わん奴は一生使わんだろうな

**デフォルトの名無しさん** · 2017/12/01(金) 11:15:01.09

>>122
>>102の法則が発動され中途半端な馬鹿がグダグダで複雑怪奇な(当然バグだらけ)の
コードを量産するから困るのです。

**デフォルトの名無しさん** · 2017/12/01(金) 11:19:59.25

馬鹿と一緒に仕事しなければOK
馬鹿に仕事を発注しなければOK
そうもいかないって？馬鹿は大変だなｗ

**デフォルトの名無しさん** · 2017/12/01(金) 11:25:04.63

受託開発なんて馬鹿の巣窟だわ

**デフォルトの名無しさん** · 2017/12/01(金) 12:24:05.27

MicrosoftがVBを切り捨てたようにC#も底辺から切り捨てられていく
底辺プログラマの未来は暗い

**デフォルトの名無しさん** · 2017/12/01(金) 12:25:29.05

C#は他の言語で便利な機能があれば取り入れようとしてる時期
rubyみたいに何でも取り入れてごちゃごちゃにならなければいいけど

本当の意味で新しい機能はしばらく追加されないだろう

**デフォルトの名無しさん** · 2017/12/01(金) 12:30:03.27

>>128
それって簡単に言えばなんの思想もないから他言語パクって見ますって話でしょｗ

**デフォルトの名無しさん** · 2017/12/01(金) 12:33:48.54

>>129
全部入れてるわけじゃない
思想的にあってるものだけ入れてる

似たような感じだったjavaとも進化の仕方から違いが見て取れるはず
C#の下位がjavaと思ってたけど今は結構違いがある

**デフォルトの名無しさん** · 2017/12/01(金) 12:42:25.56

プログラマとして大事なのはコミュ力だから

**デフォルトの名無しさん** · 2017/12/01(金) 12:52:31.16

技術力があれば不要なコミュニケーションを大幅ぬ削減できるという事実から目をそらさないで