爲什麼價格打印兩次？

我的代碼：爲什麼價格打印兩次？

package ecommerceapp; 

import java.util.Scanner; 

public class ECommerceApp { 
public static void main(String[] args) { 
    String productsCatalog = " "; 
    //double price = getPrice(); 
    bannerPrinter(); 
    productsBuilder(); 
    boolean exists = getOrder(productsCatalog); 

    if (exists == true) { 
     double salesTotal = 0; 
     printTotal(salesTotal); 
    } else { 
     System.out.println("The product not found."); 
    } 
    //double price = 0; 
    //double tax = getTax(price); 

    //getTotal(price, tax); 
} 

public static void bannerPrinter() { 
    System.out.println("******************************************"); 
    System.out.println("====== Welcome to my eCommerce app! ======"); 
    System.out.println("******************************************"); 
} 

public static String productsBuilder() { 
    String productsCatalog = "Desk  Table  Pen  "; 
    return productsCatalog; 
} 

public static boolean getOrder(String productsCatalog) { 
    String userProduct; 
    boolean exists = true; 
    Scanner scnr = new Scanner(System.in); 
    System.out.print("Please enter a product name: "); 
    userProduct = scnr.nextLine(); 
    if (productsBuilder().toLowerCase().contains(userProduct.toLowerCase())) { 
     exists = true; 
     System.out.println(exists); 
    } else { 
     exists = false; 
     System.out.println(exists); 
    } 
    return exists; 
} 

public static double getPrice() { 
    double price = 1 + Math.random() * 99; 
    price = Math.round(price * 100.0)/100.0; 
    System.out.println("Price is: " + price); 
    return price; 
} 

public static double getTax(double price) { 
    double tax = (0.1 * getPrice()); 
    tax = Math.round(tax * 100.0)/100.0; 
    System.out.println("Tax is: " + tax); 
    return tax; 
} 

public static double getTotal(double price, double tax) { 
    double salesTotal = getPrice() + getTax(price); 
    return salesTotal; 
} 

public static void printTotal(double salesTotal) { 
    double price = 0; 
    double tax = 0; 
    System.out.printf("Your sale total is: $%.2f", getTotal(price, tax)); 
    System.out.println(); 
} 

}

爲什麼我的輸出打印價格的兩倍？

======歡迎來到我的電子商務應用！ ======

請輸入產品名稱：辦公桌

真正

價格爲：64.43

價格爲：85.07

稅爲：8.51

您的銷售總額爲：$ 72.94

生成成功（總時間：3秒）

當我從兩個用getPrice和getTax除去的System.out.println，這是我的輸出

======歡迎到我電子商務應用程序！ ======

請輸入產品名稱：辦公桌

真正

稅：8.6

你的銷售總額爲：$ 38.60

來源

2017-07-16 Sharon

沒有getter方法打印任何東西。他們應該返回一個值，並沒有副作用。 –

我的提交將不會顯示價格或稅金，但我希望價格僅生成一次，無論如何。 – Sharon

你的代碼有太多的問題恕我直言，恕我直言。底線是你打電話兩次，這是打印兩次。你的許多方法返回你永遠不會使用的值。 –

因爲你打電話getPrice()，在getTax()和getTotal()中均打印價格。

來源

2017-07-16 12:37:52

因爲您正在打印getPrice的結果。而getPrice本身就是打印價格，所以你的程序打印價格兩次。刪除getPrice塊中的「System.out.println」函數。

來源

2017-07-16 12:40:53

如果我按照建議刪除println，則價格和稅款對用戶不可見。價格產生兩次的事實意味着我的方法調用存在問題，並且我想知道是否有人能夠看到該問題並幫助我解決問題。 – Sharon

當你調用getTotal（內部printTotal）你調用用getPrice其本身具有

System.out.println("Price is: " + price);

，併爲這樣的結果，當你再調用getTax你再次調用用getPrice

double tax = (0.1 * getPrice());

然後調用println，這就是爲什麼它打印兩次。

來源

2017-07-16 14:22:16 Jockie

當我用價格替換getPrice時，這裏double tax =（0.1 * getPrice（）），它將價格設置爲0，並且沒有getPrice生成的價格。如何在不使用整個方法的情況下單獨使用價格？ – Sharon

我建議你不要在get方法內部打印輸出，除非它用於調試目的。 – Jockie

你應該做的是首先刪除get方法內部的打印輸出，然後在你應該得到的printTotal裏面，把價格放在一個變量中，比如'double price = getPrice（）'，然後打印出價格是這樣的'System.out.println（「價格是：」+價格）''然後進行稅收並在那裏做同樣的事情，然後當你達到總量時，你就像你一樣通過價格和稅收已經在做了。 – Jockie

爲什麼價格打印兩次？

回答

相關問題