Discussion :

[AttilaMarkus]

Bonjour,

Occupé à faire des test unitaires sur mon code, je rencontre un soucis avec celui-ci:

Code :

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

1
2
3
4
5
6
7
@Test
    public void testGetCas1(){
         Stock stock = new Stock(new Tile(3),new Tile(-2),new Tile(1),new Tile(-4));
         Tile expected=new Tile(3);
         Tile answer=stock.get(TileType.SUSHI, 0);
         assertEquals(expected, answer);
    }

Mon test vérifie la méthode suivante:

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
 /**
     * The method show the tile number <b>pos</b>(since position 0) of Tiletype
     * get at parameter.
     *
     * @param type the tiletype(sushi or fishbone)
     * @param pos the position in the list(sushiList or fishboneList)
     * @return the tile corresponding of parameters
     */
    public Tile get(TileType type, int pos) {
        Tile domino;
        if (type.equals(TileType.SUSHI)) {
            domino = sushiList.get(pos);
        } else {
            domino = fishboneList.get(pos);
        }
        return domino;
    }

Mon test me renvoi comme erreur:

expected: g42015.sushibar.model.Tile<Valeur du domino: 3> but was: g42015.sushibar.model.Tile<Valeur du domino: 3>

junit.framework.AssertionFailedError
at g42015.sushibar.model.StockTest.testGetCas1(StockTest.java:57)

Il semble donc y avoir une erreur dans mon assertEquals. J'ai l'impression d'avoir mal rédigé mon Tile expected, mais je n'en suis pas certain.

Est-ce que quelqu'un peut m'aider?

merci

[adiGuba]

Salut,


Il faudrait voir le code de la méthode equals() de ta classe Tile...


a++

[AttilaMarkus]

Bonjour,

Voici mon code equals:

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
@Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null) {
            return false;
        }
        if (getClass() != obj.getClass()) {
            return false;
        }
        final Tile other = (Tile) obj;
        if (this.value != other.value) {
            return false;
        }
        return true;
    }

[joel.drigo]

Salut,

Donne le code complet de la classe Tile (au moins son contructeur et sa méthode toString())... Et celui du constructeur de Stock...

[AttilaMarkus]

Salut,

Voici ma classe Tile:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
public class Tile {
    //=========================================================================
    //ATTRIBUT   
 
    /**
     * A integer value represent the tile.
     */
    private int value;
 
    //=========================================================================
    //CONSTRUCTOR
    /**
     * The method instancie a object of type Tile. Each tile get a value
     *
     * @param value
     */
    public Tile(int value) {
        this.value = value;
    }
 
    //======================================================================
    //METHODS
 
    /**
     * The method indicate the TileType of a tile: Sushi or Fishbone. 
     * Sushi, if the value is negative.
     * Fishbone, if the value is positive.
     * @return the tileType of the tile
     */
    public TileType getType(){
        TileType type;
        if(value>=1){
            type=TileType.SUSHI;
        }else if(value<0)
            type=TileType.FISHBONE;
        else
            throw new IllegalArgumentException("Erreur de valeur: "+value);
        return type;
    }
 
 
    /**
     * The method get the value of the tile.
     * @return a integer number
     */
    public int getValue() {
        return value;
    }
 
    /**
     * The method display the value of the tile.
     * @return a string with the value
     */
    @Override
    public String toString() {
        return " "+value;
    }
 
    @Override
    public int hashCode() {
        int hash = 5;
        hash = 23 * hash + this.value;
        return hash;
    }
 
    @Override
    public boolean equals(Object obj) {
        if (this == obj) return true;
 
        if (obj == null) return false;
 
        if (getClass() != obj.getClass()) {
            return false;
        }
        final Tile other = (Tile) obj;
        if (this.value != other.value) {
            return false;
        }
        return true;
    }
 
}

Voici ma classe Stock:

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
27
28
29
30
31
32
33
public class Stock {
//=========================================================================
    //ATTRIBUTS
 
    /**
     * The class have two attributs, two lists wich represents two range of
     * tiles to win.
     * <b>sushiList</b> the range of sushi tiles
     * <b>fishbone</b> the range of fishbone tiles
     */
    private List<Tile> sushiList;
    private List<Tile> fishboneList;
 
    //=========================================================================
    //CONSTRUCTOR
    /**
     * The constructor receive several parameters and classify the tiles into
     * two lists.
     *
     * @param tile
     */
    public Stock(Tile... tile) {
        this.sushiList = new ArrayList<>();
        this.fishboneList = new ArrayList<>();
        for (int i = 0; i < tile.length; i++) {
            if (tile[i].getValue() > 0) {
                this.sushiList.add(tile[i]);
            } else {
                this.fishboneList.add(tile[i]);
            }
        }
 
    }

Merci!

[tchize_]

Je n'arrive pas à reporduire le problème avec le code que tu donne.

Code :

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

1
2
3
4
5
6
7
8
9
public class Main{
 
     public static void main(String []args){
         Stock stock = new Stock(new Tile(3),new Tile(-2),new Tile(1),new Tile(-4));
         Tile expected=new Tile(3);
         Tile answer=stock.get(TileType.SUSHI, 0);
         System.out.println("equals? "+expected.equals(answer));
     }
}

ça me renvoie bien "true"