初学Java：从方法中返回值

Question

在我开始之前，我想说明一下，在编写代码时，我非常习惯于初学者，所以如果我所问的内容看起来非常基本，我很抱歉。

这就是说我正在努力与我的代码中的返回方法。我可以编写这个程序，而不用像我那样将它分解成方法，但我被告知，编写代码时很好的做法是将它拆分成方法，以便调试更容易。

下面的代码似乎有一些主要缺陷。

public static boolean hybridNot() 
{ 
    String typeCar = input("Hybrid or Electric?"); 
    boolean electric = false; 

if (typeCar.equalsIgnoreCase("Electric")) 
{ 
    electric = true; 
} 

else if (typeCar.equalsIgnoreCase("Hybrid")) 
{ 
    electric = false; 
} 

else 
{ 
    print("Sorry I didn't understand that"); 
} 

return; 
} 

public static boolean solarNot() 
{ 
    String panelsMaybe = input("Solar panel?"); 
    boolean solarPanel = false; 

    if (panelsMaybe.equalsIgnoreCase("Yes")) 
    { 
     solarPanel = true; 
    } 

    else if (panelsMaybe.equalsIgnoreCase("No")) 
    { 
     solarPanel = false; 
    } 

    else 
    { 
     print("Sorry I didn't understand that"); 
    } 

    return; 
    } 

public static int discountNot() 
{ 
    final int basicPrice = 20000; 
    final int electricCost = 2000; 
    final int solarCost = 5000; 
    boolean electric = hybridNot(); 
    boolean solarPanel = solarNot(); 
    int totalPrice; 

    if ((solarPanel = true) || (electric = true)) 
    { 
     totalPrice = basicPrice + solarCost + electricCost - 500; 
    } 

    else if ((solarPanel = true) || (electric = false)) 
    { 
     totalPrice = basicPrice + solarCost; 
    } 

    else if ((solarPanel = false) || (electric = true)) 
    { 
     totalPrice = basicPrice + electricCost; 
    } 

    else 
    { 
     totalPrice = basicPrice; 
    } 

    return; 

    } 

public static void totalCost() 
{ 
    final int basicPrice = 20000; 
    final int electricCost = 2000; 
    final int solarCost = 5000; 
    final int discountCost = 500; 
    boolean hybrid = hybridNot(); 
    boolean solarPanel = solarNot(); 
    int finalPrice = 0; 

    finalPrice = discountNot(); 

    if (finalPrice >= 26500) 
    { 
     print("Basic Price: " + basicPrice + "\n" + "Electric model: " + electricCost + "\n" + "Solar Panel: " + solarCost + "\n" + "Discount: " + discountCost); 
    } 

    else if (finalPrice >= 25000) 
    { 
     print("Basic Price: " + basicPrice + "\n" + "Solar Panel: " + solarCost); 
    } 

    else if (finalPrice >= 22000) 
    { 
     print("Basic Price: " + basicPrice + "\n" + "Electric model: " + electricCost); 
    } 

    else 
    { 
     print("Basic Price: " + basicPrice); 
    } 

    print("Total: " + finalPrice); 

    } 

由于某些原因，hybridNot和solarNot似乎在转向下一个方法之前重复自己。对我来说，似乎我可能会在方法结束时返回一个问题，但我实在无法弄清楚什么是错误的。方法totalCost似乎忽略了方法discountNot中的if语句，并且布尔值没有正确传递给totalCost，我只在finalPrice >= 26500时获得该值。

再一次，我是一般的Java新手，我也是新来的stackoverflow（所以嗨!!）所以请告诉我，如果我做错了什么，我会看看下一次正确的！谢谢！！

Answer 1

你没有在你的函数中返回任何值。用途：

public static boolean hybridNot(){ 
    //... 
    return electric; 
} 

返回boolean值electric。在其他函数中使用相似的语法来返回相关的变量。

此外，方法不断地自我重复的原因是因为你给他们打电话两次：

public static int discountNot() 
{ 
    //... 
    boolean electric = hybridNot(); //here 
    boolean solarPanel = solarNot(); 
    //... 
} 

public static void totalCost() 
{ 
    //... 
    boolean hybrid = hybridNot(); //and here 
    boolean solarPanel = solarNot(); 
    //... 
} 

Answer 2

你应该根据方法的返回类型返回值。

public static boolean hybridNot() 
{ 
    String typeCar = input("Hybrid or Electric?");  
    return (typeCar.equalsIgnoreCase("Electric")) 
} 

我不是说要在检查状态的函数中获取输入 - 它本身就是邪恶的。

现在的位置：

if ((solarPanel = true) || (electric = true)) 

你做作业，不comparement。比较你可以这样做：

if (solarPanel || electric) 

因为变量已经是布尔值了。

这里：

boolean hybrid = hybridNot(); 
    boolean solarPanel = solarNot(); 
    int finalPrice = 0; 

    finalPrice = discountNot(); 

discountNot将hybridNot和solarNot再次呼吁：

public static int discountNot() 
{ 
    ... 
    boolean electric = hybridNot(); 
    boolean solarPanel = solarNot();