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

**デフォルトの名無しさん** · NG

var unko;

**デフォルトの名無しさん** · 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 とかきっちり名前付ければ間違え要がないだろ

**デフォルトの名無しさん** · 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
お前が寂しい奴だということはわかった

**515** · 2008/02/14(木) 21:21:19

俺の頭が死んでたｗｗｗ逆だ逆ｗｗｗｗ
bRetとかしか書かない奴は死ね。

**デフォルトの名無しさん** · 2008/02/15(金) 21:25:59

【放送】報道ステーション・古舘伊知郎氏が謝罪　「アダルトチルドレン」言葉の意味を知らずに発言し、関係者を傷つける★３
http://mamono.2ch.net/test/read.cgi/newsplus/1203060700/

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

**デフォルトの名無しさん** · 2008/02/15(金) 21:57:54

コイズミチルドレン

**デフォルトの名無しさん** · 2008/02/18(月) 01:52:08

ファーストチルドレン

**デフォルトの名無しさん** · 2008/02/21(木) 22:47:34

絶対可憐チルドレン

**デフォルトの名無しさん** · 2008/02/27(水) 06:17:18

ミスターチルドレン

**デフォルトの名無しさん** · 2008/02/28(木) 04:48:12

自分はまだ学生なんだけど隣の席の奴がカウンタ i ii iii iiiiっていうふうにしててすごい見にくい

**デフォルトの名無しさん** · 2008/02/28(木) 07:36:38

ループが４重になってる時点で基本的におかしい。

**デフォルトの名無しさん** · 2008/02/28(木) 10:26:43

>>524
for ループ1個ごとに変数名変えてるとか？
いずれにしろ、変数名以前の問題で痛いけど。

**デフォルトの名無しさん** · 2008/02/28(木) 21:31:48

i ii iii iv にすればいいのにと思ったｗ

**デフォルトの名無しさん** · 2008/02/29(金) 02:41:39

ゲシュタルト崩壊するからやめてやめて

**デフォルトの名無しさん** · 2008/02/29(金) 13:42:34

xivとかみたくない。

**デフォルトの名無しさん** · 2008/02/29(金) 16:11:36

>>102
Blue-rayは「青い光」で、固有名詞じゃないからアメリカで商標登録できなかったってのが原因だぜ。

**デフォルトの名無しさん** · 2008/03/01(土) 17:29:42

頭.clear();

**デフォルトの名無しさん** · 2008/03/11(火) 13:00:35

ハゲたのか

**デフォルトの名無しさん** · 2008/03/18(火) 23:01:47

ExcelVBA始めたばっかの頃

Dim 魔貫光殺法 as hoge

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

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

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

**デフォルトの名無しさん** · 2008/03/20(木) 14:59:12

漏れるって段階で会社としてダメだな

**デフォルトの名無しさん** · 2008/05/02(金) 13:05:18

まぁ、そんなこともあるさｗ

**デフォルトの名無しさん** · 2008/05/04(日) 02:02:44

漏れる事が悪いかとか何とかってのは、ここではどうでもいいがな

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

**デフォルトの名無しさん** · 2008/05/04(日) 09:41:24

意味不明

**デフォルトの名無しさん** · 2008/05/10(土) 18:46:16

変数名にこれが付いてたら大体クソ

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

**デフォルトの名無しさん** · 2008/06/16(月) 16:37:24

～オレ用関数の命名規則～Version.0.0.1
主旨：関数を副作用の強弱でレベル分けする。副作用の少ない関数を、＜純粋度が高い関数＞と呼ぶことにする。
　　　適用する命名規則は低い方に合わせる。自分の呼び出す関数の中で、もっとも低い方に合わせた規則を適用する（main関数は例外とする）
　　　

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

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

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

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

**デフォルトの名無しさん** · 2008/06/16(月) 16:39:25

↑誤爆したorzマジでごめん

**デフォルトの名無しさん** · 2008/06/16(月) 19:53:55

＞全て小文字で、最初の文字をアンダースコアにする

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

**デフォルトの名無しさん** · 2008/06/16(月) 21:55:50

>>540
最初のアンダースコアを使うのは止めておくことにします。長文の投稿失礼しました。

**デフォルトの名無しさん** · 2008/06/17(火) 07:00:34

誤爆なの？ｗ

**デフォルトの名無しさん** · 2008/06/17(火) 22:36:11

hoge を使う奴はたいてい初心者

**543** · 2008/06/17(火) 23:07:59

プロの俺はpneumonoultramicroscopicsilicovolcanoconiosisを使う

**デフォルトの名無しさん** · 2008/06/17(火) 23:10:08

珪性塵肺症か。

**デフォルトの名無しさん** · 2008/06/17(火) 23:17:08

なげえプレースホルダーだなｗ

**デフォルトの名無しさん** · 2008/06/17(火) 23:34:31

初心者の女PGでmegu1 megu2 megu3って自分の名前を付ける奴がいた
Class Megu1{
meguArg01 = ...;
meguArg02 = ...;

meguFunc(){...}
}

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

**デフォルトの名無しさん** · 2008/06/17(火) 23:46:30

>>547
中身をじろじろ見ないでくれという
初級プログラマからの心の叫び。

**デフォルトの名無しさん** · 2008/06/19(木) 04:55:47

>>547
かわいいなｗ

**デフォルトの名無しさん** · 2008/06/19(木) 09:23:29

>>540
C/C++に関してはされてないな

**デフォルトの名無しさん** · 2008/06/19(木) 11:22:47

うむ、C/C++ に関しては、アンダースコア二つで始まるのがメーカー独自拡張だな。

**デフォルトの名無しさん** · 2008/06/19(木) 11:40:06

>>550-551
いや、アンダースコア始まりで小文字が続くものもグローバルスコープで予約されてるよ。

**デフォルトの名無しさん** · 2008/06/19(木) 11:47:22

みんながあまりに安易に多用しすぎて混乱しそうだったから予約したんだろうな

**デフォルトの名無しさん** · 2008/06/19(木) 18:43:31

どうでもいいけど言語によって名前のつけ方のセオリーって全然違うんじゃないの

**デフォルトの名無しさん** · 2008/06/20(金) 01:28:19

>>554
ひまわりとか？ｗ

**デフォルトの名無しさん** · 2008/06/20(金) 07:25:03

>>552
VC++ の独自の関数とかそうだしね。
_stricmp とか。

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

**デフォルトの名無しさん** · 2008/06/20(金) 08:05:50

>>547
俺もやってた時期があったｗ
どうせバグ報告が来るなら誰が書いたかすぐ解った方が良い、と本気で思ってた。
二週間と続かなかったが。

**デフォルトの名無しさん** · 2008/06/20(金) 12:33:09

>>551
＞アンダースコア二つ
C++では、識別子に含めることが禁止されている。
先頭だけじゃなく途中でもダメ。

**デフォルトの名無しさん** · 2008/06/20(金) 20:02:32

処理系が使用する分には問題ない

**デフォルトの名無しさん** · 2008/06/21(土) 03:07:46

ハマショーが好きな上司が、クラス名に SurrenderLikeSnow って付けてた。
「悲しみは雪のように」って言おうとしたんだろうが、たぶんSadnessの間違いだと思う

**デフォルトの名無しさん** · 2008/06/21(土) 13:25:26

英訳を頑張っちゃった系のソースは読むのがツライ。
その上スペルミスもあると、もう何がなんだかわからなくなる。

**デフォルトの名無しさん** · 2008/06/21(土) 13:32:12

スペルミス大量でインデントもぐちゃぐちゃなソースがあるけど
意外と読めるもんだな・・・。
イライラはするが。

**デフォルトの名無しさん** · 2008/06/21(土) 16:28:26

ローカル変数ならまだいいけどDBのテーブル名、カラム名でスペルミスは勘弁してほしい。
使ってる方が恥ずかしくなってくるぜ。

**デフォルトの名無しさん** · 2008/06/21(土) 21:59:08

なんにせよuwariteよりはマシ。

**デフォルトの名無しさん** · 2008/07/14(月) 00:08:28

変数名のセンスが無いやつほど
無駄な一時変数を好む傾向にある。

**デフォルトの名無しさん** · 2008/07/14(月) 09:32:27

名前ではないけど、ヘッダファイル(当然いくつかのファイルで include されている)で
static 変数を定義して、値が変わらないと悩んでいた人がいた。そら変わらんわなあ。

**デフォルトの名無しさん** · 2008/07/14(月) 09:52:44

それは、多重定義のエラーが出てstaticをつけたらエラーが消えたからそのまま使ってるんだろうｗ

**デフォルトの名無しさん** · 2008/07/15(火) 00:52:58

>>566
ヘッダファイルに変数を書く時点で既に変。
（つか、普通はextern 宣言だけだろ？）

**デフォルトの名無しさん** · 2008/07/15(火) 01:39:59

>>567
あるある

**デフォルトの名無しさん** · 2008/07/16(水) 21:44:13

>>566-569
それ（static無）やってる職場に以前いたんだが、(当然ながら)全く同じ定義のせいかエラーが出ずにリンク出来てた。
動作も正常。むしろエラーになってくれる方がありがたかったな。という思い出。

**デフォルトの名無しさん** · 2008/07/17(木) 01:33:32

>>568も変だけどな。

**デフォルトの名無しさん** · 2008/07/17(木) 20:12:07

変じゃないだろ
ヘッダでメモリ確保しちゃダメなのは常識

**デフォルトの名無しさん** · 2008/08/22(金) 14:10:24

>>572
ｋｗｓｋ

**デフォルトの名無しさん** · 2008/08/22(金) 22:19:51

送信済みデータ保存関数

saveSendedData

**デフォルトの名無しさん** · 2008/08/22(金) 22:34:56

>>574
読み込み済みデータでreadedDataってのもあったぞ。

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

>>573
ヘッダファイルとして分離する目的は何でしょう？
複数のモジュールから、同一の定義を参照する事が目的。

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

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

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

>>547
getmegu
Insert_megu
open_megu (man, co)
せくはら

**デフォルトの名無しさん** · 2008/08/23(土) 21:37:33

VB質問スレから
Dim strFikeNamae

これは秀逸だと思ったので
今までfnameやfilenameにしていたけど
これからはfnamaeやfilenamaeにしようと思った。

**デフォルトの名無しさん** · 2008/08/23(土) 22:42:06

一瞬単なるタイポかと思わせておいて「なまえ」とは･･

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

public void Create_String_Procedure( string arg1, string arg2 ){
・・・・・
}

PCをけっ飛ばしたくなった。

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

-aeで終わるとラテン語の複数形みたいに見えるな
ラテン語でnameはnomenだが。

**デフォルトの名無しさん** · 2008/08/24(日) 00:11:50

aglicora, aqua

**デフォルトの名無しさん** · 2008/08/24(日) 00:32:55

programa

**デフォルトの名無しさん** · 2008/08/24(日) 00:34:34

COM次第だな

悪のCOMをインストールするのだ

**デフォルトの名無しさん** · 2008/08/25(月) 21:31:48

>>578
Fikeってなんかカッッコイイ

**デフォルトの名無しさん** · 2008/08/26(火) 00:11:43

>>580
それだけだと何が悪いのか分からん

**デフォルトの名無しさん** · 2008/08/28(木) 00:08:42

>>580
C#か？キャメルケースなのにアンダーバーでくくられているのが気になるのか、
副作用がある作りにしているのが気になるのか、
argうんちゃっらっていう意味もへったくれもない変数が気になるのか、
Procedureとかわざわざ名前をつけているのが気になるのかハッキリ汁。

あとお前、この業界にいるには
気が短すぎるのとコミュニケーション能力が低すぎると思われるから
他人のこと言う資格ないと思うぞ。

**デフォルトの名無しさん** · 2008/08/28(木) 01:39:19

まースレタイ通り、変数名・関数名を言ってんだろう。
予想："どんな"StringをCreateするProcedureなのか分からない。ってことじゃないか？

**デフォルトの名無しさん** · 2008/08/28(木) 09:57:27

dataやarg1, arg2みたいな名前が適切な場合もあるよね。
GTKでは任意のデータを受け付ける(void *) dataっていう引数を
引き回していたような記憶。
Lispでも高階関数を定義するときにarg1, arg2みたいな引数名を使ったりする。
（この場合は可変長のargsとかが良いのかもしれんが。）

**デフォルトの名無しさん** · 2008/08/28(木) 19:53:39

自分だったらl,r使うかなあ

**デフォルトの名無しさん** · 2008/08/28(木) 22:00:25

lhs rhs

**デフォルトの名無しさん** · 2008/08/28(木) 23:46:27

>>591
それ、3文字略語はマナー悪いよなぁとも思うし、
もはや専門用語化してて略語じゃないと思える気もするし。

**デフォルトの名無しさん** · 2008/08/29(金) 06:28:31

3つ以上あったらどうするの

**デフォルトの名無しさん** · 2008/08/29(金) 08:48:05

まあ、lhs, rhs は2項演算に対して使う用語だし。

**デフォルトの名無しさん** · 2008/08/30(土) 06:52:15

>>594
ソートに使用する比較関数の変数とかね。

>>593
3つの引数が等価な計算とか無理やりな例以外見たことない。

**デフォルトの名無しさん** · 2008/08/30(土) 08:49:39

>>580の引数がたまたま2つだったから二項演算の話になってるのかな
ちょっと理解に苦しむ

**デフォルトの名無しさん** · 2008/08/31(日) 04:47:26

正直、Effective C++ではじめてrhsとlhsが何の略なのか知った。

**デフォルトの名無しさん** · 2008/08/31(日) 06:12:26

ロハスとロハスだろ。

**デフォルトの名無しさん** · 2008/09/01(月) 13:40:01

Perlだと比較関数で使う仮変数は$a, $bだね。
sort { $b <=> $a } @values
みたいに指定すると降順ソートができる。安直だけど分かりやすい。

**デフォルトの名無しさん** · 2008/09/03(水) 18:46:51

でも、$a, $bを特殊変数に指定してしまったのはかなり汚い。

**デフォルトの名無しさん** · 2008/09/04(木) 22:35:23

int debu1 ;
int debu2 ;
int debu3 ;
同僚でこんな変数を作っているやつがいた。

「デブデブデブっておまいデブ専なのか？」って聞いたら
debuはdebugの略だと言っていた。

**デフォルトの名無しさん** · 2008/09/05(金) 05:57:30

でぶまで書いたんならデバッグまで書けよでぶ

**デフォルトの名無しさん** · 2008/09/07(日) 15:59:30

>>600
どんな変数名がいいと思う？