【質問】ASP.NETスレ Part6【雑談】
■ このスレッドは過去ログ倉庫に格納されています
0001nobodyさん
2009/09/01(火) 20:06:04ID:???Microsoft .NET Frameworkの一連のテクノロジの一つです。
技術の移り変わりの早い分野ですので、みんなで質問、相談しつつ、より理解を深めていきましょう。
●ASP.NET関連サイト
マイクロソフトASP.NETデベロッパーセンター
http://msdn.microsoft.com/ja-jp/asp.net/default.aspx
ASP.NETオフィシャル（英語）
http://www.asp.net/
VisualStudioホームページ
http://www.microsoft.com/japan/msdn/vstudio/
SQLServerホーム
http://www.microsoft.com/japan/sqlserver/2005/default.mspx
IISオフィシャル（英語）
http://www.iis.net/
ASP.NETにAJAX技術を取り入れるASP>NET AJAX（英語）
http://www.asp.net/ajax/
ASP.NETにMVCアーキテクチャを取り入れるASP.NET MVC（英語）
http://www.asp.net/mvc/
ASP.NETでのお役立ちの定番サイト
http://www.atmarkit.co.jp/channel/aspnet/aspnet.html
●前スレ
【質問】ASP.NETスレ Part5【議論】
http://pc11.2ch.net/test/read.cgi/php/1232671611/
006759
2009/09/13(日) 21:35:45ID:???>そんなチェックはしないことが多いのかな？
登録対象商品がまだ有効かどうかのチェックはする。よくする。
ただし、普通、マスタの更新（UPDATE）なんてしない。ありえない。
やるなら削除後に新規登録、または新規登録のみ。
そうすれば登録対象が急に別物にすりかわるなんてことはなくなる。
1:nの関係はもちろんよくあるが、親が子のIDを持つことはありえないと思うんだが。
支社テーブルは社員IDを全部持ってるの？各社員（子）が支社コードもってればよくない？
006865
2009/09/13(日) 22:27:45ID:???１：ｎの関係はよくあるが、親が子のIDもった状態でどうやって
親：子が１：ｎの関係を作れるんだ？
その親に対する子はその親がもってるIDの子１件だけだろ
まさか同一IDの子が複数いるとか言わないだろうな
後半に関しては、個人的意見だがIDとコードを両方持つ設計は少ないと思うぞ
データとしては入力されるコードを格納すればいいんだし
コードとID別持ちで、コードからID引いて格納するって設計がまあ特殊なんだと
在庫引当の例とかは、ビジネスロジックとしてのチェックの問題だ
ビジネスロジックをコードレスなんてもともと無理だと思うがな
それは更新の問題じゃなくて入力内容チェックの問題
FormViewだろうがなんだろうが画面を表示する機能に直接関係ない
0069nobodyさん
2009/09/14(月) 05:04:37ID:???>登録対象商品がまだ有効かどうかのチェックはする。よくする。
するよね？
するってーと、ObjectDataSourceなんかでチェックして、
だめなら、だめの返値返して、エラー表示する処理とかになるよね。
>1:nの関係はもちろんよくあるが、親が子のIDを持つことはありえないと思うんだが。
>支社テーブルは社員IDを全部持ってるの？各社員（子）が支社コードもってればよくない？
ごめん。急いで書いたので、一部、不適切な関係があるね。
というか、自分が、どうやら、親と子を逆に捉えてるのかな。
他のテーブルの主キーを持ってる方を親だと、ずっと思ってたよｗｗ
商品マスタ->商品区分マスタ、取引先->都道府県マスタ、社員マスタ->支社マスタとか。
0070nobodyさん
2009/09/14(月) 05:11:29ID:???>親：子が１：ｎの関係を作れるんだ？
上でも書いたけど、どうやら俺が逆に書いてるようだｗ許してくれ(´；ω；`)ｳｯ…
でも、在庫引き当て時とかのチェックは1:1でも1:nで・・も(ry
>後半に関しては、個人的意見だがIDとコードを両方持つ設計は少ないと思うぞ
どっちにしろ、コードの存在確認はしなきゃなんないでしょ？
編集されたかの楽観的ロックじゃなく、
存在しないコードを入力させない存在チェックとして。
>FormViewだろうがなんだろうが画面を表示する機能に直接関係ない
というかね
>45 名前： 23 [sage] 投稿日： 2009/09/09(水) 08:04:03 ID:???
>俺のところだと SQL 手書きが発生するのは全体の 2 割以下だな。それもこれも最新の
>ASP.NET が使えるおかげ。
から、formView+dataSourceでクエリの手書きが2割以下になると思ったのよ。
そのぐらいすごいんだと。
ほんとは、調べるとそうでもなくて、
過去の経験からは、ほとんどチェックが必要だったりして、
ObjectDataSourceでコード書かないといけないので、
自作のクラスで処理するのと、あんま差はないなと思ったということ。
で、dataSourceを利用しないで、formViewを使うメリットってなに？
という話になるとおもうんだけど、
上のサンプルみると、各コントロールのインスタンスからデータを取得してるから、
ポトペタするのと変わらないようにみえる。
フィールド値のバインドを自動的にやってくれるぐらい？
という感じがしてるんだけど、という感じ。
うーん、なんかいまいちよくわかんないな・・
007165
2009/09/14(月) 10:05:53ID:???DB更新の機能(=DataSource)の範疇でもない（入力チェックをObjectDataSourceでやるのは
俺は間違ってると思う。DataSouceのもとになってるクラスでチェックするべきだと）
で、SQLの手書きが２割以下ってのは、俺は言いすぎだとは思う
ただ、純粋にSQLを書くという作業に限ってみれば、SQL書くところすべてに
SQLDataSourceつかえば、確かに自分でSQL書くことは少ないとはおもうが
それはアクセスでクエリビルダ使ったらSQL手書きしなくていいです、ってレベルと同じ話
必要なSQL文の数が減るわけじゃなくて、それを書く作業が減るだけ
DataSource使わないFormViewのメリットなんてないと思う。というか
FromViewはDataSource前提に設計されたコントロールだと思うが
DataSourceつかうことにメリットがあるんじゃなくて、使わないことにデメリットがある
0072nobodyさん
2009/09/14(月) 19:50:49ID:???DataSourceの元になるクラスという？
こういった場合にはObjectDataSourceでSELECT、INSERTなどのクエリを生成して、
コントロールにバインドするんじゃないの？
0073nobodyさん
2009/09/14(月) 19:52:33ID:???↓
DataSourceの元になるクラスというと？
007423
2009/09/14(月) 21:41:06ID:???入力チェックは aspx.cs の仕事だと思うがなぁ。SQL 手書き 2 割以下というのは
ObjectDataSource で TableAdapter を使うパターンが 8 割を占める、という意味。
たとえ複数テーブルの更新が発生したとしても、内部的には TableAdapter を使い
回す。手書きが必要なのはバッチ処理ぐらい。まあ、プロジェクトにもよるがうちは
テーブル数と画面数が大体 200 ぐらいだな。コードマスタが 50 以上あったはず。
0075nobodyさん
2009/09/14(月) 23:51:43ID:???うちは今一切使わない方針だけど。
なんかたまにASP.NET本来の原初的な組み方をしなきゃならないって考える波がくる。
で、MS本読み直すと「あー間違ってたのかな」と思ったり。
「Javaから来たヤツは全てを自前クラスで用意しようとする。
そうする利点は認めるがASP.NETでは…」みたいな論調だし。
で、軽く組み直してみたら、やっぱりハマるんだけどw
0076nobodyさん
2009/09/15(火) 00:04:23ID:???それならDataSourceと切り離してFormViewだけのメリットって何？
って話だな？
これとは別にTableAdapterを使うのはいいけど、データのソートとか抽出とかはどうしてる？
クエリかかないならTableAdapter.Fill(DataTable)して、DataTable.Select("")してるってこと？
これだとSQLからデータ抽出して、メモリに蓄えて、そこからまたSelectして
無駄が多いような気がするんだが。
0077nobodyさん
2009/09/15(火) 02:15:05ID:???SQL書く作業がTableAdapter定義する作業になっただけ
昔のADO.NETでは、DataAdapterでのUPDATEは使えねえってのが定説だった気がするが
TableAdapterになって使いものになるようになったのかな？
0078nobodyさん
2009/09/15(火) 03:43:03ID:???複雑なことしようとすると、TableAdapter用のクエリの手書き必須
挿入時に論理削除を意味するIsDeleteをいじられたくないのでfalseで固定したいとか
サブクエリで抽出した内容を取得して挿入したいとか。
挿入したときの主キーを取得するのも手書きが必要だったような。
あと上にもあるけど動的にクエリを発行できないので
検索条件に従ってWhere句を作成するとかは無理だったはず。
かといってDataTableのSelectメソッドをWhere句の動的生成の
変わりに利用すると、いちど全部のデータを取得するので、
行数が多いとデータの取得に時間がかかる。
そのたありがLinqToSQLやEntityFrameworkで解決してると思うんだけど、
LinqToSQLは終了の方向だし、EFもなんとかしてくれって言う人が多くて、
まだ微妙なところ。
007923
2009/09/15(火) 07:39:06ID:???うちでは Excel の仕様書から TableAdapter を自動生成していて、SelectList メソッド
にソート引数も指定できるようにしてるよ。内部的には ORDER BY に変換する。
>>77
やっかいなのは、SELECT するのは VIEW だけど更新もしたいケース。そんな時は
Update メソッドの中で、個々の TableAdapter を使い回す。それで対応できなければ
SQL 手書き。
>>78
WHERE の動的作成は頻出パターンなので、TableAdapter の自動生成時に作り込んで
いる。つまり、検索条件を引数にもらう SelectList メソッドの中で、引数が null じゃ
なければ WHERE に追加している。こうすると、画面側では検索条件をバインドする
だけで済む。
0080nobodyさん
2009/09/15(火) 17:37:59ID:???>TableAdapter を自動生成していて
>WHERE の動的作成は頻出パターンなので、TableAdapter の自動生成時に作り込んで
それでSQL手書きが２割以下になるのか。それなら納得
しかし、自動生成されたSQLを使うのはどうもあまり乗り気になれん
TableAdapterにしてもDataSourceにしても、SQLは完全にラップされて
アプリ側から見えなくしてるわけだし、その方向が正しいのはわかるんだけど
アプリ側でSQLもシームレスに使いたいっていうと、LINQな方向に行くのかねぇ
でもあれもSQLがそのまま通るわけじゃないしなぁ
0081nobodyさん
2009/09/15(火) 19:25:00ID:???>TableAdapter の自動生成時に作り込んでいる。
>つまり、検索条件を引数にもらう SelectList メソッドの中で、
>引数が null じゃなければ WHERE に追加している。
これどうやるの？
TableAdapterで、条件に従ってWHEREを追加とかできたっけ？
0082nobodyさん
2009/09/15(火) 23:02:34ID:???0083nobodyさん
2009/09/15(火) 23:29:57ID:???とかじゃないだろうなｗ
008423
2009/09/15(火) 23:52:09ID:???Excel マクロでコード生成するんだからパターンさえ決まれば何でもできる。
Fill 系のメソッドで引数チェックして WHERE を組み立てるなんざどこでも
やってることでしょ。
0085nobodyさん
2009/09/16(水) 01:27:00ID:???そのFillなんちゃらの引数によって、クエリを作ったりできたっけ？
自分はできないと思っていたので、できるのなら教えてほしい。
環境はVS2005+SQLServer2005
0086nobodyさん
2009/09/16(水) 01:32:33ID:???環境はVS2005+SQLServer2005で、TableAdapterのFillなんちゃらのメソッドで、
引数に従ってWHEREを作成するなんて無理だと思ってた。
Fillなんちゃらがストアドを呼び出してて、
ストアド側で引数によってクエリのWHEREを組み立ててるならわかるけど、
それはクエリを書いてるから手書きクエリの削減じゃないしなぁ。
0087nobodyさん
2009/09/16(水) 06:02:19ID:???>Fill 系のメソッドで引数チェックして WHERE を組み立てるなんざどこでも
>やってることでしょ。
Fillなんちゃらって、TabelAdapterのか？
SQL指定したらメソッドの中身はウィザードで勝手に作られてるぞ
すくなくとも自分でコードは書いてないから、動的にSQL作ったりはしてない
これをいじるぐらいなら俺ならTableAdapterなんて使わん
>>86
実現させる方法をいろいろ考えたが
部分クラスか継承させたクラスでFillなんちゃらを全部自前で実装すればできそう
SelectList メソッド ってのもよくわからんし、自動生成されてるのは
TableAdapterだけじゃないのかもしれんが、そのへんは>84じゃないのでわからんｗ
たとえストアドで操作してても、そのストアドが自動生成されているなら
「手書き」クエリは減ってるとは言える
まあ、なんかしらの開発用フレームワークあるんじゃないかって感じだな
008823
2009/09/16(水) 07:26:21ID:???TableAdapter は Excel の仕様書からマクロで自動生成してるんだって。Fill とか
SelectList とか名前はどうでも良くて、要するに ObjectDataSource の Select
メソッドに選択できるメソッドの中で、引数をチェックして WHERE を組み立てる
ロジック込みで、コードを自動生成している。TableAdapter をウィザードで作って
いるわけではない。これを「手書き」と思うなら、まあどうぞ御自由に。
0089155
2009/09/16(水) 16:20:34ID:???すまん、どの引数なのか、ウィザードとは何のことかさっぱりわからん。
よければ質問に答えてくれないか？
>ObjectDataSource の Selectメソッドに選択できるメソッドの中で、
>引数をチェックして WHERE を組み立てるロジック込みで、コードを自動生成している。
Ｑ１ どの引数をチェックしてWHERE文を作成してるの？
１．ObjectDataSourceのSelectメソッドの引数
２．TableAdapterのFillなんちゃらメソッドの引数
３．その他のメソッドの引数（どのメソッド？）
Ｑ２ WHERE文を組み立ててるのはどこ？
１．ObjectDataSourceのSelectメソッド
２．TableAdapterのFillなんちゃらメソッド
３．Excelのマクロ
４．その他（どこ？）
Ｑ３ 自動生成するクエリの範囲は？
１．すべてをExcelのマクロで作成
２．WHERE文以外をExcelのマクロで作成、WHERE文のみQ2のメソッドで、Q1の引数から作成
３．すべてをQ2のメソッドで、Q1の引数から作成
４．その他（どこ？）
0090nobodyさん
2009/09/16(水) 16:21:32ID:???続き
>TableAdapter をウィザードで作っているわけではない。
Ｑ４ TableAdapterそのものはどうやって作ってるの？
１．データセットデザイナ
２．その他（なに？）
Ｑ５ 作成したWHERE文から前のクエリとWHERE文はどこに登録してるの？
１．すべてのクエリをTableAdapterに登録
２．WHERE文から前をTableAdapterに登録、WHERE文はDataTable.Selectメソッドで登録
３．その他（どこ？）
Ｑ６ クエリをどこに登録してるの？
１．拡張子.xsdのxmlファイルのに手書きで作成したクエリを追加
２．データセットデザイナのクエリ構成ウィザードを使って作成したクエリを登録
３．データセットデザイナのクエリ構成ウィザードからクエリビルダを使って作成したクエリを登録
４．その他（なに？）
0091nobodyさん
2009/09/16(水) 16:35:02ID:???その自動生成されてるものはTableAdapterといえるのだろうか
そもそもTableAdapterって何だって話になるんだが、MSDNによると
>TableAdapter は、DataAdapter の機能を向上させるためにデザイナで生成されるコンポーネントです
らしい。
当然TableAdapterと１００％互換のあるクラスも作成可能なんだろうが
それを生成したからといってTableAdapterを自動生成っていうと誤解を招くとおもうな
ObjectDataSourceでの使用が前提なら、あえてTableAdapterと互換のあるものにする必要もないし
SQL手書き量が減ってる最大の要因はこのデータアクセス層クラスの自動生成のおかげで
けっして最新のASP.NETのおかげではないってのも誤解を招いた原因の一つだな
0092nobodyさん
2009/09/16(水) 18:30:04ID:???じゃないと話がつながらないし、
もしExcelのマクロとやらでクエリを書いてるから自動だってのなら、
これはツール（Excel）のおかげでASP.NETのおかげじゃないから
2割以下なんて結論に至るはずがない。
>45 名前： 23 [sage] 投稿日： 2009/09/09(水) 08:04:03 ID:???
>俺のところだと SQL 手書きが発生するのは全体の 2 割以下だな。それもこれも最新の
>ASP.NET が使えるおかげ。
DataSetデザイナでは、WHERE文を動的に発行することは不可能だから、
みんなどうやってるのかを聞きたいんだと思うのだが。
※DataSetで動的にWHERE文を作るのは不可能というと誤解を招くと思うが、
考えられる場合の数だけ引数の異なるFill〜メソッドを作れば可能だが、
これは半固定なのでこれは動的発行じゃないし、
ストアドプロシージャでも可能だけど、これも事実上、クエリは手書きとかわらんし、
DataTableのSelect使えばソートやフィルタリングはできるが、これもクエリの動的発行じゃない
009323
2009/09/16(水) 21:41:11ID:???Q1=2, Q2=2, Q3=1
>>90
Q4=Excel マクロ, Q5=TableAdapter の定数, Q6=Q5 と同じ
要するに、TableAdapter.cs 全体を Excel マクロで吐き出している。全部。WHERE
の動的生成もその中にある。種も仕掛けもあるよ。
>>91, 92
あーなるほど、>>45 の発言が混乱させてたのか。たしかに ASP.NET のおかげじゃ
ないね。手書き 2 割以下は ASP.NET のおかげではなく、Excel マクロのおかげ。
元レスは ASP.NET 1.1 の環境だったから、単純に FormView が 使える3.5(2.0 も)は
いいぜ、という程度のノリだった。すまんかった。
0094nobodyさん
2009/09/16(水) 22:11:35ID:???ヽ(･ω･)/ ｽﾞｺｰ
＼(.＼ ノ
､ﾊ,,､ ￣
￣
でDataSourceと関係なく、FormViewのメリットってなに？
本来、ポトペタするコントロールに、フィールド名を登録するだけでバインドしてくれるところ？
0095nobodyさん
2009/09/16(水) 22:50:54ID:???FormView(DetailsViewも）のメリットは、データソースの単一レコードにバインドでき
レコードのナビゲーションを操作する機能まで取り込まれているところがメリットかと
1.1で単一レコード表示させたら、レコード移動と画面の更新を全部自分でやらんとダメだからな
双方向バインドで自動更新なんておまけみたいなもんですよｗ
009623
2009/09/16(水) 23:02:19ID:???それもあるが、レイアウトを自由にカスママイズできるってのと、あとは登録処理が
FormView.InsertItem(false) で OK なところとか、Update の時に元々の入力値も
引数で一緒にもらえるところとかね。使ったことないなら使ってみたら?
>>95
ナビゲーションは逆に使ってないな。GridView との連動パターンしかない。
それと、双方向バインドで自動更新は不可欠だろ! これのおかげでどんだけ楽に
なってるか。
0097nobodyさん
2009/09/16(水) 23:33:22ID:???>レイアウトを自由にカスママイズできるってのと
レイアウトの自由度はポトペタ>>>>>>>>>>>>>>>>>>>>>>>>(越えられない壁)>FormViewだろ？
なんでレイアウトの自由度がポトペタに対してFormViewのほうがメリットあるんだ？
>あとは登録処理がFormView.InsertItem(false) で OK なところとか、
そんなのFormViewじゃなくても作り方次第
>Update の時に元々の入力値も引数で一緒にもらえるところとかね。
そんなのFormViewじゃなくても作り方次第
>それと、双方向バインドで自動更新は不可欠だろ! これのおかげでどんだけ楽になってるか。
多くの人はなってないんだよ。
おまえだけがエクセルでやってるから楽なの。
FormViewの利点を述べたいがために強弁してないか？
ASP.NETでクエリを書くのが2割になったとか、
レイアウトを自由にカスタマイズできるとか、
作り方次第でどうとでもなることをFormViewのメリットだと述べたりとか、
自分がやっている方法に固執してFormViewは便利だと述べたりとか。
009995
2009/09/17(木) 01:07:19ID:???>レイアウトを自由にカスママイズできる
レイアウトのカスタマイズなんてFormView以外でもカスタマイズできる
これはFormViewのメリットでも何でもない
>ナビゲーションは逆に使ってないな。GridView との連動パターンしかない。
ナビゲーションの真意が伝わってない気がするな。FormViewでレコードを移動するって話じゃないぞ
レコードが移動されたときに新しいレコードにバインドし直すって話だ
いちど、FormViewつかわないで一覧からカレント行取得して詳細表示するコード書いてみ
このコードを自動でやってくれるのはかなり楽
データ更新は、SQL書くのなんてパターン決まってるから難しくはない
（それこそ自動生成で８割まかなえるほどに）
ただ、ビジネスロジックのチェックはそうはいかんし
単純な更新でないと使えないってのが感想だ
>>97
作り方でどうにでもなるのはその通りなんだが、問題はその作りこみが
FormViewで不要や楽に実現できるようになるかどうかだろ
レイアウトの件はまあ同意だが、FormViewにもポトペタすればある程度自由にできるぞ
登録処理は、単純な更新に限れば楽なる
UPDATEの元値は、DataSet使わないならかなり有効な機能だと思う
1.1には単一レコードを想定したバインドコントロールはないんで、
その点でFromViewにはメリットはあるから、使えるとこなら使えばいいかと
逆にデメリットは、自由度が下がるってことか
それでもある程度コードかけばカバーできる範囲だと思う
0100nobodyさん
2009/09/17(木) 05:26:24ID:???上のほうにナイスなたとえがあるけど、まさにその通りだと思ったな
FormViewは麺つゆ
ウドンやソバを作るのには便利だし美味しい
だけどいくら加工してもベースが麺つゆだから味が似てしまう（応用度が低い）し
麺つゆだから酢醤油や砂糖醤油にはならない。
ポトペタの醤油は手間はかかるがより多くの料理に利用できる。
こんなとこだろ
0101nobodyさん
2009/09/17(木) 10:42:52ID:???ありますでしょうか？repeaterの中にあってIDが固定じゃないのでべた書きすることが
出来ません。
サーバ側でならRepeater.ItemDataBound イベントで処理する。
クライアント側でならJavaScriptで走査して処理する。
0103nobodyさん
2009/09/17(木) 15:36:36ID:???俺ならサーバー側もforeachで回す。
0104nobodyさん
2009/09/17(木) 16:26:00ID:???Repeaterの中をIDでFindControl
0106nobodyさん
2009/09/17(木) 23:30:47ID:???ハードコーディングしたくない俺は大体再帰で回す。
protected void button_Click(object sender, EventArgs e) {
clear(this.Repeater1.Controls);
}
protected void clear(ControlCollection controlCollection) {
foreach (Control control in controlCollection) {
if (control.GetType().Equals(typeof(CheckBox))) {
((CheckBox)control).Checked = false;
}
if (control.HasControls()) {
clear(control.Controls);
}
}
}
0107nobodyさん
2009/09/18(金) 05:33:22ID:???まあ俺なら、ページ出力時にクライアントサイドのスクリプトを動的に生成して出力しとく
チェックボックスオフにするためだけにポストバックさせたくない
今ならAjaxでやるのもありなのかもしれん。updatepanelだっけ？
その場合、Repeaterの中全部更新されるよね？
JQueｒyとか使うと楽ちんだわな
0110nobodyさん
2009/09/18(金) 15:09:35ID:???その場合はその項目もリセットされるわな
それがまずいかどうかは>101の判断だろう
まあ、一番楽な方法としてリセットボタン考慮する価値はあるかと
0111nobodyさん
2009/09/18(金) 15:15:51ID:???そういえばそんなのもあったな。すっかり忘れてたわ。
0112nobodyさん
2009/09/18(金) 19:03:46ID:???0113nobodyさん
2009/09/19(土) 03:13:45ID:ZjH1gzNN自分で実装してて思うんだが、こんなの俺以外に絶対に保守出来ない。
つーか俺自身でも3か月後には多分保守出来ないｗ
難しいシステムを無理やりWebで作るのはほんとアホとしか思えないぜ。
0114nobodyさん
2009/09/19(土) 04:39:22ID:???5行で書ける処理でもサーバー側でできるなら、そっちでやってもらってる。
>難しいシステムを無理やりWebで作るのはほんとアホとしか思えないぜ。
特に帳票とか帳票とか帳票とかな。まずデザインで死ねる。
更に、あたかも関数だらけのExcelのような動作を求められたりして死ねる。
入力項目50個超で1つ入力すると各項目を色々再計算／再描画とか言ってくるけど、
50個AutoPostback = trueな状態にするとブーブー言ってくるのは確定的に明らか。
こうなるとJavaScriptの出番になってしまい>>113みたいになって死ねる。
で、仕様変更があった時にJavaScript側で更に色々判定する必要がでてきてまた死ねる。
この経験からうちではクライアントスクリプト禁止令と、
「出来る出来ないで言えば出来ますが…は、ハッキリNoと言う」という
超基本的なことを徹底するようになった(´;ω;`)
0115nobodyさん
2009/09/19(土) 15:13:10ID:???コードビハインドで作られてるんですけど、3つのaspxが指すコードが全て同じものを指してる
んですが、これっていいんですか？まあ動けばOKという格言もありますが。
Form1.aspxとForm2.aspxとForm3.aspxが全部FormCom.aspx.csを指してます。
ちなみにFormX.aspxは3つとも微妙に違っていて、載ってるコントロールなんかも違います。
その辺はFindcontrolしてnullかどうかをちゃんとチェックしてるようなんですが。
でもまあ、通常は基底クラスに全部詰め込んで、あとは各画面に対応するクラスを継承するのが
正しい方法だと思うんですが。
0117nobodyさん
2009/09/19(土) 19:01:05ID:???0118nobodyさん
2009/09/19(土) 19:39:44ID:???単純にコード記述量を減らせる。つまり試験工数も減るし、バグも減る。いい事尽くし。
3つのパターンで画面入力させるんだけど、画面上の項目が微妙に違う。(画面上の100項目の
うち10項目ほど）無論、3パターンを1画面でまかなって、区分によって項目のVisibleを制御
するのでもいいんだけど、いっそ3画面分のaspxを用意して、裏のcsは共通にしてしまおう、
と。デザイン指定が超絶シビアなので、Visibleで出したり隠したりとかしたくなかった。
基底クラスを継承、の場合でも、例えばボタンをクリックした場合のイベントはやっぱ3画面
それぞれ必要だよね。csが1つならとことんコード量を減らせる訳で。
まあ、「コード量が少ない」と「メンテしやすい」は等価じゃないけど。
0120nobodyさん
2009/09/19(土) 20:00:26ID:???開いてる画面によってはコントロールがあったり無かったりするので、不用意に
TextBox1.Text = "ほげ〜";
とか書けなくなる。全画面共通で必ず存在しているコントロールじゃない限り、一々FindControl
でコントロールを探さなきゃならない。
デメリットってこれぐらいだと思うンすけど。
0121nobodyさん
2009/09/19(土) 21:09:11ID:???コボラ相手にしてる気分だ。
いいと思うならやればいいんじゃないンすか？
0122nobodyさん
2009/09/19(土) 21:16:18ID:???ページが最終的にコンパイルされる仕組みを理解していれば、特に何の問題も無いわけだが？
理解出来ないなら黙ってた方が無知を晒さずに済むと思われ。
0123nobodyさん
2009/09/19(土) 21:22:24ID:???コントロール名を変更しても反映されなかったりとか
不必要なイベントハンドラメソッドが増えるとか
インテリセンスが意味をなさなくなってバグの温床になるとか
0124nobodyさん
2009/09/19(土) 21:32:36ID:???ってだけの話で、やってはいけない。という理由にはならない。
でもまあ、個人的には動けば正義だと思ってる
ちなみに「不必要なイベントハンドラメソッドが増えるとか」これだけ意味不明。
0125nobodyさん
2009/09/19(土) 21:45:24ID:???ボタンの数だけイベントハンドラメソッドが増えるでしょうが。
各ページに5個のボタンがあって、それぞれ別動作してたら15個のメソッドが存在することになる。
>でもまあ、個人的には動けば正義だと思ってる
保守性が下がるからやってはいけない
他人が見てもわけわからないことになるからやってはいけない
重複させるとインスタンス時に余計なサーバ資源を消費するからやってはいけない。
インテリセンスの動作が無駄になりバグの温床になるからやってはいけない。
エラー発生時にハイライトされた行が、どのページのエラーなのか一別しか分かりにくいからやってはいけない。
ページ初期化時に表示ページとは関係無い初期化にリソースが消費されるのでやってはいけない。
>それは、そういうデメリットもあるから、メリット・デメリットを天秤にかけて考えてね。
>ってだけの話で、やってはいけない。という理由にはならない。
デメリットのほうが圧倒的に大きいから「やってはいけない」ということでしょ。
単に自分がやってることを否定されたくないから、難癖つけて認めさせたいようにしか見えない。
0126nobodyさん
2009/09/19(土) 21:46:50ID:???↓
3枚の各ページに5個のボタンがあって、それぞれ別動作してたら15個のメソッドが存在することになる。
0127nobodyさん
2009/09/19(土) 21:55:05ID:???普通、そういうケースではさすがにこんなヒネたコードは書かんだろ常考。
各画面にボタンが5個あって、ページに関係なく処理が同じ（前画面に戻るとか）
↓
15個のメソッドが必要なところを5個で済む
ていう事を言いたいんｼﾞｬﾏｲｶ？
0128121
2009/09/19(土) 22:11:27ID:???アホかw本来別にすべきものをまとめて、
何がコンパイル時には一緒になるからだ。
App_Code以下が単一dllになるからって、
1クラスに全部まとめて書くか？書かないだろ？
なぜだ？責務が異なるものは、分けるのが当たり前だからだろ？
ある画面専用の処理が追加になったらどうするんだ？
他の画面からしたら、全く関係のない処理があるクラスを実装してることになるぞ。
リファクタリングを一回でもやったことがあれば、
それがどんなにアホなことか分かるよな。
月日が経って、そのクラスを実装するaspxが増えたらどうなる？
その度にif文やFindControl判定が増えていくのか？
なんとも素晴らしい設計だな。
仕様変更時には影響範囲が特定できず、
ある画面だけの修正なのに、処理が重なっているために
全画面の動作検証を行わねばならなくなったりしないか？
つか、高凝集密結合が良くないなんて、学生でも分かるだろ？
で、業務上、そういうことにはならないように気を使ってますとでも言うのなら、
先に述べたように、お好きにどうぞってこった。
0129nobodyさん
2009/09/19(土) 22:26:02ID:???そういう意味では「注意深く作るなら、別に問題はない」が回答。
ただし「将来的なメンテとか拡張とか修正とか考えると、3画面分まとめて1ソースに
すると身動き取れなくなったりしない？止めとけば？」ってのが周りのアドバイス。
0130nobodyさん
2009/09/19(土) 23:06:31ID:???>ちなみにFormX.aspxは3つとも微妙に違っていて、載ってるコントロールなんかも違います。
って言ってるぞ。
0131nobodyさん
2009/09/20(日) 06:26:46ID:???不備を指摘されると逆ギレ
誤りを認めたくないから強弁するってガキの流行なんか？
0133nobodyさん
2009/09/20(日) 14:03:21ID:???って馬鹿の粘着ｷﾓｲ
>129 で出てる回答が全て。あとは自分で判断しろってことで終了。
0134nobodyさん
2009/09/20(日) 15:36:43ID:???技術的に問題があるかどうかなんて聞いてないよ
本人は技術的には問題ないことを理解した上で、メリットデメリットの話をしてるんだから。
技術的に問題無いことを理解している発言は>>122でしてる。（技術的に）何の問題もないと。
メリットとデメリットの話をしようとしているのは>>120を見れば分かる。デメリットうんたらかんたらと。
0135nobodyさん
2009/09/20(日) 16:07:08ID:???0136nobodyさん
2009/09/20(日) 16:48:16ID:???その3画面での共用する方法はある意味、仕組みを熟知して使い倒してますなｗ
ネイティブアプリでの共有ライブラリ、DLLの様ように。
禁止事項ではないから、開発&保守が効率的であればそれも選択肢としてアリだと思う。
0137nobodyさん
2009/09/20(日) 17:05:18ID:???よってはそういう手もあるのかと知ってちょっと感心した。
ビハインドコード共有！そういうのもあるのか
みたいなｗ
機能的に全く完全に差異がないけど、デザイン的にどうしようもない（ある仕入先と別の
仕入先で全く異なるデザインの画面）ケースなんかでは有効かも。
0138nobodyさん
2009/09/20(日) 17:09:01ID:???４：主観で決め付ける
>ネイティブアプリでの共有ライブラリ、DLLの様ように。
６：一見関係ありそうで関係ない話を始める
>禁止事項ではないから、開発&保守が効率的であればそれも選択肢としてアリだと思う。
１：事実に対して仮定を持ち出す
10：ありえない解決策を図る
12：決着した話を経緯を無視して蒸し返す
というか自演乙
0139nobodyさん
2009/09/20(日) 17:54:42ID:???ロジックを記述するパーシャルクラス(ページなんちゃら.aspx.cs)と、
コントロールなどのメンバ変数を宣言する.aspxが自動生成するパーシャルクラスの
二つが作られるわけでしょ？
後者はVSがページ毎に自動生成するからaspxと1対1になってる
コードビハインドは、そのメンバ変数を参照してる（からインテリセンスで補完してくれる）わけで
いくらpageのインスタンスを所有していて、そこからFindControlで操作したいコントロールを見つけられるとしても
メンバ変数として宣言されてるコントロールを一切使用しないなんて、
asp.net以前にオブジェクト指向の設計として間違ってるような気がするのは俺だけ？
クラスで例えれば、
メソッド内では決して参照しないまったく関係無いコントロールのインスタンスをメンバ変数として保持し、
メソッド内で操作したいコントロールのインスタンスは、すべてメソッドの引数として得て操作してるような感じ。
じゃあ、メンバ変数として所持してるインスタンスってなに？
その都度無駄にコントロールのインスタンスを生成するの？ってな感じになると思うんだ。
技術的に問題ないとか、問題なければやってもいいだろとか別次元の話だと思うんだけど。
動けば害はないし、禁止されてないからということで、1行ごとにThread.Sleepをしかけまくるみたいな。
0140nobodyさん
2009/09/20(日) 17:58:03ID:???0141139
2009/09/20(日) 18:05:49ID:???というかで始めたのがまずかったか
0142nobodyさん
2009/09/20(日) 18:13:57ID:???技術的な見地
思想的な見地
メンテや修正といった見地
で分けて議論？してくれ。じゃないと収束せんだろ。
>技術的に問題ないとか、問題なければやってもいいだろとか別次元の話だと思うんだけど。
じゃあ技術的には問題なし、思想的に不可。でいいじゃん。
0143nobodyさん
2009/09/20(日) 18:38:22ID:???いや、技術的に問題ないわけないじゃんね。
それ以前の話。
無駄なあえてわざと無駄な変数宣言をしてインスタンスを生成することは
動くけど技術以前の問題だろ？
1行ごとにSleepかませたり、ところどころ無駄な変数を宣言してインスタンスを生成したり
技術とか思想以前の問題
0144nobodyさん
2009/09/20(日) 19:12:59ID:???1行ごとにsleepかけようが、ASP.NET的には全然OK。
でもそういう実装が実際に許されるか否かは、そのアプリの目的に依存するんで可否を決
めようがない。
InProcで動いてる時にSessionに1GBのobjectを突っ込むのも、ASP.NET的には問題なし。
でもほんとにそんなことをしていいかどうかは求められてる仕様や環境次第。
技術以前の問題だっつーなら、技術以前の問題と技術的な問題に切り分けろよ。
0145nobodyさん
2009/09/20(日) 19:58:30ID:???>どうしても「俺が認めないものは認められない」つー馬鹿がいるな。
お前のことか？
>1行ごとにsleepかけようが、ASP.NET的には全然OK。
>InProcで動いてる時にSessionに1GBのobjectを突っ込むのも、ASP.NET的には問題なし。
アホかよｗ
>そのアプリの目的に依存するんで可否を決めようがない。
>ほんとにそんなことをしていいかどうかは求められてる仕様や環境次第。
ほとんど否だろ？もしくはしないほうが望ましいとされるだろうな。
自分に有利な条件を想像すんなよ。
「認められる仕様があるかもしれない」って都合の良い言い方だよな。
90-10ぐらいでほとんど認められない状況を、可否は判断できないとして
強引に50-50まで戻せるんだからｗ
なんで、そこまでしてむりくり正当化して自分の無知を認めたがらないのかね
0147nobodyさん
2009/09/20(日) 20:15:01ID:???0148nobodyさん
2009/09/20(日) 20:39:04ID:???自演の中身は文章で判断するしかない
偉い人にはそれがわからんのですよ
0149nobodyさん
2009/09/20(日) 21:09:34ID:???0152nobodyさん
2009/09/21(月) 00:39:40ID:???0154nobodyさん
2009/09/21(月) 14:04:37ID:???褒められた作りじゃないにしても、いまの所これが原因で動作がおかしくなったとかは無い。
0156nobodyさん
2009/09/21(月) 15:20:59ID:???褒められた作りじゃないが、有りといえば有り。
↓
いや無しだろ。動く動かない以前の問題だ
↓
最初に戻る
0157nobodyさん
2009/09/21(月) 16:47:03ID:???ものすごく不便に感じていたがReflection使えばよかったんだな。
http://weblogs.asp.net/ryanw/archive/2006/03/30/441529.aspx
ちょっと無理矢理な気もするが、自前で全部用意するよりはかなり楽になりそうだ。
今まで「TableAdapterつかえねー」の一念だけで、ろくに調べもしなかった自分に反省。
個人的にはこれで使わない理由はなくなった。ちょっと試してみよう。
0158nobodyさん
2009/09/21(月) 18:47:39ID:???>>157
TransactionScope使えばかけられるんじゃないの？
http://blogs.msdn.com/nakama/archive/2008/10/23/transactionscope.aspx
リフレクションは便利だけど、遅いしコンパイルのチェックが入らないから美しくない
最低減で使う分にはいいけど、メソッドの呼出とかで使いまくってる奴をみると
C#という静的言語を一体なんだと思っているのかと小一時間チクビ舐めてやる
0159nobodyさん
2009/09/21(月) 18:51:28ID:???俺も無しに一票だな
今、テレビ見てたんだが、「第二音声では英語で実況しています」というテロップが日本語で入っていた
つまり、こういうことだ
日本語でアナウンスしてしまったから、英語で聞きたい人に伝わらないけど、
いちおう第二音声で実況しているから有り
↓
いや無しだろ。英語で実況しているしていない以前の問題だ。
↓
最初に戻る←いや戻らない戻らないｗｗｗ 英語でテロップだせよｗｗ
0160157
2009/09/21(月) 18:52:15ID:???TransactionScopeは、むかーしになんかの理由で
使えないなーって判断した記憶があるが忘れたな。
もう一回調べてみる。ありがとう。
0161nobodyさん
2009/09/21(月) 19:04:48ID:???使えると便利なんだけどね。TeansactionScope。
結局なんだかんだいって、SQLサーバにすべてクエリ登録して、
アプリ側ではストアドだけ呼び出すのが正しいのかなという気がするよ。
0162nobodyさん
2009/09/21(月) 22:11:19ID:???0163nobodyさん
2009/09/22(火) 00:49:26ID:???馬鹿っぷりを曝け出してる様をみてﾆﾔﾆｱしてんの？
つーわけで
不 要 な イ ン ス タ ン ス っ て 何 ？
TextBox1,2,3があるページと、TextBox1,3,4があるページの両方が同じ分離コードをさしてる
として、片方のページを表示してるとTextBox1,2,3,4のインスタンスが出来るとでも思ってる
の？馬鹿なの？死ぬの？
0165nobodyさん
2009/09/22(火) 12:44:19ID:???> メンバ変数として宣言されてるコントロールを一切使用しないなんて、
> asp.net以前にオブジェクト指向の設計として間違ってるような気がするのは俺だけ？
逆だろ。
自動生成されたメンバ変数を使いつつ、コードを共有したいから、
共通の基底クラスを継承するのではなく、コードビハインドを共有するんだろ。
共通の基底クラスを継承する場合は、
基底クラスでは全てのコントロールをFindControlしなければならないが、
コードビハインドの共有なら、共通のコントロールに限り、メンバ変数が使える。
0166nobodyさん
2009/09/22(火) 13:11:19ID:???彼の主張する不要なインスタンスについては>>139に書いてある内容だと思う
>ロジックを記述するパーシャルクラス(ページなんちゃら.aspx.cs)と、
>コントロールなどのメンバ変数を宣言する.aspxが自動生成するパーシャルクラスの
>二つが作られるわけでしょ？
バージョンもWEBサイトかWEBアプリかも特定せずにメンバ変数を宣言するパーシャルクラスが自動生成されてるとか
パーシャルクラス（宣言のコード）なのにクラスが二つ作成されるとか
>コードビハインドは、そのメンバ変数を参照してる（からインテリセンスで補完してくれる）わけで
コードビハインドだと勝手にメンバ変数参照してるとか
メンバ変数参照してるからインテリセンスがきくとか
>メソッド内では決して参照しないまったく関係無いコントロールのインスタンスをメンバ変数として保持し、
必ず存在しているコントロールじゃない限り、一々FindControl(>120)って発言を無視してるとか
もうね、>>164のアンカーは自分に向けとけとしか
■ このスレッドは過去ログ倉庫に格納されています