Проблемы с завершением программы

Question

Я делаю программу для класса. В основном программа просматривает некоторую информацию о продажах за день и печатает отчет в конце.

Моя единственная проблема заключается в том, что программа, похоже, не закрывается в конце. Когда я набираю «готово» для имени сотрудника, он печатает отчет, но программа возвращается к началу и продолжает работать, снова запрашивая у сотрудников продажи за год.

У меня проблемы с выяснением, почему программа делает это. Если бы кто-нибудь мог помочь мне указать, что заставляет его зацикливаться, а не выходить, чтобы я мог это исправить, это было бы очень признательно. Спасибо!

import java.io.*;
import java.text.DecimalFormat;

public class Project04Driver {
    public static void main(String args[]) throws IOException {
        Project04 app;
        app = new Project04();
        app.appMain();
    }
}

class Project04 {
private static DecimalFormat df2 = new DecimalFormat ("$#,###.00");
BufferedReader stdin;   
int trans, tCount;
double ytd, amount, empTotal, empBonus, totA, totF, totS, totSales, totYtd, hiTrans, ytdSales;
String date, hiEmp, name, type;
public void appMain() throws IOException {
    rptInit();
    displayHeader();
    getDate();
    while (!name.equalsIgnoreCase("Done")) {
        salesData();
    }
}

void rptInit() {
    name = "f";
    trans = 1;
    amount = totF = totA = totS = totSales = totYtd = hiTrans = ytdSales = 0;
    stdin = new BufferedReader(new InputStreamReader(System.in));
}

void displayHeader() {
    System.out.println("--------------------------------------------");
    System.out.println("Project #04 Solution");
    System.out.println("Written by Jordan Hawkes");
    System.out.println("--------------------------------------------");
    System.out.println("");
}

void getDate() throws IOException {
    System.out.print("Enter Date: ");
    date = stdin.readLine();
}

void salesData() throws IOException {
    System.out.print("Enter Name: ");
    name = stdin.readLine();
    if (name.equalsIgnoreCase("done")) {
        rptOut();
        return;
    }

    System.out.print("Enter Year to Date Sales: ");
    ytd = Double.parseDouble(stdin.readLine());
    ytdSales = ytdSales + ytd;
    totYtd = totYtd + ytdSales;

    trans();

}

void trans() throws IOException {
    while (trans != 0) {
    System.out.print("Enter Transaction Number: ");
    trans = Integer.parseInt(stdin.readLine());

    if (trans == 0) {
        outputSalesData();
        empTotal = 0;
        empBonus = 0;
        ytdSales = 0;
        trans = 1;
        salesData();
    } else {
        System.out.print("Enter Transaction Type: ");
        type = stdin.readLine();
        System.out.print("Enter Transaction Amount: ");
        amount = Double.parseDouble(stdin.readLine());

        totSales = totSales + amount;
        totYtd = totYtd + amount;
        ytdSales = ytdSales + amount;
        empTotal = empTotal + amount;
        empBonus = empBonus + (amount * 0.05);

        outputUpdate();

        calcSalesData();
        tCount++;
    }
}
}

void calcSalesData() {
    if (type.equalsIgnoreCase("F")) {
        totF = totF + amount;
    }
    if (type.equalsIgnoreCase("A")) {
        totA = totA + amount;
    }
    if (type.equalsIgnoreCase("S")) {
        totS = totS + amount;
    }

    hiTrans();
}

void hiTrans() {
    if (amount > hiTrans) {
        hiTrans = amount;
        hiEmp = name;
    }
}

void outputUpdate() {
    System.out.println("");
    System.out.println("--------------------------------------------");
    System.out.println(name + "'s Total Sales for Day: " + df2.format(empTotal));
    System.out.println("--------------------------------------------");
    System.out.println("");
}

void outputSalesData() {
    System.out.println("");
    System.out.println("--------------------------------------------");
    System.out.println("Employee Name: \t\t" + name);
    System.out.println("YTD Sales: \t\t" + df2.format(ytdSales));
    System.out.println("Employee Bonus: \t" + df2.format(empBonus));
    System.out.println("--------------------------------------------");
    System.out.println("");
}

void rptOut() {
     System.out.println("Total Food & Soft Drink Sales: \t" + df2.format(totF));
    System.out.println("Total Alcohol Sales: \t\t" + df2.format(totA));
    System.out.println("Total Sundries Sales: \t\t" + df2.format(totS));
    System.out.println("--------------------------------------------");
    System.out.println("Total Sales for Day: \t\t" + df2.format(totSales));
    System.out.println("Total YTD: \t\t\t" + df2.format(totYtd));
    System.out.println("--------------------------------------------");
    System.out.println("Highest Trans Amount: \t\t" + df2.format(hiTrans));
    System.out.println("Employee w/ Highest Trans: \t" + hiEmp);
    System.out.println("--------------------------------------------");
    return;
}

}

Answer 1

Для естественного выхода из программы не следует вызывать методы в цикле, а разрешать методы для выхода и управления потоком из основного цикла.

Вот переработанный подход:

Новый основной цикл вызывает в паре salesData() и trans() (при условии, что имя не "выполнено")

while (!name.equalsIgnoreCase("Done")) {
    salesData();
    if (!name.equalsIgnoreCase("Done"))
        trans();
}

Теперь метод salesData() не должен вызывать trans():

void salesData() throws IOException {
    System.out.print("Enter Name: ");
    name = stdin.readLine();
    if (name.equalsIgnoreCase("done")) {
        rptOut();
        return;
    }
    // ... processing ...
 }

А также trans() не должен звонить salesData():

void trans() throws IOException {
   while (trans != 0) {
   System.out.print("Enter Transaction Number: ");
   trans = Integer.parseInt(stdin.readLine());

   if (trans == 0) {
     outputSalesData();
     empTotal = 0;
     empBonus = 0;
     ytdSales = 0;
   } else {
     System.out.print("Enter Transaction Type: ");
     type = stdin.readLine();
     System.out.print("Enter Transaction Amount: ");
     amount = Double.parseDouble(stdin.readLine());
     // ... processing ...
     outputUpdate();
     calcSalesData();
     tCount++;
   }
 }
}

Описание: теперь, когда методы salesData() и trans() не вызывают друг друга (они естественным образом возвращаются после завершения метода), и поскольку элемент управления теперь находится в главном цикле, ваше приложение завершит работу, как и ожидалось.

Answer 2

Я не могу комментировать прямо в вашем вопросе, поэтому я поставлю его здесь. Я думаю, что проблема заключается в том, что вы вызываете salesData ():

- в appMain () у вас нет выбора для этого «Done» в цикле while, поэтому он никогда не заканчивается.

• Я предлагаю сделать так, чтобы salesData возвращал логическое значение, как только вы закончите, таким образом вы можете сказать! Логическое и вернуть true, и это завершит ваш цикл while в appMain ().

Надеюсь, что это решит проблему!

Answer 3

trans () и appMain () вызывают salesData (). Если salesData () вызывается через trans (), то возврат в salesData () возвращается к trans (), который сам имеет цикл while.

Answer 4

В моей автономной Java-программе это делает следующий вызов:

System.exit(0);