Supprime les guillments d'une value Json

Version imprimable

Bonjour,

Je suis confronte a un petit probleme.

J'ai mon JsonWriter
Code:

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:

{"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:

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

l'erreur :

Code:

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:

{"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

14/06/2019, 13h19
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...
14/06/2019, 14h14
deltree
Bonjour,
Oui, indiquer ta lib pourrait nous aider, mais je pense sans trop me tromper que sur cette ligne:
Code:

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:

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.
17/06/2019, 15h15
R1beLlu2B

Bonjour,

Merci pour vos reponse.

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

Si je fait :

Code:

obj.put("clef",tableau);

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

Code:

"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
17/06/2019, 15h44
deltree

J'ai supposé que ton tableau était déjà de type JsonArray, mais il ne va pas se remplir tout seul! :D
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.

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:

{"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:

1
2
3
 "clef": [
    10
  ],

Si par contre je rentre plusieurs valeur comme ceci :

Code:

{"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:

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

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

Code:

{"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:

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:

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

17/06/2019, 20h03
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).

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:

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:

{"clef":["5"]}

ou si plusieurs valeur

Code:

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

J'ai essaye le .split comme ceci :

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
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:

{"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

Exttrait de ton code :
Code:

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:

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 }

19/06/2019, 10h25
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:

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
19/06/2019, 11h11
joel.drigo
Citation:
Envoyé par R1beLlu2B

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

Code:

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:

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

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.
25/06/2019, 00h33
R1beLlu2B

D'accord,

Merci en tout cas pour vos conseil.

Il me sont très utile.

Cordialement

Fred