Discussion :

[JeanNoel53]

Bonjour,

Je dois tester que l'itérateur ne dépasse pas la fin de la liste:

J'ai codé:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
    Connection connection = MissionDatabaseManager.OpenDatabaseCoherence();
   elementWorks.addAll(RecupereNomDesFichier());
   Iterator<ElementWork> itWorks;
   itWork.Begin()<ElementWork>;
   while( itWork != End()<ElementWork>)
   {
      elementWorks = itWork;
      itWork++;
   }

Ce qui est refusé par le compilateur.

J'ai deux constructions de refusées: itWordk.Begin et while( itWork != End()

Quelqu'un saurait-il m'indiquer comment résoudre ce problème ?

Merci d'avance pour votre aide.

[tchize_]

Ce code ne veux simplement rien dire, c'est à peine si c'est du java

http://fmora.developpez.com/tutoriel...diaire/#L4.1.2

[JeanNoel53]

J'ai repris d'après l'exemple.

J'ai déclaré ElementWork comme structure

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
package com.all4tec.matelo.linguistique.wizards;
 
public class Structure 
{
    public static class ElementWork 
    {
        int Id;
        String wsText;
    };
}

et je n'ai pas d'instance de current dans le code current.Id et current.wsText.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
    public ArrayList<ElementWork> RecupereNomDesFichier() throws ClassNotFoundException, IllegalArgumentException, NullPointerException, SQLException
    {
        if( InterbaseManager.connection == null)
            MissionDatabaseManager.OpenDatabaseCoherence();
        ArrayList<ElementWork> elementWorks = MissionDatabaseManager.ReadWork();
        // charger les fichier de la base de donnée et remplir cette liste
        for( ArrayList<ElementWork> current = elementWorks;;)
        {
            int id = current.Id;
            String str = current.wsText;
        }
        // ToDo il faut ressortir le numéro saisi par l'utilisateur
        return elementWorks;
    }

[thelvin]

Ouais...

Alors deux choses :

- C'est quel exemple exactement, que tu as adapté, là ? J'en reconnais aucun...

- Quand tu dis, avec code à l'appui :

J ai déclaré ElementWork comme structure

Alors, je sais pas comment dire ça poliment, avec pédagogie. C'est pas bien. C'est pas comme ça qu'on fait.
On veut créer une classe ElementWork on fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
public class ElementWork 
{
  int id;
  String text;
}

Y a pas d'histoire de structure ou de classe à devoir y mettre dedans. Je sais pas où tu es allé encore chercher ça.
Par contre je constate que c'est récurrent chez toi, alors que c'est pas franchement une habitude typique du débutant.
Inventer n'importe quoi au hasard, au lieu de se demander comment c'est censé marcher, ça c'est un truc qui est vraiment à toi. Et il faudrait que tu le perdes, en fait. Que tu arrêtes d'inventer des trucs. À la place il faudrait que tu cherches comment on fait (chercher pas au hasard, chercher dans les cours, les tutoriels, comment font les gens qui savent,) et que tu essaies de faire pareil et, si possible, de comprendre un peu le principe.

[JeanNoel53]

Je l'ai fait mais ça n'a rien changé.

[Amnael]

Sauf erreur de ma part les classes internes existent en java non ? Même si ici je ne vois pas l'utilité.

Sinon pour parcourir un itérateur tu as une méthode hasNext dont tu peux te servir.

[thelvin]

Je l'ai fait mais ça n'a rien changé.

Euh... Tu as fait quoi et ça a rien changé ? Je t'ai demandé de quel exemple tu t'inspires.

[tchize_]

La classe dans la classe, ça sert à rien, je sais pas d'ou tu sort ça.

Ton code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
for( ArrayList<ElementWork> current = elementWorks;;)
        {
            int id = current.Id;
            String str = current.wsText;
        }

Ne veux décidément rien dire. current est déclaré comme un ArrayList. Un ArrayList ça n'a ni champ 'id' ni champ 'wsText'. Ta boucle n'a pas de critère d'arrêt, donc même si ça compilait, ça tournerait en boucle à l'infini. Je t'ai fourni un exemple simple de partcours de list, tu as juste à le réutiliser. La seule chose qui ressemble à l'exemple dans le code, c'est qu'il y a un 'for', c'est tout.
J'ai l'impression que tu jette du code dans ton fichier sans avoir la moindre idée de ce que tu fais et que tu espère à l'arrivée que ça fasse ce que tu veux. Ce n'est pas comme ça qu'on programme. Il faut être rigoureux et savoir ce que l'on fait, étape par étape.

[JeanNoel53]

L'arrayList est constituée d'ElementWorks et donc je m'attends à retrouver les champs Id et wsText quand je parcours la liste.

Je vous mets mon code dans l'état où il se trouve:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    public ArrayList<ElementWork> RecupereNomDesFichier() throws ClassNotFoundException, IllegalArgumentException, NullPointerException, SQLException
    {
        if( InterbaseManager.connection == null)
            MissionDatabaseManager.OpenDatabaseCoherence();
        ArrayList<ElementWork> elementWorks = MissionDatabaseManager.ReadWork();
        // charger les fichier de la base de donnée et remplir cette liste
        Iterator<ElementWork> itWork;
        while( itWork.hasNext())
        {
            elementWorks = itWork;
            int id = elementWorks.Id;
            String str = elementWorks.wsText;
        }
        // ToDo il faut ressortir le numéro saisi par l'utilisateur
        return elementWorks;
    }

C'est dans la boucle while que j'ai des erreurs de compilation elemtnWorks = itWord; itword n'est pas reconnue.
Ultérieurement les champs de ElementWork ne sont pas reconnus.
Il y a un autre problème c'est que le new n'est plus reconnu dnas la couche de plus bas niveau

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
	static ArrayList<Structure.ElementWork> ReadWork() throws SQLException
	{
		   ArrayList<ElementWork> pListText;
		   int iNbResultFound;
		   pListText = new ArrayList<Structure.ElementWork>();
		   String cmd = "SELECT iIdDoc, vcDocument FROM T_WORK_INFORMATION WHERE siIdStateDocument = 3";
		   ResultSet resultset = InterbaseManager.ExecuteSelectRequest( cmd);
		   ResultSetMetaData rsmd = resultset.getMetaData();
		   iNbResultFound = 0;
		   try 
		   {
			   while( resultset.next())
			   {
				    iNbResultFound++;
				    WorkElement = new Structure.ElementWork();
					WorkElement.Id = resultset.getInt( "iIdDoc");
					WorkElement.wsText = resultset.getString( "vcDocument");
					pListText.add( WorkElement);
				}
		   }
		   catch( SQLException ex)
		   {
			   ex.printStackTrace();
		   }
		   return( pListText);
	   }

[eulbobo]

Quelques corrections sur le premier bout de code avec des commentaires

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    public ArrayList<ElementWork> RecupereNomDesFichier() throws ClassNotFoundException, IllegalArgumentException, NullPointerException, SQLException
    {
        if( InterbaseManager.connection == null)
            MissionDatabaseManager.OpenDatabaseCoherence();
        ArrayList<ElementWork> elementWorks = MissionDatabaseManager.ReadWork();
        // charger les fichier de la base de donnée et remplir cette liste
        Iterator<ElementWork> itWork;
        while( itWork.hasNext()) // erreur de compilation ici : itWork n'est pas instancié. il faudrait le déclarer juste au dessus avec itWork = elementWorks.iterator();
        {
            elementWorks = itWork; // erreur de compilation ici : elementWorks est une ArrayList et tu essayes de le faire pointer vers un ElementWork ... Ce que tu veux faire, c'est : ElementWork element = itWork.next(); (une fois que tu auras récupéré correctement on iterator)
            int id = element.Id; 
            String str = element.wsText;
        }
        // ToDo il faut ressortir le numéro saisi par l'utilisateur
        return elementWorks;
    }

Pour le deuxième bout de code : fait déjà disparaitre tous les "Structure" de ton code si tu as créé une classe ElementWork comme indiqué par thelvin..

[JeanNoel53]

J'ai repris le code; Ça va mieux, mais il reste une erreur.
Je te met les déclarations et le code
D'abord les déclarations:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
	private LinguistiqueImportWizardPage linguistiquePage;
	private ElementWork elementWork;
	private boolean initialized = false;

puis le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
	public ArrayList<ElementWork> RecupereNomDesFichier() throws ClassNotFoundException, IllegalArgumentException, NullPointerException, SQLException
	{
		if( InterbaseManager.connection == null)
			MissionDatabaseManager.OpenDatabaseCoherence();
		ArrayList<ElementWork> elementWorks = MissionDatabaseManager.ReadWork();
		// charger les fichier de la base de donnée et remplir cette liste
		Iterator<ElementWork> itWork;
		while( itWork.hasNext())
		{
			elementWork = itWork;
			int id = elementWork.Id;
			String str = elementWork.wsText;
		}
		// ToDo il faut ressortir le numéro saisi par l'utilisateur
		return elementWorks;
	}

la déclaration de itWork est juste avant le while
while (itWork.hasNext()) est reconnu par le compilateur
elemntwork = itWork; sort en erreur
après le code est bien compilé.
Je dois résoudre l'itWork pour rééssayer

[eulbobo]

Regarde bien ce que j'ai posté, il y a la réponse à ta question dedans

[JeanNoel53]

J'ai répondu à toutes tes indications. J'ai toujours une erreur. Mais itWork est bien initialisé

[eulbobo]

Relis bien tout ce qui est indiqué sur la ligne 10 de mon code... Et si tu ne trouves toujours pas, poste ton code (parce que ma boule de cristal est en réparation)

[JeanNoel53]

tu dis que la ligne while( itWork.hasNext()) ne compile pas
dans la réalité il compile
en fait il fallait juste initialiser itWork à null avant la boucle while et il est de bon ton de caster itWork dans son utilisation

[thelvin]

Oui donc ça compilait pas avant que tu le mettes à null. Donc il s'est pas du tout trompé et tu racontes n'importe quoi.

Et quand tu le mets à null ça compile mais ça marche pas. Donc c'est pas ce qu'il fallait faire.
Il n'est absolument pas bon de caster quoi que ce soit, mais on est pas à ça près .

Je t'ai déjà demandé de regarder un tutoriel sur comment on boucle sur une Collection, et de faire comme ce tutoriel.
Si tu veux qu'on t'aide, fais ce qu'on te dit.