java - 方法在主类中返回错误值

Question

每当我运行 InschrijvingApplicatie 时，我都会得到一个错误的值 System.out.printf("Hoeveel broodjes wil je bestellen? (max %d) ", maxBroodjes); 因为 int 应该当我在此行中输入“p”时为“10”

System.out.printf("Tot welke categorie behoort u?\nTyp w voor een werknemer, p voor een werknemer met partner, g voor een gast: ");

我假设 int maxBroodjes = (inschrijving.geefAantalPersonen() * 5); 行有问题，但似乎无法弄清楚是什么。

How the output should look like

练习是:对于邀请员工(代码中的'w')的公司，员工有合作伙伴('p')和 guests ('g') 并让他们填写他们的名字，他们是哪种访客(员工+合作伙伴，客人或员工)，然后询问这个人想要多少个三明治(客人和员工最多需要 5 个三明治，员工+合作伙伴可以要求 10)，最大值以整数形式显示 (max %d)。所有这一切都在一个循环中，直到用户在询问“Zijn er nog inschrijvingen”时写入“否”(但使用了第一个字符=>导致'n')，如果答案是肯定的，它会重复。

Inschrijving.java

package domein;
public class Inschrijving {

private String naam;
private char categorie;
private int aantalBroodjes;

public Inschrijving(String naam, char categorie) {
    setNaam(naam);
    setCategorie(categorie);
}

public String getNaam() {
    return naam;
}

private void setNaam(String naam) {
    this.naam = naam;
}

public char getCategorie() {
    return categorie;
}

private void setCategorie(char categorie) {
    if (categorie == 'w' || categorie == 'p' || categorie == 'g') {
        this.categorie = categorie;
    } else {
        this.categorie = 'g';
    }

}

public int getAantalBroodjes() {
    return aantalBroodjes;
}

public void setAantalBroodjes(int aantalBroodjes) {

    if (aantalBroodjes <= (geefAantalPersonen() * 5)) {
        this.aantalBroodjes += aantalBroodjes;
    } else {
        this.aantalBroodjes += (geefAantalPersonen() * 2);
    }
}

public int geefAantalPersonen() {
    switch (categorie) {
        case 'w':
        case 'g':
            return 1;
        default:
            return 2;

    }
  }
}

InschrijvingApplicatie

package ui;

import domein.Inschrijving;
import java.util.Scanner;

public class InschrijvingApplicatie {
 public static void main(String[] args) {

    Scanner invoer = new Scanner(System.in);
    String antwoord;
    char eersteLetter;

    System.out.println("Zijn er nog inschrijvingen? ");
    antwoord = invoer.nextLine();
    eersteLetter = antwoord.toLowerCase().charAt(0);

    String naam = null;
    String categorie;
    char categorieEersteLetter = 0;

    int werknemer = 0;
    int werknemerMetPartner = 0;
    int gast = 0;

    int aantalBroodjes;
    int tijdelijk;

    Inschrijving inschrijving = new Inschrijving(naam, categorieEersteLetter);

    if (eersteLetter != 'n') {
        do {
            System.out.println("Wie mag ik inschrijven? ");
            naam = invoer.next();

            do {
                System.out.printf("Tot welke categorie behoort u?\nTyp w voor een werknemer, p voor een werknemer met partner, g voor een gast: ");

                categorie = invoer.next();
                categorieEersteLetter = categorie.toLowerCase().charAt(0);


                switch (categorieEersteLetter) {
                    case 'w':
                        werknemer++;
                        break;
                    case 'p':
                        werknemerMetPartner++;
                        break;
                    case 'g':
                        gast++;
                        break;
                }

            } while (categorieEersteLetter != 'w' && categorieEersteLetter != 'p' && categorieEersteLetter != 'g');
            int maxBroodjes = (inschrijving.geefAantalPersonen() * 5);
            do {
                System.out.printf("Hoeveel broodjes wil je bestellen? (max %d) ", maxBroodjes);
                tijdelijk = invoer.nextInt();
            } while (tijdelijk > maxBroodjes);
            aantalBroodjes = tijdelijk;
            inschrijving.setAantalBroodjes(aantalBroodjes);

            System.out.println("Zijn er nog inschrijvingen? ");
            antwoord = invoer.next();
            eersteLetter = antwoord.toLowerCase().charAt(0);
        } while (eersteLetter != 'n');

    }
    System.out.printf("Er komen %d werknemer(s) zonder partner, %d werknemer(s) met partner en %d gast(en) naar de receptie. Er dienen %d broodjes besteld te worden.", werknemer, werknemerMetPartner, gast, inschrijving.getAantalBroodjes());

}
}

Answer 1

您的方法存在一些问题，它可能有效，但您不应该那样做。

首先，您只在一个Inschrijving 对象中存储所有受邀人请求的三明治总数，这没有意义! (我需要知道请求的三明治总数还是只需要知道我请求的三明治？)。因此，将 Inschrijving 类中的 setAantalBroodjes 更改为:

public void setAantalBroodjes(int aantalBroodjes) {
  this.aantalBroodjes = aantalBroodjes;
}

其次，要求是获取人员列表并用他们做一些事情，因此您应该考虑使用支持存储人员列表的数据结构，例如 Array 或 ArrayList ，那么一旦用户停止输入(eersteLetter == 'n')，您就可以对数据做任何您想做的事情。

List<Inschrijving> inschrijven = new ArrayList<>();
try (Scanner invoer = new Scanner(System.in)) { // http://tutorials.jenkov.com/java-exception-handling/try-with-resources.html
  System.out.println("Zijn er nog inschrijvingen? ");
  String antwoord = invoer.nextLine();
  char eersteLetter = antwoord.toLowerCase().charAt(0);
  while (eersteLetter != 'n') {

    Inschrijving inschrijving = null;
    System.out.println("Wie mag ik inschrijven? ");
    String naam = invoer.nextLine();
    char categorieEersteLetter = 0;
    do {
      System.out.printf(
          "Tot welke categorie behoort u?\nTyp w voor een werknemer, p voor een werknemer met partner, g voor een gast: ");
      String categorie = invoer.nextLine();
      categorieEersteLetter = categorie.toLowerCase().charAt(0);
    } while (categorieEersteLetter != 'w' && categorieEersteLetter != 'p' && categorieEersteLetter != 'g');
    inschrijving = new Inschrijving(naam, categorieEersteLetter);

    int maxBroodjes = (inschrijving.geefAantalPersonen() * 5);
    int tijdelijk;
    do {
      System.out.printf("Hoeveel broodjes wil je bestellen? (max %d) ", maxBroodjes);
      tijdelijk = invoer.nextInt();
      invoer.nextLine(); // https://stackoverflow.com/questions/13102045/scanner-is-skipping-nextline-after-using-next-or-nextfoo 
    } while (tijdelijk > maxBroodjes);
    inschrijving.setAantalBroodjes(tijdelijk);
    inschrijven.add(inschrijving);

    System.out.println("Zijn er nog inschrijvingen? ");
    antwoord = invoer.nextLine();
    eersteLetter = antwoord.toLowerCase().charAt(0);
  }
}

当用户完成输入时:

// Do stuffs with your list of people here
int werknemer = 0;
int werknemerMetPartner = 0;
int gast = 0;
int aantalBroodjes = 0;
for (Inschrijving inschrijving : inschrijven) {
  char categorie = inschrijving.getCategorie();
  switch (categorie) {
    case 'w':
      werknemer++;
      break;
    case 'p':
      werknemerMetPartner++;
      break;
    case 'g':
      gast++;
      break;
  }
  aantalBroodjes += inschrijving.getAantalBroodjes();
}
System.out.printf(
        "Er komen %d werknemer(s) zonder partner, %d werknemer(s) met partner en %d gast(en) naar de receptie. Er dienen %d broodjes besteld te worden.",
        werknemer, werknemerMetPartner, gast, aantalBroodjes);

因为你是 java 新手，我在这里使用 foreach 循环来做例子，在学习了 OOP 和熟悉 java 之后，我建议你重新研究 Java 8 Stream api 和 lambda 表达式来处理集合类型。