Arbre Lexicographique, création d'un jeu "pendu"

Bonjour à tous, je suis étudiant en 3ème année d’informatique à Amiens et j’ai a réaliser un projet qui consiste en la création d’un pendu. J’ai comme consigne d’utiliser un arbre lexicographique comme base de donnée afin de stocker mes mots.

Je vous présente donc mon code :

package Model;

import java.io.*;



public class ArbreLexicographique {
	private  Noeud courant;
	private  Noeud tete;
	private char lettre;



	public ArbreLexicographique() {

		Noeud[] fils= new Noeud[30];
		this.tete= new Noeud('', false, 1, fils);
		this.courant=tete;
		this.RemplirArbre();


	}

	private void ajouterMot(String mot){
		int i, nbFils;
		System.out.println(mot);
		if (mot.equals("")){

			courant.setMot(true);	
			this.courant=tete;
			
			
		}
		else
			lettre=mot.charAt(0);
		if (possedeFils(this.courant.getFils(), this.lettre)){
			i=chercherFils(this.courant.getFils(), this.lettre);
			this.courant = this.courant.getFils()[i];
			System.out.println(mot);
			if (mot.equals("")){

				courant.setMot(true);	
				this.courant=tete;
			}
			else {
				mot=mot.substring(1,mot.length());
				ajouterMot(mot);	
			}
		}
		else {

			nbFils = nbFils(this.courant.getFils());
			Noeud[] fils;
			this.courant.getFils()[nbFils]= new Noeud(lettre, false, this.courant.getNiveau()+1, fils= new Noeud[30]);
			i=chercherFils(this.courant.getFils(), lettre);
			this.courant = this.courant.getFils()[i];
			if (mot.equals("")){

				courant.setMot(true);	
				this.courant=tete;
			}
			else {
				mot=mot.substring(1,mot.length());
				ajouterMot(mot);
			}
		}

	}

	String GenererMotAleatoire() {
		int random, random2, i;
		this.courant=tete;
		char [] mot = null;
		String mot2 = null;
		random=nbAleatoire(1, 6);
		for (i=0; i<random; i++){
			random2=nbAleatoire(0, nbFils(courant.getFils()));
			System.out.println(random2);
			char[] cs = mot;
			cs[i]= courant.getFils()[random2].getLettre();
			if (courant.getFils()[i].isMot()){
				mot2= new String(cs);
			}
			courant= courant.getFils()[random2];
		}
		return mot2;
	}

	private int nbAleatoire(int lower, int higher){
		int random = (int)(Math.random() * (higher-lower)) + lower;
		return random;
	}

	
	private boolean possedeFils(Noeud[] fils, char c){
		int i;
		Boolean bool=false;
		for (i=0; i<fils.length; i++){
			if(fils[i]== null){
				continue;
			}
			else if(fils[i].getLettre() == c){
				bool=true;
			}
		}
		return bool;
	}

	private int chercherFils(Noeud[] fils, char c){
		int i, j;
		j=0;

		for (i=0; i<fils.length; i++){
			if(fils[i]== null){
				continue;
			}
			else if(fils[i].getLettre() == c){
				j=i;
			}
		}
		return j;
	}

	private int nbFils (Noeud[] fils){
		int i, cpt;
		cpt=0;
		for (i=0; i<fils.length; i++){
			if (fils[i] != null ){
				cpt++;
			}
		}
		return cpt;
	}

	private void RemplirArbre(){
		String fichier ="C:/Documents and Settings/Propriétaire.VIN-1D00C6226C7/Bureau/eclipse-java-indigo-SR1-win32/Pendu/src/Model/dico.txt";


		try{

			InputStream ips=new FileInputStream(fichier); 
			InputStreamReader ipsr=new InputStreamReader(ips);
			BufferedReader br=new BufferedReader(ipsr);
			String ligne;
			int i=0;
			while ((ligne=br.readLine())!=null){
				this.ajouterMot(ligne);
				i++;
				System.out.println(i);
				
			}
			br.close(); 
		}		
		catch (Exception e){
			e.printStackTrace();
		}
	}		


	public Noeud getCourant() {
		return courant;
	}

	public void setCourant(Noeud courant) {
		this.courant = courant;
	}

	public Noeud getTete() {
		return tete;
	}

	public void setTete(Noeud tete) {
		this.tete = tete;
	}

	public char getLettre() {
		return lettre;
	}

	public void setLettre(char lettre) {
		this.lettre = lettre;
	}
}

J’ai un fichier texte contenant tout les mots de la langue francaise. A partir de celui-ci, j’utilise une fonction qui permet de lire le fichier ligne par ligne (voir la fonction remplirArbre()) et d’ajouter le mot correspondant à l’arbre (voir la fonction ajouterMot()). Cependant j’ai un soucis, Dans la fonction qui doit me permettre de générer un mot aléatoirement et qui est stocké dans l’arbre, j’ai une java.lang.NullPointerException. Elle se situe dans la fonction GénererMotAléatoire() à cette ligne :

cs[i]= courant.getFils()[random2].getLettre();

Le noeud en question est vide, c’est pour cela que je ne peux pas appliquer getLettre. Pourtant grâce a Random2 j’ai bien le nombre de fils du noeud père et donc je ne peux pas déborder sur une case suivante du tableau qui serait à null.

Je pense donc que le problème vient de l’ajout des mots qui ne se fait pas comme je le souhaite. J’ai essayer de faire des modifications mais sans succès, j’aimerais bien un petit coup de main.

Arf mince l’indentation n’est pas très belle.

Ma structure de donnée est un arbre lexicographique, chaque noeud contiendra une lettre et leur fils les lettres suivantes. En suivant un chemin et en ajoutant les lettres on obtient un mot. Pour modéliser ca, j’ai créer un type Noeud qui a comme attribut
un char, et un tableau de Noeud qui permettra de stocker les fils.

package Model;



public class Noeud {
	private char lettre;
	private boolean mot;
	int Niveau;
	
	private Noeud[] fils;

	public Noeud[] getFils() {
		return fils;
	}
	public void setFils(Noeud[] fils) {
		this.fils = fils;
	}
	public Noeud(char lettre, boolean mot, int niveau, Noeud[] fils) {
		this.lettre = lettre;
		this.mot = mot;
		this.Niveau = niveau;
		this.fils=fils;
			}
	public char getLettre() {
		return lettre;
	}
	public void setLettre(char lettre) {
		this.lettre = lettre;
	}
	public boolean isMot() {
		return mot;
	}
	public void setMot(boolean mot) {
		this.mot = mot;
	}
	public int getNiveau() {
		return Niveau;
	}
	public void setNiveau(int niveau) {
		Niveau = niveau;
	}
	
	


}

Pour l’indentation et même la coloration, utilise pastebin en mode privé.

Et pour ton problème, question super conne : tu alloue toujours des tableaux de 30 valeurs, mais tu n’y mets pas forcément toujours des noeuds… du coup, tes NPE sont certainement liée au fait que tu ne remplie jamais ces nœuds là, mais que ta fonction d’aléatoire - de par sa nature - tape pile poil où il ne faut pas.

Vu que tu as l’air d’avoir des consignes, tu devrais gérer un nombre de fils et éviter d’aller taper au mauvais endroit;

Je ferais ça pour ta classe Noeud :

int nbNoeud;
Noeud[] fils;

public Noeud getNoeud(int i) {
  if (i > nbNoeud) throw new RuntimeException("plop");
  return fils[i];
}

public void addNoeud(Noeud noeud) {
  if (nbNoeud >= fils.length) throw new RuntimeException("On ne peut plus ajouter de noeuds");
  fils[nbNoeud++] = noeud;
}

Tu peux aussi envisager d’utiliser une ArrayList qui fera ce travail pour toi.

La fonction aléatoire fait en sorte de prendre un nombre compris entre 0 et le nombre de noeud fils du noeud courant (pour cela j’ai une fonction qui calcule le nombre de fils du noeud courant en question).

Je pensais alloué un tableau de 26 cases pour toutes les lettres de l’alphabet. Etant donné que je ne peux pas l’agrandir au fur et à mesure.


``` return fils; ```

que signifie les chevrons ?

Merci pour le code !

Heu, les chevrons signifient []. Il semblerait qu’il y ait - encore - un bug sur le forum. i (entre crochets) est remplacé par i (entre chevrons).