Java

Excusez moi je jouer au boulet mais je capte pas une erreur java ...

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Excusez moi je jouer au boulet mais je capte pas une erreur java ...

EpoK

Let's burn

Exception in thread "main" java.lang.ClassCastException

je compren ke c un de mes cast ki merde mais cette erreur survient quand on essai de faire koi de mal ?

merci

Publicité

EpoK

Let's burn

Code :

while(st.hasMoreElements())
{
EstDedan = false;
mot = (String)st.nextElement();
for(i=0;i<lex.size();i++)
{
EstDedan = (mot.equals(lex.get(i)));
}
if(!EstDedan)
{
sb = new StringBuffer(mot);
lex.add(sb);
}
}

le truc c a partir dun stringtokenziker créé en vector avec dedan les mot mais pas ajouté le redondan mon blem c ke :
EstDedan = (mot.equals(lex.get(i)));
revoi tjrs false et je c pas pk :sweat:

lorill

ca réponds pas a la question, mais en passant :
t'as pas besoin de caster en string pour utiliser .equals(), c'est une methode d'object

Taz

bisounours-codeur

a causede ta boucle: quand tu as trouve, faut sortir de la boucle, sinon estdedans quiv aut vrai risque d'etre mis a faux lors de la prochaine itération. donc rajoute un autre test dans la condition de ton for

Cherrytree

cn=?

Tu compares des String à des StringBuffer, pourquoi donc ? [:zebra33]

C'est Taz qui a la solution, en revanche, je ne comprends toujours pas pourquoi tu ajoutes des StringBuffer dans ton Vector et pas des String toutes bêtes.

Message édité par Cherrytree le 01-02-2003 à 17:33:22

---------------
Le site de ma maman

Cherrytree

cn=?

++Taz a écrit :

Bien vu !

---------------
Le site de ma maman

EpoK

Let's burn

Cherrytree a écrit :

Tu compares des String à des StringBuffer, pourquoi donc ? [:zebra33]

ué jvien dle voir
now ca pass :

Code :

public static void ajouter(Vector lex, StringTokenizer st)
{
int i;
String mot;
StringBuffer sb;
boolean EstDedan;
String m;
while(st.hasMoreElements())
{
EstDedan = false;
mot = (String)st.nextElement();
for(i=0;i<lex.size();i++)
{
m = (lex.get(i)).toString();
EstDedan = (mot.equals(m));
}
if(!EstDedan)
{
sb = new StringBuffer(mot);
lex.add(sb);
}
}
}

prochaine étape les trié en ordre alphabétik

Taz

bisounours-codeur

+1 pourquoi des StringBuffer?

Taz

bisounours-codeur

c toujours pas bon....

EpoK

Let's burn

++Taz a écrit :

rololo sui a la mass

Code :

public static void ajouter(Vector lex, StringTokenizer st)
{
int i;
String mot;
StringBuffer sb;
boolean EstDedan;
String m;
while(st.hasMoreElements())
{
EstDedan = false;
mot = (String)st.nextElement();
i=0;
while((i<lex.size()) && (!EstDedan))
{
m = (lex.get(i)).toString();
EstDedan = (mot.equals(m));
i++;
}
if(!EstDedan)
{
sb = new StringBuffer(mot);
lex.add(sb);
}
}
}

thx

Publicité

Taiche

(╯°□°）╯︵ ┻━┻

Deux choses :
* ici, c'est pas un SMS ou un chat, tu peux écrire autant de lettres que tu veux, c'est la fête ! Donc si tu pouvais éviter "d'écrir kom ça", ça serait bien pour tout le monde, merci :jap:
* Java c'est bien mais à ta place et pour plus de lisibilité, je déclarerais mes variables du genre : StringBuffer sb = new StringBuffer();
En gros, mets le type de ta variable devant son nom comme ça on sait en un coup d'oeil à quoi ça correspond (d'ailleurs, je savais même pas que c'était possible de faire autrement ).

Enfin et surtout, j'ai pas très bien compris ton problème [:ddr555] Tu pourrais sortir la trace de l'Exception ou dire à quelle ligne elle se réfère, steuplé ?

EDIT : roh putain, 5 min au phone et chu grillaid 2000 fois [:ruisseau de larmes]

Message édité par Taiche le 01-02-2003 à 17:38:35

Taz

bisounours-codeur

tu pouvais laisser un for, y a pas de mal. ca serait d'aillerus peut etre plus lisible, surtout en initialisant estdedans=false dans le premier statement du for

Taz

bisounours-codeur

comme dit taiche, oublie le pascal: déclare tes varaibles au plus pres de leur initialisation

EpoK

Let's burn

++Taz a écrit :

comme dit taiche, oublie le pascal: déclare tes varaibles au plus pres de leur initialisation

Code :

while(st.hasMoreElements())
{
mot = (String)st.nextElement();
for(i=0,EstDedan=false; (i<lex.size()) && (!EstDedan) ;i++)
{
m = (lex.get(i)).toString();
EstDedan = (mot.equals(m));
}
if(!EstDedan)
{
sb = new StringBuffer(mot);
lex.add(sb);
}
}

pr son exemple j'avai pas tro le choi ... je cré le stringbuffer ke kan je suis sur ke je le mot rentre ds le vector

Taiche

(╯°□°）╯︵ ┻━┻

EpoK a écrit :

pr son exemple j'avai pas tro le choi ... je cré le stringbuffer ke kan je suis sur ke je le mot rentre ds le vector

Ouais non mais c'que je veux dire c'est qu'il ne faut pas écrire

Code :

sb = new StringBuffer();

mais plutôt :

Code :

StringBuffer sb = new StringBuffer();

lorill

EpoK a écrit :

pr son exemple j'avai pas tro le choi ... je cré le stringbuffer ke kan je suis sur ke je le mot rentre ds le vector

déclaration != instanciation

EpoK

Let's burn

Taiche a écrit :

Ouais non mais c'que je veux dire c'est qu'il ne faut pas écrire

Code :

sb = new StringBuffer();

mais plutôt :

Code :

StringBuffer sb = new StringBuffer();

ok

mais là j'écrit sb = new StringBuffer(mot), je peu pas trop le faire avant d'avoir mot

lorill

EpoK a écrit :

mais là j'écrit sb = new StringBuffer(mot), je peu pas trop le faire avant d'avoir mot

mais rien ne t'empeche de ne déclarer sb que dans ton if, plutot qu'en tete de fonction [:sinclaire]

Taz

bisounours-codeur

mais bordel, déclare tes varaibles la ou tu en as besoin et pas dans un bloc massif a chaque nouvelle fonction :bounce:

EpoK

Let's burn

ouais :sweat:

Taz

bisounours-codeur

peut etre que tu ferais bien de changer le types de lex ou de l'ordonner, par ce que la recherche linéaire, c'est pas top.

personnellement, moi je penserais à un Set

http://java.sun.com/j2se/1.4/docs/ [...] l/Set.html

ton traitement serait alors

Code :

public static void ajouter(TreeSet lex, StringTokenizer st)
{
while(st.hasMoreElements())
{
lex.add(new StringBuffer((String)st.nextElement());
}
}

EpoK

Let's burn

++Taz a écrit :

Code :

public static void ajouter(TreeSet lex, StringTokenizer st)
{
while(st.hasMoreElements())
{
lex.add(new StringBuffer((String)st.nextElement());
}
}

hum hum jai pas encore vu cette class

v jeté un oeil merci

BifaceMcLeOD

The HighGlandeur

EpoK a écrit :

Code :

(...)
for(i=0,EstDedan=false; (i<lex.size()) && (!EstDedan) ;i++)
{
m = (lex.get(i)).toString();
EstDedan = (mot.equals(m));
}
(...)

Taz> Désolé de te contredire, mais normalement, le "for" ne doit être utilisé que quand on connait à l'avance le nombre d'itérations. Sinon, il perd énormément de sa lisibilité, et il vaut beaucoup mieux utiliser "while".

Epok> Comme l'a dit lorill en tout début du topic, equals() s'invoque sur un Object et attend un Object en argument. Donc tu peux écrire :

Code :

EstDedan = lex.get(i).equals(mot);

Code :

EstDedan = mot.equals(lex.get(i));

Et puis, en français, "dedans" prend un 's' final...

Enfin, à mon avis, il n'a pas besoin de booléen ici :

Code :

List lex = ...;
(...)
int numMots = lex.size();
while (st.hasMoreElements()) {
Object mot = st.nextElement();
int i = numMots;
while (i >= 0 && !mot.equals(lex.get(i))) {
i--;
}
// Pas trouve
if (i < 0) {
lex.add(new StringBuffer(mot));
numMots++;
}
}

Maintenant, 2 choses :

A moins que tu aies besoin que plusieurs threads accèdent en même temps ton vecteur "lex", n'utilise pas "java.util.Vector", mais "java.util.ArrayList", c'est préférable. Dans tous les cas, ne déclare ton objet "lex" que comme une "java.util.List". Ca pourra être utile à l'avenir.
Si tu veux trier "lex", je te conseille de regarder du côté de "java.util.Collections.binarySearch(java.util.List list, Object key)". Tu pourras l'utiliser aussi bien pour trouver que pour insérer au bon endroit...

Taz

bisounours-codeur

BifaceMcLeOD a écrit :

:heink: étant donné que ici le parcours est fait par index, le for me parait au contraire sémantiquement plus correcte et plus lisible qu'un while avec une variable index initialisée on ne sait ou

BifaceMcLeOD

The HighGlandeur

Tout dépend de quoi on parle : du "for" C ou du "for" algorithmique. Parce que je te rappelle qu'en algorithmique pure, le "for" s'écrit :

for i in [value1, value2] loop ...

Il n'est donc pas question de pouvoir y loger une autre condition que "i < value2"...

EpoK

Let's burn

c'est bon là j'ai plus besoin de while ni de for, falai juste trouvé la bonne méthode dans la doc ...

Code :

public static void ajouter(Vector lex, StringTokenizer st)
{
int i;
boolean EstDedan;
String mot;
String motvector;
while(st.hasMoreElements())
{
mot = (String)st.nextElement();
EstDedan=false;
EstDedan=lex.contains(mot);
if(!EstDedan)
{
lex.add(mot);
}
}
}

benou

EpoK a écrit :

boolean EstDedan;

les variables avec des une majuscule au début ca sux !

Cherrytree

cn=?

benou a écrit :

les variables avec des une majuscule au début ca sux !

Ouais ça c'est la convention MS.

---------------
Le site de ma maman

BifaceMcLeOD

The HighGlandeur

Oui, et puis, il y a des lignes de code inutiles ici.
A quoi ça sert par exemple d'initialiser "EstDedan" puisqu'à l'instruction suivante, tu lui affectes de toute façon une valeur ? Et à quoi servent "i" et "motvecteur" ?

Par ailleurs, Vector, comme sa superclasse List, contient des instances d'Object, pas de String, donc il est inutile de caster le retour de nextElement() en String (ou alors utilise la méthode nextToken(), qui te renvoie directement une instance de String).
Rappelle-toi qu'un cast alourdit l'écriture et n'est jamais gratuit à l'exécution en Java. Alors autant les éviter lorsqu'ils sont inutiles.

En fait, je te suggère plutôt d'écrire :

Code :

public static void ajouter(Vector lex, StringTokenizer st) {
while (st.hasMoreElements()) {
Object mot = st.nextElement();
if (!lex.contains(mot)) {
lex.add(mot);
}
}
}

Tu noteras que ce code est strictement équivalent au tien, mais il ne contient que le code nécessaire. Or moins de lignes de code, c'est d'autant de moins de code à maintenir ensuite.

Message édité par BifaceMcLeOD le 04-02-2003 à 14:30:16

benou

et temps qu'à faire, autant utiliser un Set puisque c'est ca que tu veux (HashSet par exemple).

Code :

public static void ajouter(Set lex, StringTokenizer st) {
while (st.hasMoreElements()) {
lex.add(st.nextElement());
}
}

en plus ce sera plus rapide ...

BifaceMcLeOD

The HighGlandeur

Yup ! :jap:

Taz

bisounours-codeur

benou a écrit :

et temps qu'à faire, autant utiliser un Set puisque c'est ca que tu veux (HashSet par exemple).

Code :

public static void ajouter(Set lex, StringTokenizer st) {
while (st.hasMoreElements()) {
lex.add(st.nextElement());
}
}

en plus ce sera plus rapide ...

ce que j'ai déjà proposé 1 miyion de posts auparavant. mais moi j'aurais plutot dit un SortedSet (TreeSet)

benou

++Taz a écrit :

ce que j'ai déjà proposé 1 miyion de posts auparavant. mais moi j'aurais plutot dit un SortedSet (TreeSet)

ha oui tiens ...

pourquoi un TreeSet ?
c'est plus lent et ca a juste un intérêt si tu veux ordonner la collection ...

Taz

bisounours-codeur

ben je sais pas , histoire de montrer que ça existe et peut etre pour faire une démo. pi faut voir que c'est souvent plus facile de faire une fonction de comparaison qu'une bonne fonction de hashage

benou

ok mais bon, la c'est des String. Elle est déjà tout faite la fonctione de hashage

Taz

bisounours-codeur

tout en effet

Publicité

FORUM HardWare.fr

Programmation

Java

Excusez moi je jouer au boulet mais je capte pas une erreur java ...

Sujets relatifs
Applet Java pour connection SSH	processeur java...
[JAVA - ORACLE] Développement en java à la place de PL/SQL	Msg queue en java
Erreur de compilation avec un makefile	Differences entre Java - J2EE - J2SE
[Java] Format de Date - resolu - soltion inside	[ Java ] gestion du temps
[JAVA] programmation d'un bot de moteur de recherche	[C] message d erreur a la compilation (linking)
Plus de sujets relatifs à : Excusez moi je jouer au boulet mais je capte pas une erreur java ...

Page générée en 0.094 secondes