Класс класса Робот класса

Question

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

Вот мой Keyboard.java методом pressEscape():

public class Keyboard { 

    private static Robot robot; 
    private static int time = 1000; 

    public Keyboard(){ 
     try { 
      robot = new Robot(); 
     } catch (AWTException e) { 
      e.printStackTrace(); 
     } 
    } 

    public void pressEscape() throws TestException { 
     if (!getSession().CanRun()) 
      throw new TestException(InvalidStateMessage); 
     robot.delay(time); 
     robot.keyPress(KeyEvent.VK_ESCAPE); 
     robot.delay(time); 
     robot.keyRelease(KeyEvent.VK_ESCAPE); 
     robot.delay(time); 
    } 

} 

А вот ошибка из журнала:

[2015/08/13 15:43:29] [ID: 7F4315A] [Селен] : java.lang.NullPointerException
[2015/08/13 15:43:29] [ID: 7F4315A] [Селен]: в Kodak.AutoTest.Framework.Keyboard.pressEscape (Keyboard.java:128)

Изменение немного метод:

public void pressEscape() throws TestException { 
     if (!getSession().CanRun()) 
      throw new TestException(InvalidStateMessage); 
     try { 
      new Robot(); 
      robot.setAutoDelay(time); 
      robot.keyPress(KeyEvent.VK_ESCAPE);  
      robot.setAutoDelay(time); 
      robot.keyRelease(KeyEvent.VK_ESCAPE); 
     } catch (AWTException e) { 
      e.printStackTrace(); 
     } 

    } 

, но по-прежнему имеет ту же самую ошибку на линии:

robot.keyPress(KeyEvent.VK_ESCAPE); 

Answer 1

Это выглядит так, как будто единственное место, вы можете реально получить исключения нулевого указателя в том, что метод в эта линия:

if (!getSession().CanRun()) 

, который бросит NPE, если getSession() возвращается null. На самом деле другой вариант - если вы не смогли инициализировать robot, но тогда вы увидите AWTException в журналах.

Но вы должны подтвердить это, посмотрев номер строки, которую дали вам журналы. Если это так, я предлагаю изменение

if (!getSession().CanRun()) 
      throw new TestException(InvalidStateMessage); 

в

Session session = getSession(); 
if ((session==null) || !session.CanRun()) 
      throw new TestException(InvalidStateMessage); 

Это будет затвердевать эту часть, и означает, что вы получите TestException, если сеанс является недействительным. (Я угадываю тип, который возвращает getSession().)

Кроме того, хотя вы инициализируете robot в своем конструкторе, вы должны знать, что это статическое поле. Это означает, что если у вас много экземпляров Keyboard, каждый раз, когда вы создаете экземпляр, он перезаписывает предыдущий экземпляр robot новым, потому что все они имеют один и тот же.

Вы должны либо изменить его на поле экземпляра (удалить модификатор static), либо инициализировать его только один раз (проверьте, имеет ли он значение null, и инициализирует его, только если это так).

Answer 2

Для этого ваша переменная robot должна иметь значение null в строке 128. Это может произойти только в new Robot(). Проверьте свои журналы, чтобы убедиться, что это так. А также см. Комментарии к своим вопросам от MadProgrammer и действовать на них.

Answer 3

Он сейчас работает, вот полный код:

public class Keyboard { 

    private static Robot robot; 

    public void pressEscape() throws TestException { 
    if (!getSession().CanRun()) 
     throw new TestException(InvalidStateMessage); 
    try { 
     robot = new Robot(); 
     robot.setAutoDelay(time); 
     robot.keyPress(KeyEvent.VK_ESCAPE);  
     robot.setAutoDelay(time); 
     robot.keyRelease(KeyEvent.VK_ESCAPE); 
    } catch (AWTException e) { 
     e.printStackTrace(); 
    } 

} 

}