他の人の「プログラム」を読んで「こいつ素人だな」と思う時

例(マジックナンバーが入ってるとか、ソースとコメントの不一致とか・・・)

回答の条件
  • 1人1回まで
  • 13歳以上
  • 登録:2018/10/25 00:30:29
  • 終了:2018/11/01 00:35:05

回答(9件)

id:kaoato No.1

kaoato回答回数155ベストアンサー獲得回数632018/10/25 12:33:52

ポイント56pt

・正常ケースしか考慮されてない書き方
・エラー時の処理がない、丁寧でない
・教科書のお手本からは、程遠い書き方



追記:

こいつ素人だな = 職業プログラマーでない

という意味で判断するのなら、
小手先では超えられない部分で判断しないと、素人かどうかはわかりにくいと思う。

id:uunfo No.2

uunfo回答回数58ベストアンサー獲得回数62018/10/25 12:48:00

ポイント56pt

インデントがおかしい
無駄な改行が多い

id:portal18 No.3

portal18回答回数84ベストアンサー獲得回数332018/10/25 13:24:14スマートフォンから投稿

ポイント56pt

おもいつくままにソースを入力していったら、いつのまにかあとからみなおすのが面倒なソースになっている。(わたし)

id:a-kuma3 No.4

a-kuma3回答回数4749ベストアンサー獲得回数20372018/10/25 15:27:18

ポイント56pt
  • ひとつの関数やメソッドがやたら長い
  • ループや条件分岐のインデントがやたら深い
  • 同じようなコードが何回も出てくる
  • インデントが、タブと空白が混じってる
  • 空白や括弧の開始位置とかの書き方が、ひとつのソースの中でバラバラ


あと、質問で挙げられているマジックナンバーで思い出したのですが、ルールを教条的に守る、というのも青いなあと思います。

例えばこんなやつ。

    response = http.get(...);
    if (response.status == CODE_404) {
        ...

ここは、こうか、

    if (response.status == 404) {
        ...

それでもマジックナンバーがダメだというなら、こうだろうと思います。

    if (response.status == NOT_FOUND) {
        ...

# プロだけど、素人くさいのにありがち :-p

id:outofjis No.5

outofjis回答回数29ベストアンサー獲得回数92018/10/25 18:06:17

ポイント56pt

どっかからのコピペで作っているせいで、
ぜんぜん必要のない余計な処理が残っているプログラム。

同様に、ものの数十行で済むような処理を
ぜんぜん不必要な、でかいライブラリを組み込んで
わけもわからずにブラックボックスとして使っているやつね。


んん、だけどこういう考え方は
古い手作り派の人間のたわごとなのかもしれないなぁ。
今は最適な処理手順とか、高速化とか、あまり考えなくていいものなぁ。

id:UME

ハードの進化がソフトの作り手をカバーしてるっていう意味だと、掃除機とか洗濯機とかが出てきた時と大して変わらない

2018/10/29 22:46:24
id:outofjis

若い者にちょっとした処理をお願いしたりすると
どこぞで拾ってきたコードを、内容もよく確認せずくっつけて、
ハイ出来ました、と来るのを
日頃苦々しく感じているオイラは、
ああ、トシヨリなんだなぁと感じる。

イヤ、それで事足りるのは解るんだけどさ。
それじゃ身につかないじゃんって・・愚痴愚痴。。

老害はここらでオイトマします。

2018/10/29 23:37:34
id:nmori No.6

morinatsu回答回数75ベストアンサー獲得回数82018/10/25 20:12:23

ポイント55pt

標準ライブラリにあるものを、わざわざ自分で書いちゃってるとか。
「勉強する時間が惜しいので、自力で書きました」的な、自己流で頑張ってる感。

id:SyN No.7

SyN回答回数3ベストアンサー獲得回数02018/10/26 15:19:14

ポイント55pt

プログラム的には大したことをやっていないのに、試したコードがコメントで残されてたり、コメントがやたら詳細だったりして試行錯誤をしたあとが見られると素人(or 初心者)なのかなと思いますね。

id:kota2009 No.8

Kota回答回数1ベストアンサー獲得回数02018/10/27 18:37:47

ポイント55pt

変数や関数名の命名規則が一貫していない時、甘いなあって思います。

id:iPlus No.9

iPlus回答回数1ベストアンサー獲得回数02018/10/30 10:56:37

ポイント55pt

ハードコーディングが多い

コメントはまだありません

この質問への反応(ブックマークコメント)

トラックバック

「あの人に答えてほしい」「この質問はあの人が答えられそう」というときに、回答リクエストを送ってみてましょう。

これ以上回答リクエストを送信することはできません。制限について

絞り込み :
はてなココの「ともだち」を表示します。
回答リクエストを送信したユーザーはいません