problème attributs

problème attributs - Java - Programmation

Marsh Posté le 08-06-2012 à 23:20:51    

Bonjour,
 
J'ai un souci sur mon code qui affiche:
 
Exception in thread "main" java.lang.NullPointerException
 at java.io.Writer.write(Unknown Source)
 at java.io.PrintStream.write(Unknown Source)
 at java.io.PrintStream.print(Unknown Source)
 at java.io.PrintStream.println(Unknown Source)
 at Combat.ChangerCap(Combat.java:141)
 at TestCombat.main(TestCombat.java:8)
 
je sais que c'est une histoire de valeur null dans la méthode ChoisirCap() que je n'arrive pas à résoudre.
 
J'aimerai savoir comment faire pour récupérer mes attributs de choisirCap avec leur propre valeur (j'ai aussi une classe enum composer de caps, d'où le tableau)
 
 
 
 

Code :
  1. import java.util.Scanner;
  2. import java.util.Random;
  6. public class Combat {
  9.       public SousMarin Realiser;
  10. private Object CapInitial;
  11. private Object nbrAl;
  14.      
  15.      
  16.       public void TirerTorpille() {
  17.       
  18.       
  19.  
  20.  
  21.  
  23.   }
  25.   public void Sonder() {
  30.    }
  33.   public void CommencerCombat() {
  34.   System.out.println("Le combat commence!\n\n Vous recevez 20 points de vie" );
  35.  
  36.  
  37.  
  38.   /*Le cap est tiré aléatoirement*/
  39.   }
  41.   public void ChoisirActionCombattre() {
  42.   System.out.println("Choisissez l'action à réaliser" );
  43.  
  44.   System.out.println("1- Tirer Torpille\n 2-Sonder\n 3-Changer de Cap\n 4-Capituler" );
  45.  
  46.   Scanner sc = new Scanner(System.in);
  47.  
  49.  
  50.   int choix = sc.nextInt();
  51.  
  52.   if(choix==1){
  53.    System.out.println("Tirer la torpille" );
  54.    TirerTorpille();
  55.   }
  56.  
  57.   else if(choix==2){
  58.    System.out.println("\nSonder" );
  59.    Sonder();
  60.   }
  61.  
  62.   else if(choix==3){
  63.    System.out.println("\nChanger de Cap" );
  64.    ChangerCap();
  65.   }
  66.  
  67.   else{
  68.    System.out.println("\nCapituler" );
  69.   }
  70.  
  71.  
  72.  
  73.   }
  75.   public void MenuCombattre() throws Exception {
  76.   /*Menu du combat*/
  77.   System.out.println("MENU COMBATTRE" );
  78.  
  79.   System.out.println("Voulez-vous démarrer le combat?" );
  80.  
  81.   System.out.println("Si oui, tapez 1 si non tapez 2\n" );
  82.  
  83.   Scanner s= new Scanner(System.in);
  84.   int choix;
  85.   choix=s.nextInt();
  86.  
  87.   if(choix==1){
  88.    CommencerCombat();
  89.   }
  90.  
  91.   else{
  92.    Partie p= new Partie();
  93.    p.AfficherMenuPrincipal();
  94.   }
  95.  
  96.  
  97.   }
  99.   public void Capituler() {
  100.   }
  102. public void ChoisirCap(){
  104.   
  105.   
  107.    Random r = new Random();
  109.    int nbrAl = r.nextInt(8);
  111.    Cap[] valeur = Cap.values();
  113.    Cap CapInitial =  valeur[nbrAl];
  116.    System.out.println("Votre cap de départ est: " );
  118.    System.out.println( CapInitial);
  121.   }
  122.   public void ChangerCap() {
  123.  
  124.  
  125.  /**
  126.   * Si on change de cap on bouge de 45°.
  127.   *  
  128.   */  Combat j=new Combat();
  129.     j.ChoisirCap();
  131.                                Cap CapInitial= int valeur[i+1];
  133.  System.out.println(CapInitial);
  135. //System.out.println(i);
  138.   }
  140.   public void AfficherMenuCombattre() {
  141.  
  142.  
  143.   }
  146. }

Reply

Marsh Posté le 08-06-2012 à 23:20:51   

Reply

Marsh Posté le 10-06-2012 à 13:44:23    

Bonjour,
 
Première chose, merci pour la stack trace, par contre faites attention le numéro de ligne indiqué dans la stacktrace n'est pas compatible avec le listing de votre code!
 
Malgré ça je vous répond sans vous donnez directement la bonne réponse que je n'ai pas.
 
On commence par les attributs, à moins que celà soit absolument nécessaire ne créez jamais d'objet avec le type Object!
 

Code :
  1. private Object CapInitial;
  2. private Object nbrAl;


 
non!
 
À la place préférez un type explicite (int, String, MonObject, ...), qui vous aidera vous, le lecteur mais aussi votre IDE/compilateur! (tout le monde quoi!)
 
Ensuite pensez à respecter les conventions Java en utilisant la camel case.
 
Les attributs commencent avec une minuscule tout comme le nom des méthodes!
 
Dans votre méthode "ChangerCap" pourquoi créez vous un nouvel objet "Combat"? vous êtes déjà dans un objet "Combat".
 
Le problème que j'identifie est que dans "ChangerCap" vous instanciez un objet "Combat", vous appellez la méthode "ChoisirCap" sur cette instance, et de retour dans "ChangerCap" vous lisez l'attribut de l'instance courrante et non de l'instance créée.
 
Un code un peu trop confus,
 
Bon courage tout de même!
 
Cordialement,
 
Sébastien Millot

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed