[表示 : 全て 最新50 1-99 101- 201- 301- 401- 501- 601- 2chのread.cgiへ]
Update time : 05/09 15:07 / Filesize : 121 KB / Number-of Response : 678
[このスレッドの書き込みを削除する]
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧] [類似スレッド一覧]


↑キャッシュ検索、類似スレ動作を修正しました、ご迷惑をお掛けしました

【汚物】痛い変数名・関数名【破廉恥】



1 名前:デフォルトの名無しさん [04/02/19 05:59]
var unko;

403 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 03:02:55 ]
typedef unsigned char UInt8;
typedef unsigned short UInt16;
typedef unsigned long UInt32;
typedef char SInt8;
typedef short SInt16;
typedef long SInt32;
typedef float Float32;
typedef double Float64;


404 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 05:36:41 ]
>>403
それはアリだろ。
プログラムが多様な環境に移植されうるなら、環境間で異なる型の特徴を合わせやすくするため
その他諸々の理由でtypedefするのはよくあること。痛くも汚くもない。

405 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 12:22:37 ]
むしろ正しい。

406 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 13:25:23 ]
普通にstdint.h使えって言いたいんじゃない?

407 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 13:55:14 ]
なんというC99

408 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 19:52:00 ]
>>404
環境ごとの #ifdef がないとかいうことなんじゃね?

409 名前:デフォルトの名無しさん mailto:sage [2007/08/24(金) 23:34:36 ]
>>404
普通になしだろ。
なんでstdintつかわずにわざわざそんなことするんだよ

410 名前:デフォルトの名無しさん mailto:sage [2007/08/25(土) 00:35:37 ]
いつでもどこでもstdint.hが使えると思ったら大間違い。
ちょうど手元にあるVC8では

fatal error C1083: include ファイルを開けません。'stdint.h': No such file or directory


余談だが、そのかわりstddef.hでintptr_tなど一部が使えるようだが
それでもint32_tなど長さ指定型はない。

411 名前:デフォルトの名無しさん mailto:sage [2007/08/25(土) 06:16:47 ]
stdint.h が無い環境でビット幅に応じた整数型が欲しいなら、
自分で stdint.h を作るのがいいよね。 stdint.h がある環境に簡単に
移植できるように。



412 名前:デフォルトの名無しさん mailto:sage [2007/08/25(土) 13:05:40 ]
stdint.hがない環境なんてww

413 名前:デフォルトの名無しさん mailto:sage [2007/08/28(火) 04:06:32 ]
boost/cstdint.hppも思い出してあげてください。 アレがどのくらいのコンパイラに対応してるのか知らないけど。

414 名前:デフォルトの名無しさん [2007/09/01(土) 16:34:43 ]
ゲームプログラムで、
prayer1 祈る人1
FlameTransformMatrix 炎の変換行列
なんてのは尽きないな。見るたびに笑っちまうよ(w

415 名前:デフォルトの名無しさん mailto:sage [2007/09/01(土) 16:43:23 ]
司祭クラスのインスタンスとか
火の精霊魔術で何かを変質させる際の母体となるものとか
ゲームプログラムでなければアレなネーミングではあるな

416 名前:デフォルトの名無しさん mailto:sage [2007/09/01(土) 16:51:58 ]
>>415
typoのことじゃね?

417 名前:デフォルトの名無しさん mailto:sage [2007/09/01(土) 16:56:26 ]
うむ ゲームプログラムでtypoということなら
player1
FrameTransformMatrix
だね
でも炎の変換行列はおいておいてファンタジーRPGならPrayerクラスはあっても不思議じゃないかも

418 名前:デフォルトの名無しさん mailto:sage [2007/09/01(土) 21:52:02 ]
prayerといえばZONEですね

419 名前:414 mailto:sage [2007/09/01(土) 22:22:48 ]
>>416
>>417
おお、その通りデス。サンクス!
ゲームプログラマじゃない人にももう少し配慮して
書くべきだったね。

420 名前:デフォルトの名無しさん mailto:sage [2007/09/07(金) 08:06:06 ]
炎の変換行列カコイイ

421 名前:デフォルトの名無しさん mailto:sage [2007/09/08(土) 09:50:17 ]
final int ONE=1
とか
boolean executeFlagIsTrue=true
とか見たことある

素人じゃあるまいし、値を変数名にしないで欲しい。



422 名前:デフォルトの名無しさん mailto:sage [2007/09/08(土) 21:34:54 ]
pi=3.141592
とか、もうアホかと

423 名前:デフォルトの名無しさん mailto:sage [2007/09/08(土) 21:52:53 ]
それはM_PIを使えという話か?



424 名前:デフォルトの名無しさん mailto:sage [2007/09/08(土) 22:19:02 ]
変数の値と名前が一致しているのは素人臭がしてキモイ
ONE=2 とか iTrue=fales が望ましい

425 名前:デフォルトの名無しさん [2007/09/08(土) 23:40:09 ]
>422
DATA文の主な目的は、定数に名前を付けることである。
円周率πを使うたびに3.141592653589793と記述する代わりに、変数 pi にDATA文を使って値を割り当て、 毎回 3.14...という長い定数を記述する必要なく pi を使うことが出来る。
さらに、円周率が変わった場合に、プログラムの変更が容易になるという利点がある。

- FORTRAN manual for Xerox computers


426 名前:デフォルトの名無しさん mailto:sage [2007/09/09(日) 00:58:27 ]
それ書いた人って、単に円周率の精度が変わった場合を意図していたん
だろうけど、いつまでもさらされて気の毒ではあるな。




427 名前:デフォルトの名無しさん mailto:sage [2007/09/09(日) 01:12:31 ]
アメリカ合衆国のインディアナ州の下院で医者のエドウィン・グッドウィンによる
円積問題解決方法を盛り込んだ議案264号が満場一致で通過した。
グッドウィンの方法から得られる値は π = 3.1604, 3.2, 3.232, 4 であり、
このうち 4 については公式に認められた最も不正確な円周率の値として
ギネスブックに記載された。

428 名前:デフォルトの名無しさん mailto:sage [2007/09/09(日) 04:51:18 ]
某参考書
int[] ints

429 名前:デフォルトの名無しさん mailto:sage [2007/09/09(日) 11:01:02 ]
>>426
ジョークじゃなかったのか

430 名前:デフォルトの名無しさん mailto:sage [2007/09/11(火) 00:40:33 ]
>>429
ぐにゅのドキュメントならきっとジョークだと思えたが、
ゼロックスってそのへんどうなの?

431 名前:デフォルトの名無しさん mailto:sage [2007/09/22(土) 23:50:31 ]
でもちょっと痒い



432 名前:デフォルトの名無しさん [2007/10/16(火) 04:19:56 ]
「○○」って書くべきところを「ほげほげ」って書いたら
プログラマじゃない友達に笑われた。
「ほげほげ」を常用するのってプログラマだけなのか?

433 名前:デフォルトの名無しさん mailto:sage [2007/10/16(火) 04:31:46 ]
ほげほげって言葉は70〜80年代の香りがするよな

434 名前:デフォルトの名無しさん [2007/10/16(火) 19:58:03 ]
俺は結構「ほげほげ」使うけどな

435 名前:デフォルトの名無しさん [2007/10/17(水) 23:04:03 ]
foo bar baz
hoge moge fuga

436 名前:デフォルトの名無しさん mailto:sage [2007/10/18(木) 03:16:31 ]
moe gao piyo

437 名前:デフォルトの名無しさん mailto:sage [2007/10/19(金) 23:17:58 ]
hogeは俗っぽいので使わないことにしてる。

438 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 05:08:27 ]
foo bar はクールでかっこいいけどhogeはアホっぽい

439 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 08:55:26 ]
foo bar も hoge もアホっぽい。
思考停止してますって主張してるようなもんじゃん。

440 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 12:16:54 ]
そもそも何も考える必要がないとこに使うためのものじゃんか

441 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 12:25:48 ]
>>439は破廉恥な汚物





442 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 15:25:06 ]
何も考えてないときは a, b, c, x, y, z でよくね?

443 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 15:38:50 ]
ja.wikipedia.org/wiki/メタ構文変数

中でも42みたいなのはあえて言えばメタ構文定数とでも呼べばいいんだろうか。


444 名前:デフォルトの名無しさん mailto:sage [2007/10/20(土) 21:37:03 ]
>>442
それは一般の変数として使われる可能性が普通にあるからダメだろ

445 名前:デフォルトの名無しさん mailto:sage [2007/10/21(日) 08:59:30 ]
01 HEDDAA-LINE.



01 HUTTAA-LINE.

446 名前:デフォルトの名無しさん [2007/10/23(火) 16:28:00 ]
rink

profeel

rogo

447 名前:デフォルトの名無しさん mailto:sage [2007/10/23(火) 21:24:55 ]
samunail

448 名前:デフォルトの名無しさん mailto:sage [2007/10/23(火) 22:07:25 ]
>>442
ループ用なら i,j,k が一般的だと思うが…?

痛い変数名
temp1,temp2,temp3
痛い関数名
func1,func2,func3

プロ言の穴埋めテストですかって気になります…。

449 名前:デフォルトの名無しさん mailto:sage [2007/10/24(水) 07:52:34 ]
誰もループカウンタの話なんかしてないし・・

450 名前:デフォルトの名無しさん mailto:sage [2007/10/24(水) 11:36:03 ]
> temp1,temp2,temp3
これで思い出した
TeX の tempdima, tempdimb, ... は気持ち悪くて仕方がない

451 名前:デフォルトの名無しさん mailto:sage [2007/10/24(水) 12:41:52 ]
文芸的プログラミングとはほど遠いな。



452 名前:デフォルトの名無しさん mailto:sage [2007/10/25(木) 02:10:17 ]
C/C++ で、 float 版の abs を FAbs と命名した奴がいた(今もその FAbs がアチコチで使われてる)。
そこまでなら構わんのだが、float 版の sin が SinF で cos が CosF だ。
なら AbsF だろ。
fabs って C の標準関数にあるよな?
(非標準の) sinf や cosf には倣うのに、fabsf は無視か。

453 名前:デフォルトの名無しさん mailto:sage [2007/10/25(木) 02:32:39 ]
fabs、あるね。
C89 以降の標準化だけど。

てか、C++ なら abs をオーバーロードしろよ。

454 名前:デフォルトの名無しさん mailto:sage [2007/10/25(木) 02:33:33 ]
標準でオーバーロードされてるんだが。

455 名前:デフォルトの名無しさん [2007/11/07(水) 04:28:24 ]
>>422
どこがアホなん?kwsk

456 名前:デフォルトの名無しさん mailto:sage [2007/11/07(水) 10:46:28 ]
>>455
>422がどういう趣旨で書いたかは知らんが、定数置くなら桁数をもっと考えろとは言いたい。
その桁までで丸めるなら値もおかしいしね。

457 名前:デフォルトの名無しさん mailto:sage [2007/11/08(木) 21:29:35 ]
modf, modff, fmodf, fmod

阿呆かと

458 名前:デフォルトの名無しさん [2007/11/27(火) 14:53:46 ]
hyde

459 名前:デフォルトの名無しさん mailto:sage [2007/11/27(火) 18:12:57 ]
ふもっふ ふもっふ

460 名前:デフォルトの名無しさん mailto:sage [2007/11/27(火) 20:04:34 ]
ゲシュタルト崩壊してきた

461 名前:デフォルトの名無しさん mailto:sage [2007/12/22(土) 12:42:20 ]
>>442
i,j,kくらいはグローバルでいいよな?
ループごといちいち宣言するのもめんどくせーし



462 名前:デフォルトの名無しさん mailto:sage [2007/12/22(土) 22:01:01 ]
>>461
 マルチスレッドで動かないなら。
 あと、ループ内で別関数呼ばないって制限もつく。
(呼んだ先にループがあったら帰ってきたときどうなるかは自明だろ)

463 名前:デフォルトの名無しさん mailto:sage [2007/12/22(土) 22:11:55 ]
つーか、一番スコープを限定したいのがループ制御変数だと思うのだがw

464 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 02:27:12 ]
最初は小さい処理で i を使ってたけど
機能追加していっていつの間にかその中でもう一度 i を使ってしまいましてですね

465 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 02:32:27 ]
>>464
 1関数内で、その状態なら、
 グローバル変数は一切使わない方が身のためでは?

# 変数管理が出来ないなら「台帳」つくれ。

466 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 03:01:52 ]
250行の関数に150行のfor文があって、その中の3行のfor文が同じ変数名のループカウンタを使ってたんだよね
実質オープンソース状態で管理者が複数いたからってのもあるけど、
1画面に収まらない関数があるともう思考停止してきちゃうね

1年半も放置されていたのが不思議でならないよ

467 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 10:17:07 ]
>管理者が複数いた
 それって誰も管理してないって事でわ?
 プロジェクト管理がなってない

>250行の関数に150行のfor文があって、その中の3行のfor文が同じ変数名のループカウンタを使って
 まともなテストを1回でも行っていれば気が憑く筈。
 追加機能のテストしてない+現行機能のテストしてないってことだな。
 コーディング担当無能すぎ。(単体テストくらい行え!)

>1年半も放置されていた
 機能追加以前に、*誰も使ってない* って事でしょ?
 そんなアプリ、メンテするだけ無駄。

468 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 11:18:45 ]
そうだね、管理されてないね
利用者も多くて30人程度だろうしなぁ

469 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 15:20:25 ]
>>466
そういうケースこそ、forで変数を宣言するのが一番。それならまともに動くはず。
# 勿論、内側のループ内で外側の制御変数を使っていなければ、ね。

470 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 15:42:34 ]
>>469
コンパイラがマトモなら、ね。
世の中には古いコンパイラとか、中途半端なコンパイラがあるんだな。
 forで変数宣言できないのと、
 forで変数宣言すると、それ以降有効になるのと、
 for文のブロック内のみ有効なのと。
従って、別々の識別名を使用するのが安全確実。

そもそも、250行程度の関数で変数管理できないのが異常。
コーディングに向いていないじゃないか?
(あるいは修正時間が3分しか与えられていないとか)

471 名前:デフォルトの名無しさん mailto:sage [2007/12/23(日) 17:38:17 ]
俺が書いたわけじゃないんだ
あんまり責めないでくれ・・・orz



472 名前:デフォルトの名無しさん mailto:sage [2007/12/24(月) 13:52:17 ]
うちにいる中華のコード

public static String ZERO = "0";
public static String ZERO_ZERO = "00";

ダメだって言っても聞かない。プライド「だけ」は高い


473 名前:デフォルトの名無しさん mailto:sage [2007/12/24(月) 19:35:33 ]
それconstでなくていいのか

474 名前:デフォルトの名無しさん mailto:sage [2007/12/26(水) 02:45:08 ]
typedef enum
{
DEF_NAE,
DEF_NORMAL,
DEF_BOKKI,
DEF_EJACULATION
}length;

class Chinko
{
public:
ulong m_hair[INT_MAX][INT_MAX][INT_MAX];
state m_state;
private:
Chinko():m_state(nae){memset(m_hair,0x00,sizeof(m_hair)*sizeof(ulong);}
state doSex(){while(m_state !=DEF_EJACULATION)m_state++;return m_state};
};

475 名前:デフォルトの名無しさん mailto:sage [2008/01/01(火) 17:36:42 ]
ローカルでネーミングに困ったら
oppai
anal
tinntinn
を使ってる

476 名前:デフォルトの名無しさん mailto:sage [2008/01/10(木) 05:46:40 ]
てぃんてぃん

477 名前:デフォルトの名無しさん [2008/01/27(日) 01:41:00 ]
>>475
仕事で?

478 名前:デフォルトの名無しさん mailto:sage [2008/01/28(月) 09:44:05 ]
古いコンパイラでも

void hoge(void){
 ...
 {
  for(int i=0; ...)
   ...
 }
 ...
ってやれば int i は実質 for のローカル変数扱いになるんじゃないの?

479 名前:デフォルトの名無しさん mailto:sage [2008/01/28(月) 10:32:07 ]
>>478
えーと、それはforのすぐ外側でブロックを閉じるということ?
つまり、こう?
void hoge(void)
{
...
...
{
for (int i = 0; ...) {
...
...
}
}
...
}
それならその通りだけど、forをブロック化してない>478でそれをやるのは危険だと思う。

480 名前:デフォルトの名無しさん mailto:sage [2008/01/29(火) 01:39:02 ]
#define for if(1) for

481 名前:デフォルトの名無しさん mailto:sage [2008/01/29(火) 10:22:27 ]
#define _ int
#define __ return
#define ___ printf
#define ____ main
#define _____ argv
#define ______ argc

_ ____(_ *______, char* _____[]){
 ___("Hello world\n");
 __ 0;
}





482 名前:デフォルトの名無しさん [2008/02/06(水) 01:11:34 ]
↑解説きぼんぬ

483 名前:デフォルトの名無しさん mailto:sage [2008/02/06(水) 01:43:13 ]
/* # は #define に置換してください
 そうしないとなぜか書き込めない */
# d char
# e "d"
# f "e"
# g "H"
# h int
# i "l"
# j main
# k "\n"
# l "o"
# m printf
# n "r"
# o return
# p "w"
# q {
# r (
# s ;
# t ,
# u )
# v }
h j r h w t d b b x u q m r g f i i l a p l n i e u s o c s v

484 名前:デフォルトの名無しさん mailto:sage [2008/02/06(水) 01:45:46 ]
/* >>483は失敗
このコメントを入れれば書き込めるみたいです */
#define a " "
#define b *
#define c 0
#define d char
#define e "d"
#define f "e"
#define g "H"
#define h int
#define i "l"
#define j main
#define k "\n"
#define l "o"
#define m printf
#define n "r"
#define o return
#define p "w"
#define q {
#define r (
#define s ;
#define t ,
#define u )
#define v }
h j r h w t d b b x u q m r g f i i l a p l n i e u s o c s v

485 名前:デフォルトの名無しさん mailto:sage [2008/02/06(水) 11:59:46 ]
プログラミング言語 Whitespace
スペースとタブ文字だけが意味のあるトークンであり
それ以外の文字は全て無視される
ttp://ja.wikipedia.org/wiki/Whitespace

486 名前:デフォルトの名無しさん mailto:sage [2008/02/06(水) 15:46:11 ]
>>485
これはひどいwww

487 名前:デフォルトの名無しさん mailto:sage [2008/02/07(木) 12:55:27 ]
ハカー向け言語だなwwwwww

488 名前:デフォルトの名無しさん mailto:sage [2008/02/09(土) 02:41:02 ]
ループカウントに使う変数名がc(Count),d,e,...な俺は異端なんだろうか
i,j,k,...パターンだとiとjがややこしくなるし
あと、for (c = 0 ; c < 100; c++《←ここがC++になる。重要》)

489 名前:デフォルトの名無しさん mailto:sage [2008/02/09(土) 10:20:14 ]
そこ重要なのかwwwww

漏れも「i と j って似てるよな問題」は忌々しいと感じていて
個人的なコードでは、i の内側では j を使わずに k を使ったりしてみた事もある。
でも、普通の人なら、つい j のループがどこかにある事を期待してしまうわけで
漏れの戦略は大っぴらにできないんだよな。公然猥褻だよ。社会の窓!みたいな。チン開の神秘!みたいな。
漏れも c d e にしてみるか。

490 名前:デフォルトの名無しさん mailto:sage [2008/02/09(土) 12:25:49 ]
iとjが区別しやすいフォント使えよ。
……と言って済ませたいところだが、実は俺もi,jは避けて、

for (int r = 0; (略)) // Row Number
  for (int f = 0; (略)) // Field Number

とかやっているのだ。

491 名前:デフォルトの名無しさん mailto:sage [2008/02/09(土) 17:16:07 ]
>>490 すなおに row, field にしとけよ。



492 名前:デフォルトの名無しさん mailto:sage [2008/02/09(土) 17:20:24 ]
>>490
>for (int r = 0; (略)) // Row Number
少なくとも、本当にこう書いているならスレタイ通りの汚物だな。

493 名前:デフォルトの名無しさん mailto:sage [2008/02/10(日) 03:30:46 ]
>>489
よし、洗脳完了
っていうか、実際オススメ

>>490
フォントはConsolas





かHG行書体

494 名前:デフォルトの名無しさん mailto:sage [2008/02/11(月) 11:46:49 ]
Dim bcd ' 部コード
Dim kcd ' 課コード
Dim scd ' 社員コード
これならまだ分かるんだ。  むしろ統一されていれば覚えやすい気もする

Dim TTL

どうも合計値(ToTaL)らしい。
Time To Liveかと思ったよ。

COBOLer が ASP を触り始めた現場(´・ω・)

495 名前:デフォルトの名無しさん mailto:sage [2008/02/11(月) 22:28:46 ]
Transister Transister Logicかと思いそう

496 名前:デフォルトの名無しさん mailto:sage [2008/02/11(月) 23:59:16 ]
OTLの亜種かと思った

497 名前:デフォルトの名無しさん [2008/02/12(火) 05:19:57 ]
つーか、おまいら変数名にプリフィクス付けないの?

498 名前:デフォルトの名無しさん mailto:sage [2008/02/12(火) 08:36:16 ]
ポインタ以外はつけない

499 名前:デフォルトの名無しさん mailto:sage [2008/02/12(火) 11:09:28 ]
なんでポインタだけ特別なんだよ?

500 名前:デフォルトの名無しさん [2008/02/12(火) 17:20:49 ]
>>497
 宣言見れば解るし、名前がまともなら察しがつく。クラスメンバーとローカル変数程度なら、ローカル変数の宣言確認するだけで済む。そもそも宣言確認できない規模で変数が点在する設計は糞。
 因みに、昨今のOOライブラリならプリフィクスはタブー。

501 名前:デフォルトの名無しさん mailto:sage [2008/02/12(火) 23:41:04 ]
別に接頭辞=ハンガリアンでもないだろうが



502 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 00:35:41 ]
個人用である程度ウンコプログラム作るならハンガリアンも全然あり

503 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 01:29:47 ]
コーディング規約も定められてない個人用なら
ハンガリアンなぞよけい使わんわ






[ 続きを読む ] / [ 携帯版 ]

前100 次100 最新50 [ このスレをブックマーク! 携帯に送る ] 2chのread.cgiへ
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧]( ´∀`)<121KB

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef