[CICLO 10] Storia 1

[^TiGeRShArK^]

nope...
mi pareva troppo bello...
ancora c'è lo stesso problema....

[^TiGeRShArK^]

ora mi ha creato una terza griglia sotto la prima!

[^TiGeRShArK^]

ma porka troia...
mi legge le variabili

private static final int GRID_P2_TOP = 32;
    private static final int GRID_P2_LEFT = 524;

invertite!
ma com'è possibile???
ho provato col debug e mi dice GRID_P2_TOP = 20 e GRID_P2_LEFT = 524
mai vista una cosa del genere...

[cionci]

Guarda che basta usare due Factory Method diversi...uno per il primo controlelr e uno per il secondo...

[^TiGeRShArK^]

boh..giuro ke non ci capisco niente...come fa a cambiare il valori di variabile final e a ritrovarselo sbagliato???
il codice incriminato è il seguente:

private static GridController createWithGemGenerator(Config config, Input input, GemGenerator gemGenerator, boolean isPlayer1)
    {
        Grid grid;
        if(isPlayer1)
        {
        grid = new Grid(
            new Rectangle(
                GRID_P1_LEFT, 
                GRID_P1_TOP, 
                WIDTH + GRID_P1_LEFT - 1, 
                HEIGTH + GRID_P1_TOP - 1), 
            config);
        } 
        else 
        {
            grid = new Grid(
                new Rectangle(
                    GRID_P2_LEFT, 
                    GRID_P2_TOP, 
                    WIDTH + GRID_P2_LEFT - 1, 
                    HEIGTH + GRID_P2_TOP - 1), 
                config);
        }

        InputReactor inputReactor = new InputReactor(
            input, 
            config.getInteger("NormalRepeatDelay"), 
            config.getInteger("FastRepeatDelay"));

        GridController controller = new GridController(
            grid, 
            inputReactor,gemGenerator
            );

        controller.setDelay(config.getInteger("NewGemDelay"));
        return controller;
    }

ho controllato col debug per vedere se prendeva il ramo giusto dell'if ma mi da quel valore assurdo per uqelle costanti....
non ci capisco + niente...

[^TiGeRShArK^]

Originariamente inviato da cionci

Guarda che basta usare due Factory Method diversi...uno per il primo controlelr e uno per il secondo...

si ok... ma ks xkè non funge???

[cionci]

Boh...

Io l'ho messo così:

    public static GridController createFirst(Config config, Input input)
    {
        return createWithGemGenerator(config, input, 
            new GemQueue(config), new Rectangle(20, 32, 275, 479));
    }
    
    
    public static GridController createSecond(Config config, Input input)
    {
        return createWithGemGenerator(config, input, 
            new GemQueue(config), new Rectangle(524, 32, 779, 479));
    }

[^TiGeRShArK^]

ah.. gli hai messo proprio i valori numerici nel metodo?
hai provato ad usare le costanti per vedere se ti da lo stesso problema mio???

[cionci]

Le costanti le ho proprio tolte perchè non venivano lette da nessuno...

[^TiGeRShArK^]

ah... pensavo fossero messe per una questione di leggibilità in modo da averle sempre sott'okkio in cima alla classe.....

[^TiGeRShArK^]

cmq ora lo faccio col factory method...
uso le costanti o metto direttamente i valori numerici????

[cionci]

Fai te... Comunque potrebbero bastare anche le costanti vecchie... Poi basta sfasare 304 pixel il primo e il terzo parametro del rectangle...

[^TiGeRShArK^]

fatto....però ancora le gemme sono solo nella prima griglia... ora vedo se capisco ke mi dimentico : muro:

[^TiGeRShArK^]

risolto...
oa vedo se passa il checkstyle e committo

[^TiGeRShArK^]

perfetto... green..
l'unico dubbio..
ho mantenuto il metodo creat di gridcontroller per compatibilità con i test...
ke faccio lo sostituisco con createFirst? forse sarebbe piu' logico...
anzi ... ora lo faccio

[^TiGeRShArK^]

ok perfetto...
Task 1 completato...
1 grazie a cionci per l'aiuto.....
e 1 maledizione a quel maledetto bug delle costanti sfasate ke mi ha fatto perdere la salute stasera

[Bonfo]

Spero di essermi perso qualcosa....altrimenti ti devo sgridare Tiger.

LA TEST LIST ???



Spero di non aver rubato la frase a uno dei project manager

[VICIUS]

Originariamente inviato da Bonfo

Spero di essermi perso qualcosa....altrimenti ti devo sgridare Tiger.

LA TEST LIST ???



Spero di non aver rubato la frase a uno dei project manager

Non è stato introdotta nessuna nuova funzione quindi è gia tutto testato

Ora che siamo partiti mi aspetto di vedere tutti task completati fra due giorni. Quindi dateci sotto su che sono facili.

ciao

[^TiGeRShArK^]

Originariamente inviato da VICIUS

Non è stato introdotta nessuna nuova funzione quindi è gia tutto testato

Ora che siamo partiti mi aspetto di vedere tutti task completati fra due giorni. Quindi dateci sotto su che sono facili.

ciao

infatti era un semplice refactoring
vabbè.. semplice a parte il mio solito ke bekko le situazioni piu' inspiegabili... tipo quelle costanti ke cambiano valore da sole...

[Bonfo]

LA TEST LIST ???

ops...che figuraccia !!!