Javaでこんなメソッドを作りたい

Question

public List<String> arrayzAsList(....){

//....にはStringまたはList<String>のいずれかがいくつか並ぶ。Arrays.asList(.....)と同じパターン。

List<String> rtrn = new ArrayList<String>;

//List<String> は分解してString1個ずつrtrnにaddする。
//Stringはそのままrtrnにaddする。
//最終的にrtrnとして返す。

return rtrn;
 
}

List<String> a = new ArrayList<String>();

a.add("a");
a.add("b");
a.add("c");

List<String> z = arrayzAsList("d", a, "e", a);

System.out.println(z);
//["d", "a", "b", "c", "e", "a", "b", "c"]

メソッドarrayzAsListを作れたらありがたいです。よろしくお願いいたします。上記2つとも同じクラス内にあるものとします。

Comments

[gentaro]

> メソッドarrayzAsListを作れたらありがたいです。よろしくお願いいたします。上記2つとも同じクラス内にあるものとします。

https://teratail.com/help/avoid-asking
「何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。」

[y_waiwai]

どんなメソッドでしょうか。いまいち意味不明です

[LouiS0616]

arrayzAsList(a, "d", "e") とか arrayzAsList("d", a, a) にも対応させたいのでしょうか？

[退会済みユーザー]

?

https://docs.oracle.com/javase/jp/8/docs/api/java/util/Arrays.html#asList-T...-

これ?

[tiqua_nibio]

>arrayzAsList(a, "d", "e") とか arrayzAsList("d", a, a) にも対応させたいのでしょうか？

はい、そうです。

[tiqua_nibio]

>これ?

そうです。(T... a)のところには、いくつも並ぶという意味ですよね。これをStringまたはList<String>などが並ぶということです。この部分でハマりました。

[tiqua_nibio]

>どんなメソッドでしょうか。いまいち意味不明です

SQLのデータベースのカラムが多くなりすぎたので、SQLを生成するコードを書いています。また、リファクタリングでもあります。

[tiqua_nibio]

>「何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。」

それをおっしゃるということであれば、答えられる自信があるのではと推測します。
学校の課題などではありません。一歩進んだ目的があってのことです。

[LouiS0616]

別の言語を使うことも選択肢に入れてみてはどうでしょうか。
前回の質問 https://teratail.com/questions/253768 もそうですけれど、容易に実現できないのは言語の思想も絡んでいるわけで。

[gentaro]

> 学校の課題などではありません。一歩進んだ目的があってのことです。

じゃあそれを説明し、ガイドラインに目を通した上で「丸投げの質問ではない」証明をお願いします。

[退会済みユーザー]

一歩進んだなら　JPAとかつかうから詰め替えの実装なんて発生しないよ

[tiqua_nibio]

>じゃあそれを説明し、ガイドラインに目を通した上で「丸投げの質問ではない」証明をお願いします。

Object o
o instanceof List<String>

がエラーになるから、質問しているのです。丸投げと決めつけないでください。

[tiqua_nibio]

>別の言語を使うことも選択肢に入れてみてはどうでしょうか。

ご意見ありがとうございます。

[退会済みユーザー]

試してエラーになったことを書かなければそれはやってないことと同じです。

[swordone]

> Object o
o instanceof List<String>

がエラーになるから、質問しているのです。丸投げと決めつけないでください。

じゃあ最初からそう書きなさいよ。

[tiqua_nibio]

皆さん色々と批判されているようですが、私にとっては重要な質問の一つであり、答えを提示いただくことで色々と前進することにつながります。
批判する前に、まず技術的に難しいのではないか、などの建設的なコメントを頂けないかと少しがっかりしております。

Answer 1

public class Formatter {


	public static List<String> arrayzAsList(Object... a) {

		List<String> list_str = new ArrayList<String>();

		for (Object o : a) {

			if (o instanceof String) {

				list_str.add((String) o);

			} else if (o instanceof ListString) {

				ListString ls = (ListString) o;

				for (String temp_str : ls.getList_Str()) {

					list_str.add(temp_str);

				}

			}

		}

		return list_str;
	}

	public class ListString {

		public ListString(List<String> list_str) {

			this.list_str = list_str;

		}

		private List<String> list_str;

		public List<String> getList_Str() {

			return list_str;

		}

		public void setList_Str(List<String> list_str) {

			this.list_str = list_str;

		}

	}

	public static void main(String[] args) throws NoSuchAlgorithmException {

		ListString ls_1 = new Formatter().new ListString(Arrays.asList("tata", "nana", "haha"));
		ListString ls_2 = new Formatter().new ListString(Arrays.asList("yaya", "rara"));
		
		System.out.println(arrayzAsList("sasa", ls_1, "mama", ls_2));
		
		//[sasa, tata, nana, haha, mama, yaya, rara]

	}

}

Answer 2

	public static List<String> arrayzAsList(Object... objs) {
		List<String> ans = new ArrayList<String>();
		for (Object obj : objs) {
			if (obj instanceof String) {
				ans.add((String) obj);
			} else if (obj instanceof List) {
				for (String s : (List<String>) obj) {
					ans.add(s);
				}
			}
		}
		return ans;
	}

実行例：

追記 2020--4-19
exception 処理する例をつくりました。
(でも本当は、 可変長引数にするのをやめて、 List<String> の配列を引数にするのが良いです。
(String は 要素数 1 の List<string> にすればよいです。そうすれば、 instanceof でチェックする必要がなくなります。)

import java.util.ArrayList;
import java.util.List;

public class Main {
	public static void main(String[] args) {
		List<String> a = new ArrayList<String>();
		a.add("a");
		a.add("b");
		a.add("c");
		List<String> z = arrayzAsList("d", a, "e", a);
		System.out.println(z);

		List<Object> b = new ArrayList<Object>();
		b.add(1);
		b.add(1.2);
		b.add('c');
		arrayzAsList(b, 10, 12.3, 'Z');
	}

	public static List<String> arrayzAsList(Object... objs) throws IllegalArgumentException {
		String errors = "";
		List<String> ans = new ArrayList<String>();
		for (Object obj : objs) {
			if (obj instanceof String) {
				ans.add((String) obj);
			} else if (obj instanceof List<?>) {
				for (Object e : (List<?>)obj) {
					try {
						ans.add((String)e);
					} catch (ClassCastException exp) {
						errors += "\n  " + e + " (" + obj.getClass().toString() + ")";
					}
				}
			} else {
				errors += "\n  " + obj + " (" + obj.getClass().toString() + ")";
			}
		}

		if (errors.length() > 0) {
			throw new IllegalArgumentException(errors);
		}
		return ans;
	}
}

実行例：

追記 その２

メソッドの引数を 可変長でなく、 List<List<String>> にしてみました。

Comments

[tiqua_nibio]

ありがとうございます。
私も回答を書いてみたのですが、私の回答よりもかんたんに書かれていますね。ジェネリクスを書かないのがポイントのようですね。

[katoy]

こらはタイプチェックが甘いです。
lList<String> や String 以外が渡ってきたときは破綻します。

[jimbe]

いっそ for 文の中は
if(obj instanceof List) {
for(Object o : (List<?>)obj) ans.add(o.toString());
} else {
ans.add(obj.toString());
}
でも良いのではないでしょうか.

[tiqua_nibio]

ありがとうございます。

>いっそ for 文の中は

Exception処理はどうやって入れたらよろしいでしょうか？
katoyさんのコードに最後にelse節があればできそうですね。

[tiqua_nibio]

>lList<String> や String 以外が渡ってきたときは破綻します。

Exception処理を入れられればなんとかなりそうです。
ですが、コード的に私のように別のクラスを作らなくてすむという意味で、スッキリしていると思いました。参考にさせていただきたいと思います。

[katoy]

exception 処理する例を回答に追記しました。
ここでは すべての不正データをみつけられうようにしてますが、
単位 全体を try .. catch で囲むだけでも十分かもしれません。

[jimbe]

> Exception処理はどうやって入れたら
String も List も, そして引数も List の中身も Object ですので, Object として扱う限りはキャスト例外は発生しないと思うのですが...(回答に追加されました数値を含む List を渡されても発生しません.)
他の例外で何が発生するとお考えでしょうか.

[katoy]

2 つめの回答例では、 List<object> の要素に String 以外 int, float を入れています。
メソッドが返す List<STring> には String だけしか入れられません。
int や fllot なども .toString() して STring に変換していれるという案もありますが、
ここでは、(string)e として 入れるようにしています。このキャスト部分で例外が発生します。

[jimbe]

私のコメントは >いっそ for 文の中は で提案させて頂いた ".toString() して String に変換していれるという案" でのことです. 紛らわしくてすいません.

[jimbe]

ただ, キャストによる例外の対処まで含めるとなりますと, java としては「戦略の不味さを戦術でカバー」しているような印象を受けます.
このメソッドが必要になるという対処方法自体, もしくはより根本的なレベルから再考される余地があるのではとも考えてしまいます.
"リファクタリング" とも言われていますので, その過程での一時的な対処であれば仕方ないのかも知れませんが...

[tiqua_nibio]

> exception 処理する例を回答に追記しました。
ありがとうございます。こちらも大変参考になります。

[tiqua_nibio]

>"リファクタリング" とも言われていますので, その過程での一時的な対処であれば仕方ないのかも知れませんが...

当方、データベースでupdateをするときに、特定のカラムのみを更新したいというときに、List<String>にそのカラム名のリストを入れ、それを更新後の値とともにあるメソッドに入れれば、更新してくれるというものを作動させています。

カラムが多くなりすぎましたので、一部のカラムをグループ化して、List<String>にして他のカラムとarrayzAsListで一つのList<String>にしているというものです。
大変便利で、数百行のコードが一行になりました。

[jimbe]

なるほど. ということは扱われる各要素はまず String 以外あり得ないのですね.
キャスト例外が発生するとすれば(カラム名に数値が入るとは思えませんので)使い方を間違えている場合ということになりますか.
とすればキャスト例外が発生したとすればバグということになりますし, それはそのまま上位メソッドに伝搬するだけで良いのかもしれません.

[tiqua_nibio]

はい、ユーザー側の操作によるバグというよりは開発者側のバグということになりそうです。カラム名ですので、String型のみとなります。
上位へ伝える形に進めております。

[katoy]

可変長引数でなく、 List<List<String>> にしたもの回答の 追記 2 として書きました。

[tiqua_nibio]

List<List<String>>という方法もありますね。

昔、少し複雑なコードを書いたときに試してみたことがありました。
ジェネリクスが重なっているので、見栄えを考慮して新しいクラスを作ったりなどをしていました。
またstreamなどのメソッドは奥が深いと思います。

Answer 3

StringとListを同列に扱うにはObjectを使うしかないので、やるとしたらこうです。

public List<String> arrayzAsList(Object... objs)

ただし、色々と面倒です。