2014-10-08 4 views
0

Я работаю над своим первым Java-приложением, игрой. Что я имею теперь компилируется нормально, но когда я запускаю его в командной строке Командной строки: требуемое количество раз печатаемОшибки в командной строке при загрузке и отображении изображений

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException     
at Spike.loadpic(Spike.java:50)             
at Spike.paintspike(Spike.java:55)            
at Move.paint(Move.java:106)              
at javax.swing.JComponent.paintToOffscreen(JComponent.java:5217)     
at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(RepaintManager.java:1532)                   
at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1455)                      
at javax.swing.RepaintManager.paint(RepaintManager.java:1252)     
at javax.swing.JComponent._paintImmediately(JComponent.java:5165)    
at javax.swing.JComponent.paintImmediately(JComponent.java:4976)     
at javax.swing.RepaintManager$3.run(RepaintManager.java:811)      
at javax.swing.RepaintManager$3.run(RepaintManager.java:794)      
at java.security.AccessController.doPrivileged(Native Method)     
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)                   
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:794)                       
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:769)                       
at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:718)                      
at javax.swing.RepaintManager.access$1100(RepaintManager.java:62)    
at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1680)                     
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)    
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:744)     
at java.awt.EventQueue.access$400(EventQueue.java:97)       
at java.awt.EventQueue$3.run(EventQueue.java:697)        
at java.awt.EventQueue$3.run(EventQueue.java:691)        
at java.security.AccessController.doPrivileged(Native Method)     
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)                   
at java.awt.EventQueue.dispatchEvent(EventQueue.java:714)      
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)                    
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)                    
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)                    
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)                       
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)   
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)   

И это не запустить приложение (Когда я загружаю один меньше изображений он в конце концов останавливается, и это делает Работа).

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

 @SuppressWarnings("serial") 
     public class Move extends JPanel implements Runnable { 
     private synchronized void start(){ 
     if(running){ 
      return; 
     } 
     running = true; 
     thread = new Thread(this); 
     thread.start(); 
    } 

    public void run(){ 
     private Thread thread; 
     boolean running = false; 
     long tijd = System.nanoTime(); 
     double ticks = 60.0; 
     double ns = 1000000000/ticks; 
     double delta=0; 
     while(running){ 
      long tijd2 = System.nanoTime(); 
      delta+=(tijd2-tijd)/ns; 
      tijd=tijd2; 
      if(delta>1){ 
      player.move(); 
      for(int i=0;i<zappelin.length;i++){ 
      zappelin[i].movezappelin(); 
      } 
      repaint(); 
      delta--; 
       } 
      } 
     } 
     @Override 
    public void paint(Graphics g) { 
     super.paint(g); 
     Graphics2D g2d = (Graphics2D) g; 

     if(player.x<400){g2d.translate(0,0);} 
     else if(player.x+400>levellength){g2d.translate(-levellength+800,0);} 
     else{g2d.translate(-(player.x-400),0);} 
     player.paintplayer(g2d); 
     for(int i=0;i<ground.length;i++){ 
      ground[i].paintground(g2d); 
     } 
     for(int j=0;j<spike.length;j++){ 
      spike[j].paintspike(g2d); 
     } 
     for(int k=0;k<zappelin.length;k++){ 
      zappelin[k].paintzappelin(g2d); 
     } 
    } 
     public static void main(String[] args){ 
     /*Set frame, create Move game, etc */ 
     game.start(); 
    } 
     /*Some other stuff*/ 
     } 

public class Player{ 
loaded=false;  
private void loadpic(){ 
     player[0] = new ImageIcon("Images/Playa.png").getImage(); 
     player[1] = new ImageIcon("Images/Playal.png").getImage(); 
     player[2] = new ImageIcon("Images/Playar.png").getImage();  
     loaded = true; 
     game.repaint(); 
    } 

public void paintplayer(Graphics2D g){ 
    if(!loaded){ 
     loadpic(); 
    } 
    if(loaded){ 
     if(xa==-1){ 
      g.drawImage(player[1], x, y, null); 
     } 
     else if(xa==1){ 
      g.drawImage(player[2], x, y, null); 
     } 
     else{ 
     g.drawImage(player[0], x, y, null); 
     } 
    } 
} 
/*Other stuff*/ 
} 

public class Zappelin{ 
private void loadpic(){ 
     zappelin[0] = new ImageIcon("Images/zappelinl.png").getImage(); 
     zappelin[1] = new ImageIcon("Images/zappelinr.png").getImage(); 
     zappelin[2] = new ImageIcon("Images/explosion.png").getImage(); 
     loaded = true; 
     game.repaint(); 
    } 
    public void paintzappelin(Graphics2D g) { 
     if(dead){return;} 
     if(!loaded){ 
      loadpic(); 
     } 
     else{ 
      if(xaz==-1||xaz==0){g.drawImage(zappelin[0], xz, yz, null);} 
      else{g.drawImage(zappelin[1],xz,yz,null); 
       //g.drawImage(zappelin[2], 10,10, 64, 64, 0,0,64,64,null); 
      } 
     } 
     if(bullet){ 
      g.setColor(Color.RED); 
      g.fillOval(xbullet,ybullet,4,4); 
     } 
    } 
/*Other stuff*/ 
} 

public class Spike{ 
private void loadpic(){ 
     switch (dimension){ 
     case "normaal":  
      spike = new ImageIcon("Images/Spike.png").getImage(); 
      test=1337; 
      break; 

     case "links": 
      spike = new ImageIcon("Images/spikel.png").getImage(); 
      break; 

     case "rechts": 
      spike = new ImageIcon("Images/spiker.png").getImage(); 
      break; 

     case "boven": 
      spike = new ImageIcon("Images/spikeb.png").getImage(); 
      break; 

     default: 
      spike = new ImageIcon("Images/Spike.png").getImage(); 
      break; 
      } 

     loaded = true; 
     game.repaint(); 
    } 
public void paintspike(Graphics2D g) { 
     if(!loaded){ 
      loadpic(); 
     } 
     else{ 
      if(dimension=="normaal"||dimension=="boven"){ 
       for(int i=0; i<amount;i++){ 
        g.drawImage(spike, xg+15*i, yg, null);} 
       } 
      if(dimension=="links"||dimension=="rechts"){ 
       for(int i=0; i<amount;i++){ 
        g.drawImage(spike, xg, yg+20*i, null); 
       } 
      } 
     } 
    } 
} 

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

Я не могу найти решение этого в любом месте, поэтому было бы здорово, если кто-то знает, как это исправить.

EDIT: Я пробовал некоторые вещи, и оказывается, что командная строка в конечном итоге перестает печатать эту ошибку и запускает приложение, когда я использую только методы g.drawImage (image, x, y, null), но он повторяет его бесконечно, когда я использую методы g.drawImage (image, dx1, dy1, dx2, dy2, sx1, sy1, sx2, sy2, null).

+0

Проверить строку 50 в файле Spike.java. У вас есть исключение Null-указателя. – Anon

+0

** (1) ** - ['dimension ==" normaal "') (http://stackoverflow.com/questions/513832/how-do-compare-strings-in-java). ** (2) ** Является ли папка «Images» в «рабочем каталоге»? Если нет, то ваши изображения будут пустыми –

ответ

0

У вас есть NPE (исключение NullPointerException) в вашей paint() - методе. Это означает, что вы используете переменную, которая не инициализирована (она просто равна нулю). Поскольку ваш код не является полным, мы не знаем, где строка 50, поэтому я не могу сказать вам, где именно проблема.

Другое примечание: вы вызываете метод repaint вашего JPanel изнутри Thread, что является плохой практикой. Swing не является потокобезопасным. Вместо этого вам нужно использовать SwingUtilities.invokeAndWait(runnable) или SwingUtilities.invokeLater(runnable).

+1

_ «Это означает, что вы используете переменную, которая не инициализирована» _ Not _always_ true. Изображение может быть нулевым. Попытка «g.drawImage (null, ...)» приведет к NPE –

+0

Да - вы совершенно правы. Но AFAIK Exception должен указывать на этот вызов в другом стеке. –

Смежные вопросы