[Task 9.1] 71104 Vs. TigerShark Pair Reloaded

[Ufo13]

mi spiego meglio... se la gemma che non deve uscire è la nr 3. se esce 4 diventa 5 se esce 5 diventa la 0. Ma se esce 0 rimane 0. Quindi la 0 ha probabilità doppia di uscire!!! Giusto?

[71104]

Originariamente inviato da cionci

71104: quando avete finito testa che sia possibile estratte un diamond_gem dopo un diamond_chest

l'ho appena testato
adesso Bonfo deve risolvere...

[Ufo13]

Originariamente inviato da Ufo13

mi spiego meglio... se la gemma che non deve uscire è la nr 3. se esce 4 diventa 5 se esce 5 diventa la 0. Ma se esce 0 rimane 0. Quindi la 0 ha probabilità doppia di uscire!!! Giusto?

ahhh no perchè c'è il -1 nel module... ho capito... però continuo a non essere sicuro sul firstGemExtraction e secondGemExtraction... boh

[Ufo13]

e ora ho capito anche quelli... però ora li rinomino

[cionci]

Originariamente inviato da Ufo13

mi spiego meglio... se la gemma che non deve uscire è la nr 3. se esce 4 diventa 5 se esce 5 diventa la 0. Ma se esce 0 rimane 0. Quindi la 0 ha probabilità doppia di uscire!!! Giusto?

Sì questo se la popolazione fosse N...ma grazie a firstExtraction la setto a N-1...quindi 5 non può uscire

Se la gemma estratta è la 3 e N = 5 (quindi al massimo posso estrarre 4) di fatto la popolazione su cui estraggo è: 0, 1, 2, 4

Infatti passando al random generator N-1 (cioè 4) posso tirare fuori N-1 numeri da 0 a 3...ma se l'ultima gemma estratta è 3, tutti i numeri che posso estrarre >= 3 vengono incrementati...quindi mi riporto in una situazione corretta, esattamente come sopra: 0, 1, 2, 4...

[Ufo13]

sìsì capito tutto... era il first ed il second che non mi ritornava più che altro... ma ora ricordo anche che la gemma deve essere diversa non dall'ultima ma dalla penultima

[cionci]

Originariamente inviato da Ufo13

ahhh no perchè c'è il -1 nel module... ho capito... però continuo a non essere sicuro sul firstGemExtraction e secondGemExtraction... boh

Nell'estrazione bisogna fare in modo che la gemma estratta debba essere DIVERSA, non dall'ultima estratta, ma dalla PENULTIMA gemma estratta... Come fare altrimenti te ?

PS: questa cosa è testata...

[Bonfo]

Modificata GemFactory:

    private int firstGemRandomIndex = GEM_TYPES.length;
    private int secondGemRandomIndex = GEM_TYPES.length;
    
    private int firstChestRandomIndex = CHEST_TYPES.length;
    private int secondChestRandomIndex = CHEST_TYPES.length;

    private boolean firstGemExtraction = true;
    private boolean firstChestExtraction = true;

Ora vengono usati indici differenti per createRandomGem e createRandomChest

    public Gem createRandomGem()
    {
        int module = GEM_TYPES.length - 1;
        
        if(firstGemExtraction)
        {
            module = GEM_TYPES.length;
            firstGemExtraction = false;
        }

        int randomIndex = randomGenerator.extract(module);

        if(randomIndex >= secondGemRandomIndex)
        {
            randomIndex = (randomIndex + 1) % GEM_TYPES.length;
        }

        secondGemRandomIndex = firstGemRandomIndex;
        firstGemRandomIndex = randomIndex;       

        return create(GEM_TYPES[randomIndex]);
    }
    
    public Gem createRandomChest()
    {
        int module = CHEST_TYPES.length - 1;
        
        if(firstChestExtraction)
        {
            module = CHEST_TYPES.length;
            firstChestExtraction = false;
        }

        int randomIndex = randomGenerator.extract(module);

        if(randomIndex >= secondChestRandomIndex)
        {
            randomIndex = (randomIndex + 1) % CHEST_TYPES.length;
        }

        secondChestRandomIndex = firstChestRandomIndex;
        firstChestRandomIndex = randomIndex;
        
        return create(CHEST_TYPES[randomIndex]);
    }

Prossimo Test:

    public void testNotTwoEqualChestSequence()
    {
        int[] percentages={1, 1};
        factory = GemFactory.createForTesting(new MockRandomGenerator(percentages));
        
        assertNotSame(factory.createRandomChest(),factory.createRandomChest());
    }

[cionci]

Testate anche questa sequenza di estrazioni che DEVE funzionare: diamond_chest, diamond_gem, diamond_chest

[Ufo13]

Originariamente inviato da cionci

Testate anche questa sequenza di estrazioni che DEVE funzionare: diamond_chest, diamond_gem, diamond_chest

Penso sia già testata con questo no?

    public void testSameIndexForGemAndChest()
    {
        int[] percentages={1, 0, 1};
        factory = GemFactory.createForTesting(new MockRandomGenerator(percentages));
        
        Gem gem = factory.createRandomGem();
        assertSame(gem.getType(), RUBY);

        factory.createRandomGem();
        
        gem = factory.createRandomChest();
        assertSame(gem.getType(), RUBY_CHEST);
    }

[cionci]

Secondo me no...basta provare

[cionci]

Originariamente inviato da cionci

Testate anche questa sequenza di estrazioni che DEVE funzionare: diamond_chest, diamond_gem, diamond_chest

Ah...nel mezzo ci puoi mettere anche un emerald o qualsiasi altra gemma o chest...l'importante è che il primo e il terzo siano un chest dello stesso tipo...

[fek]

Originariamente inviato da Bonfo

    public Gem createRandomGem()
    {
        int module = GEM_TYPES.length - 1;
        
        if(firstGemExtraction)
        {
            module = GEM_TYPES.length;
            firstGemExtraction = false;
        }

        int randomIndex = randomGenerator.extract(module);

        if(randomIndex >= secondGemRandomIndex)
        {
            randomIndex = (randomIndex + 1) % GEM_TYPES.length;
        }

        secondGemRandomIndex = firstGemRandomIndex;
        firstGemRandomIndex = randomIndex;       

        return create(GEM_TYPES[randomIndex]);
    }
    
    public Gem createRandomChest()
    {
        int module = CHEST_TYPES.length - 1;
        
        if(firstChestExtraction)
        {
            module = CHEST_TYPES.length;
            firstChestExtraction = false;
        }

        int randomIndex = randomGenerator.extract(module);

        if(randomIndex >= secondChestRandomIndex)
        {
            randomIndex = (randomIndex + 1) % CHEST_TYPES.length;
        }

        secondChestRandomIndex = firstChestRandomIndex;
        firstChestRandomIndex = randomIndex;
        
        return create(CHEST_TYPES[randomIndex]);
    }

Ullalla' quanto bel codice duplicato. Urge refactoring alla fine del task.

[71104]

cancellare post plz

[Ufo13]

che post?

[71104]

Originariamente inviato da Ufo13

che post?

il numero 74; stavo per scrivere una sciocchezza, ma ho editato in tempo

[Ufo13]

mi spiego meglio... manca ancora un test per il task... Ovvero quello che testa che Grid inserisca una GemsPair con un Chest (o 2 )

[71104]

Originariamente inviato da Ufo13

mi spiego meglio... manca ancora un test per il task... Ovvero quello che testa che Grid inserisca una GemsPair con un Chest (o 2 )

ok, allora mettiamo questo affare in TestGemQueue

    public void testGemQueueCreatesChestsToo()
    {
        int mockValues[] = {0, 1, 99, 1};
        MockRandomGenerator randomGenerator = new MockRandomGenerator(mockValues);
        gemQueue = GemQueue.createForTesting(randomGenerator);
        
        gemQueue.fillQueueRandomly();
        assertSame(gemQueue.extract().getType(), RUBY);
        assertSame(gemQueue.extract().getType(), RUBY_CHEST);
    }

[cionci]

Originariamente inviato da cionci

Ah...nel mezzo ci puoi mettere anche un emerald o qualsiasi altra gemma o chest...l'importante è che il primo e il terzo siano un chest dello stesso tipo...

Mi sento un po' ignorato

    
    public void testChestGemChestSequence()
    {
        int indexes[] = {99, 1, 0, 4, 99, 1};
        factory = GemFactory.createForTesting(new MockRandomGenerator(indexes));
        
        GemType firstChest = factory.createRandomDroppable().getType();
        factory.createRandomDroppable().getType();
        GemType lastChest = factory.createRandomDroppable().getType();
        assertSame("first chest was not of same type of last chest", firstChest, lastChest);
    }
    
    
    public void testGemGemChestGemSequence()
    {
        int indexes[] = {0, 1, 0, 2, 99, 3, 0, 1};
        factory = GemFactory.createForTesting(new MockRandomGenerator(indexes));
        
        GemType firstGem = factory.createRandomDroppable().getType();
        factory.createRandomDroppable().getType();
        factory.createRandomDroppable().getType();
        GemType lastGem = factory.createRandomDroppable().getType();
        assertSame("first Gem was not of same type of last Gem", firstGem, lastGem);        
    }

Questi test secondo me non devono fallire...

[Ufo13]

falliscono?