[TASK 8.2.2] Pair Programming: Ufo13 + thebol

[Ufo13]

Inizio con QuickFix sul draw di GemGroup per far compilare il tutto...

Build rossa... NullPointerException... Implemento il draw

    public void draw(PositionType position, EngineInterface engine)
    {
        Point origin = POSITIONTOPOINT.get(position);
        
        sprite.setOrigin((int)origin.getX(), (int)origin.getY());
        
        sprite.draw(engine);
    }

Ho aggiunto una HashMap per tenere le posizioni di ogni tile all'interno della Texture.

Build verde

[Ufo13]

Bene preparo il prossimo test

    public void testDrawInCorrectPositions()
    {
        MockEngine engine = MockEngine.createForTesting(800, 600);
        
        Gem gem9 = Gem.create(gem3.getType(), 3500);
        Gem gem10 = Gem.create(gem3.getType(), 3500);
        Gem gem11 = Gem.create(gem3.getType(), 3500);
        
        insertAndUpdate(gem3, 13, 1);
        insertAndUpdate(gem4, 13, 2);
        insertAndUpdate(gem5, 13, 3);
        insertAndUpdate(gem6, 12, 1);
        insertAndUpdate(gem7, 12, 2);
        insertAndUpdate(gem8, 12, 3);
        insertAndUpdate(gem9, 11, 1);
        insertAndUpdate(gem10, 11, 2);
        insertAndUpdate(gem11, 11, 3);
        
        grid.draw(engine);
        
        assertEquals("GemGroup wasn't drawn correctly", 10, engine.getNumberOfQuadsDrawn());
    }

I quad sono 10 perchè c'è anche il Background

Build rossa...

[Ufo13]

thebol è uscito e Fek ha chiesto di finire il task stasera... Finisco io...

cambio il draw in Grid

    public void draw(EngineInterface engine)
    {
        background.draw(engine);

        for(int y = grid.length - 1; y >= 0; y--)
        {
            for(int x = 0; x < grid[y].length; x++)
            {
                Gem gem = getGemAt(y, x);
                if(gem != null)
                {
                    gem.draw(engine);
                    
                    if(gem.getGroup() != null)
                    {
                        gem.getGroup().draw(gem.getGroup().getPositionType(gem), engine);
                    }
                }
            }
        }
    }

Build Verde

[Ufo13]

Mi manca ancora un test... la draw di un tile deve venire fatta alle coordinate della gemma corrispondente nel gemgroup.

Hmmm pensandoci bene preferisco spezzarlo in due test... Prima controllo che la draw venga fatta in una certa posizione.

    public void testGridDrawsTilesInRightPositions()
    {
        MockEngine engine = MockEngine.createForTesting(800, 600);
        
        insertAndUpdate(gem3, 13, 1);
        insertAndUpdate(gem4, 13, 2);
        insertAndUpdate(gem5, 12, 1);
        insertAndUpdate(gem6, 12, 2);
        
        gem3.getGroup().draw(PositionType.BottomLeftCorner, engine, new Point(10, 20));
        
        assertEquals("Tile wasn't drawn in correct position (X)", 10, (int)engine.getQuadPosition().getX());
        assertEquals("Tile wasn't drawn in correct position (Y)", 20, (int)engine.getQuadPosition().getY());
        
        gem3.getGroup().draw(PositionType.BottomLeftCorner, engine, new Point(30, 40));
        
        assertEquals("Tile wasn't drawn in correct position (X)", 30, (int)engine.getQuadPosition().getX());
        assertEquals("Tile wasn't drawn in correct position (Y)", 40, (int)engine.getQuadPosition().getY());        
    }

Il codice che fa passare il test è questo:

    public void draw(PositionType positionType, EngineInterface engine, Point position)
    {
        Point origin = POSITIONTOPOINT.get(positionType);
        
        sprite.setOrigin((int)origin.getX(), (int)origin.getY());
        
        sprite.moveTo(position.getX(), position.getY());
        
        sprite.draw(engine);
    }

[Ufo13]

Mi serve un test che dimostri la correttezza della posizione della draw...

Non sono riuscito a trovare modo migliore per fare questo test

    public void testGridDrawsEveryTileInCorrespondingGemPosition()
    {
        MockEngine engine = MockEngine.createForTesting(800, 600);
        
        insertAndUpdate(gem3, 13, 1);
        insertAndUpdate(gem4, 13, 2);
        insertAndUpdate(gem5, 12, 1);
        insertAndUpdate(gem6, 12, 2);
        
        grid.draw(engine);
        
        assertEquals("TopRight tile wasn't drawn correctly (X)", gem6.getX(), engine.getQuadPosition().getX());
        assertEquals("TopRight tile wasn't drawn correctly (Y)", gem6.getY(), engine.getQuadPosition().getY());
        
        gem6.moveTo(60, 50);
        
        grid.draw(engine);
        
        assertEquals("TopRight tile wasn't drawn correctly (X)", gem6.getX(), engine.getQuadPosition().getX());
        assertEquals("TopRight tile wasn't drawn correctly (Y)", gem6.getY(), engine.getQuadPosition().getY());
    }

Il codice che fa passare il test è semplicissimo:

    public void draw(EngineInterface engine)
    {
        background.draw(engine);

        for(int y = grid.length - 1; y >= 0; y--)
        {
            for(int x = 0; x < grid[y].length; x++)
            {
                Gem gem = getGemAt(y, x);
                if(gem != null)
                {
                    gem.draw(engine);
                    
                    if(gem.getGroup() != null)
                    {
                        Point position = new Point(gem.getX(), gem.getY());
                        
                        GemGroup gemGroup = gem.getGroup();
                        
                        gemGroup.draw(gemGroup.getPositionType(gem), engine, position);
                    }
                }
            }
        }
    }

E finalmente gli agglomerati si vedono in gioco

[fek]

Appena provata la build. Che belli i gemmoni!