Fehler bei Zufallsgeneration von Zahlen

[der_Schokomuffin]

Hallo erstmal,
ich bin gerade dabei, etwas zu programmieren. Nun treffe ich auf folgenden Fehler: Ich habe im Code einen Zufallsgenerator mit java.util.Random. Darum ist eine Schleife, die verhindern soll, dass er 0 ergibt. Hier der Code:

int body_weight;

while(body_weight < 45) {
     body_weight = 1 + gen1.nextInt(94);
}

Nun soll "body_weight" mit JOptionPane.showMessageDialog(); ausgegeben werden.
Der Fehler ist nun, dass immer noch vereinzelt 0 für body_weight ausgegeben wird.
Ich selbst hab schon mehrere Sachen ausprobiert, die den Fehler beheben sollten, wie z.B. eine "do" - Schleife verwendet. Leider wurde immer noch ab und zu 0 ausgegeben.

Über Antworten wäre ich sehr erfreut.

PS: Ich benutze Eclipse, falls es jemand wissen möchte.

 

[Robat]

Kannst du mal den kompletten relevanten Code posten, bitte?
So wie du den Code jetzt gepostet hast würde er nicht mal compilieren, weil die lokale Varaible body_weight nicht initialisiert ist.

 

[der_Schokomuffin]

Nur zur Info, das ist eigentlich ein Spiel wo ich dran arbeite. Ist aber ein bisschen viel Code.

Die Datei mit Main - Class:

package ccv03;

import javax.swing.*;                                   
import java.util.Random;                               
                                                       
public class CashCommanderV03 {                           

    public static void main(String[] args) {
       
        int start = JOptionPane.showConfirmDialog(null, "Wilkommen bei CashCommander!\n\nNeue Schicht starten?");
       
        int time = 3;
        int customer_counter = 0;
        int rule_fix_attempt = 0;
        while(start >= 0) {
            Random gen0 = new Random();
            ProductV01 wrong_product = new ProductV01();
            if(time == 0) {
                JOptionPane.showMessageDialog(null, "Schicht zuende!");
                start = JOptionPane.showConfirmDialog(null, "Neue Schicht starten?");
                if(start != 0) {
                    JOptionPane.showMessageDialog(null, "Gut, dann nicht. Bis zum nächten mal bei\n\nCashCommander\n\n: ]");
                    break;
                }
                else {
                    time = 3;
                }
            }
            JOptionPane.showMessageDialog(null, "Folgendes Produkt darf nicht verkauft werden: \n\n" + wrong_product.getProduct());
            System.out.println(wrong_product.getProduct());
            while(time != 0) {
                int number_products = 1 + gen0.nextInt(12);
           
                CustomerV01 customer = new CustomerV01();
               
                if(customer.gender == 'w') {
                    JOptionPane.showMessageDialog(null, "Neue Kundin!");
                }
                else {
                    JOptionPane.showMessageDialog(null, "Neuer Kunde!");
                }
                System.out.println(customer.gender);
                System.out.println(number_products);
                while(number_products != 0) {
                    ProductV01 product = new ProductV01();
                    int scan_result = JOptionPane.showConfirmDialog(null, "Ware: " + product.getProduct() + "\n\nWare abscannen?");
                    System.out.println(product.getProduct());
                    if(scan_result == 0) {
                        JOptionPane.showMessageDialog(null, "Ware abgescannt!");
                    }
                    else {
                        rule_fix_attempt = Integer.parseInt(JOptionPane.showInputDialog("Optionen:\n\n1.: Ware darf nicht verkauft werden\n\n2.: Alter prüfen\n\n"
                                                                                        + "3.: Gewicht prüfen\n\n4.: Abbrechen\n\nBitte Nummer eingeben:"));
                       
                        if(rule_fix_attempt == 1 && product.getProduct().equals(wrong_product.getProduct())) {
                            JOptionPane.showMessageDialog(null, "Kunde: \"Oh, tut mir leid, ich wusste das nicht :[\"");
                        }
                        else if (rule_fix_attempt == 1 && !product.getProduct().equals(wrong_product.getProduct())) {
                            JOptionPane.showMessageDialog(null, "Ich habe gerade nochmal nachgefragt und ich kann dieses Produkt kaufen.");
                        }
                        else if (rule_fix_attempt == 2) {
                            JOptionPane.showMessageDialog(null, "Personalausweis:\n\nName: ...\n\nAlter: " + customer.age + "\n\nGeschlecht: " + customer.gender);
                            int sell = JOptionPane.showConfirmDialog(null, "Ware verkaufen?");
                            if(sell != 0 && customer.rule_number == 1) {
                                JOptionPane.showMessageDialog(null, "Kunde: \"Verdammt!\" \n\n *Kunde läuft weg*");
                                break;
                            }
                            else if (sell == 0 && customer.rule_number == 1) {
                                JOptionPane.showMessageDialog(null, "Boss: \"RAUS!!!\"");
                                time = 0;
                                break;
                            }
                            else if (sell == 0 && customer.rule_number != 1) {
                                JOptionPane.showMessageDialog(null, "Kunde: ...");
                            }
                            else {
                                JOptionPane.showMessageDialog(null, "Kunde: \"Aber ich bin doch volljährig...\"");
                            }
                        }
                        else if (rule_fix_attempt == 3) {
                            JOptionPane.showMessageDialog(null, "\"Bitte stellen sie sich einmal auf diese Waage.\"\n\nWaage zeigt " + customer.body_weight + " kg");
                            int sell = JOptionPane.showConfirmDialog(null, "Ware verkaufen?");
                            if(sell != 0 && customer.rule_number == 2) {
                                JOptionPane.showMessageDialog(null, "Kunde: \"Sie haben Recht. Ich bin einfach zu schwer. Deswegen gehe ich jetzt und tuhe was dagegen!\"\n\n"
                                        + "*Kunde läuft weg*");
                                break;
                            }
                            else if (sell == 0 && customer.rule_number == 2) {
                                JOptionPane.showMessageDialog(null, "Boss: \"RAUS!!!\"");
                                time = 0;
                                break;
                            }
                            else if (sell == 0 && customer.rule_number != 2) {
                                JOptionPane.showMessageDialog(null, "Kunde: \"...\"");
                            }
                            else {
                                JOptionPane.showMessageDialog(null, "Kunde: \"So schwer bin ich doch gar nicht...\"");
                            }
                        }
                        else {
                            JOptionPane.showMessageDialog(null, "Ware abgescannt!");
                        }
                    }
                    if(product.getProduct().equals(wrong_product.getProduct()) && scan_result == 0) {
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else if(product.getProduct().equals("Rotmützchen Wein") && scan_result == 0 && customer.rule_number == 1){
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else if(product.getProduct().equals("Schokolade") && scan_result == 0 && customer.rule_number == 2) {
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else if(product.getProduct().equals("Torte") && scan_result == 0 && customer.rule_number == 2) {
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else if(product.getProduct().equals("Donuts") && scan_result == 0 && customer.rule_number == 2){
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else if(product.getProduct().equals("Nutella") && scan_result == 0 && customer.rule_number == 2) {
                        JOptionPane.showMessageDialog(null, "Boss: \"Sie wissen schon, dass sie gerade gegen die Regeln verstoßen haben? Deswegen: RAUS!!!\"");
                        time = 0;
                        break;
                    }
                    else {
                        number_products -= 1;
                    }
                }
                customer_counter++;
                System.out.println(customer_counter);
                if(customer_counter == 5) {
                    if(time > 1) {
                        time -= 1;
                        customer_counter = 0;
                        JOptionPane.showMessageDialog(null, "Noch " + time + " Stunden!");
                    }
                    else if (time == 1) {
                        JOptionPane.showMessageDialog(null, "Noch eine Stunde!");
                        time -= 1;
                    }
                }
                else if(number_products == 0 && customer_counter != 5) {
                    customer_counter++;
                }
       
            }
        }
    }
}

Klasse ProductV01:

package ccv03;

import java.util.*;

public class ProductV01 {
   
    public ProductV01() {
        Random gen = new Random();
        int nProduct = 1 + gen.nextInt(15);
       
        switch(nProduct) {
            case 1: product = "Äpfel";
                break;
            case 2: product = "Butter";
                break;
            case 3: product = "Donuts";
                break;
            case 4: product = "Eier";
                break;
            case 5: product = "Fischstäbchen";
                break;
            case 6: product = "Geröllsteiner Mineralwasser";
                break;
            case 7: product = "Harribo Gummibärchen";
                break;
            case 8: product = "Kloßteig";
                break;
            case 9: product = "Müllbeutel";
                break;
            case 10:product = "Nutella";
                break;
            case 11:product = "Pompers Windeln";
                break;
            case 12:product = "Rotmützchen Wein";
                break;
            case 13:product = "Schokolade";
                break;
            case 14:product = "Torte";
                break;
            case 15:product = "Voll & Fit Energydrink";
                break;
        }
    }
    private String product;
   
    String getProduct() {
        return product;
    }
}

Klasse CustomerV01:

package ccv03;

import java.util.*;

public class CustomerV01 {
   
    char gender;                                       
    int age;
    int body_weight;
    boolean rule;
    int rule_number;
   
    public CustomerV01() {
        Random gen1 = new Random();
        int nGender = 1 + gen1.nextInt(2);
       
        if(nGender == 1) {
            gender = 'w';
        }
        else {
            gender = 'm';
        }
        int nRule = 1 + gen1.nextInt(3);
        if(nRule <= 2) {
            rule = true;
            rule_number = gen1.nextInt(2) + 1;
            if(rule_number == 1) {
                while(age < 8  || age == 0) {
                    age = gen1.nextInt(18) + 1;
                }
            }
            else if (rule_number == 2) {
                while(body_weight < 95) {
                    body_weight = gen1.nextInt(140) + 1;
                }
            }
        }
        else if(nRule == 3) {
            rule = false;
            do {
                age = 1 + gen1.nextInt(85);
            }while(age < 18);
            do {
                body_weight = 1 + gen1.nextInt(94);
            }while(body_weight < 45);
        }
    }                                               
}

Am besten ist das Problem jedoch im Spiel selbst zu sehen. Wenn du möchtest, kann ich es auch nomal als .jar posten und erklären, wie man den Fehler zu Gesicht bekommt.

 

[Robat]

body_weight kann bei dir aus 2 Gründen noch 0 sein:
- Wenn nRule kleiner oder gleich 2 ist UND rule_number 1 ist, dann machst du mit body_weight nichts. Da body_weight mit 0 initialisiert wird, kommt 0 heraus
- Wenn nRule kleiner oder gleich 2 ist UND rule_number 2 ist, musst du auch eine do-while-Schleife benutzen, anstatt einer while-Schleife. (du willst ja, dass die Schleife mindestens 1 mal betreten wird).

 

[der_Schokomuffin]

body_weight kann bei dir aus 2 Gründen noch 0 sein:
- Wenn nRule kleiner oder gleich 2 ist UND rule_number 1 ist, dann machst du mit body_weight nichts. Da body_weight mit 0 initialisiert wird, kommt 0 heraus

Dass macht natürlich Sinn. Also mal wieder ein Logischer Fehler.
Jedenfalls Danke für deine Hilfe! Ich probiere das jetzt mal aus und wenn es noch Fehler gibt, melde ich mich noch mal. Wenn ich mich nicht melde, dann hat alles funktioniert.

PS: Ich bin echt froh, dass es Leute gibt, die ein helfen

 

[MoxxiManagarm]

Schön ist der Code aber nicht

Hier schon mal Verbesserungsvorschläge (könnte Fehler enthalten, habs schnell im Text Editor gemacht) für die Klassen Produkt und Kunde

public class ProductV01 {
    private String product;
   
    public String getProduct() {
        return product;
    }
   
    private ProductV01(String product) {
        this.product = product;
    }
   
    public static ProductV01 getRandomProduct() {
        List<String> names = Arrays.asList("Äpfel", "Butter", "Donuts", "Eier", "Fischstäbchen",
            "Geröllsteiner Mineralwasser", "Harribo Gummibärchen", "Kloßteig", "Müllbeutel", "Nutella",
            "Rotmützchen Wein", "Pompers Windeln", "Schokolade", "Torte", "Voll & Fit Energydrink");
           
        return new ProductV01(names.get(new Random().nextInt(names.size())));
    }   
}

public class CustomerV01 {
    private char gender;                                       
    private int age;
    private int body_weight;
   
    /* getter & setter */
   
    public boolean canApplyRule() {
        return age <= 18;
    }
   
    private CustomerV01(char gender, int age, int body_weight) {
        this.gender = gender;
        this.age = age;
        this.body_weight = body_weight;
    }   
   
    public static CustomerV01 getRandomCustomer() {
        Random r = new Random();
       
        char gender = r.nextBoolean() ? 'w' : 'm';
        int age;
        int body_weight;
       
        switch(r.nextInt(3)) {
            case 0: // young
                age = r.nextInt(18) + 1;
                body_weight = r.nxtInt(50) + 45;
                break;
            case 1: // young and fat
                r.nextInt(18) + 1;
                body_weight = r.nxtInt(66) + 95;
                break;
            case 2: // adult
                r.nextInt(67) + 19;
                body_weight = r.nxtInt(50) + 45;
                break;
        }
       
        return new CustomerV01(gender, age, body_weight);
    }
}

 

[mihe7]

dass es Leute gibt, die ein helfen

Es gibt auch Leute, die zwei helfen

Was soll eigentlich das V01 (u. ä.) an den Klassennamen?!?

 

[Xyz1]

Außerdem könnte man durch einen Teil deines Einkaufs deinen kompletten Einkauf "intiutivieren".