バージョン選択

フォーラム

メニュー

オンライン状況

47 人のユーザが現在オンラインです。 (38 人のユーザが フォーラム を参照しています。)
登録ユーザ: 0
ゲスト: 47
もっと...

サイト内検索

機能要望 > その他 > リファクタリング 変数名について

その他

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
seasoft
投稿日時: 2011/1/30 1:23
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7367
リファクタリング 変数名について
http://svn.ec-cube.net/open_trac/wiki/%E3%83%AA%E3%83%95%E3%82%A1%E3%82%AF%E3%82%BF%E3%83%AA%E3%83%B3%E3%82%B0%E3%82%AC%E3%82%A4%E3%83%89%E3%83%A9%E3%82%A4%E3%83%B3
テーブルカラム名と共通の変数名は, キャメル式を使用せず, アンダーバーで区切ること

<?php

/**
 * テーブルカラム名と共通の変数名は, 連想配列の添字で使用されるため,
 * アンダーバー区切りの方が扱いやすい
 */

/**
 * NG
 */
$productId = getProductId();

/**
 * OK
 */
$product_id = getProductId();


例のように直接連想配列を扱っているわけでないケースでは、カラム名か否かを意識する必要があるのは、実装が煩雑となるので避けたいです。また、「変数名の一部にカラム名を含む」というケースで、曖昧さが発生すると感じます。

「リテラルを格納する (つまり配列・オブジェクト以外の) 変数名は, キャメル式を使用せず, アンダーバーで区切ること」としてはいかがでしょうか?

また「キャメル式を使用せず」ではなく、「すべて小文字を使用し」の方が明確だと思います。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2011/1/30 16:26
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2314
Re: リファクタリング 変数名について
引用:

seasoftさんは書きました:

「リテラルを格納する (つまり配列・オブジェクト以外の) 変数名は, キャメル式を使用せず, アンダーバーで区切ること」としてはいかがでしょうか?

また「キャメル式を使用せず」ではなく、「すべて小文字を使用し」の方が明確だと思います。


方向性には相違ないと思いますので賛成です
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBE公式 Amazon Payプラグイン

統計情報

総メンバー数は89,041名です
総投稿数は110,020件です

投稿数ランキング

1
seasoft
7367
2
468
3217
3
AMUAMU
2712
4
nanasess
2314
5
umebius
2085
6
yuh
1819
7
h_tanaka
1646
8
red
1570
9
mcontact
1296
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
799
14 ramrun 789
15 karin 689
16 sumida 641
17
homan
633
18 DELIGHT 572
19
patapata
502
20
flealog
485


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.