svnno****@sourc*****
svnno****@sourc*****
2009年 9月 24日 (木) 22:08:31 JST
Revision: 3667
http://sourceforge.jp/projects/jiemamy/svn/view?view=rev&revision=3667
Author: daisuke_m
Date: 2009-09-24 22:08:31 +0900 (Thu, 24 Sep 2009)
Log Message:
-----------
refactor.
また、@sinceのjavadocタグを付けるのは、public / protected のみとします。API提供開始バージョンをあらわすと共に、公開済みAPIであるため、メソッドシグネチャや戻り値を変更することが許されない、というマーキングにも使っています。
Modified Paths:
--------------
eros/maven-jiemamy-plugin/trunk/src/main/java/org/jiemamy/maven/ExportMojo.java
Modified: eros/maven-jiemamy-plugin/trunk/src/main/java/org/jiemamy/maven/ExportMojo.java
===================================================================
--- eros/maven-jiemamy-plugin/trunk/src/main/java/org/jiemamy/maven/ExportMojo.java 2009-09-24 12:46:19 UTC (rev 3666)
+++ eros/maven-jiemamy-plugin/trunk/src/main/java/org/jiemamy/maven/ExportMojo.java 2009-09-24 13:08:31 UTC (rev 3667)
@@ -197,7 +197,9 @@
Map<String, Method> removedDuplicateMethod = removeDuplicateMethod(interfaceMethods);
// サポートされている戻り値であるかチェックする
for (String methodName : removedDuplicateMethod.keySet()) {
- isSupportType(removedDuplicateMethod.get(methodName).getReturnType());
+ if (isSupportType(removedDuplicateMethod.get(methodName).getReturnType()) == false) {
+ throw new IllegalArgumentException("parameter type not supported: " + clazz);
+ }
}
// 実装クラスを生成
@@ -247,7 +249,6 @@
* @param clazz {@link Exporter}の実装クラス
* @return {@link Exporter}の要素型
* @throws MojoExecutionException {@link Exporter}が見つからない場合
- * @since 0.3
*/
private Type getExporterGenericInterface(Class<?> clazz) throws MojoExecutionException {
for (Type type : clazz.getGenericInterfaces()) {
@@ -272,7 +273,6 @@
*
* @param clazz 検査対象の{@code Class}
* @return {@code Method}の一覧
- * @since 0.3
*/
private List<Method> getInterfaceMethods(Class<?> clazz) {
List<Method> methods = new ArrayList<Method>();
@@ -297,25 +297,20 @@
}
/**
- * サポートされているクラスかチェックします。
+ * サポートされているクラスかチェックする。
*
* @param clazz 検査対象の{@link Class}
- * @return サポートしている場合{@code true}
- * @since 0.3
+ * @return サポートしている場合は{@code true}、そうでない場合は{@code false}
*/
private boolean isSupportType(Class<?> clazz) {
- if (suppertReturnTypes.contains(clazz) == false) {
- throw new IllegalArgumentException("parameter type not supported: " + clazz);
- }
- return true;
+ return suppertReturnTypes.contains(clazz);
}
/**
- * {@code Method}の一覧から重複を削除したMapを作成します。
+ * {@code Method}の一覧から重複を削除したMapを作成する。
*
* @param interfaceMethods {@code Method}のリスト
* @return メソッドの{@code Map}
- * @since 0.3
*/
private Map<String, Method> removeDuplicateMethod(List<Method> interfaceMethods) {
Map<String, Method> methodMap = new HashMap<String, Method>();