アカウント名:
パスワード:
> 同僚の書く酷いコード、どうやって気づかせる?気づいても直らないかも。
気づかせるだけなら、スマートなコードを書いて比べさせればいいと思う。メンテナンスのしやすさ、修正が来たときの直しやすさ、良く使うならクラス化とか、そういうのは比べさせれば、とりあえず気づくと思う。一番いいのはダメージが大きくてもハッキリ言うこと。
でも直らないんじゃなかろうか。私は人並みにしかコードが書けませんが、人並み以下のコードを指摘しても直った試しがないです。
自分もそう思います。この人は1にしか対処できない人で、複数が見えないのだと思います。
1にしか対処できないので、世界をそれで対処可能な様に再定義しないと生きて行けないので、自然と再定義能力に富み、頭がいいと思われているだけかも、知れません。
あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタすればよいだけでは?
#人間の認識形態の問題になったら、なんであれ、直らないと思います。
> あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ> すればよいだけでは?
分野にもよるんだろうけど、リファクタリングの機会って多い?「動いているコードは修正するな」なことが多いので、ベースがクソコードだと、それに対する機能追加・bug fixもadhocになりがち。
テストコードの資産が十分にあればリファクタリングしても比較的安全なんだろうけど、クソコードに対するテストコードが十分とは考えにくい...
この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして引数なしの新設PROCEDUREにペーストするだけをしました。理由は、その一部分で何度も使う、そこだけでしか使わないカーソルを定義したかったからです。
(引数で引き回したり、変数名の変更をしたりの)大げさなリファクタリングでないやつでも、それなりに役立つと思います。
あと、
自分が、「そのソースの全面的権限を得た時に」と言ったのは、「その位してもOKな改修時」を想定していたので、もちろん、なんの件名も無く修正するのは絶対禁忌だと思います。
そういう変更って、コード書いている人がどういいう意図でそう記述しているかを確認しないと怖くないですか?
貴方の知らない技術的な意図があるのかもしれないし、意味の無いことであればそれを気づかせる必要がある。
>この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして>引数なしの新設PROCEDUREにペーストするだけをしました。
の様な修正方法なら、「現行通り」だとおもいます。「現行通り」なら「知らない技術的な意図」が有っても問題ないと思います。
#そうでない変更なら、たしかに怖いです。
>>この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして>>引数なしの新設PROCEDUREにペーストするだけをしました。
>の様な修正方法なら、「現行通り」だとおもいます。>「現行通り」なら「知らない技術的な意図」が有っても>問題ないと思います。
いや、Oracleの場合はたったそれだけでなにか変な現象が発生することがあるの。出会ったことない?
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
海軍に入るくらいなら海賊になった方がいい -- Steven Paul Jobs
気づいても直らないかも。 (スコア:5, すばらしい洞察)
> 同僚の書く酷いコード、どうやって気づかせる?
気づいても直らないかも。
気づかせるだけなら、スマートなコードを書いて比べさせればいいと思う。
メンテナンスのしやすさ、修正が来たときの直しやすさ、良く使うならクラス化とか、
そういうのは比べさせれば、とりあえず気づくと思う。
一番いいのはダメージが大きくてもハッキリ言うこと。
でも直らないんじゃなかろうか。
私は人並みにしかコードが書けませんが、人並み以下のコードを指摘しても直った試しがないです。
Re: (スコア:0)
自分もそう思います。この人は1にしか対処できない人で、複数が見えないのだと
思います。
1にしか対処できないので、世界をそれで対処可能な様に再定義しないと生きて
行けないので、自然と再定義能力に富み、頭がいいと思われているだけかも、
知れません。
あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ
すればよいだけでは?
#人間の認識形態の問題になったら、なんであれ、直らないと思います。
Re: (スコア:0)
> あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ
> すればよいだけでは?
分野にもよるんだろうけど、リファクタリングの機会って多い?
「動いているコードは修正するな」なことが多いので、ベースがクソコードだと、
それに対する機能追加・bug fixもadhocになりがち。
テストコードの資産が十分にあればリファクタリングしても比較的安全なんだろうけど、
クソコードに対するテストコードが十分とは考えにくい...
Re: (スコア:0)
この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして
引数なしの新設PROCEDUREにペーストするだけをしました。
理由は、その一部分で何度も使う、そこだけでしか使わないカーソルを
定義したかったからです。
(引数で引き回したり、変数名の変更をしたりの)大げさなリファクタリングで
ないやつでも、それなりに役立つと思います。
あと、
自分が、「そのソースの全面的権限を得た時に」と言ったのは、「その位
してもOKな改修時」を想定していたので、
もちろん、なんの件名も無く修正するのは絶対禁忌だと思います。
Re: (スコア:0)
そういう変更って、コード書いている人が
どういいう意図でそう記述しているかを確認しないと怖くないですか?
貴方の知らない技術的な意図があるのかもしれないし、
意味の無いことであればそれを気づかせる必要がある。
Re:気づいても直らないかも。 (スコア:0)
>この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして
>引数なしの新設PROCEDUREにペーストするだけをしました。
の様な修正方法なら、「現行通り」だとおもいます。
「現行通り」なら「知らない技術的な意図」が有っても
問題ないと思います。
#そうでない変更なら、たしかに怖いです。
Re: (スコア:0)
>>この前、PL/SQLで、長いトップレベルBEGIN〜END;の一部をカットして
>>引数なしの新設PROCEDUREにペーストするだけをしました。
>の様な修正方法なら、「現行通り」だとおもいます。
>「現行通り」なら「知らない技術的な意図」が有っても
>問題ないと思います。
いや、Oracleの場合はたったそれだけでなにか変な現象が発生することがあるの。出会ったことない?