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


■
追記：

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

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

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

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

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

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

例えばこんなやつ。

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

ここは、こうか、

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

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

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

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

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

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


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

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

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

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

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