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

■ このスレッドは過去ログ倉庫に格納されています
1デフォルトの名無しさん
垢版 |
NGNG
var unko;
2007/09/01(土) 16:56:26
うむ　ゲームプログラムでtypoということなら
player1
FrameTransformMatrix
だね
でも炎の変換行列はおいておいてファンタジーRPGならPrayerクラスはあっても不思議じゃないかも
2007/09/01(土) 21:52:02
prayerといえばZONEですね
419414
垢版 |
2007/09/01(土) 22:22:48
>>416
>>417
おお、その通りデス。サンクス！
ゲームプログラマじゃない人にももう少し配慮して
書くべきだったね。
2007/09/07(金) 08:06:06
炎の変換行列ｶｺｲｲ
2007/09/08(土) 09:50:17
final int ONE=1
とか
boolean executeFlagIsTrue=true
とか見たことある

素人じゃあるまいし、値を変数名にしないで欲しい。
2007/09/08(土) 21:34:54
pi=3.141592
とか、もうアホかと
2007/09/08(土) 21:52:53
それはM_PIを使えという話か?

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
2007/09/09(日) 00:58:27
それ書いた人って、単に円周率の精度が変わった場合を意図していたん
だろうけど、いつまでもさらされて気の毒ではあるな。


2007/09/09(日) 01:12:31
アメリカ合衆国のインディアナ州の下院で医者のエドウィン・グッドウィンによる
円積問題解決方法を盛り込んだ議案264号が満場一致で通過した。
グッドウィンの方法から得られる値は π = 3.1604, 3.2, 3.232, 4 であり、
このうち 4 については公式に認められた最も不正確な円周率の値として
ギネスブックに記載された。
2007/09/09(日) 04:51:18
某参考書
int[] ints
2007/09/09(日) 11:01:02
>>426
ジョークじゃなかったのか
2007/09/11(火) 00:40:33
>>429
ぐにゅのドキュメントならきっとジョークだと思えたが、
ゼロックスってそのへんどうなの？
2007/09/22(土) 23:50:31
でもちょっと痒い
432デフォルトの名無しさん
垢版 |
2007/10/16(火) 04:19:56
「○○」って書くべきところを「ほげほげ」って書いたら
プログラマじゃない友達に笑われた。
「ほげほげ」を常用するのってプログラマだけなのか？
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
2007/10/18(木) 03:16:31
moe gao piyo
2007/10/19(金) 23:17:58
hogeは俗っぽいので使わないことにしてる。
2007/10/20(土) 05:08:27
foo bar はクールでかっこいいけどhogeはアホっぽい
2007/10/20(土) 08:55:26
foo bar も hoge もアホっぽい。
思考停止してますって主張してるようなもんじゃん。
2007/10/20(土) 12:16:54
そもそも何も考える必要がないとこに使うためのものじゃんか
2007/10/20(土) 12:25:48
>>439は破廉恥な汚物

2007/10/20(土) 15:25:06
何も考えてないときは a, b, c, x, y, z でよくね？
2007/10/20(土) 15:38:50
http://ja.wikipedia.org/wiki/メタ構文変数

中でも42みたいなのはあえて言えばメタ構文定数とでも呼べばいいんだろうか。
2007/10/20(土) 21:37:03
>>442
それは一般の変数として使われる可能性が普通にあるからダメだろ
2007/10/21(日) 08:59:30
01 HEDDAA-LINE.



01 HUTTAA-LINE.
446デフォルトの名無しさん
垢版 |
2007/10/23(火) 16:28:00
rink

profeel

rogo
2007/10/23(火) 21:24:55
samunail
2007/10/23(火) 22:07:25
>>442
ループ用なら i,j,k が一般的だと思うが…？

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

プロ言の穴埋めテストですかって気になります…。
2007/10/24(水) 07:52:34
誰もループカウンタの話なんかしてないし・・
2007/10/24(水) 11:36:03
> temp1,temp2,temp3
これで思い出した
TeX の tempdima, tempdimb, ... は気持ち悪くて仕方がない
2007/10/24(水) 12:41:52
文芸的プログラミングとはほど遠いな。
2007/10/25(木) 02:10:17
C/C++ で、 float 版の abs を FAbs と命名した奴がいた（今もその FAbs がアチコチで使われてる）。
そこまでなら構わんのだが、float 版の sin が SinF で cos が CosF だ。
なら AbsF だろ。
fabs って C の標準関数にあるよな？
(非標準の) sinf や cosf には倣うのに、fabsf は無視か。
2007/10/25(木) 02:32:39
fabs、あるね。
C89 以降の標準化だけど。

てか、C++ なら abs をオーバーロードしろよ。
2007/10/25(木) 02:33:33
標準でオーバーロードされてるんだが。
455デフォルトの名無しさん
垢版 |
2007/11/07(水) 04:28:24
>>422
どこがアホなん？ｋｗｓｋ
2007/11/07(水) 10:46:28
>>455
>422がどういう趣旨で書いたかは知らんが、定数置くなら桁数をもっと考えろとは言いたい。
その桁までで丸めるなら値もおかしいしね。
2007/11/08(木) 21:29:35
modf, modff, fmodf, fmod

阿呆かと
458デフォルトの名無しさん
垢版 |
2007/11/27(火) 14:53:46
hyde
2007/11/27(火) 18:12:57
ふもっふ　ふもっふ
2007/11/27(火) 20:04:34
ゲシュタルト崩壊してきた
2007/12/22(土) 12:42:20
>>442
i,j,kくらいはグローバルでいいよな？
ループごといちいち宣言するのもめんどくせーし
2007/12/22(土) 22:01:01
>>461
　マルチスレッドで動かないなら。
　あと、ループ内で別関数呼ばないって制限もつく。
（呼んだ先にループがあったら帰ってきたときどうなるかは自明だろ）
2007/12/22(土) 22:11:55
つーか、一番スコープを限定したいのがループ制御変数だと思うのだがw
2007/12/23(日) 02:27:12
最初は小さい処理で i を使ってたけど
機能追加していっていつの間にかその中でもう一度 i を使ってしまいましてですね
2007/12/23(日) 02:32:27
>>464
　１関数内で、その状態なら、
　グローバル変数は一切使わない方が身のためでは？

# 変数管理が出来ないなら「台帳」つくれ。
2007/12/23(日) 03:01:52
250行の関数に150行のfor文があって、その中の3行のfor文が同じ変数名のループカウンタを使ってたんだよね
実質オープンソース状態で管理者が複数いたからってのもあるけど、
1画面に収まらない関数があるともう思考停止してきちゃうね

1年半も放置されていたのが不思議でならないよ
2007/12/23(日) 10:17:07
＞管理者が複数いた
　それって誰も管理してないって事でわ？
　プロジェクト管理がなってない

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

＞1年半も放置されていた
　機能追加以前に、*誰も使ってない* って事でしょ？
　そんなアプリ、メンテするだけ無駄。
2007/12/23(日) 11:18:45
そうだね、管理されてないね
利用者も多くて30人程度だろうしなぁ
2007/12/23(日) 15:20:25
>>466
そういうケースこそ、forで変数を宣言するのが一番。それならまともに動くはず。
# 勿論、内側のループ内で外側の制御変数を使っていなければ、ね。
2007/12/23(日) 15:42:34
>>469
コンパイラがマトモなら、ね。
世の中には古いコンパイラとか、中途半端なコンパイラがあるんだな。
　forで変数宣言できないのと、
　forで変数宣言すると、それ以降有効になるのと、
　for文のブロック内のみ有効なのと。
従って、別々の識別名を使用するのが安全確実。

そもそも、250行程度の関数で変数管理できないのが異常。
コーディングに向いていないじゃないか？
（あるいは修正時間が３分しか与えられていないとか）
2007/12/23(日) 17:38:17
俺が書いたわけじゃないんだ
あんまり責めないでくれ・・・orz
2007/12/24(月) 13:52:17
うちにいる中華のコード

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

ダメだって言っても聞かない。プライド「だけ」は高い
2007/12/24(月) 19:35:33
それconstでなくていいのか
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};
};
2008/01/01(火) 17:36:42
ローカルでネーミングに困ったら
oppai
anal
tinntinn
を使ってる
2008/01/10(木) 05:46:40
てぃんてぃん
477デフォルトの名無しさん
垢版 |
2008/01/27(日) 01:41:00
>>475
仕事で？
2008/01/28(月) 09:44:05
古いコンパイラでも

void hoge(void){
　...
　{
　　for(int i=0; ...)
　　　...
　}
　...
ってやれば int i は実質 for のローカル変数扱いになるんじゃないの？
2008/01/28(月) 10:32:07
>>478
えーと、それはforのすぐ外側でブロックを閉じるということ?
つまり、こう?
void hoge(void)
{
...
...
{
for (int i = 0; ...) {
...
...
}
}
...
}
それならその通りだけど、forをブロック化してない>478でそれをやるのは危険だと思う。
2008/01/29(火) 01:39:02
#define for if(1) for
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
↑解説きぼんぬ
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
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
2008/02/06(水) 11:59:46
プログラミング言語 Whitespace
スペースとタブ文字だけが意味のあるトークンであり
それ以外の文字は全て無視される
ttp://ja.wikipedia.org/wiki/Whitespace
2008/02/06(水) 15:46:11
>>485
これはひどいｗｗｗ
2008/02/07(木) 12:55:27
ﾊｶｰ向け言語だなｗｗｗｗｗｗ
2008/02/09(土) 02:41:02
ループカウントに使う変数名がc(Count),d,e,...な俺は異端なんだろうか
i,j,k,...パターンだとiとjがややこしくなるし
あと、for (c = 0 ; c < 100; c++《←ここがC++になる。重要》)
2008/02/09(土) 10:20:14
そこ重要なのかｗｗｗｗｗ

漏れも「i と j って似てるよな問題」は忌々しいと感じていて
個人的なコードでは、i の内側では j を使わずに k を使ったりしてみた事もある。
でも、普通の人なら、つい j のループがどこかにある事を期待してしまうわけで
漏れの戦略は大っぴらにできないんだよな。公然猥褻だよ。社会の窓！みたいな。ﾁﾝ開の神秘！みたいな。
漏れも c d e にしてみるか。
2008/02/09(土) 12:25:49
iとjが区別しやすいフォント使えよ。
……と言って済ませたいところだが、実は俺もi,jは避けて、

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

とかやっているのだ。
2008/02/09(土) 17:16:07
>>490 すなおに row, field にしとけよ。
2008/02/09(土) 17:20:24
>>490
＞for (int r = 0; (略)) // Row Number
少なくとも、本当にこう書いているならスレタイ通りの汚物だな。
2008/02/10(日) 03:30:46
>>489
よし、洗脳完了
っていうか、実際オススメ

>>490
フォントはConsolas





かHG行書体
2008/02/11(月) 11:46:49
Dim bcd ' 部コード
Dim kcd ' 課コード
Dim scd ' 社員コード
これならまだ分かるんだ。　　むしろ統一されていれば覚えやすい気もする

Dim TTL

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

COBOLer が ASP を触り始めた現場(´･ω･)
2008/02/11(月) 22:28:46
Transister Transister Logicかと思いそう
2008/02/11(月) 23:59:16
OTLの亜種かと思った
497デフォルトの名無しさん
垢版 |
2008/02/12(火) 05:19:57
つーか、おまいら変数名にプリフィクス付けないの？
2008/02/12(火) 08:36:16
ポインタ以外はつけない
2008/02/12(火) 11:09:28
なんでポインタだけ特別なんだよ？
500デフォルトの名無しさん
垢版 |
2008/02/12(火) 17:20:49
>>497
　宣言見れば解るし、名前がまともなら察しがつく。クラスメンバーとローカル変数程度なら、ローカル変数の宣言確認するだけで済む。そもそも宣言確認できない規模で変数が点在する設計は糞。
　因みに、昨今のOOライブラリならプリフィクスはタブー。
2008/02/12(火) 23:41:04
別に接頭辞=ハンガリアンでもないだろうが
2008/02/13(水) 00:35:41
個人用である程度ウンコプログラム作るならハンガリアンも全然あり
2008/02/13(水) 01:29:47
コーディング規約も定められてない個人用なら
ハンガリアンなぞよけい使わんわ
504デフォルトの名無しさん
垢版 |
2008/02/13(水) 03:15:49
ハンガリアンなしでどうやって型の間違いを知るの？

たとえば、
a=b+c
ってあって、a、cが整数、bが文字列であっても
実行するまで気づかないの？
2008/02/13(水) 03:23:57
そもそも適切な名前であれば型情報をその名前に含まなくても型の誤認識をしないのが一般の人
2008/02/13(水) 05:00:53
ジェネリックプログラミングなら可能な限り型を限定しないのが普通
型名を名前に含めるにしてもtraitsとかcontainerとか抽象化する
2008/02/13(水) 13:45:17
ハンガリアンは型の識別よりもプログラマの質の識別ができて便利
2008/02/13(水) 14:30:25
>>504
その場合は a と b と c の定義や代入のとこを見ないとなかなか気づかない
一応断っておくが a や b や c という1文字アルファベットの変数名だという前提の話だからな

>>506
list とかいう名前だとちょっと困る
2008/02/13(水) 15:23:30
>>504
だから宣言を見える範囲に納めろと…。
スクリプトなら BookName とか SerialNumber とか BackgroundImage とかきっちり名前付ければ間違え要がないだろ
510デフォルトの名無しさん
垢版 |
2008/02/13(水) 16:11:06
なんとかtmpとか、なんとかworkという変数名がいちばん痛いな。
2008/02/13(水) 16:36:03
C++やDelならまず問題ない。
2008/02/13(水) 21:34:26
>>504
言語によるね。Javaだったらまず問題にならない。
2008/02/14(木) 00:56:52
>>510
普通に使うだろ
変数aの値と変数bの値を入れ替えたい時とか
いちいち意味のある変数名なんて考えねーよｗ
2008/02/14(木) 02:59:17
>>510の言ってることは分かる気がする。
なんとか、の後に続いてるのもポイントだな。
変数の置換みたいな、まさにtempでいいような状況より、
もうちょっと複雑なことをしてるときにも、
そういう意味の無い名前ばっかり付けてコードを分からなくする奴がいる。
一時的な値を保存する変数が複数になると目も当てられない。
そういう奴に共通する特徴として、ありふれた処理を
独特のアルゴリズムで実装する、というのがあって、
そこに暗号的な変数名が加わると･･･
2008/02/14(木) 20:35:55
まだ一緒のソースいじってないからいいけど、
bRetとかiRetとか書かない奴、頭死んでるんじゃね？
2008/02/14(木) 21:12:31
>>515
お前が寂しい奴だということはわかった
■ このスレッドは過去ログ倉庫に格納されています
5ちゃんねるの広告が気に入らない場合は、こちらをクリックしてください。
ニューススポーツなんでも実況