Comment récupéré une variable qui est dans une autre classe

Version imprimable

Bonjour,

Je rencontre un problème pour faire remonter ma variable jusqu'au main.
Mon problème se situe dans le fichier Menu.java dans la méthode public int choixPerso()

Pour que cela soit plus compréhensible voici un code :

Fichier App.java qui contient le main.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
package test;
 
public class App 
{
    public static void main( String[] args )
    {
        Menu menu = new Menu();
        Game game = new Game();
 
        int nMenuVal = 0;
        boolean nLoop = true;
 
        while(nLoop)
        {
            switch(nMenuVal)
            {
                case 0:
                    nMenuVal = menu.welcome(); //Permet de choisir de créer : son perso, jouer ou de quitter.
                    break;
                case 1:
                    nMenuVal = menu.choixPerso(); //Permet de donner un nom à son perso.
                    break;    
                case 2:
                    game.game(); //De jouer.
                    nMenuVal = 0;
                    break;
                case 3:
                    default:
                        nLoop = false;
            }
        }
    }
}

Le fichier Menu.java

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
package test;
 
import java.util.Scanner;
 
public class Menu{
 
    private String sChoice;//Récupération de la chaine de caractère entré par l'utilisateur.
    private int nChoice; //Choix de l'utilisateur après conversion de String en int.
    private boolean bLoop;
 
    Scanner scan = new Scanner(System.in);
 
    public Menu(){
        sChoice = "";
        nChoice = -1;
        bLoop = true; 
    }
 
//Récupère le nom rentré par l'utilisateur.
    public String nameGamer(){
        sChoice = scan.nextLine();
        return sChoice;
    }
 
//Accueil demandant de donner un nom à son perso.
    public int choixPerso(){
 
        String sNameGamer;
        System.out.println(" Donne un nom a ton personnage : ");
 
        sNameGamer = nameGamer();//MON PROBLEME SE SITUE ICI :comment faire pour renvoyer sNameGamer dans le main ?
 
        System.out.println("Nom du perso " + sNameGamer + ".\n");
 
        return 0;
    }
 
//Le menu.
    public int welcome(){
 
        System.out.println("/******** Bienvenue dans WARRIOR GAMER ********/");
 
        do{    
            System.out.println(" - 1 - Creer son personnage.");
            System.out.println(" - 2 - Game.");
            System.out.println(" - 3 - Quitter.");
 
            sChoice = scan.nextLine();
            if(Exceptions.isInteger(sChoice)){
                nChoice = Integer.parseInt(sChoice);
                bLoop = Tools.isChoiceValidInt(nChoice, 1, 3);
            }
            else
                bLoop = true;
        }while(bLoop);
        return nChoice;
    }
}

Le fichier Perso.java :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
package test;
 
public class Perso{
    private String sName;
 
    public void setName(String s_Name){
        sName = s_Name;
    }
 
    public String toString(){
        return "Mon nom est : " + sName + ".";
    }
}

Pendant que j'y suis : si dans une class je dois utiliser dans plusieurs méthode l'objet Scanner est-ce que je peut instancier la class Scanner comme un attribut comme j'ai fait dans le Menu.java ou bien dois-je l'instancier dans chaque méthode ?

30/03/2018, 04h34
CoDuSu
Tu dois créer une méthode dans ta classe menu, méthode qu'on appelle un Getter:
Code:

1 2 3 public String getSChoice() { return this.sChoice; }
et dans ta classe main:

Code:

menu.getSChoice();

(Une manière de faire encore plus rapide est tout simplement Menu.sChoice (avec le M en majuscule, il ne sagit pas ici de l'objet que tu as crée dans la classe Main) mais ça peut vite foutre le bordel d'utiliser directement les variables de la classe plutot que celle de l'objet crée donc conventionnellement on doit créer des Getters et/ou des Setters pour manipuler les variables d'autres classes)

Salut,

Citation:

Envoyé par hbx360

Il faut éviter de créer plusieurs instances de Scanner sur System.in parce que cette classe est bufferisée : lorsqu'on lit par exemple un int (avec nextInt()), des caractères vont être lus en avance pour déterminer la fin du int dans le buffer. La lecture par une autre instance de Scanner va donc être incorrecte.

Solutions possibles (mono thread):

Créer une instance et la passer en argument de la construction des classes

Code:

1
2
3
4
5
6
7
8
9
public class Machin {
    private Scanner input;
    public Machin(Scanner input) {
            this.input=input;
    }
     public void methodeQuiLit() {
            int valeur = input.nextInt();
     }
}

Code:

1
2
3
4
5
6
public class Truc {
    private Scanner input;
    public Truc(Scanner input) {
            this.input=input;
    }
}

Code:

1
2
3
4
5
6
7
public class Programme {
     public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        Machin machin = new Machin(scanner);
        Truc truc = new Truc(scanner);
     }
}

Utiliser un singleton, avec accès static :

Code:

1
2
3
4
5
6
7
8
9
public class Programme {
 
     public static Scanner INPUT = new Scanner(System.in);
 
     public static void main(String[] args) {
        Machin machin = new Machin();
        Truc truc = new Truc();
     }
}

Code:

1
2
3
4
5
6
7
public class Machin {
    public Machin() {
    }
     public void methodeQuiLit() {
            int valeur = Programme.INPUT.nextInt();
     }
}

Faire une classe utilitaire pour lire au clavier :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public class Clavier {
 
     private static Scanner INPUT = new Scanner(System.in);
 
     public static int lireInt(String question) {
            System.out.print(question);
            while(INPUT.hasNext()) {
                  if ( INPUT.hasNextInt() ) {
                       int valeur = INPUT.nextInt();
                       INPUT.nextLine(); // vide le buffer jusqu'au prochain retour chariot
                       return valeur;
                  } 
                  System.out.println("Vous devez saisir un nombre.  Recommencez s'il vous plait.");
                  System.out.print(question);
            }
     }
 
     public static int lireInt(String question, int min, int max) {
          /*...*/
     }
 
 
     public static String lireString(String question) {
          /*...*/
     }
 
}

Code:

1
2
3
4
5
6
7
8
9
public class Programme {
 
 
     public static void main(String[] args) {
        Machin machin = new Machin();
        Truc truc = new Truc();
        machin.methodeQuiLit();
     }
}

Code:

1
2
3
4
5
6
7
public class Machin {
    public Machin() {
    }
     public void methodeQuiLit() {
            int valeur = Clavier.lireInt("Veuillez saisir machin...");
     }
}

Cette dernière version présente l'avantage de pouvoir décorer la lecture de différents traitements (affichage d'un prompt, traiterment d'erreur de saisie, etc) qu'on peut réutiliser partout. En multi-thread on pourrait ajouter des synchronized sur les méthodes pour éviter des lectures et affichages mélangés.

30/03/2018, 09h06
joel.drigo
Citation:

Envoyé par CoDuSu

(Une manière de faire encore plus rapide est tout simplement Menu.sChoice (avec le M en majuscule, il ne sagit pas ici de l'objet que tu as crée dans la classe Main)

Attention, Menu.sChoice implique que sChoice est static :
- mauvais si plusieurs instances de Menu, parce que même valeur partagée entre toutes les instances
- mauvais si une seule instance de Menu parce que des méthodes non static vont manipuler une variable static : autant faire toutes les méthodes static et ne plus faire de new (voire interdire la possiblité de faire new pour éviter des soucis)
30/03/2018, 10h10
professeur shadoko
Citation:

Solutions possibles (mono thread):

on peut même pousser le bouchon très loin avec des interfaces définissant des services:
Code:

1 2 3 4 5 public interface AgentSaisie { // ou autres public String saisie() ; }
et
Code:

1 2 3 4 5 6 public interface AgentAffichage { public void invite( String invite) ; public void affichage(String résultat) ; public void incident(String messageErreur) ; }
ok c'est plus tordu ;) (et dépasse le niveau débutant) mais ça donne l'habitude de découpler "métier" et interactions.
(le même code pourra être utilisé avec des interactions console, des scripts de test ou des interfaces graphiques)
30/03/2018, 10h18
hbx360

Merci beaucoup pour vos réponses.
14/04/2018, 19h39
hbx360

@joel.drigo Donc j'ai fait une class Keyboard pour ouvrir qu'une instance Scanner mais dans ton exemple tu n'utilise pas la méthode close() pour fermer System.in je voulais savoir si c'est obligatoire ou optionnel et ou dois-je la mettre.

Merci par avance.
14/04/2018, 20h38
joel.drigo

La règle : fermer les flux qu'on a ouvert. System.in est géré par Java, pas besoin de le fermer.
15/04/2018, 20h40
hbx360

Ok merci.