Discussion :

[R1beLlu2B]

Bonjour,

Je suis confronte a un petit probleme.

J'ai mon JsonWriter

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
            JSONObject obj = new JSONObject();
 
 
 
            obj.put("clef",tableau.getText());  
 
             try {
                FileWriter out = new FileWriter("src/Games/"+Main.GameSelected+"/"+NomArme.getText()+".json");
 
                obj.writeJSONString(out);
 
                out.close();
 
               } catch (IOException H) {
                H.printStackTrace();
            }

qui écrit ceci dans mon fichier .json

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":"[0,0,0,0]"}

Jusque la rien d'anormale mais, quand je charge mon Json depuis mon programme,

j'ai une erreur. L'erreur est logique puisque mon tableau lis

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"[0,0,0,0]"

l'erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_ARRAY but was STRING at line 1 column 87 path $.tableau

Si je vais manuellement dans mon Json et je supprime les guillemets comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":[0,0,0,0]}

Mon tableau se remplis sans problème, je n'es plus d'erreur

L'erreur je l'es trouvé mais je ne sais pas comment réglé le problème.

- Soit j'arrive a écrire dans mon Json le tableau sans les "" mais avec les crochet []
- Soit a la lecture une méthode qui retire les "".
- Soit une solution que je n'es pas envisagé


J'ai cherche de part et autre rien trouvé a ce sujet.

Je vous remercie.


Fred

[joel.drigo]

Salut,

La solution que tu n'as pas envisagée c'est d'utiliser la bonne classe pour affecter la valeur de type tableau. Tu n'as précisé l'API que tu utilises, ça aurait été mieux vu qu'il y en a pléthore, mais il doit exister à priori une classe JSONArray dans cette API pour construire un tableau...

[deltree]

Bonjour,
Oui, indiquer ta lib pourrait nous aider, mais je pense sans trop me tromper que sur cette ligne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
     obj.put("clef",tableau.getText());

getText renvoie ton tableau au format String, probablement en le "Jsonifiant", donc c'est normal, tu peux tenter

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
     obj.put("clef",tableau);

si tableau est de type JSONArray et que ta lib l'autorise sur object.put.

Sinon, je te recommande d'utliser si tu peux de la sérialisation de plus au niveau avec Jackson ou Gson, qui permet de sérialiser directement tes objets custom sans avoir à manipuler des JsonObject et JsonArray.

[R1beLlu2B]

Bonjour,

Merci pour vos reponse.

J'utilise la lib : import org.json.simple.JSONObject;

Si je fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

obj.put("clef",tableau);

Çà fonctionne mais étant donne que mes données viennent d'un textfield editable, je recois ceci dans mon Json:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

"Clef":javax.swing.JTextField[,110,298,550x30,layout=javax.swing.plaf.basic.BasicTextUI$UpdateHandler,alignmentX=0.0,alignmentY=0.0,border=javax.swing.plaf.BorderUIResource$CompoundBorderUIResource@bb7daa9,flags=296,maximumSize=,minimumSize=,preferredSize=,caretColor=sun.swing.PrintColorUIResource[r=51,g=51,b=51],disabledTextColor=javax.swing.plaf.ColorUIResource[r=184,g=207,b=229],editable=true,margin=javax.swing.plaf.InsetsUIResource[top=0,left=0,bottom=0,right=0],selectedTextColor=sun.swing.PrintColorUIResource[r=51,g=51,b=51],selectionColor=javax.swing.plaf.ColorUIResource[r=184,g=207,b=229],columns=0,columnWidth=0,command=,horizontalAlignment=LEADING]

Peut etre un JTable a la place d'un Texfield? Mais j'ai essaye et ca na pas fonctionne..



je vais essaye les solutions propose si dessus.

je reviens vers vous peu importe l'issue


Cordialement


Fred

[deltree]

J'ai supposé que ton tableau était déjà de type JsonArray, mais il ne va pas se remplir tout seul!
Il faut convertir ton tableau en bouclant sur les valeurs: lire la doc du "textfield " Utilisé.

Il faut bien séparer tes données de tes objets écrans, tu ne peux pas "pousser" un objet graphique au milieu de tes données, c'est le 1er niveau de séparation de "concern" (notion de programmation SOLID".

Bref, après cet aparté, il faut donc que tu arrive par un moyen ou un autre à boucler sur les valeurs "String" de tes données pour pouvoir remplir ton JsonArray. (c'est compliqué car comme je disais, avec Jackson il suffirait d'avoir une List<String> et il se débrouille pour le convertir, là ce serait plus simple.)

Sans connaitre le type du textfield que tu utilise, difficile d'en dire plus.

[R1beLlu2B]

Merci pour vos reponse,

J'utilise un simple JtextField dont je récupère les données tapées par l'utilisateur.

Apres pas mal d'essais sur mon code, je suis presque au resultat voulue.

C'est a dire que si mon Json se retrouve ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":["5"]}

mon class qui utilise cette valeur arrive a la lire et réalise le resultat souhaité. (une bonne chose)

code dans la console :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 "clef": [
    10
  ],

Si par contre je rentre plusieurs valeur comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":["5,2,8,5"]}

La ca ne fonctionne pas.

Au lieu de cherche a tout prix de retire les ("")pour avoir un tableau

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":[5,2,8,5]}

Ne serai t'il pas possible que je fasse un "split" comme ceci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":["5","2","8","5"]}

au chargement des données.

Si cela est faisable, les données seront lus.(testé en ayant ecrit un .json manuellement ca fonctionne)

Voici comment je charge mon .jSON

Dans mon Main

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 static void setupConfigs() {
                       try {
                           File file = new File("src/Games/"+GameSelected);
                       if (!file.exists()) {
                           FileWriter fileWriter = new FileWriter(file);
                           gson.toJson(new Config(), fileWriter);
                           fileWriter.close();
                       }
                       ConfigManager1.getInstance1().readConfigs();
                       ConfigManager2.getInstance2().readConfigs();
 
                   } catch (IOException e) {
                       e.printStackTrace();
                }
        }

Et ensuite la class ConfigManager1.getInstance1().readConfigs(); (identique pour la deuxieme)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
60
61
62
63
64
65
66
import com.google.gson.Gson;
import com.google.gson.GsonBuilder;
 
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.util.ArrayList;
 
 
class ConfigManager1 {
 
  private static ArrayList<Config> configs = new ArrayList<>();
  private static ConfigManager1 ourInstance1 = new ConfigManager1();
  private int currentConfig = 0;
 
  private ConfigManager1() {
 
  }
 
  static ConfigManager1 getInstance1() {
    return ourInstance1;
  }
 
  void readConfigs() {
    configs.clear();
    Gson gson = new GsonBuilder().setPrettyPrinting().create();
    File dir = new File("src/Games/"+Main.GameSelected);
    File[] files = dir.listFiles();
    if (files != null) {
      for (File file : files) {
        try {
 
          BufferedReader br = new BufferedReader(new FileReader(file));
          Config config = gson.fromJson(br, Config.class);
          config.name = file.getName().replace(".json", "");
          configs.add(config);
          System.out.println("Loaded Config-1:\n" + gson.toJson(config));
          br.close();
        } catch (Exception e) {
          e.printStackTrace();
        }
 
      }
    }
  }
 
  Config getCurrentConfig() {
    if (configs.size() <= 0) {
      return new Config();
    } else {
      return configs.get(currentConfig);
    }
  }
 
  void cycleConfigs() {
    if (configs != null && configs.size() > 0) {
      if (currentConfig == configs.size() - 1) {
        currentConfig = 0;
      } else {
        currentConfig++;
      }
    }
  }
 
 
}

Je vous remercie


Fred

[joel.drigo]

Je ne sais pas pourquoi tu cherches à bidouiller la lecture plutôt que d'écrire du json correct dans ton fichier (et que du coup tu nous montres le code de lecture et pas celui d'écriture !). Ensuite peu importe que tu utilises GSon, Jackson, ou autre, ce sera du json, et ça sera lisible comme du json.

Ensuite, pour proprement écrire du json, soit :

• avec org.json.simple, comme org.json.simple.JSonArray, c'est une List, il te suffit de parser la saisie dans ton JTextField (c'est pas le plus adéquate pour saisir proprement une liste, mais bon admettons), selon ce qui est contenu (par exemple, si tu fais confiance et que le gars qui saisit est censé entrer des nombres, séparés par des virgules, tu fais un jtextfield.getText().split(","), et tu convertis chaque valeur du tableau obtenu pour les ajouter dans le JSonArray avec add...
• avec org.json, ça serait, une boucle sur jtextfield.getText().split(","), conversion dans le bon type, et jsonarray.put( la valeur )
• avec Gson ou Jackjson, et un bean, tu passerais par un ArrayList pour stocker la liste de "machins" saisis, par exemple, et ça serait pareil que dans le premier cas pour la remplir,
• ou en streaming (via jackson par exemple), bah, pareil, split, conversion, boucle et appel des méthode writexxx qui vont bien

En gros, la technique est la même : split de la saisie, conversion vers le type qui t'intéresse, boucle pour remplir une List ou un JSonArray et zou, écriture du fichier...

Ensuite, ton json est du json, donc pas besoin de bidouiller pour le relire (et en plus, bonus, tu peux le lire avec des éditeurs qui comprennent le json).

[R1beLlu2B]

Bonsoir,

Merci joel.drigo pour toute les information.

J'ai essayé de mettre en application les conseils données mais je coince avec le split...


Tableau1 et Tableau2 sont des JTextField.

Voici comment j'ecris mon .Json au clic de mon bouton :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
   public  void    actionPerformed(ActionEvent e)
    {
 
 
        JSONArray obj2 = new JSONArray();
        obj2.add((Tableau1.getText()));
 
 
        JSONArray obj3 = new JSONArray();
        obj3.add((Tableau2.getText()));
 
 
        JSONObject obj1 = new JSONObject();
 
        obj1.put("clef1",obj2);
        obj1.put("clef2",obj3);
 
        try {
            FileWriter out = new FileWriter("src/Games/"+Main.GameSelected+"/"+NomArme.getText()+".json");
 
            obj1.writeJSONString(out);
 
            out.close();
 
           } catch (IOException H) {
            H.printStackTrace();
        }
 
        //  Main.setupConfigs();
 
    }

La je sort un .json comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":["5"]}

ou si plusieurs valeur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":["1,1,1,1"]}

J'ai essaye le .split comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
public  void    actionPerformed(ActionEvent e)
    {
 
 
        JSONArray obj2 = new JSONArray();
        obj2.add((Tableau1.getText().split(",")));
 
 
        JSONArray obj3 = new JSONArray();
        obj3.add((Tableau2.getText().split(",")));
 
 
        JSONObject obj1 = new JSONObject();
 
        obj1.put("clef1",obj2);
        obj1.put("clef2",obj3);
 
        try {
            FileWriter out = new FileWriter("src/Games/"+Main.GameSelected+"/"+NomArme.getText()+".json");
 
            obj1.writeJSONString(out);
 
            out.close();
 
           } catch (IOException H) {
            H.printStackTrace();
        }
 
        //  Main.setupConfigs();
 
    }

Et je me retrouve dans mon fichier .json avec ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

{"clef":[[Ljava.lang.String;@10cd84ad]}

J'ai l'impression qu'il y a une histoire de conversion a faire maintenant mais je tourne en rond.

Merci

Fred

[joel.drigo]

Exttrait de ton code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
JSONArray obj3 = new JSONArray();
        obj3.add((Tableau2.getText().split(",")));

Chaque élément doit être ajouté séparément dans le JSONArray, alors que ton code ajoute le résultat du split, un tableau de String donc, en une fois, donc tu te retrouves avec un tableau qui contient juste un élément qui est un tableau.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
JSONArray obj3 = new JSONArray();
for(String item : Tableau2.getText().split(",")) { // pour chaque sous partie du tableau résultat du découpage...
        obj3.add(item); // ... on l'ajoute dans le JSONArray
}

[R1beLlu2B]

Bonjour,

Merci, je ne voyais pas sous cet angle...

Çà marche beaucoup mieux maintenant ,


Je suis sous IntelliJ IDEA et il m'a propose la modification de votre code de la manière suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  JSONArray obj2 = new JSONArray();
            obj2.addAll(Arrays.asList(Tableau1.getText().split(",")));

ceci est juste une simplification de syntaxe ou il y a un impact en performance? memoire?


Encore merci,

Bonne Journée

[joel.drigo]

Je suis sous IntelliJ IDEA et il m'a propose la modification de votre code de la manière suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  JSONArray obj2 = new JSONArray();
            obj2.addAll(Arrays.asList(Tableau1.getText().split(",")));

ceci est juste une simplification de syntaxe ou il y a un impact en performance? memoire?

Il y'a des dizaines de façons de faire la même chose. On pourrait aussi faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
JSONArray obj3 = new JSONArray();
Arrays.stream(Tableau2.getText().split(",")).forEach(obj3::add);

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
JSONArray obj3 = new JSONArray();
Pattern.compile(",").splitAsStream(Tableau2.getText()).forEach(obj3::add);

etc...

je t'ai juste donné l'écriture qui me semblait la plus adaptée au contexte, la plus basique aussi, un bon vieux foreach . Elle est à priori aussi la moins consommatrice de mémoire, surtout comparé à cette suggestion étonnante de ItellijIDEA, qui va instancier une ArrayList. La boucle qu'il y a dans le addAll à priori doit faire un for sans Iterator contrairement à ma première proposition. La différence de consommation de mémoire et de temps d'exécution doit être ridicule (en tout non remarquable sur l'export d'un json) surtout pour le volume de données à priori saisissable dans le champ. IntellijIDEA fait une proposition standard qui ne résulte pas d'un processus automatique analysant le contexte et cherchant à optimiser quoique soit. Et si ça simplifie, je ne suis pas sûr... une recherche de faire le truc en une "ligne" ou plutôt en un statement, probable, mais est-ce que ça simplifie la lecture du code, c'est toi qui voit.

[R1beLlu2B]

D'accord,

Merci en tout cas pour vos conseil.

Il me sont très utile.


Cordialement

Fred