問題タブ [code-cleanup]

VB.net と、SQL データベースへの rsData 接続の使用について質問があります。

基本的に、私の教育機関が運営するコースのコース情報を表示するインライン ページがいくつかあります。コードは SQL DB に接続し、ライブ データを次の形式で直接取得します。

今、人々が開いている DB 接続から直接プルすることを提案するか、RsData の結果を文字列にマップするかどうか疑問に思っていましたか? すべてのデータ接続は、必要な部分が完了した後に開いたり閉じたりします。ページ内で発生する約 5 つの異なる手順があります。

私は、コードが可能な限りクリーンではないことを心配しており、この継承された悪夢を本当に片付けたいと思っています. また、インライン コードと複数のデータ接続を使用して、ベスト プラクティスを捨てることはできますか?

ありがとう！

3 つの異なるプロジェクトをビルドするために、このビルド ファイルを作成しました。

これは現在機能しています。しかし、ビルドを行う前にプロジェクトをきれいにしたいです。

実際、私はこれを達成したい: C:/work/project1 ant clean build C:/work/project2 ant clean build C:/work/project3 ant clean build

前もって感謝します。

更新: Alex からの迅速な対応のおかげで、次のように新しい build.xml ファイルを作成しました。そして、うまく機能していると思いますが、どう思いますか?.

ありがとう。

これは一見単純な問題ですが、私はそれをきれいに行うのに苦労しています。私は次のようなファイルパスを持っています：

/ this / is / an / abstract / path / to / the / location / of / my / file

私が必要としているのは、上記のパスから/ of / my / fileを抽出することです。これは、私の相対パスだからです。

私がそれをすることを考えている方法は次のとおりです：

これはおそらく私の当面のニーズに応えるでしょうが、誰かがJavaで提供されたパスからサブパスを抽出するより良い方法を提案できますか？

ありがとう

私は Visual Studio を初めて使用しますが、それを使用してファイル内のメソッドの順序を変更したいと考えています。ReSharper をインストールし、コードのクリーンアップをセットアップして、1 つのこと (メソッドの並べ替え) のみを実行しました。今私の問題は、Resharper -> Tools -> Cleanup Code の下でコードのクリーンアップを選択すると、選択できないことです。クリーンアップ コードはグレー表示されています。ファイルをクリーンアップできるようにするために、ファイルに対して何かしなければならないことはありますか? コンピューターから C++ ファイルを開き、クリーンアップ コードを選択しようとしています。プロジェクトを作成してその中にファイルを入れようとしましたが、それでもうまくいきません。

助けはありますか？

ありがとう

私は自分のコーディングスタイルを改善しようとしていますが、自分のコーディングスタイルがひどいことを理解しています。私の機能を改善する方法を提案してもらえますか？

コードに問題があります。実際には動作しますが、より適切にするためにきれいにしたいです。

したがって、フロート x とフロート y を含むクラス Coord があります。

コンストラクターは次のとおりです。

その方法で必要なすべてのポイントを作成します。

いくつかの点の配列を作成する必要がある後、たとえば、5 番目の 5 つの点が配列に割り当てられ、他の 4 つの点が別の配列に割り当てられ、他の 2 つの点が別の配列に割り当てられます...

この配列は、たとえば、クラスのコンストラクターの引数として指定する必要があります。

同じ方法でいくつかの Sef を作成し、それらの配列を作成します。

クラスユニバースにはいくつかの sef が含まれます：

私がたくさんのポイント、たくさんのセフを持っているとき、それはたくさんの汚いコードであると想像できます...

どうすれば改善できますか？私のコードをより良くするために...

私たちのチームには、さらなるメンテナンスと開発のためのレガシーシステムが与えられています。これは真の「レガシー」なものであるため、テストの数は非常に少なく、ほとんどががらくたです。これはWebインターフェースを備えたアプリであるため、コンテナー管理コンポーネントと、必要に応じていちこちで「新しく」なプレーンJavaクラス（フレームワークなどに関連付けられていない）の両方があります。

このシステムで作業するとき、特定の部分に触れるたびに、すべてのものを小さな断片に分割し、依存関係を検出してリファクタリングし、依存関係をコードにプルするのではなくプッシュしようとします。

私の質問は、そのようなシステムをどのように操作するか、依存関係を破る方法、コードをよりテストしやすくする方法などです。いつ停止し、これにどのように対処するのですか？

例を示しましょう：

このコンストラクターは間違いなくやりすぎです。また、これをさらにリファクタリングするためにテストするには、PowerMockなどで魔法をかける必要があります。私が行うことは、次のように変更することです。

または、依存関係のセッターのみをコンストラクター設定ゾーンに提供します。

私が見ている問題は、コンストラクターで依存関係を提供する場合でも、どこかに依存関係を提供する必要があるということです。つまり、問題を1レベル上に移動しているだけです。すべての依存関係を配線するためのファクトリを作成できることは知っていますが、アプリのさまざまな部分に触れると、それぞれに異なるファクトリが発生します。明らかに、すべてのアプリを一度にリファクタリングして、Springなどを紹介することはできません。

セッターの公開（おそらくデフォルトの実装が提供されている）も同様であり、さらにテスト専用のコードを追加するようなものです。

だから私の質問はあなたがそれをどのように扱うかです？オブジェクト間の依存関係を一度に実行せずに、より良く、より読みやすく、テスト可能にする方法は？

テスト ソリューションに問題があります。テストは正常に実行されますが、テストの実行後にクリーニングを行うクラスとメソッドに関しては、基本的に、SQL データベース上のサイトとグループにインデックスを付けているため、sharepoint グループ、サイト、および DB インデックス エントリを削除します。メインからクリーニング メソッドを実行すると、すべてがうまくいきます。代わりに ClassCleanup からメソッドを呼び出すと、問題が発生します。

メソッドはデリゲートまで実行されます。

その時点で例外が発生します：

独自に呼び出された場合、メソッドは問題なく実行されますが、唯一の問題は、クリーンアップ メソッドが呼び出しを行うときです。

ありがとうございました。

psプロジェクトは、SP 2010を搭載したwin 2008でcpuがANYに設定されたフレームワーク3.5で実行されています。

http://msdn.microsoft.com/en-us/library/ee782531.aspxから CPU 実行モードを x64 に変更する指示に従いましたが、まだ何も実行できません。

Error loading C:\Users\Administrator\Documents\WSTests\WSTests\bin\Debug\WSTests.dll: Unable to load the test container 'C:\Users\Administrator\Documents\WSTests\WSTests\bin\Debug\WSTests.dll' or one of its dependencies. If you build your test project assembly as a 64 bit assembly, it cannot be loaded. When you build your test project assembly, select "Any CPU" for the platform. To run your tests in 64 bit mode on a 64 bit processor, you must change your test settings in the Hosts tab to run your tests in a 32 bit process. Error details: Could not load file or assembly 'file:///C:\Users\Administrator\Documents\WSTests\WSTests\bin\Debug\WSTests.dll' or one of its dependencies. An attempt was made to load a program with an incorrect format.

I have a a couple of different radio buttons which return an ethnicity and a gender. The script runs inside an internal application so rather than returning "boy", "girl" or "both" I get back 7707330, 7707333, and 7707336. Similar from the ethnicity radio button.

I then need to validate data based on the combonation of ethnicity and gender. This was a pretty simple task but I have ended up with 15 if statements! It all works as it should, but there must be a cleaner solution?

How would I go about cleaning this up?

コード内で名前を3回繰り返すと、ConfigurationPropertyAttribute本当に気になります。
スペルミスを見逃したり、プロパティをコピーして貼り付けたりして、名前の1つのインスタンスを更新するのを忘れるのは簡単です。

定数を宣言すると、これらの問題の1つだけが解決されます。もっと良い方法はありますか？

リフレクションを試しましたが、属性を列挙するのはもっと面倒で醜いようでした。

DRYは単なる原則であり、現実の世界では、原則が実用主義に取って代わらなければならないことがよくあります。しかし、誰かがよりクリーンな方法を持っていると確信していますか？