[debutant] Structure de mon fichier correcte ?

Structure de mon fichier correcte ? [debutant] - Java - Programmation

Marsh Posté le 04-05-2006 à 18:01:56    

Bonjour,
 
je vous soumet la structure de mon fichier en java. L'idee est que vous me disiez si les utilisations que je fais des static, private et compagnie sont effroyables, maladroites ou ok.
 

Code :
  1. /**
  2. *  
  3. */
  4. package packHou;
  5. import com.csvreader.CsvReader;
  6. import com.csvreader.CsvWriter;
  7. import com.csvreader.CsvWriter.FinalizedException;
  8. import java.io.IOException;
  9. import java.io.*;
  10. public class Trans {
  11. // reader pour le fichier en entree
  12. private static CsvReader reader;
  13. // writer pour un fichier tampon  
  14. private static CsvWriter writer = new CsvWriter("c:/out.csv" );
  15. // Nom du fichier en entree
  16. private static String fileName;
  17. // number of columns of a record
  18. private static int numOfColumns;
  19. // creation date of the BAI file
  20. private static String creationDate;
  21. // creation time of the BAI file
  22. private static String creationTime;
  23. private static void arrange88() {
  24. ...
  25. }
  26. public static void main(String[] args) {
  27.   fileName = args[0];
  28.   //arrange88 to rearrange the file without 88 fields
  29.   arrange88();
  30.  }
  31. }


 
parce que ma classe contient un main, je declare mes methodes static pour pouvoir les utiliser dans le main.  
 
Merci d'avance

Reply

Marsh Posté le 04-05-2006 à 18:01:56   

Reply

Marsh Posté le 04-05-2006 à 23:16:19    

réponse : plutôt maladroit...
Le main() est plutôt fait pour créer les objets de base de ton application, et
non pas pour "être" ton application.
Tu devrais plutôt avoir quelque chose comme ça :
 

Code :
  1. /**
  2. *  
  3. */
  4. package packHou;
  5. import com.csvreader.CsvReader;
  6. import com.csvreader.CsvWriter;
  7. import com.csvreader.CsvWriter.FinalizedException;
  8. import java.io.IOException;
  9. import java.io.*;
  10. public class Trans {
  11. // reader pour le fichier en entree
  12. private CsvReader reader;
  13. // writer pour un fichier tampon  
  14. private CsvWriter writer = new CsvWriter("c:/out.csv" );
  15. // Nom du fichier en entree
  16. private String fileName;
  17. // number of columns of a record
  18. private int numOfColumns;
  19. // creation date of the BAI file
  20. private String creationDate;
  21. // creation time of the BAI file
  22. private String creationTime;
  23. // construteur de l'objet
  24. public Trans(String nomFichier)
  25. {
  26.  fileName = nomFichier;
  27.  writer = new CsvWriter("c:/out.csv" );
  28. }
  29. public void arrange88() {
  30. ...
  31. }
  32. public static void main(String[] args)
  33. {
  34.  Trans monObj = new Trans(args[0]);
  35.  monObj.arrange88();
  36. }
  37. }


 
 
Edit: au fait, pas de 'new' possible quand on déclare une variable !  :hello:

Message cité 1 fois
Message édité par post_it le 04-05-2006 à 23:32:05

---------------
You need to get to the bottom of things...
Reply

Marsh Posté le 04-05-2006 à 23:27:40    

ok compris
 
Merci pour le coup de main, ca va bien m'aider

Reply

Marsh Posté le 05-05-2006 à 03:32:53    

post_it a écrit :


 
Edit: au fait, pas de 'new' possible quand on déclare une variable !  :hello:


:heink: n'imp' :heink:


---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Marsh Posté le 05-05-2006 à 06:45:18    


Ah?  :sweat:  
Bon, mais alors en tout cas ce n'est pas très beau !...  :D


---------------
You need to get to the bottom of things...
Reply

Marsh Posté le 05-05-2006 à 12:53:51    

bof, c'est très bien, sauf en l'occurence, où il a un chemin vers le file system, par exemple, mais sinon [:manust]


---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Sujets relatifs:

Leave a Replay

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