パスワードを忘れた? アカウント作成
この議論は賞味期限が切れたので、アーカイブ化されています。 新たにコメントを付けることはできません。

年収を査定してもらうためとして、SMBC証券の業務コードを公開していたことが発見される」記事へのコメント

  • この人がアップしたコードを見たところ、どうしようもないほどの糞コードだった。
    これあげて年収評価してもらうとか何を考えているんだろう。

    https://pbs.twimg.com/media/Es3-1IsVEAA7Fpz?format=png&name=small [twimg.com]
    https://pbs.twimg.com/media/Es3-1IsVEAA7Fpz?format=png&name=small [twimg.com]

    boolean にすべき変数を string にして "○" とか "×" を入れてる。
    断指有無が crossFinger とか英語も滅茶苦茶。

    1行で書けば良いコメントを何故か3行にしているし無駄な2行空け多数。
    これ1行いくらでコード書いてるから無駄に長くして水増ししているのかな。

    こんなコードを書く「プロプログラマー」がこの世に存在するとは信じられない。
    年収0、不採用が妥当。

        /**
         *  断指有無
         */
        public String crossFinger;
     
        /**
         *  入墨有無
         */
        public String tatoo;
     
        /**
         *  身上有無
         */
        public String sosialPosition;
     
        /**
         *  本籍有無
         */
        public String domicile;

    • by Anonymous Coward

      とりあえず動くコードを書いた奴が正義
      派遣ごときが後からメンテする奴のことを考える必要はない

    • by Anonymous Coward
      テストデータがクソなだけで、実際は "右小指" とか ”背中に桜吹雪模様” とか入るんじゃないかな?
      知らんけど。
      • by Anonymous Coward

        それなら納得だが、となると「断指有無」というコメントが不適切だな。
        プログラマー的には有無といったらflagであってboolだからね。

    • by Anonymous Coward

      > これのどごが?と疑問を持つ人が多いと思いますが、
      > 作ったにんげんですら、わからない事も多々あるし、
      > こんな簡単たことはないでしょう。・・・そうありたいね。
      > 今回のロシアワールドカップはベスト4にヨーロッパ勢しかいないのは、
      > まさに、「勝チームが強い」を象徴しているとおもっています。
      > ブラウザ版の艦これとしてば、1季ぶりの海域攻略イベントで、
      > しわじわ人気がでてきた、「はたらく細胞」
      http://blog.livedoor.jp/sabure0001/ [livedoor.jp]

      同一人物と思しきブログが発掘されてるが、ちょっとなんというか…
      小学生みたいな誤字の仕方、文章になってる

      • by Anonymous Coward

        なんか今見たらブログ削除されててワロタw

    • by Anonymous Coward

      全体見えないのでなんとも言えないけど、JSPで${data.crossFinger}なんて使ってるのかも。

    • by Anonymous Coward

      > 1行で書けば良いコメントを何故か3行にしているし

      おもしろおかしい

    • by Anonymous Coward

      それ名前上がってる会社の規約。
      承認貰うにはこういう書き方になるだけ。

    • by Anonymous Coward

      クソであることに異論はないがこいつが設計したのかはわからんよね

    • by Anonymous Coward
      断指有無がcrossFingerなのはまあ婉曲的な表現なのでは
      正しくは何と言うんだろうchoppedFingerとかか
    • by Anonymous Coward

      この指摘でも不十分なぐらい残念なコード

      まず、Javaなら指摘すべきは
      "なんでpublic変数なのか"じゃないでしょうか。
      明らかに属性情報っぽいのでクラスの外から参照できる必要はないと思う。
      getter/setter不要論とか一時期流行ったけどデファクトスタンダードに寄せるならクラスのprotectedかprivateにすべきかと思う。

      > 断指有無が crossFinger とか英語も滅茶苦茶。
      英語が無茶苦茶なのもあるけど、変数名としてもおかしい
      切断指を表してると思うから英語ならfinger amputationっぽい?から
      booleanにするならtrueになったときに意味が通るようにhasFingerAmputation

UNIXはただ死んだだけでなく、本当にひどい臭いを放ち始めている -- あるソフトウェアエンジニア

処理中...