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

[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]

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