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

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

ありそうでなかったスレ

**デフォルトの名無しさん** · 2009/11/04(水) 10:58:34

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

**デフォルトの名無しさん** · 2009/11/17(火) 17:00:10

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

**デフォルトの名無しさん** · 2009/11/24(火) 03:48:54

まぁなぁ

**デフォルトの名無しさん** · 2009/11/24(火) 04:51:58

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

**デフォルトの名無しさん** · 2009/12/27(日) 17:01:01

変数を初期化していない

**デフォルトの名無しさん** · 2010/05/18(火) 14:28:59

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

**デフォルトの名無しさん** · 2011/02/01(火) 14:29:53

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

**デフォルトの名無しさん** · 2011/02/02(水) 03:28:51

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

**デフォルトの名無しさん** · 2011/02/02(水) 04:58:24

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

**デフォルトの名無しさん** · 2011/02/03(木) 07:19:23

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

**デフォルトの名無しさん** · 2011/02/03(木) 08:08:22

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

**デフォルトの名無しさん** · 2011/02/04(金) 21:11:45

>>224
スレタイ嫁

**デフォルトの名無しさん** · 2011/02/08(火) 19:39:16

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

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

**デフォルトの名無しさん** · 2011/02/09(水) 09:35:46

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

**デフォルトの名無しさん** · 2011/02/09(水) 17:53:59

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

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

**デフォルトの名無しさん** · 2011/02/09(水) 20:30:26

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

**デフォルトの名無しさん** · 2011/02/09(水) 21:32:26

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

**のみねーと** ◆myadMFZ/7k · 2011/04/04(月) 12:32:31.66

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

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

**ななし。** · 2011/07/27(水) 18:27:45.01

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

**デフォルトの名無しさん** · 2011/07/28(木) 02:14:27.16

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

**デフォルトの名無しさん** · 2011/07/28(木) 02:16:42.07

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

**デフォルトの名無しさん** · 2011/07/28(木) 02:24:11.34

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

**デフォルトの名無しさん** · 2011/07/28(木) 02:27:20.32

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

**デフォルトの名無しさん** · 2011/07/28(木) 02:46:19.16

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

**デフォルトの名無しさん** · 2012/01/13(金) 11:08:01.33

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

**デフォルトの名無しさん** · 2012/04/15(日) 07:15:15.25

if
if
if
if

uy · 2012/04/29(日) 20:05:23.25

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

**デフォルトの名無しさん** · 2012/05/16(水) 00:26:15.84

-- 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;

**デフォルトの名無しさん** · 2012/07/30(月) 17:27:25.99

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

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

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

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

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

**デフォルトの名無しさん** · 2012/07/30(月) 18:57:53.11

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

**デフォルトの名無しさん** · 2012/08/30(木) 21:28:49.61

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

**デフォルトの名無しさん** · 2012/08/30(木) 21:35:03.30

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();
  }
}

**デフォルトの名無しさん** · 2012/10/24(水) 00:07:41.75

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間違ってますよ」

**デフォルトの名無しさん** · 2012/11/08(木) 05:36:41.54

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();

忍法帖【Lv=15,xxxPT】(2+0：5) · 2013/12/10(火) 13:07:06.69

あへ

**デフォルトの名無しさん** · 2013/12/25(水) 22:31:58.20

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

**デフォルトの名無しさん** · 2014/01/03(金) 00:24:24.28

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

**デフォルトの名無しさん** · 2014/01/03(金) 00:32:45.33

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

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

**デフォルトの名無しさん** · 2014/02/05(水) 10:05:30.46

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

**デフォルトの名無しさん** · 2014/02/18(火) 03:37:47.79

_GLOVAE_VAER
int hoge;
int foo;
int bar;

**デフォルトの名無しさん** · 2014/02/18(火) 03:40:19.62

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

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

**デフォルトの名無しさん** · 2014/04/03(木) 13:52:23.60

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

**デフォルトの名無しさん** · 2014/04/03(木) 13:53:01.55

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

**デフォルトの名無しさん** · 2014/10/28(火) 01:38:09.27

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

**デフォルトの名無しさん** · 2016/04/24(日) 10:49:59.38

if (flag == true)

**デフォルトの名無しさん** · 2016/05/01(日) 02:22:36.63

if (a > 100)
return true;
else
return false;

**デフォルトの名無しさん** · 2016/05/23(月) 20:06:30.08

ド・モルガンの法則くらい覚えろよな
おまい、中卒か？

**デフォルトの名無しさん** · 2016/06/11(土) 20:18:32.93

namae = "taro"
ubin = "123-4567"
adoresu = "tokyo"
denwa = "4126"

**デフォルトの名無しさん** · 2016/06/17(金) 20:06:35.57

**デフォルトの名無しさん** · 2018/05/23(水) 23:00:16.95

僕の知り合いの知り合いができたパソコン一台でお金持ちになれるやり方
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法　モニアレフヌノ』

T0R7Q

**デフォルトの名無しさん** · 2018/07/04(水) 23:08:11.71

N5U