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

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

ありそうでなかったスレ

**デフォルトの名無しさん** · 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