salvare file xml [Java]

[Metflar]

nel salvare un file xml mi sn accorto che nn mi da il risultato sperato...ma mi da 'n' volte gli stessi dati
ecco il codice che penso dia il problema:

Element mast= new Element("Libro_mastro");

                

                for(int i=0;i<MTabbedP.mp.getComponents().length;i++){

                    MastP co=(MastP)MTabbedP.mp.getComponent(i);//questo è un pannello con tre campi di testo name,d,a

                    Element m = new Element("Mastro");

                    m.setAttribute("id",String.valueOf(i));

                    m.setText(co.name.getText());

                    

                    Element d =new Element("Dare");

                    d.setText(co.d.getText());

                    

                    Element a =new Element("Avere");

                    a.setText(co.a.getText());

                    

                    m.addContent(d);

                    m.addContent(a);

                    mast.addContent(m);

                } 

[lucke84]

scrivi il set di risultati che ti da

[Metflar]

questo è un esempio ho creatp due conti software e fondo ammortamento software ma quando salvo:

<?xml version="1.0" encoding="UTF-8"?>

<Libro_mastro>

  <Mastro id="0">

    Fondo ammortamento software

    <Dare />

    <Avere>1000.0</Avere>

  </Mastro>

  <Mastro id="1">

    Fondo ammortamento software

    <Dare />

    <Avere>1000.0</Avere>

  </Mastro>

</Libro_mastro>

[lucke84]

guarda, l'unica cosa che ti posso consigliare è di portare fuori dal ciclo le dichiarazioni dei tipi.. in generale non va bene istanziare delle variabili all'interno di un ciclo.. ossia dovresti:

MastP co; //quello che è
Element m;
for(int i=0;i<MTabbedP.mp.getComponents().length;i++){

      co=(MastP)MTabbedP.mp.getComponent(i); //questo è un pannello...

      m = new Element("Mastro");

      m.setAttribute("id",String.valueOf(i));

      m.setText(co.name.getText());

...

} 

per il resto, nn saprei.. mi spiace

[Metflar]

Originariamente inviato da lucke84
guarda, l'unica cosa che ti posso consigliare è di portare fuori dal ciclo le dichiarazioni dei tipi.. in generale non va bene istanziare delle variabili all'interno di un ciclo.. ossia dovresti:

MastP co; //quello che è
Element m;
for(int i=0;i<MTabbedP.mp.getComponents().length;i++){

      co=(MastP)MTabbedP.mp.getComponent(i); //questo è un pannello...

      m = new Element("Mastro");

      m.setAttribute("id",String.valueOf(i));

      m.setText(co.name.getText());

...

} 

per il resto, nn saprei.. mi spiace

grazie del consiglio...spero che qualcuno possa capire ciò ke nn va

[Alex'87]

Originariamente inviato da lucke84
guarda, l'unica cosa che ti posso consigliare è di portare fuori dal ciclo le dichiarazioni dei tipi.. in generale non va bene istanziare delle variabili all'interno di un ciclo..

Perchè non va bene? Alla fine sono locali al ciclo, fuori non servono...

[lucke84]

nessun errore, solo bello stile

[Alex'87]

Originariamente inviato da lucke84
nessun errore, solo bello stile

Ah, vabbè ^^' Io preferisco dichiararle localmente

[lucke84]

sì sì, ha senso, diciamo che dipende da che uso ne farai, spesso quando dichiari in locale poi ridichiari e in seguito ridichiari ancora.. al di là dei contatori io porto fuori quasi tutto.. ma è corretto dichiarare localmente quello che è locale se nn serve altrove quindi, onone e gloria ad alex! muahaha!

[Alex'87]

Originariamente inviato da lucke84
sì sì, ha senso, diciamo che dipende da che uso ne farai, spesso quando dichiari in locale poi ridichiari e in seguito ridichiari ancora.. al di là dei contatori io porto fuori quasi tutto.. ma è corretto dichiarare localmente quello che è locale se nn serve altrove quindi, onone e gloria ad alex! muahaha!

Certo, ovviamente dipende dai casi