使用布尔值创建选项>保留<？

Question

好吧，我让我的程序"完成"到我无法解决这个简单问题的地步。

我得到了这个：

package hotel;
import java.util.Scanner;
/**
*
* @author Defalt
*/
public class Hotel {
/**
 * @param args the command line arguments
 */
public static void main(String[] args) {
    Prices prices = new Prices();
    Scanner user_in = new Scanner(System.in);
    String method;
    boolean taken = false;
    boolean taken2 = false;
    boolean taken3 = false;
    boolean again = true;
    //***********Introduction to the hotel*************
    System.out.println("Welcome to Hotel Vermond's Heaven!");
    System.out.println("We are happy to see that you would like to stay with us.");
    //***********Creating of a Loop*************

    do {
    System.out.println("Please, type which room you would like to book: ");
    System.out.println("Single Bed, Double Bed, President Suit");
    method = user_in.nextLine();
    //***********Choices of Rooms and Question to book another one*************
    if ("Single Bed".equals(method)) {
            System.out.println(prices.describe1());
    if ("y".equals(user_in.nextLine())){
         if(taken=true){
                System.out.println("We are sorry, this room is already booked. Please choose another one");
            }else again=true;
    } else again=true;
        } else if ("Double Bed".equals(method)){
            System.out.println(prices.describe2());
    if ("y".equals(user_in.nextLine())){
            taken2 = true;
            again=true;
        } else again=true;
    } else if ("President Suit".equals(method)){
            System.out.println(prices.describe3());
    if ("y".equals(user_in.nextLine())){
            taken3 = true;
            again=true;
    } else again=true;
    } else {
            System.out.println("Please choose one of the rooms above.");
    }
            System.out.println("Would you like to book another Room(y/n)n");
    //***********Outbreak if User declines another booking*************
    //***********Otherwise redo the whole process************* 
    if ("y".equals(user_in.nextLine())){
        again=true;
    } else{
        break;
    }
 }while(again);
    System.out.println("Thank you and goodbye!");
   }
}

我的另一堂课是这样的：

package hotel;
import java.util.Scanner;
/**
*
* @author Defalt
*/
public class Prices {
Scanner user_in = new Scanner(System.in);
int price1 = 300;
int price2 = 800;
int price3 = 2500;
//***********Information of the Room including Validation Question*************
public String describe1() {  
       return
        "You Choose the Single Bed.ntThis room Contains 1 Bed, 1 Fridge a Bathroom but no View on the Ocean.ntThis room will cost CHF " + price1 + ".-.ntWould you like to book this room?(y/n)";
   }
public String describe2() {  
   return
    "You Choose the Double Bed.ntThis room Contains 1 Queen-size Bed, 1 Fridge a bathroom, an Icemaker but no View on the Ocean.ntThis room will cost CHF " + price2 + ".-.ntWould you like to book this room?(y/n)";
   } 
public String describe3() {  
   return
    "You Choose the President Suit.ntThis room Contains 1 King-size Bed, 1 Fridge, XXL Bathroom, Private Entertainment-System, 65inch Flatscreen and a Balcony with View on the Ocean.ntThis room will cost CHF " + price3 + ".-.ntWould you like to book this room?(y/n)";
   } 
}

正如你所看到的，我试图在Single Bed被占用后将其作为保留的房间。然而，即使在第一次预订时，System.out.println("We are sorry, this room is already booked. Please choose another one");也会显示出来。

为什么即使我的taken布尔值的初始值为假？我还可以采取哪些其他选择来实现我的想法？

PS！：我想我第一次让它工作，但无论我多久关闭一次程序并重新启动它，它都会保持获取的值为 true。（只是一个旁注）

Answer 1

你需要改变

if(taken=true){
            System.out.println("We are sorry, this room is already booked. Please choose another one");

自

if(taken==true){
            System.out.println("We are sorry, this room is already booked. Please choose another one");

请注意 if 条件中的双等号。这将检查taken的值，而不仅仅是总是将其设置为 true ，这是当前正在发生的事情。

或者，由于taken已经是一个布尔值，您可以将其简化为：

if(taken){
            System.out.println("We are sorry, this room is already booked. Please choose another one");

如果您修复了格式和缩进，可能会帮助您解决这些问题。我倾向于避免在任何时候省略大括号。这通常是 java 标准格式：

if(condition) {
    if(condition) {
        // do the thing
    }
    else {
        // do the other thing
    }
}
else if(condition) {
    // do another thing
}
else {
    // do the third thing
}

请注意，同一嵌套级别的 ifs/else ifs/else 也保持在同一缩进级别。这对保持它们直截了当有很大帮助。

编辑：

我想指出的另一件事是，您有again布尔值，但您没有使用它。如果用户完成，则不要break出圈，而是使用您应该拥有的again布尔值并将其设置为 false。这将导致循环条件失败并很好地退出循环。使用休息是丑陋的。