[表示 : 全て 最新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;

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 ]
コーディング規約も定められてない個人用なら
ハンガリアンなぞよけい使わんわ

504 名前:デフォルトの名無しさん [2008/02/13(水) 03:15:49 ]
ハンガリアンなしでどうやって型の間違いを知るの?

たとえば、
a=b+c
ってあって、a、cが整数、bが文字列であっても
実行するまで気づかないの?

505 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 03:23:57 ]
そもそも適切な名前であれば型情報をその名前に含まなくても型の誤認識をしないのが一般の人



506 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 05:00:53 ]
ジェネリックプログラミングなら可能な限り型を限定しないのが普通
型名を名前に含めるにしてもtraitsとかcontainerとか抽象化する

507 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 13:45:17 ]
ハンガリアンは型の識別よりもプログラマの質の識別ができて便利


508 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 14:30:25 ]
>>504
その場合は a と b と c の定義や代入のとこを見ないとなかなか気づかない
一応断っておくが a や b や c という1文字アルファベットの変数名だという前提の話だからな

>>506
list とかいう名前だとちょっと困る

509 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 15:23:30 ]
>>504
だから宣言を見える範囲に納めろと…。
スクリプトなら BookName とか SerialNumber とか BackgroundImage とかきっちり名前付ければ間違え要がないだろ

510 名前:デフォルトの名無しさん [2008/02/13(水) 16:11:06 ]
なんとかtmpとか、なんとかworkという変数名がいちばん痛いな。

511 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 16:36:03 ]
C++やDelならまず問題ない。


512 名前:デフォルトの名無しさん mailto:sage [2008/02/13(水) 21:34:26 ]
>>504
言語によるね。Javaだったらまず問題にならない。

513 名前:デフォルトの名無しさん mailto:sage [2008/02/14(木) 00:56:52 ]
>>510
普通に使うだろ
変数aの値と変数bの値を入れ替えたい時とか
いちいち意味のある変数名なんて考えねーよw

514 名前:デフォルトの名無しさん mailto:sage [2008/02/14(木) 02:59:17 ]
>>510の言ってることは分かる気がする。
なんとか、の後に続いてるのもポイントだな。
変数の置換みたいな、まさにtempでいいような状況より、
もうちょっと複雑なことをしてるときにも、
そういう意味の無い名前ばっかり付けてコードを分からなくする奴がいる。
一時的な値を保存する変数が複数になると目も当てられない。
そういう奴に共通する特徴として、ありふれた処理を
独特のアルゴリズムで実装する、というのがあって、
そこに暗号的な変数名が加わると・・・


515 名前:デフォルトの名無しさん mailto:sage [2008/02/14(木) 20:35:55 ]
まだ一緒のソースいじってないからいいけど、
bRetとかiRetとか書かない奴、頭死んでるんじゃね?



516 名前:デフォルトの名無しさん mailto:sage [2008/02/14(木) 21:12:31 ]
>>515
お前が寂しい奴だということはわかった

517 名前:515 mailto:sage [2008/02/14(木) 21:21:19 ]
俺の頭が死んでたwww逆だ逆wwww
bRetとかしか書かない奴は死ね。

518 名前:デフォルトの名無しさん mailto:sage [2008/02/15(金) 21:25:59 ]
【放送】報道ステーション・古舘伊知郎氏が謝罪 「アダルトチルドレン」言葉の意味を知らずに発言し、関係者を傷つける★3
mamono.2ch.net/test/read.cgi/newsplus/1203060700/

アダルトチルドレンも相当糞なネーミングだな

519 名前:デフォルトの名無しさん mailto:sage [2008/02/15(金) 21:57:54 ]
コイズミチルドレン

520 名前:デフォルトの名無しさん mailto:sage [2008/02/18(月) 01:52:08 ]
ファーストチルドレン

521 名前:デフォルトの名無しさん mailto:sage [2008/02/21(木) 22:47:34 ]
絶対可憐チルドレン

522 名前:デフォルトの名無しさん [2008/02/27(水) 06:17:18 ]
ミスターチルドレン

523 名前:デフォルトの名無しさん mailto:sage [2008/02/28(木) 04:48:12 ]
自分はまだ学生なんだけど隣の席の奴がカウンタ i ii iii iiiiっていうふうにしててすごい見にくい

524 名前:デフォルトの名無しさん mailto:sage [2008/02/28(木) 07:36:38 ]
ループが4重になってる時点で基本的におかしい。

525 名前:デフォルトの名無しさん mailto:sage [2008/02/28(木) 10:26:43 ]
>>524
for ループ1個ごとに変数名変えてるとか?
いずれにしろ、変数名以前の問題で痛いけど。



526 名前:デフォルトの名無しさん mailto:sage [2008/02/28(木) 21:31:48 ]
i ii iii iv にすればいいのにと思ったw

527 名前:デフォルトの名無しさん mailto:sage [2008/02/29(金) 02:41:39 ]
ゲシュタルト崩壊するからやめてやめて

528 名前:デフォルトの名無しさん mailto:sage [2008/02/29(金) 13:42:34 ]
xivとかみたくない。

529 名前:デフォルトの名無しさん [2008/02/29(金) 16:11:36 ]
>>102
Blue-rayは「青い光」で、固有名詞じゃないからアメリカで商標登録できなかったってのが原因だぜ。

530 名前:デフォルトの名無しさん [2008/03/01(土) 17:29:42 ]
頭.clear();

531 名前:デフォルトの名無しさん mailto:sage [2008/03/11(火) 13:00:35 ]
ハゲたのか

532 名前:デフォルトの名無しさん mailto:sage [2008/03/18(火) 23:01:47 ]
ExcelVBA始めたばっかの頃

Dim 魔貫光殺法 as hoge

if ピッコロ大魔王 hogehoge then
魔貫光殺法
end if

とかやってたら取引先に書式を渡したボケがいて
「ピッコロ大魔王って何ですか?」って真面目に
尋ねられた事がある。

例え自分のためだけに作ろうとも、
自分のPCの中だけにあっても、油断はできない。
一度作ったものはどこへ誰を(何を)経由して漏れるか分からない。
いい勉強になった。

533 名前:デフォルトの名無しさん mailto:sage [2008/03/20(木) 14:59:12 ]
漏れるって段階で会社としてダメだな

534 名前:デフォルトの名無しさん [2008/05/02(金) 13:05:18 ]
まぁ、そんなこともあるさw

535 名前:デフォルトの名無しさん mailto:sage [2008/05/04(日) 02:02:44 ]
漏れる事が悪いかとか何とかってのは、ここではどうでもいいがな

アルファベット1文字または2文字 + 数字
で数字がものによってはabcd...になることがあった
まあネタにする程面白くはないが...大層読みづらかった



536 名前:デフォルトの名無しさん mailto:sage [2008/05/04(日) 09:41:24 ]
意味不明

537 名前:デフォルトの名無しさん mailto:sage [2008/05/10(土) 18:46:16 ]
変数名にこれが付いてたら大体クソ

data
arg
num
my
temp
object
flag
old
new
ret
ptr
sign
of
by

538 名前:デフォルトの名無しさん [2008/06/16(月) 16:37:24 ]
〜オレ用関数の命名規則〜Version.0.0.1
主旨:関数を副作用の強弱でレベル分けする。副作用の少ない関数を、<純粋度が高い関数>と呼ぶことにする。
   適用する命名規則は低い方に合わせる。自分の呼び出す関数の中で、もっとも低い方に合わせた規則を適用する(main関数は例外とする)
   

<純粋度が高い関数>
純粋度4 全て小文字で、最初の文字をアンダースコアにする (例:atoi()→_atoi()、sin()→_sin()、strlen()→_strlen())
   「ローカル変数のみを使う。」「ポインタでない引数を使う(もちろん、使わなくても良い)。」
   「ポインタの引数を使い逆参照も行うが、中身は書き換えない。」のいずれかのみに該当

純粋度3 単語の先頭が小文字の名前にする (例:strcat()→strcat()、memmove()→memmove())
   「ポインタ型の引数を受け取り、逆参照して操作する(要素がポインタでない1次配列と構造体まで)。」に該当
   ※純粋度4の関数Aがローカル変数を純粋度3の関数Bにアドレス渡しする場合は、関数Aは純粋度4に留まって良いとする。
    例えば、関数Aのローカルな配列をソート関数Bに渡す場合、B。

純粋度2 単語の先頭が大文字の名前にする (例:malloc()→MAlloc()、free()→Free()、rand()→Rand()(←たぶん静的変数を使ってるから))
   「構造体が持つ、ポインタ型のメンバ変数を逆参照し読み書きする。(二重、三重ポインタの使用、リスト操作など)」
   「malloc、freeやそれに類似したAPIなどを使う。」
   「静的変数の読み書きを行う。」のいずれかに該当

純粋度1 単語の先頭が大文字の名前で 最後にアンダースコアをつける (例:fopen()→FOpen_()、fprintf()→FPrintF_())
   「グローバル変数を読み書きする。」
   「画面への出力や、キーボードからの入力、ネットワーク通信、ファイルを開く、保存、などを行う(主記憶領域以外との通信)」
   のどちらかに該当
<純粋度が低い関数>
これから使うつもり('A`;)。変じゃないだろうか。

539 名前:デフォルトの名無しさん [2008/06/16(月) 16:39:25 ]
↑誤爆したorzマジでごめん

540 名前:デフォルトの名無しさん mailto:sage [2008/06/16(月) 19:53:55 ]
>全て小文字で、最初の文字をアンダースコアにする

グローバルな識別子として処理系に予約されています。
絶対に使用しないでください。

541 名前:デフォルトの名無しさん mailto:sage [2008/06/16(月) 21:55:50 ]
>>540
最初のアンダースコアを使うのは止めておくことにします。長文の投稿失礼しました。

542 名前:デフォルトの名無しさん [2008/06/17(火) 07:00:34 ]
誤爆なの?w

543 名前:デフォルトの名無しさん [2008/06/17(火) 22:36:11 ]
hoge を使う奴はたいてい初心者

544 名前:543 [2008/06/17(火) 23:07:59 ]
プロの俺はpneumonoultramicroscopicsilicovolcanoconiosisを使う

545 名前:デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:10:08 ]
珪性塵肺症か。



546 名前:デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:17:08 ]
なげえプレースホルダーだなw

547 名前:デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:34:31 ]
初心者の女PGでmegu1 megu2 megu3って自分の名前を付ける奴がいた
Class Megu1{
meguArg01 = ...;
meguArg02 = ...;

meguFunc(){...}
}

って感じで。もう何やってんのか全く分からん

548 名前:デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:46:30 ]
>>547
中身をじろじろ見ないでくれという
初級プログラマからの心の叫び。

549 名前:デフォルトの名無しさん [2008/06/19(木) 04:55:47 ]
>>547
かわいいなw

550 名前:デフォルトの名無しさん mailto:sage [2008/06/19(木) 09:23:29 ]
>>540
C/C++に関してはされてないな


551 名前:デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:22:47 ]
うむ、C/C++ に関しては、アンダースコア二つで始まるのがメーカー独自拡張だな。

552 名前:デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:40:06 ]
>>550-551
いや、アンダースコア始まりで小文字が続くものもグローバルスコープで予約されてるよ。

553 名前:デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:47:22 ]
みんながあまりに安易に多用しすぎて混乱しそうだったから予約したんだろうな

554 名前:デフォルトの名無しさん mailto:sage [2008/06/19(木) 18:43:31 ]
どうでもいいけど言語によって名前のつけ方のセオリーって全然違うんじゃないの

555 名前:デフォルトの名無しさん [2008/06/20(金) 01:28:19 ]
>>554
ひまわり とか?w



556 名前:デフォルトの名無しさん mailto:sage [2008/06/20(金) 07:25:03 ]
>>552
VC++ の独自の関数とかそうだしね。
_stricmp とか。

C++ だとローカルスコープでは予約されてないけど
やっぱりグローバルスコープでは予約されている。

557 名前:デフォルトの名無しさん mailto:sage [2008/06/20(金) 08:05:50 ]
>>547
俺もやってた時期があったw
どうせバグ報告が来るなら誰が書いたかすぐ解った方が良い、と本気で思ってた。
二週間と続かなかったが。

558 名前:デフォルトの名無しさん mailto:sage [2008/06/20(金) 12:33:09 ]
>>551
>アンダースコア二つ
C++では、識別子に含めることが禁止されている。
先頭だけじゃなく途中でもダメ。

559 名前:デフォルトの名無しさん mailto:sage [2008/06/20(金) 20:02:32 ]
処理系が使用する分には問題ない

560 名前:デフォルトの名無しさん [2008/06/21(土) 03:07:46 ]
ハマショーが好きな上司が、クラス名に SurrenderLikeSnow って付けてた。
「悲しみは雪のように」って言おうとしたんだろうが、たぶんSadnessの間違いだと思う

561 名前:デフォルトの名無しさん mailto:sage [2008/06/21(土) 13:25:26 ]
英訳を頑張っちゃった系のソースは読むのがツライ。
その上スペルミスもあると、もう何がなんだかわからなくなる。

562 名前:デフォルトの名無しさん mailto:sage [2008/06/21(土) 13:32:12 ]
スペルミス大量でインデントもぐちゃぐちゃなソースがあるけど
意外と読めるもんだな・・・。
イライラはするが。

563 名前:デフォルトの名無しさん mailto:sage [2008/06/21(土) 16:28:26 ]
ローカル変数ならまだいいけどDBのテーブル名、カラム名でスペルミスは勘弁してほしい。
使ってる方が恥ずかしくなってくるぜ。

564 名前:デフォルトの名無しさん mailto:sage [2008/06/21(土) 21:59:08 ]
なんにせよuwariteよりはマシ。


565 名前:デフォルトの名無しさん mailto:sage [2008/07/14(月) 00:08:28 ]
変数名のセンスが無いやつほど
無駄な一時変数を好む傾向にある。



566 名前:デフォルトの名無しさん mailto:sage [2008/07/14(月) 09:32:27 ]
名前ではないけど、ヘッダファイル(当然いくつかのファイルで include されている)で
static 変数を定義して、値が変わらないと悩んでいた人がいた。そら変わらんわなあ。

567 名前:デフォルトの名無しさん mailto:sage [2008/07/14(月) 09:52:44 ]
それは、多重定義のエラーが出てstaticをつけたらエラーが消えたからそのまま使ってるんだろうw

568 名前:デフォルトの名無しさん mailto:sage [2008/07/15(火) 00:52:58 ]
>>566
ヘッダファイルに変数を書く時点で既に変。
(つか、普通はextern 宣言だけだろ?)

569 名前:デフォルトの名無しさん mailto:sage [2008/07/15(火) 01:39:59 ]
>>567
あるある

570 名前:デフォルトの名無しさん mailto:sage [2008/07/16(水) 21:44:13 ]
>>566-569
それ(static無)やってる職場に以前いたんだが、(当然ながら)全く同じ定義のせいかエラーが出ずにリンク出来てた。
動作も正常。むしろエラーになってくれる方がありがたかったな。という思い出。



571 名前:デフォルトの名無しさん mailto:sage [2008/07/17(木) 01:33:32 ]
>>568も変だけどな。

572 名前:デフォルトの名無しさん mailto:sage [2008/07/17(木) 20:12:07 ]
変じゃないだろ
ヘッダでメモリ確保しちゃダメなのは常識

573 名前:デフォルトの名無しさん [2008/08/22(金) 14:10:24 ]
>>572
kwsk

574 名前:デフォルトの名無しさん [2008/08/22(金) 22:19:51 ]
送信済みデータ保存関数

saveSendedData

575 名前:デフォルトの名無しさん mailto:sage [2008/08/22(金) 22:34:56 ]
>>574
読み込み済みデータでreadedDataってのもあったぞ。



576 名前:デフォルトの名無しさん mailto:sage [2008/08/23(土) 00:03:00 ]
>>573
ヘッダファイルとして分離する目的は何でしょう?
複数のモジュールから、同一の定義を参照する事が目的。

ゆえに変数定義など、唯一でなければならないモノをヘッダに書いてはならない。

そもそも、static変数をヘッダで定義して
あちこちのモジュールでインクルードせにゃならんってのは、
構造的に間違ってる。 うつくしくない。

577 名前:デフォルトの名無しさん [2008/08/23(土) 21:21:27 ]
>>547
getmegu
Insert_megu
open_megu (man, co)
せくはら







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

前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