포럼: Open Discussion (Thread #6620)

Designerのソースについて (2004-12-17 10:55 by gibap #12226)

Designer は Viewer に比べてあまり力を入れて作られていないのかもしれないのですが、SQL文を組み立てるのに文字列連結後の代入を繰り返していたり、変数宣言をメソッドの最初に固めたりなど、あまりにもひどいソースです。
OpenOLAP には期待していますので、このような一目見て汚いと思われてしまうようなソースは、できるだけ早く改善して欲しいと希望します。

RE: Designerのソースについて (2004-12-20 19:21 by gibap #12293)

Viewer のほうもチェックしてみましたが、文字列をequals()メソッドではなく、== 演算子で比較しているところが、7箇所ありました(NPEが発生するところも1箇所ありました。)。
FindBugs などのツールを使用すれば、このような初歩的なバグは防げるのではないでしょうか?
参考:http://findbugs.sourceforge.net/
Reply to #12226

RE: Designerのソースについて (2005-01-20 18:14 by gibap #12571)

これらの指摘に対して、反応がありませんが、ソースの問題を指摘する人は必要ないのでしょうか?何のためのオープンソースなのでしょうか?
Reply to #12293