Dotaz: Java Arraylist polí a přidávání hodnot

[Raaďa]

Zpracovávám výstup z ERP, který dostanu přes http a který je v xml. Z toho si povytahám údaje co mě zajímají (vbeln, posnr) a tyto chci naskládat do arraylistu (každý item je pole {vbeln, posnr}. Do arraylistu vkládám tyto údaje ve for smyčce. Úryvky z kódu:

...
ArrayList<String[]> processedXML = new ArrayList<String[]>();
...

void processPreview()throws Exception {
     String vbeln = "";
     String posnr = "";
     String[] values = new String[2];

     String[] s1 = this.previewXML.split("</d:Vbeln><d:Posnr>",-1);

     int len = s1.length;
     for (int i=0; i<len-1; i++) {
         int len2 = s1[i].length();
         vbeln = s1[i].substring(len2-10, len2);
         posnr = s1[i+1].substring(0, 6);
 
         values[0] = vbeln;
         values[1] = posnr;
         this.processedXML.add(values); //tady přidávám values do arraylistu
 
         System.out.println(values[0] + ", " + values[1]);  //kontrolní výpis
     }

     System.out.println();

     for (int i=0; i<len-1; i++) {
         values = this.processedXML.get(i);
         System.out.println(values[0] + ", " + values[1]);  //ještě jeden kontrolní výpis
     }

 }

Výpis bohužel vypadá následovně:
klmnopqrst, 000020
uvwxyz0123, 000010
uvwxyz0123, 000020
uvwxyz0123, 000040

uvwxyz0123, 000040
uvwxyz0123, 000040
uvwxyz0123, 000040
uvwxyz0123, 000040

Zdá se, že do arraylistu takto nevkládám konkrétní hodnoty, ale ukazatele, které pak všechny ukazují na původní pole values, ve kterém zůstávají poslední vložené hodnoty. 

Jak mám do arraylistu vložit konkrétní hodnoty a ne ukazatele?

předem dík za rady

---

---

Odpovědi

[Bherzet]

Vždyť sis to zodpověděl skoro sám. Vkládáš tam referenci na pole values, jehož obsah se neustále mění. Jak by asi tak mohlo vypadat řešení? Možná tam třeba nevkládat referenci na jedno a totéž pole?

[Bherzet]

Jak mám do arraylistu vložit konkrétní hodnoty a ne ukazatele?

To mimochodem v Javě není možné pro nic jiného než primitivní typy (mezi něž java.lang.String nepatří). Což neznamená, že tvoje úloha nemá řešení (viz výše), jen je potřeba vkládat tam referenci na správný objekt.

[OldFrog {Ondra Nemecek}]

Možná tam třeba nevkládat referenci na jedno a totéž pole

Tým sa myslí, že miesto

String[] values = new String[2];
for (...)
  ...
  processedXML.add(values)

potrebuješ

for (...)
  ...
  String[] values = new String[2];
  ...
  processedXML.add(values)

[Bherzet]

Chtěl jsem ho navést, aby na to přišel sám…

[Kit]

Vkládej tam klony.

[xkucf03]

Ad patička:

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

to sedí :-)

this.processedXML.add(values); //tady přidávám values do arraylistu

[OldFrog {Ondra Nemecek}]

Tady je špatně snad úplně všechno…

Nejrychlejší oprava: řádek this.processedXML.add(values); //tady přidávám values do arraylistu ti do toho seznamu přidává pořád to samé pole – tzn. máš jedno pole, v něm si přepisuješ hodnoty a dáváš ho opakovaně do seznamu. Přesuň si řádek String[] values = new String[2]; dovnitř cyklu.

Nevím, v čem jsi programoval dřív, ale podle stylu mi to přijde jako že v céčku před třiceti lety :-)

• Proměnné si nemusíš deklarovat na začátku metody a cpát do nich prázdný řetězec.
• Java je primárně objektový jazyk, využívej toho. Když vidím někde seznam polí řetězců, tak je to téměř vždy chyba návrhu. Není to čitelné, kód nedokumentuje sám sebe, když k tomu přijdu, tak nevím, co má být na první pozici v poli, co na druhé, jaký je význam těch prvků… Vytvoř si třídu, která bude mít dva atributy, a dej jí nějaký výstižný název. Pak do toho seznamu dávej instance té třídy.
• Neviděl jsem zbytek kódu, ale ty this.previewXML a this.processedXML vypadají trochu jako globální proměnné resp. trpí stejným nešvarem. Nebylo by lepší předávat vstup jako parametr metody a výstup vracet jako její návratovou hodnotu? (tzn. byla by to spíš funkce, bez stavu, bez vedlejších efektů) Nebo si rozmysli návrh té třídy, jaký stav si bude držet, jak se bude řešit přístup z více vláken nebo opakované volání metody processPreview()
• Proč len-1? Tím přeskočíš poslední prvek. Nebo používej for-each (for s dvojtečkou).
• Pro práci s XML existují knihovny, jsou součástí Javy. Pracovat s XML jako s textem a dělit ho podle </d:Vbeln><d:Posnr> fakt není dobrý nápad. Když budou na vstupu navíc mezery mezi elementy nebo odsazení, tak to bude pořád platné XML se stejným významem, ale tvůj „parser“ přestane fungovat. Totéž, když se změní třeba prefix jmenného prostoru. O nějakém escapování ani nemluvě. Prostě použij knihovnu, možností máš plno, buď si to namapuj na objekty přes JAXB, nebo to zpracuj přes SAX, DOM, XPath…

[OldFrog {Ondra Nemecek}]

Přehled možností viz třeba https://www.baeldung.com/java-xml Jackson má výhodu, že umožňuje totožnou práci s formátem JSON, XStream a XStream a Simple XML mají zase výhodu v jednoduchosti.

[Raaďa]

Díky za všechny komentáře. Vskutku nejsem javista. Bohužel, i v dnešní době se někde musí používat jazyk, proti kterému je céčko před 30-ti lety úžasně (super)moderní jazyk.

[Kit]

Když budeš v Javě psát jako v C, tak to vždy bude mizerné.

[Bherzet]

Není nic jednoduššího než svou neznalost a neschopnost svést na jazyk, že? Pro arogantní debily:

JavaScript:

> var arr1 = []
undefined
> var arr2 = arr1
undefined
> arr1[0] = 123
123
> arr1[1] = 456
456
> arr2
[ 123, 456 ]

Python:

>>> arr1 = []
>>> arr2 = arr1
>>> arr1.append(123)
>>> arr1.append(456)
>>> arr2
[123, 456]

C:

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[]) {
	int *arr1 = malloc(sizeof(int) * 2);
	int *arr2 = arr1;

	arr1[0] = 123;
	arr1[1] = 456;

	printf("%i %i\n", arr2[0], arr2[1]);
}

Groovy:

groovy:000> arr1 = []
===> []
groovy:000> arr2 = arr1
===> []
groovy:000> arr1[0] = 123
===> 123
groovy:000> arr1[1] = 456
===> 456
groovy:000> arr2
===> [123, 456]

A mohl bych pokračovat.