ふらっと 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/12/20(水) 17:57:31.91

50個のクラスができるものをクラス使わずにごちゃ混ぜにしたら余計にわけわからんだろ
こういう話題で数が増えたらどうするの？って反論するやつが必ず現れるけどそれ墓穴掘ってるだけだぞ

**752** · 2017/12/20(水) 18:12:27.89

>>843
レートクラスは必要かも知れないですね
まだまだ考慮不足でした

**デフォルトの名無しさん** · 2017/12/20(水) 18:25:24.19

データが構造を持ってしまうと、別の使い方をする時に手間が増えるだけなんだよ
データはデータでクラスにする必要なんかまったく無い

**デフォルトの名無しさん** · 2017/12/20(水) 18:31:13.93

>>850
> クラス図どうなるの？クラス50個作るん？
必要なら作るだけだろ

> 上下のわからんクラスって粒度を
> そんなちっぽけなもんに使うと
> 全体が見えなくなっちゃうんだよ
それお前さんの管理能力の問題...

> 費目は少なくてってそれお前の環境がそうってだけの偶然でしょ？
世の中の会計ソフトとか見てみな
もしくは自分の会社の経理担当者に聞けばわかると思うよ

> この方針はオススメできないね俺は
別にお前もそうしろなんて言ってないから勝手にやってなよ

**デフォルトの名無しさん** · 2017/12/20(水) 18:47:00.16

>>853
別の使い方する時点で分析設計ミス

**デフォルトの名無しさん** · 2017/12/20(水) 18:56:33.13

>>854
残念ながら、世の中の会計ソフトなんかほぼ100%トランザクションスクリプトだよ

**デフォルトの名無しさん** · 2017/12/20(水) 19:01:31.53

>>851
好きなようにすればいいよ
ただ、お前のクラス図に50個のクラスが並ぶだけ

**デフォルトの名無しさん** · 2017/12/20(水) 19:08:20.84

>>856
なんの話をしてるんだよ w

**デフォルトの名無しさん** · 2017/12/20(水) 19:13:04.16

費目、時間単価、時間がそれぞれ独立したクラスか
めまいを禁じ得ない

**デフォルトの名無しさん** · 2017/12/20(水) 19:35:42.09

それらが同じクラスに混在する方が目眩するわ

**デフォルトの名無しさん** · 2017/12/20(水) 19:59:30.76

で、最終顧客がのぞむ要求に答えられるアプリケーションなのかと

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

全体を把握しないと作業に取り掛かれないなら
既にクラス設計として破綻してるとしか

**デフォルトの名無しさん** · 2017/12/20(水) 20:45:21.17

結局要件があいまいだから何とも言えんけど普通に考えれば費目ごとに派生するのはないね。

せっかくC#にはデリゲートがあるんだから、費目で違う処理はコンストラクタで
デリゲート食わせて対応できるし、その上でプロトタイプパターン適用した方が分かりやすいと思うよ。

もちろん費目ごとの固有の処理が費目固有のプロパティに依存するような場合は
継承を使うしかないけどね

**デフォルトの名無しさん** · 2017/12/20(水) 21:00:51.80

費目はクラスじゃなくてデータにするべきものでは在るが、初心者が「そう言われたから」って理由でやるより
何故そうなのかを理解してやるのは悪いことじゃない
自分で考える人を挫くスタイルはあまり感心しないな

**デフォルトの名無しさん** · 2017/12/20(水) 21:05:35.30

種類によって振る舞いが違うって要件なのにデータで扱うとか呆れるわ
どんだけswitchする気だよ

**デフォルトの名無しさん** · 2017/12/20(水) 21:15:55.31

> 自分で考える人を挫くスタイルはあまり感心しないな
マウントしたいだけやろ
今回は質問者の方がレベル高いからアホみたいなことになってるけど w

**デフォルトの名無しさん** · 2017/12/20(水) 21:16:44.49

>>865
アホ
全く別のデータ項目をどうやったら混同できるんだ

**デフォルトの名無しさん** · 2017/12/20(水) 21:22:03.45

>>867
全く別のものを混同して同じデータ構造でごちゃ混ぜに扱おうとしてるのがデータフロー派だぞ
俺や質問者は異なる物は異なるクラスにしろっていうオブジェクト指向派な
むしろ真逆だわ

**デフォルトの名無しさん** · 2017/12/20(水) 21:33:44.71

>>868
何でこんなくだらないものいちいちクラスにするんだ
クラス図書かないからこんな蜘蛛の巣みたいなクラス図作って平気な面してんだろお前

**デフォルトの名無しさん** · 2017/12/20(水) 21:35:40.32

>>868
混ぜて使いたいからクラスにするんだと思ってたんだが違うのか？
全く別物として区別して扱うんなら行全体を1クラスにして複数のフィールドで持つのと大差ないと思うけど？

**デフォルトの名無しさん** · 2017/12/20(水) 21:49:04.68

費用と言っても材料費、人件費、、減価償却費、貸倒引当金など性格や扱いの異なるものを同じ基底から派生させるのはどだい無理がある
集計したいだけなら金額を取得するインターフェイスを実装する方が賢い

**デフォルトの名無しさん** · 2017/12/20(水) 22:10:01.13

>>869
アホほど条件分岐書いて平気な顔してるやつに言われたくない
きみさクラス図どころかオブジェクト指向すらしたことないだろ？

>>870
同じ規約で異なるものを集めるのと
異なるものをただひとつのものとするのと
全く違うことだ
混ぜるという言葉の使い方を見極めろ

**デフォルトの名無しさん** · 2017/12/20(水) 22:10:03.47

List〈string［］〉の型をstring［］の特定の列をキーにしてソートしたいんですけど
List.Sort（x => x［1］）;
のような書き方はできないのでしょうか？
List〈class〉を用意するしかないのでしょうか？多分Listの中の配列にもLINQ使えそうなんですが・・・

テストで試しに動かしたいだけなのにわざわざclassを書くのが大変なので・・・

**デフォルトの名無しさん** · 2017/12/20(水) 22:18:17.41

>>873
Sortは過去の遺物
OrderBy使っときなさい

**デフォルトの名無しさん** · 2017/12/20(水) 22:26:57.08

>>872
でもさ
クラス図書いたら君のはこんなくだらない処理でクラスが50個も並んでるんだぜ
どの辺がいいプログラムなの？
他のも合わせれば5-600は行くんじゃない？
正しいクラス図なんて一度も書いたことないっしょ？
おそらく資料見てもこのクラスは記述があるけどこのクラスは記述がないなぁ
ってもんを平気で作っちゃうでしょ？

**デフォルトの名無しさん** · 2017/12/20(水) 22:31:22.52

>>875
そのドメインがそういうものなら何百クラスだろうがクラスで書くのが正しい
それがそのドメインでは最も簡潔で正しい姿なんだよ

そこを曲げてデータ区分や条件分岐で雑なコードを書いたら
クラス数は増えないかもしれないがコードの複雑度が爆発的に高まるだけ

なぜなら数百クラスが個々に持つべき振る舞いが同じクラスに区別なく同居してしまうんだぜ？
ろくなことにならないよ

**デフォルトの名無しさん** · 2017/12/20(水) 22:32:30.67

>>874
OrderByも試したんですがこのラムダ式だと意図した動作にならないです
とりあえずList〈class〉で素直に書いてみますが、ラムダ式のキーに配列やListのインデックス指定して使うのは無理なんでしょうか？

**デフォルトの名無しさん** · 2017/12/20(水) 22:33:13.38

>>873
>型をstring［］の特定の列をキーにしてソートしたいん
ってどういう事ですか？
型をソートするの意味が分かりません。
初心者なので詳しく教えて下さい。

**デフォルトの名無しさん** · 2017/12/20(水) 22:35:03.12

意図した動作にならないのをバグっていうんだと思いますw

**デフォルトの名無しさん** · 2017/12/20(水) 22:36:47.64

>>876
そもそも設計書で四角で囲って強調すべき単位なの？費目って
オブジェクト指向も自分解釈でおおよそ一般人がオブジェクトなんてしない単位でオブジェクトを作ってるよね？

**デフォルトの名無しさん** · 2017/12/20(水) 22:40:38.82

>>876
結局DBに入れて永続化しなくちゃいかんわけで、その用途考えると数百のクラスとかありえんのさ
EFで扱える範囲で物を考えたほうが良いよ

**デフォルトの名無しさん** · 2017/12/20(水) 22:42:06.35

>>878
ちょっと説明不足してたので補足

List〈string［］〉
｛
｛"1", "aa", "1個目"｝,
｛"2", "bbb", "2個目"｝,
｛"3", "abc", "3個目"｝,
｛"4", "dddd", "4個目"｝
｝
の形があった場合、string［1］のところを基準にソートさせたいってことですね

public class　test
｛
public string id｛get; set;｝
public string text｛get; set;｝
public string discription｛get; set;｝
｝
みたいな形式だった場合
list.OrderBy（x => x.text）;
と同じ動作にしたいということです

**デフォルトの名無しさん** · 2017/12/20(水) 22:54:00.94

>>882
そこが分からない人はいないと思うよw
逆に何が難しいの

list.OrderBy(x => Integer.Parse(x[0]));

とか、

list.Sort((x, y) => Integer.Parse(x[0]) - IntegerParse(y[0]));

ではダメなんですか？

**デフォルトの名無しさん** · 2017/12/20(水) 22:55:43.46

あ、やってもーたw
IntegerじゃなくてInt32ね

**デフォルトの名無しさん** · 2017/12/20(水) 22:59:37.49

>>883
classでも試してだめでした
それで動かなかったので質問したのですが、classもソートされなかったのでデータ側の問題かもです
さっきのダミーデータでちゃんと挙動するか試します

**デフォルトの名無しさん** · 2017/12/20(水) 23:03:12.47

>>880
質問者のドメインでは書くべきものだよ
振る舞いが異なる別のものだからね
それが数百種類でそれぞれ異なるような複雑なドメインなら書くんだよ
無理やり1つのレコードに押し付けちゃダメ
逆にドメインそのものがシンプルで1つのクラスで表現すべきならそうする

きみのおかしな点は質問者のドメインを無視してきみの頭の中にあるシンプルなドメインを前提に話を進めようとするところだな
独りよがりってこと

>>881
実装はどうとでもなる
RDBにこだわる必要もない

**デフォルトの名無しさん** · 2017/12/20(水) 23:06:38.88

>>885
classってのが何のことかよくわからないけど、OrderByの方使ってる？

LINQのメソッドはコレクションをのものを並び替えるのではなくて
並び替えた列挙オブジェクトを返すんだけど、そこを誤解してるなんてオチではなくて？

**デフォルトの名無しさん** · 2017/12/20(水) 23:17:47.14

>>885
https://ideone.com/5Vgtw7
できたよ

**デフォルトの名無しさん** · 2017/12/20(水) 23:21:21.55

書き方がわかりませんということだから並び替え方法とは違うように読めたけどな
list = list.OrderBy(x => x[1]).ToList();
もしくは
list.Sort((x, y) => x[1].CompareTo(y[1]));
とかだろ
OrderByはシーケンスが返ってくるからListにしたい場合はToListが要る
Sortは直接並び替えてるので前の状態も欲しい場合は事前にコピーしておくかな

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

>>888
ありがとうございます
var a = ～
って感じで受け取っててVSで中身無いけどどうなってるの？？
IOrderedEnumerabel〈～〉型ってなんだ？ってなってましたけどforeachに直接入れないとダメなんですね
使い方間違えてました

>>887
これでした、すいません

**デフォルトの名無しさん** · 2017/12/20(水) 23:26:14.46

>>889
あぁToList（）を忘れてました
あやふやな記憶で使っててToList（）使うって教わったのにすっかり忘れてました
型の返り方の説明見て、正しい扱い方覚えないといけないですねすいません
ICollectionみたいなやつとか、抽象化されてるからToListでListに明示的に変換しないとあかんやつでした

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

>>890
てかその構造のデータをstring[]で扱わなきゃいけない理由が何かあんの？

**デフォルトの名無しさん** · 2017/12/20(水) 23:31:09.24

あ、ごめん書いてたね

**デフォルトの名無しさん** · 2017/12/20(水) 23:35:13.47

>>892
試しにデータ群扱う時にclass宣言したり、大量のデータを型分けするのがめんどいからこういう扱い方できると楽ってだけですね
この使い方ならcsv形式にsplitでListぶち込むだけでやれるので宣言項目少なくてテストするなら楽ってだけです
DBから持って来るときは文字列型じゃなければToStringかけたりとかありそうですが
最近はEF使ってるので楽ですけどね

**デフォルトの名無しさん** · 2017/12/20(水) 23:38:15.25

Kotlin では、ある列の値で、レコードをソートするのは以下。
年齢・名前でソートする

https://mevius.5ch.net/test/read.cgi/tech/1509462463/18

**デフォルトの名無しさん** · 2017/12/20(水) 23:44:31.66

>>894
Visual Studioならjsonからクラスを自動生成できるからクラスの方が扱いやすいけどね。stringとHashtableまみれのコードとか絶対メンテできない…

**デフォルトの名無しさん** · 2017/12/20(水) 23:49:35.45

>>896
こんな機能あるんですね！
interfaceのメソッド自動実装ぐらいしか知らなかったのでこういうの積極的に使いたいです

**デフォルトの名無しさん** · 2017/12/21(木) 00:47:58.94

http://yossk.hatenablog.com/entry/2015/02/23/070546

ここの「4-3散在する情報」のサンプルコード
川俣晶の本からのコピーなんだけど

Runメソッド内で値設定したあと
CalcDistメソッド先頭で値をリセットしてるから
常に計算結果0になるように見えるんだけど
なんか俺読み間違えてる？

**デフォルトの名無しさん** · 2017/12/21(木) 00:54:31.04

たしかに

**デフォルトの名無しさん** · 2017/12/21(木) 03:28:09.32

>>886
はぁ？人件費がクラスに？
センス悪いじゃん

**デフォルトの名無しさん** · 2017/12/21(木) 05:44:12.11

>>900
人件費と光熱費が異なる振る舞いをするなら別のクラスにするのが正しい
センスの問題じゃない常識の問題

**デフォルトの名無しさん** · 2017/12/21(木) 06:04:58.06

>>901
はぁ？
どう組んでも動きますが？

**デフォルトの名無しさん** · 2017/12/21(木) 06:50:24.48

オブジェクト指向は物事を複雑にするためにあるモノじゃ無いぞ。
オブジェクト指向を目的にしてどうするwww

**デフォルトの名無しさん** · 2017/12/21(木) 06:50:56.44

動く動かないとセンスの有無は別の話
理解力無さすぎだろ w

**デフォルトの名無しさん** · 2017/12/21(木) 06:56:56.39

どの世界でも原理主義者は物事を複雑にするw

**デフォルトの名無しさん** · 2017/12/21(木) 06:57:03.08

>>904
正しいとかおかしいよな
どう組んでも動くし

**デフォルトの名無しさん** · 2017/12/21(木) 07:00:31.05

クラス図書かないやつは客観的に自分の設計を見ることができない
だからこんなプロパティ単位のものクラスにして
クラス図を蜘蛛の巣にしちゃっても
それを汚いと反省する機会がない

ただし、どう組んでも動くのであくまで俺の持論

**752** · 2017/12/21(木) 07:13:38.51

費用クラスの者ですが、色々なアドバイスありがとうございます
参考になります

>>902
どう組んでも動くの精神で、前任のパートさんが凄いの組んでくれたので、オブジェクト指向を意識し始めました

クラスどころかイベントリスナーのメソッドというのでしょうか?
そこから共通関数すらなく処理されているので、なんとかしたかったのです

ドメイン層とアプリ層とインフラ層が渾然一体となっていると、どう手を付けて良いのやら

**デフォルトの名無しさん** · 2017/12/21(木) 07:25:44.67

必要最低限だけ修正しろ
コードのきれいさは主観
他人のコードはすべて糞コード

**デフォルトの名無しさん** · 2017/12/21(木) 07:27:47.47

>>908
お前のはプロパティ単位もクラスになってるからさらにカオスだな
初見で見た人間がそのたくさんあるクラスの依存関係がわかるといいな

**デフォルトの名無しさん** · 2017/12/21(木) 07:42:54.80

と、プロパティ単位とか意味不明のことをつぶやいており当局は措置入院の必要性があると判断している模様です

**デフォルトの名無しさん** · 2017/12/21(木) 07:56:29.75

>>908
おまえ本当に未経験者かよｗｗｗ

**デフォルトの名無しさん** · 2017/12/21(木) 08:08:25.86

残念なのはおそらくこいつはコードレビューとかしたことないね
クラス図と基本設計書との不一致とか設計書のレビューもやったことない
このレベルでクラス図なんて書いたらごちゃごちゃしてなんだかわからないだろ

見難いよって誰からも指摘されたことないから当然伸びない
修正が聞かないまま今に至る

**デフォルトの名無しさん** · 2017/12/21(木) 08:09:15.38

(ｷﾘｯ

**デフォルトの名無しさん** · 2017/12/21(木) 08:17:58.61

新参だがここで聞くのは無駄だってことは分かった

**デフォルトの名無しさん** · 2017/12/21(木) 08:37:19.47

>>908
え～、今更オブジェクト指向なの?
c#はマルチパラダイム言語なのに

**デフォルトの名無しさん** · 2017/12/21(木) 08:40:09.55

>>915
ちゃんとした質問ならまともな答えが帰って来る。
トンデモ質問はスルーされるかネタにされて遊ばれるw

**デフォルトの名無しさん** · 2017/12/21(木) 08:54:24.69

最近クラス図キチガイが湧いてるな

**デフォルトの名無しさん** · 2017/12/21(木) 09:04:04.27

>>912
素人だろうね。
ギョーミーなアプリ開発の経験者に費目毎のクラス化なんて発想はない。
社内のパソコンマニアに白羽の矢が立っただけ。

**デフォルトの名無しさん** · 2017/12/21(木) 10:15:36.99

>>919
俺が言ったのは本当にプログラミング初体験なのかよってことな。

**デフォルトの名無しさん** · 2017/12/21(木) 10:19:43.29

javaの匂いが消えない

**デフォルトの名無しさん** · 2017/12/21(木) 10:33:22.61

>>920
大学出てりゃプログラミング初体験なんてことは無いだろうな。
プログラミング教育義務化なのでプログラミング初体験者ゼロの世の中になるw

**デフォルトの名無しさん** · 2017/12/21(木) 10:41:24.50

>>805
＞急遽プログラムをやれと言われた人間なのでセンスとか言われると厳しいです

**デフォルトの名無しさん** · 2017/12/21(木) 12:03:45.23

>>908
昨日も書いた((>>863)けど、費目ごとに派生クラス作れ、なんて話は真に受けない方がいいと思うよw
処理の委譲はデリゲートでもできる。

**デフォルトの名無しさん** · 2017/12/21(木) 12:50:57.69

>>918
自称上級者なんだろ
今度はデリゲートとか言い出しちゃってるし
自分の知ってる単語ならべてるって感じ w

**752** · 2017/12/21(木) 12:55:56.30

>>923
急遽と言っても半年以上OJTの形で作っているので、変数、関数、クラスが何なのか程度は理解しています

で、費用と費目は抽象クラスを使うのに絶好の課題ではと初めて使ってみて、世間様ではどう思われるか聞いてみたら、そんな関係には使わないと指摘された次第です

**752** · 2017/12/21(木) 13:00:11.33

ズブの素人でも手続き型プログラムを一ヶ月で作れるようになったのは、visual studioの支援とC#の解りやすさのおかげだと感謝してます

ゆえに急遽突っ込まれたのかも知りませんが

**デフォルトの名無しさん** · 2017/12/21(木) 13:29:10.16

>>926
結局ね、データってのはDBに保存したり手で入力したりするものなので
データごとにクラスが変わったりすると破綻するんですよ
クラス分けは集計には便利かもしれんが、全ての工程でどうなるか考えて設計しないとね

**デフォルトの名無しさん** · 2017/12/21(木) 13:38:55.98

>>927
ズブの素人が１ヶ月でc#をそれなりにマスターするなんて凄いわ。
オレは無理だ (ーー;)

**752** · 2017/12/21(木) 17:30:18.01

>>929
最初の二ヶ月は前任の人が引き継ぎ兼トレーニングしてくれたのでブートは早かったと思います
そこから半年でやっと継承を触ったので、成長曲線はそこまで急では無いかと

比較が解らないですが仮に成長が早いとするとC#が非常に取っ付きやすかったのが大きいと思います

基本情報試験のアルゴリズムとかさっぱり解らないので、プログラム能力は経験相応に低いと思います

**752** · 2017/12/21(木) 17:42:42.42

>>913
PM交え前任者とコードレビューをしましたが、三項演算子とLINQは見難いと指摘されました

手続き型より宣言型のが良いとネットで見たのですが

逆に前任者が書いた4重ネストのfor文にifelseが連ねてあるコードはいくら頭をひねっても理解できず
数字の動きはステップ実行で見れるのですが、だから何って感じになります
ゆえに知識を整理したくなり初めて抽象クラスを意識した次第です

**デフォルトの名無しさん** · 2017/12/21(木) 17:50:29.58

優秀だね
ここにくると馬鹿になるからもう来ない方がいいよ

**752** · 2017/12/21(木) 17:56:25.54

>>907
素朴な疑問ですがプロパティが大量に並ぶのとクラスが大量に並ぶのはどちらが好まれるのでしょうか

個人的には、クラス内にプロパティがギッシリという光景も凄いなと思うのですが

私の知らないテクニックで隠蔽できるのかも知れないので解りませんが

なおデリケートはイベントリスナとして使われてる程度の理解しかなく使いどころがピンと来ていません
精進します

**デフォルトの名無しさん** · 2017/12/21(木) 18:05:03.28

プログラミングしたので評価してください
archive.isに定期的にアーカイブさせるプログラムです
https://dotup.org/uploda/dotup.org1417700.zip.html 👀
Rock54: Caution(BBR-MD5:1322b9cf791dd10729e510ca36a73322)

**デフォルトの名無しさん** · 2017/12/21(木) 18:12:09.40

間違えてプログラムだけ上げてしまいました
こっちはプロジェクトです
https://www.axfc.net/u/3872901

**デフォルトの名無しさん** · 2017/12/21(木) 18:14:54.74

GitHubにあげてくれ

**デフォルトの名無しさん** · 2017/12/21(木) 18:56:17.83

ギフハブ？

**デフォルトの名無しさん** · 2017/12/21(木) 19:01:00.92

>>933
そんなのトレードオフに決まってんじゃん

**デフォルトの名無しさん** · 2017/12/21(木) 19:04:17.37

>>933
何べんも言うけどさ、結局要件の詳細が分からないとどういう実装方法が
ベストかなんて何とも言えんよ

妄想込みの印象論としては費目ごとに派生クラス作るなんて筋悪に感じるなとは言えるけど

**デフォルトの名無しさん** · 2017/12/21(木) 19:09:52.65

(´･ω･`)
https://github.com/as123414/AutoArchiver

**デフォルトの名無しさん** · 2017/12/21(木) 19:21:07.79

md書け

**デフォルトの名無しさん** · 2017/12/21(木) 19:22:33.42

いつから「評価してもらうスレ」になったんだよ
せめて相談室でやれよ

**デフォルトの名無しさん** · 2017/12/21(木) 19:37:28.56

ちょっとだけ見たけどやはり初心者ですねとまだ短いにも関わらず読みづらいコードになりそうですねということかな
一行が長いところは絶対ダメとはいわないけど大体クソコード

**デフォルトの名無しさん** · 2017/12/21(木) 19:41:21.29

>>933
もう手続き型の組み方で良いんじゃないの?
オブジェクト指向はプログラムを分かりやすく保守しやすくするために出来たもの。
変なオブジェクト指向に拘って物事を複雑にするのは本末転倒。

**デフォルトの名無しさん** · 2017/12/21(木) 19:46:25.90

意味なく一行が長いのを見るとどんなエディタで書いてるんだろうかと思う
あとは一行でつなぎまくってるのを見るとデバッグがめんどくさいだろうなと思う

**デフォルトの名無しさん** · 2017/12/21(木) 19:47:38.41

>>945
120行目のことでしょうか
それとも出だしのメッセージボックスでしょうか

**デフォルトの名無しさん** · 2017/12/21(木) 19:52:03.68

もう見るつもりはないから忘れたけど後ろの方にあったメソッドは内容をもっと見直したらいいと思う
けど初心者はこんなもんだろ

**デフォルトの名無しさん** · 2017/12/21(木) 19:52:34.81

(´･ω･`)

**デフォルトの名無しさん** · 2017/12/21(木) 19:57:53.60

>>946
メソッドの結果を受ける段で概念的にわかりやすい名前で一度受けたほうがいいと思う
何々Textとか何々uriとか

あとは個人的な感想として成功じゃなくて失敗をbool値で受ければいいんではないか
bool 失敗=false

catchで失敗=true

>>948
まあこんなところで聞くやつが悪い

**デフォルトの名無しさん** · 2017/12/21(木) 19:58:12.75

>>933
小さなクラスが大量に並ぶのがオブジェクト指向の正解

**デフォルトの名無しさん** · 2017/12/21(木) 20:10:11.41

>>950
俺はトレードオフがオススメ

ちょっとした修正でクラス図まで書き直す必要ありますはお客に印象悪い
どういう設計してんの？って聞かれる