Faire la somme de données groupées

Version imprimable

Bonjour! Je suis entrain de réaliser un programme java qui doit inscrire chaque nom de clients et la somme de leur transactions à l'écran. Je n'arrive pas à faire imprimer la somme des transactions. Quelqu'un pourrait m'aider? Merci :)

Voici le fichier XML :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?xml version="1.0" encoding="ISO-8859-1" ?>
<liste> 
<client nom="Jean Charles"> 
<transaction montant="500" />
<question>Quelle est la dernière marque?</question>
<transaction montant="1200" /> 
</client> 
<client nom="Pierre Élisabeth"> 
<transaction montant="600" /> 
<transaction montant="800" /> 
<question>Où puis-je trouver le modèle 2002?</question>
<transaction montant="2000" /> 
</client> 
</liste>

Mon code java :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
import org.w3c.dom.*;
import javax.xml.parsers.*;
 
public class transactions {
public static void main(String[] args) throws Exception {
  DocumentBuilderFactory factory = 
  DocumentBuilderFactory.newInstance();
  DocumentBuilder parser = 
  factory.newDocumentBuilder();
  Document doc = parser.parse(args[0]);
  Element racine = doc.getDocumentElement(); 
  NodeList nl = racine.getElementsByTagName("client");
  NodeList nl2 = racine.getElementsByTagName("transaction");
  for (int i = 0; i < nl.getLength(); ++i) {
    Element client = (Element) nl.item(i);
    Element transaction = (Element) nl2.item(i); 
    System.out.println("Nom du client : " + client.getAttribute("nom"));
    System.out.println("Somme : " + transaction.getAttribute("montant"));
    }
}
}

15/04/2015, 19h01
thelvin

Hello,

L'idée c'est
- pour chaque client,
-- pour chaque transaction,
-- l'ajouter à la somme des transactions.

Tu dois donc faire deux for() imbriqués : un for() qui parcourt les clients, et un autre for() qui parcourt les transactions de ce client et seulement de ce client.

Je n'arrive pas à cibler un client à la fois ce qui donne le mauvais résultat représenté ci-dessous :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
import org.w3c.dom.*;
import javax.xml.parsers.*;
 
public class transactions {
public static void main(String[] args) throws Exception {
  DocumentBuilderFactory factory = 
  DocumentBuilderFactory.newInstance();
  DocumentBuilder parser = factory.newDocumentBuilder();
  Document doc = parser.parse(args[0]);
  Element racine = doc.getDocumentElement(); 
  NodeList nl = racine.getElementsByTagName("client");
  NodeList nl2 = racine.getElementsByTagName("transaction");
  for (int i = 0; i < nl.getLength(); ++i) {
    Element client = (Element) nl.item(i);
    System.out.println("Nom du client : " + client.getAttribute("nom"));
  for (int j = 0; j < nl2.getLength(); ++j) {
    Element transaction = (Element) nl2.item(j); 
    System.out.println(transaction.getAttribute("montant"));
    }
    }  
}
}

15/04/2015, 20h00
tchize_

Les transactions sont dans l'objet client, donc c'est pas une liste prise depuis la racine mais une liste prise depuis le client que tu dois récuperer la liste imbriquée. PArce que le code que tu as donnée parcoure toutes les transactions indépendamment du client.
15/04/2015, 20h06
lsebe2

Comment dois-je faire? Je remplace NodeList nl = racine.getElementsByTagName("client") par NodeList nl = client.getElementsByTagName("client") ?
Ça ne semble pas fonctionner...

merci
15/04/2015, 21h53
lsebe2

Quelqu'un m'a conseillé d'utiliser getChildNodes() sur chaque élement <client>. Est-ce que quelqu'un sait comment faire?
16/04/2015, 01h36
thelvin

Non mais getElementsByTagName() c'est mieux.

... Par contre, quand tu cherches des éléments "transaction" il faudrait les appeler "transaction", pas autre chose.
16/04/2015, 16h29
lsebe2
Merci pour votre aide mais je n'y arrive vraiment pas!
Est-ce que le problème ce situe dans les lignes ci-dessous? :
Code:

1 2 3 4 5 Document doc = parser.parse(args[0]); Element racine = doc.getDocumentElement(); NodeList nl = racine.getElementsByTagName("client"); NodeList nl2 = racine.getElementsByTagName("transaction");
Je vois que racine = doc.getDocumentElement(); mais comment puis-je faire référence à l'élément <client> ou à l'élément <transaction> seulement?
Quelque chose comme Element transaction = doc.getElementsByTagName("transaction") ?
16/04/2015, 18h24
thelvin

Pour l'élément <client> tu y étais pourtant bien arrivé :

Code:

Element client = (Element) nl.item(i);

Ça marche parfaitement.
Il y a plus qu'à te servir de cette variable client.

Citation:

Quelque chose comme Element transaction = doc.getElementsByTagName("transaction") ?

Ben non -_-°.
- D'abord getElementsByTagName renvoie un NodeList, c'est-à-dire une liste d'éléments. Normal : il y en a plus d'un, donc il peut pas en renvoyer un, bien sûr -_-°.
- Ensuite, ce ne sont pas toutes les transactions du document, que tu veux. Ce sont toutes celles du client. Donc il ne faut pas le faire sur doc bien sûr -_-°.

Bon, maintenant je tente d'utiliser <transaction> avec getChildNode(); . Dois-je utiliser une autre NodeList que "nl" pour parcourir les transactions?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  NodeList nl = racine.getElementsByTagName("client");
  for (int i = 0; i < nl.getLength(); ++i) {
    Element client = (Element) nl.item(i);
    System.out.println("Nom du client : " + client.getAttribute("nom"));
  for (int j = 0; j < nl.getLength(); ++j) {
    NodeList nl2 = transaction.getChildNode(); 
    Element transaction = (Element) nl2.item(j);
 
    System.out.println(transaction.getAttribute("montant"));
    }
    }
}
}

17/04/2015, 00h45
thelvin

Bon c'est n'importe quoi. On arrête les frais.
Pour l'instant tu oublies les transactions. Elles t'intéressent plus, on les regarde plus.

Tu vas t'entraîner sur quelque chose de plus simple :
Un programme qui affiche le nom de tous les clients.

Fais un programme comme ça et montre-nous le code.