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

ありそうでなかったスレ

sudo kill >>1

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

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

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

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

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

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

C#なのにクラスがない

publicだらけ

コピペ

staticなメソッドしかない

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

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

メモリが溢れる

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

意味もなく個性を追求

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

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

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

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

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

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

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

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

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

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

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

コメントがない

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

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

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

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

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

int* null_ptr = NULL;
*null_ptr = 50;

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

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

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

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

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

ファイル名がうんこ

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

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

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

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

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


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

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

>>38
俺は今でも使うぞ

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

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

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

>>44
関東と近畿

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

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

確かにありそうでなかったｗ

これ、何故動かないのか悩みに悩んだソース。

10 A="PRINT"
20 A "HELLO"

run
Syntax Error in 20
■

ほんとのほんとに初心者だったからなぁ。

>>50
プログラマの素質がありません

ド・モルガンの法則を知らない

うちの会社のソースが、殆ど当てはまるなぁ
>>49とか
おわっとる

>>52
それ自体は知らなくてもしばらくやってたら自分で発見するだろ、普通

と思ってた時期が俺にもありました・・・・

集合論は数学Ａの修学過程に包含されて高校１年後期〜２年前期で習うわけだが…。

ワード単位でごっそりMOVするくせにTSTやTSTNでせこくビット展開した挙げ句、
XFERやXFRBを駆使して点在した値を再度ワードアドレスに転送したりする。
よく分からなくなったら立ち上がり微分のb接点で自己保持を切ったり、強引にRSTかましたりする。

>>56
電気屋乙

not (2 < a and a < 8)
のとき
2 >= a or a >= 8
と同値ってすぐ判るよな？

多くの言語では違うと思う

それはないｗ

>>58
お前がすぐ判らないのは判った

演算子の優先順位に依存します

>>59-62はおいといて、

2 >= a or a >= 8
ではなく
a <= 2 or 8 <= a
って書いてほしいんだけど！！！！

C風にいうと範囲を表現するときは
2 < Val && Val < 10　　　　　　　　　　2 < Val AND Val < 10
とかの表記のほうが
2 < Val < 10 　　　　という意味を視覚的に表現できてると思う

だから　　　if ( ! (2 < X && X < 10)　　　　　IF ( NOT (2 < Val AND Val < 10)
とか冗長な書き方するのは別にいいと思う
「視認性の為の冗長さをかっこわるいと思う」ってのも初心者にありがちじゃないかい

初心者に毛が生えると>>58のように凡ミスをします

>>64
それはandの場合ならね

>>64
無駄をかっこわるいと思うのは初心者じゃなく中級者じゃね

>>62
カッコつければいいだけじゃないかっ
優先順位的に必要なくてもカッコつけたほうが処理の括りとしてわかりやすい場合もあるしね

LD CF113
ANDNOT 0.00
@MOV #0 D0
LD =< #2 D0
AND =< D0 #8
OUT 0.01
++B D0
LD < D0 #9
OUT 0.00 END

―動作説明―
・毎スキャンに渡りブールデバイス0.00がOFFの時、ワードデバイスD0に１スキャンのみ#0を転送する。
・毎スキャンに渡りD0が&2〜8である時、ブールデバイス0.01をONする。
・毎スキャンに渡りワードデバイスD0をインクリメントする。
・毎スキャンに渡りワードデバイスD0が&9より大きい時、ブールデバイス0.00をONする。
・I/Oリフレッシュを行う。

>>50
ｲｷﾛ

最近の言語では
a = print
a("hello")
が普通に動いたりするし、ある意味時代を先取りじゃね？ｗ

>>71

#include <stdio.h>
main(){
int (*a)(char*, ...);
a=printf;
a("hello");
}

C++0xだとさらに

auto a = printf;
a("hello¥n");

Java でクラス一つ、メソッド多数、main() の中で手続き型構造化プログラミング。

>>73
お前いい事言うな。
出来るか？
なら今すぐやれ。

gccならできるだろ

>>73
それって型推論でもやってるの？
auto pi = 3.14;
とかもあり？
autoってauto intの略だったような記憶が。

>>77
型推論用にautoキーワードができた。
その例だと、環境にもよるけどpiはdoubleになる。

サンクス
そりゃ便利だな。g++にはfinallyもあるし結構ツボな拡張が多いよね。
gccはほとんど使ったことないんだけどｗ

>>72
#define a printf
main () {
a( "hello" );
}
こっちのが近い感じかも。

ちなみに当時小学1年か2年。子供の発想力って凄いわ。

>>79
g++というか時期C++がそうなる予定。
g++がTechnical Reportの一部をお試し実装しているということ。

C++0xでググれ

でもfinallyは現状でもかなりの処理系で実装されてるよね

ちなみにfinallyは時期C++でも非標準だからね。

init();
try{


}finally{
　done();
}
って距離が遠すぎるんだよな。

begin{
　init();
} end {
　done();
} main {


}
にしてくれ。

距離が遠すぎると感じるくらい詰め込みすぎのプログラムは初心者にありがちですね

ボトムズに乗れば勝機はある

わざわざスペース入れるの面倒臭い
詰め詰めで書いて後で整形ツールで一括整形するわ
というか構文木で保存しておいて編集するときは整形ルールを適用して表示するようにすりゃいいのだ
XMLとスキーマみたいにさ

>>88
>構文木で保存しておいて
最初からLispで書けよ

VB

foo.bar.baz.hoge.fuga1.Text = 値
foo.bar.baz.hoge.fuga2.Text = 値

…

foo.bar.baz.hoge.fuga22.Text = 値
foo.bar.baz.hoge.fuga23.Text = 値

>>90
Delphi だと

with foo.bar.baz.hoge do begin
　fuga1.Text = XXX;
　fuga2.Text = XXX;
　...
　fuga23.Text = XXX;
end;

だな　なつかしい

>91
いや、VBにもほぼ同じ機能の With があるんだよ
なのに >90 みたいなコード書く人が居るからスレタイ通りなワケで…

まぁ、withを使うのは下策だとする書籍もあるがな。曰く、検索したときにヒットしにくいとか。
あれは誰の書いた本だったか。物書きとしては兎も角、技術者としては初心者レベルだったのだろう。

どっちかっつーとwith云々よりも
fuga1〜fuga23を一々書いてる辺りが問題じゃね？
配列的な何かが使えないのか

配列を使わないでText1、Text2…
って変数作っちゃったことはあるな。そう言えば。

MFC使ってるとブトゥンウィンドウひとつごとに
CButton button1;
CButton button2;
CButton button3;
ってデータメンバ作っちゃうことなんて普通。プチスール。

>>96
クラスウィザードないと何にもできない奴とか、いるからなー。
オブジェクトがいくつあろうと配列にしてないから、
メッセージマップもひとつづつ指定しやがる。

VB6とかなら


dim a as variant

a = "123"

dim i as long

i = a

こんな感じの暗黙キャストしまくりとかあったな…

こんな酷いの専門の時よく見たなぁ

void func(int flg,int flg2)
{
if(grbFlg == OK)
if(flg == 1)
{
if(flg2 == 1)
{
処理1
} else{
処理2
}
}else if(flg == 2)
if(flg2 == 1)
{
処理1
}else{
エラー処理
}
}
}
printf("%d %d %d",grbFlg,flg1,flg2);
}

ファイル分けで躓く
とりあえず動かせるようにと、ファイル間共有する変数は全てextern
書きなおすより作りなおしたほうが早いことに気付く

自分はそうだった……

俺は昔extern知らなくてファイルが肥大傾向にあった

externは最初何でもかんでも1つのファイルに詰め込んでた時は
ワケがわからなかったが、ファイル分けをするようになってこりゃいいやと思った。
で、余計なのまで指定して動かない。

ポインタも理解できたと思い込んでこりゃすげーかっこいい！と思って
ポインタ使いまくったコードを組んだが動かない。

未だにそんなことの繰り返しですわ。

constを使わない

#define VALUE_100 100

妙にハイテンションで意味不明なコメントがある。

過去にローカルネタ書いた折れはアホ顔で逝ってよし。

調べればすぐに見つかる有名で簡単なアルゴリズムとか

たとえばソートアルゴリズムとか

自前で考えて作ってる

>>105
デスマが終わりそうな気配になるとよくやりますよね

if (flag==ture) {
　hoge();
}

一行だろうとカッコでくくる

不用意に flag == true と書く
不用意に {} を省略する
スペルミスが多い（true と ture と書くなど）

trueでは無い。tureという名の変数だ

ﾜﾛﾀｗ

booleanでない変数にflagという名前をつける。

# この前見たflag変数は構造体だった……

>>106
それがバグってるからさらに困るｗｗｗｗｗ

>>108
IDEがかってにカッコを入力してくれるんだから問題ない

うちのコーディング規約だと一行野郎以外は括ることになっているんだよね。
つまり、
if (flag) hoge();
でなければ
if (flag) {
hoge();
}
とか
if (flag)
{
hoge();
}

プログラムに関わらず物作りの初心者は模倣から入るのが基本なのに
全部自身のオリジナルで構築しようとする

そうやってダメな横並び社会になるんだなｗ

どうせみんな、模倣から入ってパクりで終わる。

三流会社の基本(笑)

エラーが出るとコンパイラのバグを疑う。

printfをはさむとなぜかバグらなくなる

ブレークポイントをやたらに設定する
関数の開始位置全部とか

玄人はデバッグなんかしないんですよね。たしかデバッグ専門の人に丸投げするんですよね。

int a,b,c,d,e,f, ......., x,y,z,aa,ab,ac,...............;
void main(){

以下コーディングスタイルの話は禁止

変数名や関数名は自由に決められないと思っている

ローマ字読みで関数名や変数名を付けるのが格好悪いと思って
無理にでも英語にする

それ俺だ

>>127
あぁ、それ未だにやるわ
コメント書いときゃIDEが説明表示してくれるから大丈夫だよね なんて言い訳しながら

むしろ初心者の方が、日本語をローマ字で書いたような変数名を平気でつける印象があるが。

業務固有の用語とか英訳できないんでローマ字。

酷いのは独学プログラマー

横並び共同体

誰かに指導してもらったとかじゃなくて
独学はソースが汚すぎ

独学でもちゃんと「名著」って言われる書籍を何冊か読んでる人なら問題ないけど
ほんっとうにその辺の（大学教授ですらない日本人が書いた）クソ本をさらっと読んだだけの人とかは
まじで大弱り。

>独学でもちゃんと「名著」って言われる書籍を何冊か読んでる人なら問題ないけど

K&Rの初版読んでスタイルが未だにそのままの人ですね、わかります。

わけのわかんない俺流よりは、いくらかマシだがな。

プログラミングは技能であるから本質的に独学以外に有り得ない。

・すぐ「本質的」という言葉を使いだす。

横並び共用体って何だと思ったじゃないか

初心者はすぐに運命共用体になりたがる。
中2の美少女プログラマが俺と共用体になりたいって言ってきたから俺は

何そのエロゲ

K&Rの初版本はC89やC99の本じゃないので、
それを読んでC89やC99を使うのは無理。
C89を使うならK&Rの第2版を読んでください。

で、C89を使うのにK&Rの第2版すらも読んでないような人は
独学以下なのでしんでください。

老害じゃー

>>135
「新人を教育するならなにも知らないやつがいい。
独学でかじったやつは変なクセがついている」

↑みたいなことを言うやつを見かけることがあるけど、

「自分で本とかネットの記事とか読んだりしません。
職場で先輩の書いたコードしかみません。先輩直伝のテクニックはすばらしいです」

↑みたいなやつばっかりの職場のほうが、ガラパゴス化して、よっぽどクセが強くなってるって
いうか、すごいことになってるな。

先輩や研修で教わったことから
もっと積極的に学ぶ姿勢みせて
たくさん覚えていくのならいいが

教えてもらうまで待つみたいな
指示待ち初心者が使えないの

入社前に独学で学んでた奴は
変な癖とかがあった場合に困る
長年染み付いた癖は中々落ちん
１人変なプログラム組んで全部
プロジェクトダメにしたりするから

明らかに変なコードなのだが手を入れる余裕がなくて
そのまま無理矢理な修正するから余計に訳がわからなくなっていく

なにも知らない新人がいい、って、とにかくプロジェクトに
突っ込んで頭数になれば稼げる時代の遺物じゃね？

何か嫌なことでもあったのだろう。そっとしておいてやれ

STLアルゴリズムを使ったコードを変なコードだと思ってたらやだなあ

関数を自分で創るという概念が理解できない

ある機能（俺の場合、文字列を逆順にするメソッド）がほしかったけど
その関数が見つけられなくって
自分で関数作ってそれをずっと使ってた
数年後に気づいたが、標準でその関数が備わってたことに気づいたとき

>>148
このネタふると、必死に反論してくるやつが必ずいるから、そう思ってるやつ多いんじゃね?
自分のダメさかげんを、新人がダメだってことにして、合理化してるんだろ。

だいたいなにも知らないやつのほうが素直なコードを書くとか、我流でやると
クセがついて直らないとか、スポーツのアナロジーで捕らえてるのも頭悪いし。

経験不足で間違うのは未熟者の浅薄
経験則で間違うのは成功者の浅薄
お互いに信じる正義をかけた血みどろの戦いになるに決まってる

初期化と代入が別のものであることを知らない

おまえらコードレビューとかしないの？
プロジェクトのコード規約に準拠してない変な癖なんて速攻で突き返すよ

コーディング規約のチェック目的のレビューって、経験無いな。
規約から外れてるのは、レビューの時に指摘する事はあるけど、
レビューに持ってくる前にチェック済ませとけだな。

ところで、規約から外れてるのと、変な癖とは別物だとおもうが。
どっかのコーディング規約のせいで変な癖がついてるやつもいる。

＞レビューに持ってくる前にチェック済ませとけだな
それ突き返すってことじゃねｗ

変な癖ってのの例が1つも出てないからこの話はよくわからん

>>157
>それ突き返すってことじゃねｗ
そう言われれば、たしかにそうｗ

http://bbs.wankuma.com/
にアクセス禁止になりました。
質問しているのに嫌な言い方をした人が
いてそれについて反論しただけです。
記事削除ならともかく酷いサイトです。
なんとか管理者にいってやりたいです。
satoという名前で投稿してました。
相手はだかぼんという名前です。

http://bbs.wankuma.com/index.cgi?mode=one&namber=42860&type=42847&space=45

>たくぼん、金もらえるとかは関係ないんじゃないか
>答える気ないなら書き込みするなや

>>159
アクセス禁止おめでとう！

お前が自ら招いた事態のようだが。
まぁ、お前がプログラミング向いてないのは確か。

>>159
ページのソースを調べたところ、
管理者と連絡をとる場合は、次のページを使えばいいらしいです。

http://blogs.wankuma.com/jeanne/contact.aspx

>162
プログラミングは初心者じゃないし、
VB.NETでは1本1000万するソフト作ったんですけどね。
向いてないですね、ありがとう。なんせASP.NETは初めて触るもんで。

>>159
自業自得。アク禁で済んでよかったね。

ありがとう。もう質問掲示板は利用しません

>>自分でどこまで調べて、どこまで理解できてて、どこがわからないかくらい書いてないとねぇ。
正論だな。

>>仕事変えた方がいいんじゃない？こんなんでお金もらえるんだからすごいよねぇ(笑)
>質問者のレベルも人それぞれやろうが。
>なんの為の質問掲示板や。
>カスが。
程度の低い釣りに釣られた結果がこれだよ！ まぁ、俺が管理者なら両方アク禁にするけど

>>164
1本1000万のソフト作った なんてこの件とは全く関係ない。
ていうか、そんだけ経験があるなら質問するより自分で調べる方が圧倒的に早いだろｗｗｗ

1本1000万円なんてはした金クラスじゃないか。
中小企業向けの販売管理ソフト作っても
最低3000万円くらいはかかるのに。

＞VB.NETでは1本1000万するソフト作ったんですけどね。
よっぽど相手が馬鹿だったのか、一部のみを作ったのか、只の妄想だろ。

＞ありがとう。もう質問掲示板は利用しません
そうしてくれ。その方が余程平和だ。

中堅どころのSIerで人月単価100万円として、最低30人月?
こりゃまたおめでたい。

平日の4時に書き込む皆様は本当に単価に詳しいですね＾＾

この後、「仕事中の息抜きだ」「今日は非番」「夜勤」「有給」「祝日」など、
あられもない言い訳が続きます↓

学校の創立記念日なんで

>>171-172
君はなんで居るの？

平日の4時は学生はお休みタイムですよ？

だから、平日の勤務時間帯に書き込むときは
無職ニートがばれないように注意しろって言ったじゃないですか！

この後、「仕事中の息抜きだ」「今日は非番」「夜勤」「有給」「祝日」など、
あられもない言い訳が続きます↓

ワロタｗ自業自得だろ

>>168 一人で1000万でしたよ。三ヶ月かかりました。システム全体の一機能で
もあります。
仕事に戻ります。では

個人事業主なのかな？

へぇ
こんな簡単に金が稼げるのか
プログラミング勉強しててよかった

大丈夫、在り得ないから。

大丈夫、皮肉だから。

25年前ならたまにあった話だな。
ただ、本人が1000万受け取ったかどうかは別だ。

もしかしたら1000万で魂売ったのかもしれないけど。
タイーホのリスクと引き換えに。

VBだし、値段で自慢してるし、一発屋みたいだし、
やっぱりプログラミング初心者だな。

>>184
会社員ですよ。一発屋??現役ですけど
プログラム難しいから任せたと言われて作る事に。
値段でいったら解りやすいでしょ。
プログラミングの才能ないやつに任せる仕事じゃないの
解る?ソフトの設計もしたしね。

わかったから、もういいよ＾＾
十分戦った。
もう、休んでいいんだよ＾＾

自分を大きく見せることに必死だなぁ

例え真実でも意味ないよ。もう書かないほうが宜しいかと。

>>159
つ「恥の上塗り」

でも、いくら回答者も立場が上とは言え、あの回答はないな。言うにしても別の言い方があったはず。質問者も減るかもね。

>ソフトの設計もしたしね。
設計した事が嬉しい初心者君なのでした。

プログラミング自体は難しくない。
難しいのはプログラムが扱う論理や事象の方なんだよ。

おまいら、ちゃんと褒めるべき所は褒めなくちゃだろ。
初心者が３ヶ月で書いた物を１０００万で売ってくる営業もすごい。
１０００万で売れるネタを初心者が３ヶ月で書ける仕様に落としたSEもすごい。

>>190
でも >>159 みたいなのは減った方がいいかもね。

＞＞193 利便性が減るね。敷居を高くしてどうするの？なら上級者専用にすればいいじゃない。
簡単なものは他いけって話でしょ。質問掲示板て名前もおかしいんだが。こうやって突っぱねてたら初心者は減るだろうね。
誰か優しい人が答えてくれてたみたいだが。どう解決済みにチェックいれるんだろね。

上級とか初心者とかいう問題じゃない。

>この人ってちょっとは自分で調べたりしてるのかな？
>自分でどこまで調べて、どこまで理解できてて、どこがわからないかくらい書いてないとねぇ。

この台詞に尽きる。

はぁ？　３ケ月あれば脱初心者だろ　初心者ヅラなんかできるのはせいぜい２週間まで

でも、いつまでたっても初心者から脱出できないやつもいるがね。

確かににその台詞だけやったらなにもおこらんかったね〜。後ろに続く言葉がなければの話。

ここ何のスレ？

マ板からの出張スレ

叩きすぎｗ。
どんだけストレス溜まってんだよｗ

平日の営業時間に書き込んでいる人たちはみんな無職だから＾＾

それしか言い返せなくなったんですね。

会社から書き込んでる給料泥棒という線も考えられるな

いいなぁ給料泥棒、がんばってるフリだけしてれば給料貰えちゃうんだろ。
自営業者としてはうらやましい限り。

VBなんかに1000万も払うか？
３年後には言語仕様変わってソースがゴミになるんだぜ

1000万円ならユーザーのところに直接納入したってことじゃろ？
それならソースをくれてやるいわれはないので
ソースが古くなろうと関係ない。
どうせ検収が終わったら、あとはしったこっちゃねーんだし。

数日で一気に糞スレ化したなぁ

マ板でやれ

>>209
>>199-200

マ板でやれ

> 敷居を高く

敷居＝ハードルとでもおもってるんじゃないの？

出入りしてた人が行きにくくなる話だから、使い方あってるんじゃね？
>>194の内容には同意しないけど。

アク禁になるような人の場合、技術が未熟なんじゃなくてマナーが未熟なんだよな。
敷居が上がったとしたら、マナーの敷居が上がっただけだろう。

>>214
マナー以前に人間性の問題。
このような人は社会的に適性がないから生き続けていると
かえって不幸になる。

まぁなぁ

【審議拒否】
　　　 　 ∧,,∧　　∧,,∧
　∧∧ (・｀　　)　(　　´・) ∧∧
(ω・｀ )　Ｕ　　)　 ( 　Ｕﾉ(　´・ω)
｜　Ｕ 　　u-ｕ　 　u-ｕ　(Ｕ　 ﾉ
　ｕ-ｕ ∧,,∧　　∧,,∧　　u-ｕ
　　　 (・ω・｀)　(´・ω・)
　　　 （l 　Ｕ) 　(Ｕ　　ﾉ
　　　 `ｕ-ｕ'.　　`ｕ-u'

変数を初期化していない

自作関数が尽くvoid型
引数は大体が自作構造体のアドレス

ライブラリにある機能(関数)を自前で実装して後で気づく

>>219
後者はそんなに問題か？
むしろ良く使うだろ

グローバル変数使うよりよっぽどいいよ

for(i=1; i<=3; i++) {
    if(i==1) {
        // i が1のときの処理
    } else if(i==2) {
        // i が2のときの処理
    } else if(i==3) {
        // i が3のときの処理
    }
}

そんなことしなくても
実行時に条件によりimportするモジュールを切り替えたいなら
if 条件: import foo as spam
else: import bar as spam
で充分だと思いますけど。

>>224
スレタイ嫁

GOSUBをGOTOで戻してスタック崩壊

10 'MAIN LOOP
20 GOSUB *SUB1
20 GOTO 10
90 END
100 *SUB1
150 GOTO 20

同じ行番号の行が上書きされてて期待通りに動かないとか

プログラマの麻疹
ttp://d.hatena.ne.jp/Yoshiori/20090706/1246849833

このエントリあるあるｗと読みつつ自分も通った道だなぁなどと思いつつ。

A.陰性(種が異なるため感染の可能性なし)
B.ちょっと触ったくらい未感染
C.罹患治癒済
D.当然罹ってるがビフィズス菌並みに善玉なので実害なし

OO 厨 C
function 厨 B
三項演算子厨 D
テスト厨 A
lambda厨，クロージャ厨 D
インターフェース厨 B
ワンライナ厨 A
演算子オーバーロードを使いすぎるとか B
mixin使いすぎるとか B
リファクタアリング厨 D
パターン厨 デザパタ厨 A
メタプログラミング厨 D
自動化厨 D
エディタの設定厨 C
jsで名前空間汚さない厨 D
低レイア厨 D
正規表現厨 C

覚えたのが嬉しくてトライグラフでコーディング

|￣:／ : : ／: : : : : : : :ｉ: : : ｉ: :＼ : :＼ 　 　＼
|: / : : : / : : : : :.ｉ: : : : |: : : :|: : : |: : : ｉ　　　 　'
: :|: : : : |: : ｉ: : : :|: : : /ｉ|: : :|:| : : :|: : : |　　　 　|
: :|: : : : |: :.|:斗 十‐/　ｉ|: :∧-‐ト|: : : |　　　　 |
: :|: : : : |イ|: :|:/ |:/　　 |:/　|ﾊ: : |: : : |　 ＼_,ノ
: :|: : : : |　ﾚヤ示ﾄ 　　 　ｒﾃｖ|:./|: : : :|
ヽ|: : : : |/{:::::::ｉ }　　　　　ﾄｲ ﾚ: :|／|/　　　関東地方
（ |: : : : 代ｃ::ﾉｿ　　　　　ﾚﾘ |: : : : |　　　　　　　東北地方　北海道地方
.〈.|: : : : |ｘ｀¨　　　　　　 ､¨ｘ.|: : : ｉ:|　　　　関西地方　九州地方
＼|ｉ: : : :|　　　　　　　 ,　　 /|: : : ｉ:|　　　　　　沖縄地方　にいる
∨ﾘ＼: :ｉﾄ ､ 　 　 　｀´　ィ　 |: : /|ﾘ　　　　　すべての　愛すべき　国民へ　告ぐ
‐┴―＼|- 〈 ＞ｒｧ 爪|Ｖ　　|／
:::::::::::::::::::::::ヽ ＼　ﾘ
::::::::::::::::::::::::＿＿＼
:::::::::::::::／ ::ｒ――‐へ　　　　 　ｒっｒ‐-､_
:::::::::／ ::／/　 　　　 ∧. |￣￣￣| | | | |￣￣￣￣￣￣￣|
:::／ ::／ / 　 　 　 　 　ヽ|.　　　　L|_|_ﾄ'　　　　　　　　　　 |
/ ::::/　 /　　　　　　　　　＼　　ただちに　にげろ　　　　 |
. :::/　|/　　　　　　　　　　　|　　　　　　　　　　　　　　　　　|...

ただちににげろのガイドライン
http://www.geocities.jp/ust7800870/index.html

カ　オ　ス　ラ　ウ　ン　ジ　ゆ　る　せ　な　ぁ　い　ー

boolean hoge(int a)
{
  if (a == 0) return true;
  else return false;
}

boolean hoge(int a)
{
  int val = a / 2;
  if (val != Math.round((double) a / 2))
    return true;
  else
    return false;
}

void hoge(String s)
{
  for (byte b : s.getBytes("Shift_JIS"))
    hage(b);
}

void hoge(String s)
{
  for (int i = 0; i < s.getBytes("Shift_JIS").length; i++)
    hage(s.getBytes("Shift_JIS")[i]);
}

StringBuilder sb = new StringBuilder("<html><body>");
sb.append("x=" + x + ", y=" + y + ", z=" + z);
sb.append("</body></html>");
System.out.println(sb.toString());

自分のPCを爆発させないか心配になる

if
if
if
if

初心者にありがちなのは綺麗さにこだわりすぎてやたら無駄に長い関数名変数名だよ
そうJAVAのように

-- qty 1〜5 を 5〜1 に付け替える処理
UPDATE `member` SET `qty`=5 WHERE `qty`=1;
UPDATE `member` SET `qty`=4 WHERE `qty`=2;
UPDATE `member` SET `qty`=3 WHERE `qty`=3;
UPDATE `member` SET `qty`=2 WHERE `qty`=4;
UPDATE `member` SET `qty`=1 WHERE `qty`=5;

javaでハッシュコードをハッシュキーに使ってしまったヤツがいて、
ダメだと言っても、なぜダメなのか納得させるのにずいぶん苦労した。
およそこんなコードなんだけど

HashMap<Integer, String> data = new HashMap<>();

void addData(String name, String value)
{
data.put(name.hashCode(), value);
}

そいついわく
「検索キーは文字列より整数の方が比較するのが高速だし、メモリー消費も少なくて済むよ」
「なんでハッシュをハッシュに使っちゃいかんの？そのためのハッシュなんでしょ？」

濃い血痔間かけて説明して、やっと折れてくれたが、たぶん理解して無いっぽい。

異なる文字列のハッシュが衝突することを示せばいいんじゃない
http://ideone.com/Wl0a4

switch (id)
{
case 1:
  hoge();
  break;
case 2:
case 3:
  if (id == 2) fuga();
  else gero();
  break;
}

static final int ITEM_A = 0;
static final int ITEM_B = 1;
static final int ITEM_C = 2;
static final int ITEM_D = 3;

if (item == ITEM_D) {
  hoge();
} else {
  if (item > 0) {
    if (item == ITEM_B)
        fuga();
    else
        gero();
  }
}

1 名前：デフォルトの名無しさん 投稿日：2012/10/09(火) 22:35:35.65
変数をつくりまくる

3 名前：デフォルトの名無しさん 投稿日：2012/10/09(火) 22:48:20.82
自作関数が全部void型

4 名前：デフォルトの名無しさん 投稿日：2012/10/09(火) 22:50:35.24
とりあえずdouble型

5 名前：C言語の名無しさん 投稿日：2012/10/10(水) 22:10:26.13
変数名を　kazu とかにする

6 名前：デフォルトの名無しさん 投稿日：2012/10/10(水) 23:30:01.91
>>5
あるある

7 名前：デフォルトの名無しさん 投稿日：2012/10/11(木) 00:02:34.01
日本人なんだから分かりやすい変数名にしたくなる…

しまいには言語がサポートするなら変数名にマルチバイト文字使う人も出てきそう

8 名前：デフォルトの名無しさん 投稿日：2012/10/11(木) 07:08:34.01
怒られたことがあるのは…
Cで
グローバル変数を使用した。
ポインタの演算が気楽すぎた。p = p + N (N = 1なら、たいていの場合は、いいと思われる)
同じ部分を関数にしろといわれたなぁ（そのとき、不満だった。
たまたま、同じに見えるのであって、むしろ同じであることを見せたかったのだ）
http://toro.2ch.net/test/read.cgi/tech/1349789735/ 初心者が作るプログラムにありがちなこと
「datが存在しません。削除されたかURL間違ってますよ」

static final int FLAG_A = 0;
static final int FLAG_B = 1;
static final int FLAG_C = 2;
static final int FLAG_D = 4;
static final int FLAG_E = 8;
static final int FLAG_F = 16;
static final int FLAG_G = 32;
static final int FLAG_H = 64;
中略
static final int FLAG_U = 524288;
static final int FLAG_V = 1048576;
static final int FLAG_W = 2097152;
static final int FLAG_X = 4194304;
static final int FLAG_Y = 8388608;
static final int FLAG_Z = 16777216;

if ((a & FLAG_A) == FLAG_A) hogeA();
else if ((a & FLAG_B) == FLAG_B) hogeB();
else if ((a & FLAG_C) == FLAG_C) hogeC();
else if ((a & FLAG_D) == FLAG_D) hogeD();
else if ((a & FLAG_E) == FLAG_E) hogeE();
else if ((a & FLAG_F) == FLAG_F) hogeF();
else if ((a & FLAG_G) == FLAG_G) hogeG();
else if ((a & FLAG_H) == FLAG_H) hogeH();
中略
else if ((a & FLAG_U) == FLAG_U) hogeU();
else if ((a & FLAG_V) == FLAG_V) hogeV();
else if ((a & FLAG_W) == FLAG_W) hogeW();
else if ((a & FLAG_X) == FLAG_X) hogeX();
else if ((a & FLAG_Y) == FLAG_Y) hogeY();
else if ((a & FLAG_Z) == FLAG_Z) hogeZ();

あへ

セミコロンが抜けている（Java）

目的がわからない。
いくら聞いてもわからない。

#define VALUE_1 func_1(なんちゃら)
として VALUE_1 をあちこちに使ってる。

「何でこんなマクロ使ってるの？」
「func_1 は結構重い処理なんですよ。こうしとけば１回だけで済むでしょ！」

　　　　　　　 　　　　　 ...........
　　　　　　　　　　　　(´･ω･`)　
　　　　　　　　　　 cく_>ｙｃく__）　
　　　　　　　　　　　（___,,_,,___,,_）　 ∬
　　　　　　　　　彡※※※※ミ　　旦
￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣
　　　＼　　　ドッ！！ 　 　　／　　＼　ワハハ！　／
　 　　　＼　　　　　　　 　　／　　　　　 ＼　　　　∞
　　　l||||||||||||||　∩,,∩　∩,,∩　　∩,,∩　ﾐ∩ハ∩彡
　　　（,　 　　）（,,　　　 ） 　　　,,）（　　　　）（　　　　）

_GLOVAE_VAER
int hoge;
int foo;
int bar;

誤射した…
グローバル変数のためのヘッダが２つある
// GLOVAL_VAR.h
int hoge;
int foo;
int bar;

// _GLOVAL_VAR.h
extern int hoge;
extern int foo;
extern int bar;

void foo(int[] array) {
&nbsp;if (array.length != 0)
&nbsp;&nbsp;for (int x : array)
&nbsp;&nbsp;&nbsp;System.out.println(x);
}

void foo(int[] array) {
　if (array.length != 0)
　　for (int x : array)
　　　System.out.println(x);
}

クラスをmallocしてコンストラクタを手動で呼び出す