回答編集履歴
3
修正
answer
CHANGED
@@ -3,15 +3,19 @@
|
|
3
3
|
まず第一にasListというメソッドはあなたのコード上存在していません。
|
4
4
|
おそらく以下のような処理を期待していると思います。
|
5
5
|
```Java
|
6
|
+
public class Test {
|
7
|
+
public static void main(String[] args) {
|
6
|
-
//
|
8
|
+
//省略
|
7
|
-
private static List<User> asList(String... emails) {
|
8
|
-
List<User> list = new ArrayList();
|
9
|
-
for(int i=0;i<emails.length;i++) {
|
10
|
-
User user = new User();
|
11
|
-
user.setEmail(emails[i]);
|
12
|
-
list.add(user);
|
13
9
|
}
|
10
|
+
private static List<User> asList(String... emails) {
|
11
|
+
List<User> list = new ArrayList();
|
12
|
+
for(int i=0;i<emails.length;i++) {
|
13
|
+
User user = new User();
|
14
|
+
user.setEmail(emails[i]);
|
15
|
+
list.add(user);
|
16
|
+
}
|
14
|
-
|
17
|
+
return list;
|
18
|
+
}
|
15
19
|
}
|
16
20
|
```
|
17
21
|
次にasListに渡す引数がシングルクォーテーションで書かれていることが問題です。
|
@@ -19,4 +23,7 @@
|
|
19
23
|
|
20
24
|
余談ですが、そもそもJavaのSetクラスは重複を自動で判定してくれます。
|
21
25
|
とりあえず全メールアドレスを突っ込めば、変数setの中はダブリのないメールアドレス一覧になっています。
|
22
|
-
重複である、というメッセージを出すことが目的であれば、判定部分はいまのままのコードで大丈夫です。
|
26
|
+
重複である、というメッセージを出すことが目的であれば、判定部分はいまのままのコードで大丈夫です。
|
27
|
+
|
28
|
+
#追記
|
29
|
+
追加したasListはstaticメソッドというものです。メソッドを定義すべき場所、staticの意味を理解できていないのでしたら、そのへんは**必ず**勉強したほうが今後のためになるでしょう。
|
2
追記、修正
answer
CHANGED
@@ -3,7 +3,8 @@
|
|
3
3
|
まず第一にasListというメソッドはあなたのコード上存在していません。
|
4
4
|
おそらく以下のような処理を期待していると思います。
|
5
5
|
```Java
|
6
|
+
//Testクラスの中
|
6
|
-
|
7
|
+
private static List<User> asList(String... emails) {
|
7
8
|
List<User> list = new ArrayList();
|
8
9
|
for(int i=0;i<emails.length;i++) {
|
9
10
|
User user = new User();
|
1
訂正
answer
CHANGED
@@ -16,5 +16,6 @@
|
|
16
16
|
次にasListに渡す引数がシングルクォーテーションで書かれていることが問題です。
|
17
17
|
Javaでのシングルクォーテーションは「文字」であり、「文字列」ではありません。2文字以上の長さの文字列はダブルクォーテーションで囲む必要があります。
|
18
18
|
|
19
|
-
|
19
|
+
余談ですが、そもそもJavaのSetクラスは重複を自動で判定してくれます。
|
20
|
-
とりあえず全メールアドレスを突っ込めば、変数setの中はダブリのないメールアドレス一覧になっています。
|
20
|
+
とりあえず全メールアドレスを突っ込めば、変数setの中はダブリのないメールアドレス一覧になっています。
|
21
|
+
重複である、というメッセージを出すことが目的であれば、判定部分はいまのままのコードで大丈夫です。
|