Discussion :

[pat-trix]

Bonjour à tous,

Enfait j'ai le rpoblème suivant:
J'ai crée un Vector qui stock un nombre de points. Ensuite, j'aimerais créer un Polygon à partir des points de ce Vector.
Voici le code que j'ai utilisé: (Trace est le Vector et dpoint les objets du Vector)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
            Trajet=new Polygon();
            int[] surfaceTrajetx= new int[Trace.size()];
            int[] surfaceTrajety= new int[Trace.size()];
 
            int for1;
 
            for (for1=0;for1<Trace.size();for1++){
                surfaceTrajetx=Trace[for1].dpoint.x;
                surfaceTrajety=Trace[for1].dpoint.y;
            }

Et enfait il me dit que des Type incompatible sont trouvés. les 2 dernière lignes ne marchent pas: "requis tableau, trouvé java.util.Vector"
Comment faire pour attribuer les points au polygon... ??
Voilà, merci à tout le monde!

Pat-trix

[le y@m's]

Pour accèder à un element d'un Vector tu ne peux pas utiliser l'opérateur []. Il te faut utiliser la méthode get(int).

Code :

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

1
2
3
4
5
6
7
8
9
10
             Trajet=new Polygon();
            int[] surfaceTrajetx= new int[Trace.size()];
            int[] surfaceTrajety= new int[Trace.size()];
 
            int for1;
 
            for (for1=0;for1<Trace.size();for1++){
                surfaceTrajetx=Trace.get(for1).dpoint.x;
 
            }

PS : _ par convention les attributs commencent par une minuscule
_ si tu n'utilise pas le caractère thread-safe du Vector je te recommande d'utiliser une List.

[pat-trix]

Merci!
J'ai essayer mais il me souligne "dpoints" comme variable introuvable"impossible de trouver le symbole; symbole: variable dpoint, emplacement: class java.lang.Object" ...bizarre, peut être la raison est que dpoint est une class que j'ai créee comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
 
class dpoint {
        public double x,y;
 
        dpoint(double xpos, double ypos){
            x=xpos;
            y=ypos;
        }
    }

et ensuite le code du premier message modifié avec "get"...

??

Merci !

[le y@m's]

Tu veus dire que ton Vector contient des dpoints ?
Si oui alors le code que je t'ai donné n'est pas bon, car la méthode get() du vector renvoi l'objet dpoint correspondant à l'objet spécifier :

Code :

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

1
2
3
4
5
6
7
Trajet=new Polygon();
int[] surfaceTrajetx= new int[Trace.size()];
int[] surfaceTrajety= new int[Trace.size()];            
int for1;
for (for1 = 0;for1 < Trace.size(); for1 ++) {
     surfaceTrajetx = ((dpoint) Trace.get(for1)).x;
}

Plusieurs remarques :

• Par convention le nom des classes commence par une majuscule

• Il vaut mieux déclarer les attributs en private et faire des accesseurs/modifieurs.

[pat-trix]

Merci pour le code! ... mais il ne marche toujours pas:

cette fois, JBuilder "m'a proposé" le 'x' quand j ai mis le '.'
( surfaceTrajetx=((dpoint) Trace.get(for1)).x; ), donc ça ne devrait pas être faux.
Cependant, le 'x' est déclarer faux par JBuilder :"types incompatibles: trouvé double, requis: int[];"

bizarre non ?

Cordialement

Pat-trix

[NeptuS]

on n'utilise pas get, mais elementAt(i), qui renvoie un objet du type contenu dans la collection (le Vector), si tu l'as typé lors de sa déclaration et de son instanciation.

voilou. ++.
[EDIT]apparemment, tu n'as pas typé ta collection. : essaye :

Code :

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

1
2
3
4
 
Vector<dpoint> Trace = new Vector<dpoint>();
Trace.addElement(monDPoint);
int monX = Trace.elementAt(0).x;

En revanche, je te conseille de relire les règles de nommage de la programmation objet. D'après ton code .... tu les as comprises à l'envers. Tu devrais écrire :

Code :

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

1
2
3
4
 
Vector<DPoint> trace = new Vector<DPoint>();
trace.addElement(monDPoint);
int monX = trace.elementAt(0).getX();

De plus, x et y devraient être déclarés en privé dans ta classe DPoint, donc tu doit y accéder par un accesseur : getX() et getY()
[/EDIT]

[le y@m's]

on n'utilise pas get, mais elementAt(i), qui renvoie un objet du type contenu dans la collection (le Vector)

C'est pareil :

public Object elementAt(int index)

Returns the component at the specified index.

This method is identical in functionality to the get method (which is part of the List interface).

[EDIT]apparemment, tu n'as pas typé ta collection.

+1 (si Java 5 bien sûr)

Cependant, le 'x' est déclarer faux par JBuilder :"types incompatibles: trouvé double, requis: int[];

Je n'avais même pas vu . Effectivement x est un double et surfaceTrajetx est un int[] (le message d'erreur est assez explicite). Alors soit du change x en int soit surfaceTrajetx en double[] et n'oublie pas l'indice.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Trajet=new Polygon();
double[] surfaceTrajetx= new double
[Trace.size()];
double[] surfaceTrajety= new double
[Trace.size()];            
int for1;
for (for1 = 0;for1 < Trace.size(); for1 ++) {
     surfaceTrajetx[for1] = ((dpoint) Trace.get(for1)).x;
 
     // si Java 5 et utilisation des générics
//     surfaceTrajetx[for1] = Trace.get(for1).x;  
}

[pat-trix]

Merci! alors le code marche, j'ai changé lées deux tabelaux (surfaceTrajetx et surfaceTrajety en double[]) et ces tableaux "arrivent" à prendre les valeurs du Vector Trace.

Mais maintenant, lorsque je veux créer mon Polygon "Trajet", je ne peux pas utiliser les 2 tableaux "surfaceTrajetx/y", car la méthode qui dessine un Polygon demande (int[],int[],int), or j'ai (double[],double[],int). Existe-t-il un moyen simple pour changer mes 2 tableaux double[] en int[]

Merci

Cordialement

Pat-trix

[le y@m's]

Dans ce cas il faudrait peut-être mieux changer les attributs x et y de ta classe Dpoint et les déclarer en int (idem pour les tableaux).

[dazz_x]

Salut pat-trix

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Trajet=new Polygon();
int[] surfaceTrajetx= new int
[Trace.size()];
int[] surfaceTrajety= new int
[Trace.size()];            
int for1;
for (for1 = 0;for1 < Trace.size(); for1 ++) {
     surfaceTrajetx[for1] = Math.round(((dpoint) Trace.get(for1)).x);
 
     // si Java 5 et utilisation des générics
//     surfaceTrajetx[for1] =  Math.round( Trace.get(for1).x);  
}

[le y@m's]

Salut pat-trix

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Trajet=new Polygon();
int[] surfaceTrajetx= new int
[Trace.size()];
int[] surfaceTrajety= new int
[Trace.size()];            
int for1;
for (for1 = 0;for1 < Trace.size(); for1 ++) {
     surfaceTrajetx[for1] = Math.round(((dpoint) Trace.get(for1)).x);
 
     // si Java 5 et utilisation des générics
//     surfaceTrajetx[for1] =  Math.round( Trace.get(for1).x);  
}

A moins que sa classe Dpoint nécessite obligatoirement l'utilisation de double, il faut mieux directement déclarer ses attributs en int plutôt que d'utiliser cette methode (qui dit transtypage dit cout en ressources, surtout dans une boucle).

[dazz_x]

tu as raison, bien evidemment, mais je pars du principe qu'il se sert déjà de sa classe Dpoint un petit peu partout et qu'il ne peux pas faire trop de modif... Je pars aussi du principe qu'il dessine ses éléments et qu'avec Graphics2D, il vaut mieux garder des coordonnées en double pour les points...

Mais effectivement, je fais beaucoup d'hypothèses

[le y@m's]

tu as raison, bien evidemment, mais je pars du principe qu'il se sert déjà de sa classe Dpoint un petit peu partout et qu'il ne peux pas faire trop de modif... Je pars aussi du principe qu'il dessine ses éléments et qu'avec Graphics2D, il vaut mieux garder des coordonnées en double pour les points...

Mais effectivement, je fais beaucoup d'hypothèses

C'est vrai qu'il est difficile de connaitre le contexte d'utilisation de la classe.
Au final je pense qu'il s'agit plus d'une erreur (manque) de conception qu'autre chose.
pat-trix as-tu d'abord réfléchi sur l'organisation de tes classes ainsi que leurs intéractions ? cela te permettrais de définir le type des attributs dont tu as besoin.

[pat-trix]

Hey!
Je vais vous expliquer le contexte de ce bout de code:
Je travaille sur une simulation de voile.
Enfait, j'ai créer le Vector Trace afin de "prendre" les valeurs sur x et sur y de la position du bateau à chaq1ue tour du thread, afin de faire une "trace" du bateau (en les dessinant à chaque fois tous).
J'aimerais maintenant créer un Polygon, obtenu en "reliant" tous les points des traces. (Ceci pour vérifier que toutes les bouées sont bien "dans " le Polygon, avec la méthode '.contains') NB dazz_x, c'est un plus par rapport à la méthode des bissectrices (qui marche très bien ), mais où l'utilisateur peut tricher (il peut revenir en arrière une fois passé la bissectrice), il fera que perdre du temps, et c'est juste un petit test en +, qui en+ de ça me fait découvrir de nouvelles chose en programmation.
(Rappelons que je suis un débutant)...!

Voilà, merci encore !

Cordialement

Pat-trix

[dazz_x]

allez, j'vais faire un p'tit commentaire, vu que t'as implémenté la méthode du determinant pour l'orientation d'un triangle...
Je ne sais pas quel algo Java implémente pour la méthode contains, mais je pense que si tu vérifies que tous les triangles bouée_i- bouée_i+1 - Bateau sont orientés dans le même sens, c'est que tu es à l'intérieur du polygone
(evidemment ça ne fonctionne que pour un polygone convexe...)


Sinon, je pense que c'est le moment pour toi de te poser et de penser un peu à la structuration de ton code... Tous ces tableaux de points... Enfin je dis ça, je ne l'ai pas eu sous les yeux, donc c'est difficile de juger, c'est une simple impression

au plaisir