の最適化、複数のJava配列の追加は

Question

私は、次のコード

JsonObject domainsObject = new JsonParser().parse(json).getAsJsonObject().get("data").getAsJsonObject(); 
     this.domain = domainsObject.get("main_domain").getAsString(); 

     this.domains.add(this.domain); 

     for(JsonElement domain : domainsObject.get("addon_domains").getAsJsonArray()) { 
      this.domains.add(domain.getAsString()); 
     } 

     for(JsonElement domain : domainsObject.get("parked_domains").getAsJsonArray()) { 
      this.domains.add(domain.getAsString()); 
     } 

     for(JsonElement domain : domainsObject.get("sub_domains").getAsJsonArray()) { 
      this.domains.add(domain.getAsString()); 
     } 

を持っています。

手動でループすることなく、3つの別々の配列と1つの文字列をすべて1つの配列にスカッシュする方法はありますか？

Answer 1

基本的に2つの方法があり、それぞれに利点と制限があります。

GrzegorzGórkiewiczは既に1つのリストとループを作成しました。代わりに、私は、配列を使用したいリストの：

String[] parameters = { "addon_domains", "parked_domains", "sub_domains" }; 

for(String parameter : parameters) { 
    for(JsonElement domain : domainsObject.get(parameter).getAsJsonArray()) { 
     this.domains.add(domain.getAsString()); 
    } 
} 

あなたがより良いリストのような場合は、Arrays.asList("addon_domains", "parked_domains", "sub_domains")を使用しています。 Chai T. Rexがコメントに言及しているように、配列またはリストをprivate static finalの外側に宣言して、メソッドを使用するたびに再度ビルドしないようにすることができます。

他の補助メソッドを呼び出すことです：

addDomainsFromParameter("addon_domains"); 
addDomainsFromParameter("parked_domains"); 
addDomainsFromParameter("sub_domains"); 

（あなたはもっと自分のプログラムを知って、より良いメソッド名を見つけることができ、私は方法を自分で書くためにあなたを信頼。。）後者の利点が来ますある時点で処理のバリエーションが必要な場合は、メソッドにさらにパラメーターを追加できます。

Answer 2

リストに入れてください。

List<String> parameters = new ArrayList<>(); 
parameters.add("addon_domains"); 
parameters.add("parked_domains"); 
parameters.add("sub_domains"); 

for(String parameter : parameters) { 
    for(JsonElement domain : domainsObject.get(parameter).getAsJsonArray()) { 
     this.domains.add(domain.getAsString()); 
    } 
}