爪哇。 方法 isOpen() 未定义为 Lock 类型。 还需要一些帮助来修复一些代码/调试

Question

import java.io.*;
public class GentCPT3
{
  public static void main (String[] args) throws IOException
  {
    BufferedReader objReader = new BufferedReader(new InputStreamReader (System.in)); 

      System.out.println("Enter key");
      int key1 = Integer.parseInt(objReader.readLine()); // set to 111  

      System.out.println("Enter key2");
      int key2 = Integer.parseInt(objReader.readLine()); // set to 222

      Lock lock1 = new Lock (key1);

      Lock lock2 = new Lock (key2);

      System.out.println(lock1.isOpen()); // prints false

      lock1.close();
      lock2.close();
      lock1.open(111); // opens lock1
      lock2.open(222); // opens lock2111
      lock1.open(123); // fails to open
      lock1.open(456); // fails to open
      lock1.open(789); // fails - prints “Alarm”
    }
  }
class Lock //Initializing class
{
  //Initializing variables
  boolean isOpen;
  int key; 
  int numAttempts = 0;

  Lock(int key) 
  {
    isOpen = false; 
    this.key = key;
  } 

  public void close()//for incorrect combo 
  {
    isOpen = false;
  } 
  public void open(int key)//for correct combo
  { 
    if(this.key == key) 
    {
      isOpen = true;
    } 
    else if(this.key != key && !isOpen) 
    {
      numAttempts++;
    } 
    if(numAttempts == 3) 
    {
      System.out.println("ALARM");//prints alarm when the combo is incorrect **enter code here**3 times
    } 
  } 
}

我需要帮助修复标题中提到的错误。 还有一些我需要帮助的其他调试事项。 如果您认为可以修复它，请随时编辑代码。 我需要帮助将锁上每个组合的尝试次数设置为 3，因为现在它设置为 2，我不知道为什么

Answer 1

问题是一旦成功打开锁，您就不会关闭它们。 将您的电话更改为：-

lock1.close();
lock2.close();
lock1.open(111); // opens lock1
lock2.open(222); // opens lock2111
lock1.close(); // NEW ADDITION
lock2.close(); // NEW ADDITION
lock1.open(123); // fails to open
lock1.open(456); // fails to open
lock1.open(789); // fails - prints “Alarm”

为了进一步改进代码，当this.key == key时在open()重置numAttempts = 0 。 并删除this.key != key因为由于先前的if条件，它始终为true 。