[[SeasarJavadocProject]]
*org.seasar.framework.container パッケージのふりかえり [#hadc6c81]

ふりかえりはML上では行いづらい部分があるためWiki上でやってみるテスト。

**2006.10.29 (Sun) org.seasar.framework.container KPT@赤坂 レポート [#h7f74ed7]
作成中…

***クラスコメント [#m6d95d8b]
--クラスコメントの先頭行は句点(。)まで改行しない。
--2行目(2文目)以降については<p>タグで括る。
--HTMLタグは小文字で記述する。
--@author タグの "(Javadoc)" は除く(そして気を引き締める (^^;))。ex. @author hoge (Javadoc) ⇒ @author hoge

***用語統一 [#mbbbdbdd]
|用語|コメント|
|引数定義||
|diconファイル||
|<arg>要素||
|インジェクション||
|コンテキスト||
|S2コンテナ||
|Expression||
|コンポーネント定義||
|アクセスタイプ(AccessTypeDef)|⇒アクセスタイプ定義|
|文字列表現||
|コンポーネント||
|状態||
|インジェクション||
|アクセスタイプ||
|ファクトリ||
|定数||
|バインディングタイプ定義||
|プロパティ定義||
|コンポーネント定義||

**文書表現統一 [#j3b26c6c]
|ポイント|表現|
|#getName()のコメント|"〜の文字列表現を返します" ⇒ "定義名を返します。"|
|||

**検討課題 [#a4af7a0b]
|No|課題|ステータス|検討結果|
|001|日本語・アルファベット混在の場合、括弧は全角で統一する?|保留||
|002|<arg>要素、<component>タグ などタグ要素の表現が不統一…|保留||
|003|初期化メソッド定義? initメソッド定義? ({@link InitMethodDef *})|保留|【方向性】初期化メソッド定義 にして、ペアとなる DestroyMethodDef の日本語表現を考える…|

作業中… Zz…
再開ポイント 15:02:50

** 修正項目一覧 [#ta015d6c]
|場所|内容|優先度|サインアップ|対応済み?|備考|
|全体|@authorの後の(Javadoc)を削除する。|||||
|package.html|Smart DeployをSMART Deployに変更する。|X||||
|AccessTypeDef/概要|サマリされる1文目の後に改行を入れる。|||||
|AccessTypeDef/全般|アクセスタイプという言葉をアクセスタイプ定義に統一する。|||||
|AccessTypeDef#getName()|「…文字列表現を返します。」ではなく、「アクセスタイプ定義名を返します。」に修正する。|||||
|ArgDef#isValueGettable()|「…いずれかが存在する場合、trueを返します。」を「…いずれかが存在し、値の取得が可能かどうかを返します。」といった表現に変更する。合わせて、戻り値の説明を「値の取得が可能な場合、trueを返す」といった表現に修正する。||vestige|vestige||
|ArgDefAware#addArgDef|登録(追加)を追加に書き換える。||jundu|||
|ArgDefAware#getArgDef()|他と統一するため、IndexOutOfBoundsExceptionの記述を削除する。||jundu|||
|AspectDef/概要|S2AOPの後のカッコ内の記述は、Javadocに不要に思われるので削除する。|||||
|AspectDef/概要|AOP一般に関する記述は、S2Containerの公式ドキュメントに任せ、そちらへの参照先のみとする。リンクにするかどうかは課題。|||||
|AspectDef/概要|インターセプタを複数登録することと、InterceptorChainを使うことの違いを記述できると良い。|||||
|AspectDefAware#addAspectDef|登録(追加)を追加に書き換える。|||||
|AutoBindingDef/概要|サマリされる1文目の後に改行を入れる。||jundu|||
|BindingTypeDef/概要|サマリされる1文目の後に改行を入れる。||jundu|||
|ComponentCreater/概要|Smart DeployをSMART Deployに変更する。|X||||
|ComponentCustomizer/概要|Smart DeployをSMART Deployに変更する。|X||||
|ComponentDef/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|ComponentDef/概要|コンストラクタ引数定義は、引数定義に修正する。|||||
|ComponentDef#getComponentClass()|サマリされる1文目の後に改行を入れる。|||||
|ComponentDef#getComponentClass()|<component/>を<component>に統一する。|||||
|ComponentDeployer/全般|コンポーネントインスタンス定義を、インスタンス定義に統一する。|||||
|ComponentDeployer/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|ConstructorAssembler/概要|サマリされる1文目の後に改行を入れる。||jundu|||
|ConstructorAssembler#assemble()|サマリされる1文目の後に改行を入れる。||jundu|||
|ContainerConstants#PACKAGE_SEP|Javadocの記述が漏れているようです。|||||
|DestoroyMethodDef/概要|「コンポーネントインスタンス定義」を「インスタンス定義」に統一する。|||||
|DestroyMethodAware#addDestoryMethodDef|登録(追加)を追加に書き換える。|||||
|ExternalContext/概要|「Webコンテナなどの外部コンテキスト」に修正してはいかがでしょうか?|||||
|ExternalContext#getInitParameterMap()|初期化設定値という言葉に違和感を覚えますが、いかがでしょうか? 初期設定値で通じる気がします。|||||
|IllegalAccessTypeDefRuntimeException/全体|「アクセスタイプ名」を「アクセスタイプ定義名」に統一する。|||||
|IllegalAccessTypeDefRuntimeException/コンストラクタ|アクセスタイプ名をアクセスタイプ定義名に統一する。|||||
|IllegalConstructorRuntimeException/概要|2行目が「コンポーネントの構築は」で始まっているのは日本語がおかしい。「この例外は」の間違いと思われる。||jundu|||
|IllegalMethodRuntimeException/概要|「適切な型にパース出来ない場合」は、「適切な型に変換出来ない場合」の方が分かりやすいように個人的には感じるのですが、いかがでしょうか?|||||
|InitMehtodDef/概要|「1個以上の任意のメソッド」のところは、「1つ以上の任意のメソッド」に変えてみてはいかがでしょうか。個人的にメソッドを1個と数えることに違和感を覚えました。|||||
|InitMethodDefAware#addInitMethodDef|登録(追加)を追加に書き換える。|||||
|InstanceDef/全般|「コンポーネントインスタンス定義」を「インスタンス定義」に統一する。|||||
|InstanceDef/概要|backport175アノテーションに関する記述は削除する。2.4以降でサポートされないため。|||||
|InterTypeDef/概要|インタフェースをインターフェースに統一する。(1行目、実装インタフェースのところ)|||||
|InterTypeDef/概要|空の@authorを削除するか、@authorの前に誰かの名前を書いたほうが良いように思います。|||||
|InterTypeDefAware/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|InterTypeDefAware/概要|空の@authorを削除するか、@authorの前に誰かの名前を書いたほうが良いように思います。|||||
|InterTypeDefAware#addInterTypeDef|登録(追加)を追加に書き換える。|||||
|MetaDef/概要|サマリされる1文目の後に改行を入れる。|||||
|MetaDefAware/概要|個人的には、メタデータの取得方法の説明は、リストではなく「または」で繋ぐだけでも良い気がします。項目が2つしかないので。|||||
|MetaDefAware#getMetaDef(String name)|サマリされる1文目の後に改行を入れる。|||||
|MetaDefAware#getMetaDefs(String name)|サマリされる1文目の後に改行を入れる。|||||
|MetaDefAware#getMetaDefs(String name)|@param nameが2つ書かれているので、1つ削除する。|||||
|MethodDef/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|PropertyAssembler#assemble|例外は、投げるではなくスローに統一する。||jundu|||
|PropertyDef/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|PropertyDefAware/概要|インタフェースをインターフェースに統一する。(1行目)|||||
|PropertyDefAware#addPropertyDef|登録(追加)を追加に書き換える。|||||
|PropertyDefAware#hasPropertyDef(String propertyName)|ここの説明は、「指定したプロパティ名のプロパティ定義があればtrueを返します。」ではないでしょうか。ちょっと自信ないですが。元の表現と言っている事は同じですか?|||||
|TooManyRegistrationComponentDef/概要|インタフェースをインターフェースに統一する。(1行目)|||||

トップ   新規 一覧 検索 最終更新   ヘルプ   最終更新のRSS