初心者の作ったプログラムにありがちなこと

[1 デフォルトの名無しさん 2008/03/08(土) 23:51:11]

ありそうでなかったスレ

[2 デフォルトの名無しさん 2008/03/08(土) 23:56:47]

sudo kill >>1

[3 デフォルトの名無しさん 2008/03/09(日) 00:51:15]

せめてマ板に立てればいいものを...

[4 デフォルトの名無しさん 2008/03/09(日) 01:26:12]

まじれすすると
「動かない」

[5 デフォルトの名無しさん 2008/03/09(日) 01:30:08]

ポインタ変数を宣言しただけで、そこにメモリの平原が広がっていると妄想する。

[6 デフォルトの名無しさん 2008/03/09(日) 01:37:15]

変数の型が何バイトかわかっていない。

[7 デフォルトの名無しさん 2008/03/09(日) 01:38:43]

タブインデックスが設定されてないとか、ウインドウは可変になってるのにコントロールの配置は固定になってるとか、
OK、キャンセル系のボタンでフォームを閉じる時には後始末の処理をやってるけど、×を押されると対応してないとか。

[8 デフォルトの名無しさん 2008/03/09(日) 01:48:52]

メニューバーがなく、OKボタンで終了する。

[9 デフォルトの名無しさん 2008/03/09(日) 02:03:21]

C#なのにクラスがない

[10 デフォルトの名無しさん 2008/03/09(日) 02:06:43]

publicだらけ

[11 デフォルトの名無しさん 2008/03/09(日) 02:07:24]

コピペ

[12 デフォルトの名無しさん 2008/03/09(日) 02:07:33]

staticなメソッドしかない

[13 デフォルトの名無しさん 2008/03/09(日) 02:11:06]

インデントがタブ／スペース混在。

[14 デフォルトの名無しさん 2008/03/09(日) 02:16:58]

そこそこの規模があるのにすべて一つのクラス

[15 デフォルトの名無しさん 2008/03/09(日) 02:23:01]

メモリが溢れる

>>10
なんという俺のプログラム

[16 デフォルトの名無しさん 2008/03/09(日) 03:33:56]

意味もなく個性を追求

[17 デフォルトの名無しさん 2008/03/09(日) 05:30:13]

main関数内でなんでも終わらせようとする
（慣れると、むしろそのほうが難しいよね・・・）

[18 デフォルトの名無しさん 2008/03/09(日) 08:57:21]

フラグ用グローバル変数の山

[19 デフォルトの名無しさん 2008/03/09(日) 12:00:29]

メソッドに「○○する関数」というコメントがついてる

[20 デフォルトの名無しさん 2008/03/09(日) 12:02:40]

>>19
それは別にいいんじゃね？

[21 デフォルトの名無しさん 2008/03/09(日) 13:23:24]

インラインアセンブラを使うが逆効果

[22 デフォルトの名無しさん 2008/03/09(日) 13:54:35]

>>20
functionでないものを関数と呼ぶ理由はないと思われ
普通に「○○するサブルーチン」でいいじゃん

[23 デフォルトの名無しさん 2008/03/09(日) 14:18:52]

コードを手直ししてもコメントはそのまま。

[24 デフォルトの名無しさん 2008/03/09(日) 14:29:15]

>>22
下の様に書く言語もあるから，メソッドもプロパティも関数でいいんじゃね？ｗ
class MyClass {
    function MyMethod() {
    }
    function get MyProperty() {
    }
    function set MyProperty() {
    }
}

[25 デフォルトの名無しさん 2008/03/09(日) 14:38:46]

メソッドというと脳がショートする同僚がいるので、
例えどのようなものでも関数と呼ぶようにする。
クラスは最近慣れたらしいが構造体とか列挙子がと言ったほうが通じる。

[26 デフォルトの名無しさん 2008/03/09(日) 14:39:52]

動きや使いどころが違うものに似た記述方法を用いると、
分かりやすい部分もあるが、分かりにくい部分もあるな

[27 デフォルトの名無しさん 2008/03/09(日) 14:41:55]

C++はメソッドをメンバ関数というな
オブジェクト指向性が中途半端なのが原因だが

[28 デフォルトの名無しさん 2008/03/09(日) 15:01:04]

コメントがない

[29 デフォルトの名無しさん 2008/03/09(日) 15:19:12]

えーむしろ初心者の方がコメントを打つような気が。
ただしコメントの内容がわかりづらいけど。

#もし戻り値がfalseだったらエラーにする
if not foo(baz) :
　　print "エラーです"
　　sys.exit(1)

いや、それは見たら分かるから、どういうチェックさせている端的に書けと。

[30 デフォルトの名無しさん 2008/03/09(日) 15:27:31]

簡単な処理をわざわざ難しく書く。

[31 デフォルトの名無しさん 2008/03/09(日) 22:08:48]

自分が書いたコードでも数日経つと読めない

[32 デフォルトの名無しさん 2008/03/10(月) 19:53:23]

int* null_ptr = NULL;
*null_ptr = 50;

[33 デフォルトの名無しさん 2008/03/10(月) 23:39:53]

ポインタだとわからなくなるので
全部配列にする。

[34 デフォルトの名無しさん 2008/03/11(火) 00:00:22]

初心者に毛が生えると>>22のようにどうでもいいことにこだわりだしますｗ

[35 デフォルトの名無しさん 2008/03/11(火) 02:38:23]

俺が初めて作ったシューティングゲームは1ファイル15000行

[36 デフォルトの名無しさん 2008/03/11(火) 14:33:12]

起動しようとすると「〜.dll(デバッグ用ライブラリやダイナミックリンク用メモリマネージャなど)が見つかりません」と出る。

[37 デフォルトの名無しさん 2008/03/11(火) 19:39:36]

>>33
逆に、高速化を狙いポインタを使ってわけのわからないことをする

[38 デフォルトの名無しさん 2008/03/11(火) 20:25:55]

ファイル名がうんこ

[39 デフォルトの名無しさん 2008/03/11(火) 21:04:21]

細かく分けすぎ＆雑な名前付けのせいで、何をする関数か当人でも即答できなくなる

[40 デフォルトの名無しさん 2008/03/11(火) 21:12:03]

>>37
その挙句に、却って遅い罠。

[41 デフォルトの名無しさん 2008/03/11(火) 22:03:10]

>>36
しかもそのdllがない客先でそれを起こす。

[42 デフォルトの名無しさん 2008/03/11(火) 22:04:36]

変数名をworkとかa,b,num等の名前から内容を推測できないものにし、しばらくしてから書いた本人も読めなくなる。

・・・というのは誰でも初めはやるわな。
むしろ早い段階でやっておいて、後悔したほうが勉強になっていい。


あと、インクリメント演算、デクリメント演算を使わない、for文を使わないなどもあるね。

最初はそれもいいと思うけど、とりあえず書けるようになってきたら、
頑張って使う努力をしないと何時になっても使えない気がする。

[43 デフォルトの名無しさん 2008/03/11(火) 22:43:19]

>>38
俺は今でも使うぞ

[44 デフォルトの名無しさん 2008/03/11(火) 22:48:52]

cntとか下品な言葉を使いまくりのやつとか。

[45 デフォルトの名無しさん 2008/03/11(火) 22:52:20]

それ、カントのことじゃないよ？

[46 デフォルトの名無しさん 2008/03/11(火) 22:58:17]

>>44
警察のどこが下品なの？

[47 デフォルトの名無しさん 2008/03/11(火) 23:13:07]

>>44
関東と近畿

[48 デフォルトの名無しさん 2008/03/11(火) 23:21:25]

巨大なクラスと巨大なメンバ関数が
ほとんどを占めていて大丈夫か？
と思うが、意外と何でもできる。
こいつ天才か？と、
でも
次の瞬間シヌ。

[49 デフォルトの名無しさん 2008/03/11(火) 23:24:55]

if の連続で、よくみたら
最初のif が閉じたのが数ページ後