Strategie um einen Bug zu finden

[herzog]

Hallo,
ich habe einen Kopfrechentrainer geschrieben bei dem man die Möglichkeit hat über eine GUI auszuwählen in welchem Bereich die beiden Zahlen liegen sollen. Per Zufallsgenerator werden dann die Aufgaben erzeugt.
Es funktioniert auch alles wunderbar, nur wenn ich für die erste Zahl auswähle, dass sie zwischen 2 und 9 liegen soll, ist die Zahl immer die selbe, sprich der Generator funktioniert nicht. Bei allen anderen Zahlen geht es. Nur nicht bei 2 und 9. Woran kann sowas liegen und vorallem wie finde ich so was raus? Das Programm funktioniert ja eigenltich. Eigentlich..
Zuerst hatte ich immer die gleichen Aufgaben, weil der Zufallsgenerator mit dem gleichen Seed erzeugt wurde. Dafür habe ich dann ein Delay eingebaut. Aber daran kann es diesmal nicht liegen...

Die Klasse Task

public class Task{
    private double figure1, figure2;
    private double solution;
    private char operator;
    private Random r;

    public Task() {
        try {
            Thread.sleep(1);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
        r = new Random();

        this.operator = r.getOperator(Options.getOperator());
        this.figure2 = r.getInt(Options.getMinF2(), Options.getMaxF2());
        this.figure1 = r.getInt(Options.getMinF1(), Options.getMaxF1());

        switch(this.operator){
            case '+': this.solution = this.figure1 + this.figure2;
                break;
            case '-': this.solution = this.figure1 - this.figure2;
                break;
            case '*': this.solution = this.figure1 * this.figure2;
                break;
            case '/': this.solution = this.figure1 / this.figure2;
                break;
         }
    }

Was auch noch kurios ist:
Wenn ich

this.figure2 = r.getInt(Options.getMinF2(), Options.getMaxF2());
this.figure1 = r.getInt(Options.getMinF1(), Options.getMaxF1());

schreibe, tritt der Fehler bei der zweiten Zahl auf und wenn ich es drehe bei der Ersten.

Hier noch der Zufallsgenerator (der böserweise auch Random heißt):

public class Random {

    private java.util.Random r;

    public Random(){
        r = new java.util.Random(System.currentTimeMillis());
    }

    public int getInt(int lower, int upper){
        return r.nextInt(upper + 1 - lower) + lower;

    }
}

Ich vermute es will sich niemand durch den kompletten wüsten Anfänger-Code wühlen, aber falls Interesse besteht, sagt Bescheid. Ich hoffe ihr könnt mir weiterhelfen.
Gruß

 

[Marco13]

Mach das mit dem Delay raus, das ist Unfug. Die Klasse umzubenennen könnte auch nicht schaden. Und wenn du die Methode änderst zu

public int getInt(int lower, int upper)
{
    int result = r.nextInt(upper + 1 - lower) + lower;
    System.out.println("Zufall zwischen "+lower+" und "+upper+": "+result);    
    return result;
}

wirst du vielleicht etwas näher an die Ursache kommen. Ich tippe auf einen Fehler in den "Options".

 

[bygones]

sonsten generell: JUnit test(s) schreiben der das Verhalten aufdeckt

 

[0x7F800000]

sonsten generell: JUnit test(s) schreiben der das Verhalten aufdeckt

Imho: nicht der tote Bug ist der beste Bug, sondern der ungeborene^^
Finde diese Taktik zur Bug-Bekämpfung eigentlich am angenehmsten: einfach keine Fehler machen :smoke: Und im Notfall hat man ja junit, asserts, RuntimeExceptions, das gute alte out.println, debugger, logger, sonstwas...

 

[herzog]

Mach das mit dem Delay raus, das ist Unfug. Die Klasse umzubenennen könnte auch nicht schaden. Und wenn du die Methode änderst zu

public int getInt(int lower, int upper)
{
    int result = r.nextInt(upper + 1 - lower) + lower;
    System.out.println("Zufall zwischen "+lower+" und "+upper+": "+result);    
    return result;
}

wirst du vielleicht etwas näher an die Ursache kommen. Ich tippe auf einen Fehler in den "Options".

Vermutlich war das mit dem Delay kein guter Stil, aber hat seinen Zweck erfüllt ich habe aber aus der dem Objekt von Random eine statische Variable gemacht und jetzt funktioniert alles. Delay weg und auch der Fehler mit der 2 und der 9 wo auch immer er herkam. Ich weiß es nicht... Übrigens war er nicht da wenn ich das ganze mit dem Debugger habe durchlaufen lassen.

sonsten generell: JUnit test(s) schreiben der das Verhalten aufdeckt

danke, davon hatte ich noch nichts gehört. eben einmal kurz angelesen und hört sich sehr hilfreich an.

Imho: nicht der tote Bug ist der beste Bug, sondern der ungeborene^^
Finde diese Taktik zur Bug-Bekämpfung eigentlich am angenehmsten: einfach keine Fehler machen :smoke: Und im Notfall hat man ja junit, asserts, RuntimeExceptions, das gute alte out.println, debugger, logger, sonstwas...

mit dem Ziel habe ich auch angefangen wäre schön wenn es immer so klappen würde.