Java: ajout d'un tableau de chaîne dans une boucle while en utilisant un lecteur de fichiers

J'ai été faire un petit programme qui lit une liste de terrains de golf qui pourrait être changeing et doit être appelée quand jamais. Voici le code: 

    public class Courses {
public String[] courselist;
public void loadCourses() throws IOException{
    int count = 0;
    int counter = 0;
    File f = new File("src//courses//courses.txt");
    BufferedReader reader = new BufferedReader(new FileReader(f));
    while(count<1){
        String s = reader.readLine();
        if(s.equalsIgnoreCase("*stop*")){
            reader.close();
            count = 5;

        }else{
            courselist[counter] = s;
            counter++;
        }
        s = "";
    }
}

}

Et maintenant, c'est ce qui est dans le fichier txt.

    RiverChase 
    Steward Peninsula 
    Lake Park 
    Coyote Ridge 
    *stop*

Maintenant lorsque j'ai jamais démarrer pour exécuter le programme, car j'ai appeler la méthode instantanément, il me donne un lancer exeption et c'est à cause de la matrice. Et j'ai besoin de rester et tableau car je l'utilise dans un JComboBox. Si vous pouvez aider ou résoudre le problème. Probablement im juste de faire le mal, je suis un noob. Juste de l'aide. Merci à l'avance.
Je sais que tous les lecteur de fichier et ça marche parce qu'il imprime au système correct, j'ai juste besoin d'aide pour écrire sur le tableau repetedly.

  • Votre problème de base est que les tableaux en Java ne sont pas dynamiques de taille moyenne. Beaucoup de bonnes infos sur ce à la question suivante: java dynamique de tableau de tailles?
  • Vous devez vérifier le résultat de readLine() null avant de faire autre chose avec elle. Si la valeur est null, c'est EOF: fermez le fichier.
InformationsquelleAutor tyty5949 | 2012-06-06
  1. 1

    Modifier votre code pour affecter un nouveau tableau au cours de loadCourses(), et ajouter un appel à loadCourses() de votre constructeur:

    public class Courses {
    public String[] courselist;

    public Courses() throws IOException {  //<-- Added constructor
        loadCourses();                     //<-- Added call to loadCourses
    }

    public void loadCourses() throws IOException {
        int count = 0;
        int counter = 0;
        File f = new File("src//courses//courses.txt");
        BufferedReader reader = new BufferedReader(new FileReader(f));
        List<String> courses = new ArrayList<String>(); //<-- A List can grow 
        while(true){
            String s = reader.readLine();
            if (s.equalsIgnoreCase("*stop*")){
                break;
            }
            courses.add(s);
        }
        courseList = courses.toArray(new String[0]); //<-- assign it here
    }
}

    Cela garantit que lorsque vous créez une instance, il commence de la vie avec le tableau initialisé. Non seulement allez-vous pas une erreur, mais les données seront toujours correct (contrairement à d'autres réponses que de créer simplement un vide (c'est à dire inutile) tableau.

    Notez que ce code fonctionne avec n'importe quel nombre de noms dans le fichier (et pas seulement de 5).

    • merci beaucoup c'est celui que j'ai utilisé et ça marche
    InformationsquelleAutor Bohemian
  2. 2

    Vous devez initialiser votre tableau avant de l'utiliser

    public static final MAX_SIZE = 100;
public String[] courselist = new String[MAX_SIZE];
    InformationsquelleAutor Luiggi Mendoza
  3. 1

    Vous feriez mieux de créer une Liste qui est plus facile à manipuler et convertir comme un tableau à la fin du processus :

        List<String> list = new ArrayList<String>();
    String [] array = list.toArray(new String [] {});

    Voici une implémentation possible de la charge à l'aide d'une Liste :

    public static String [] loadCourses() throws IOException {
    File f = new File("src//courses.txt");
    BufferedReader reader = new BufferedReader(new FileReader(f));
    List<String> courses = new ArrayList<String>();
    while (true){
        String s = reader.readLine();
        if (s == null || s.trim().equalsIgnoreCase("*stop*")){
            break;
        } else{
            courses.add(s);
        }
    }
    return courses.toArray(new String [] {});
}

    Aussi pourquoi utiliser un arrêter mot-clé ? Vous pouvez tout simplement arrêter le processus quand vous avez atteint la fin du fichier (lorsque s est null).

    • Grâce à ce qui a aidé, mais maintenant je ne peux pas fiqure comment appliquer cela à la JComboBox. S'Il Vous Plaît Répondre
    • Appeler ce code à l'endroit où vous créez votre zone de liste modifiable. Exemple : String [] values = loadCourses(); JComboBox combo = new JComboBox(values);
    • OHHHHHH je ne l'ont connu que,à la merci d'une tonne.Je vais utiliser cette connaissance à travers le reste de ce projet
    • ok c'est emmbarasing mais il n'est pas fixé, j'ai essayé de faire comme ce public String[] = loadCourses(); dans le cours de la classe et il me donne une erreur. J'ai ensuite essayer de le charger dans la classe avec la jcombobox et il dosnt charger quoi que ce soit. Qu'ai-je fait cette fois
    • Ne peut pas vous dire si je ne vois plus de votre code. Avez-vous changer la signature de la méthode de public static String [] ?
    • Non je l'ai gardé juste public
    • Et statique ? Aussi, dans l'exemple de code que vous collez, je ne vois pas le nom de la variable. Il doit être : String [] courses = loadCourses();

    InformationsquelleAutor Arnaud
  4. 1

    Voici quelques exemple, sans l'aide de la *stop*:

    public class ReadFile {

    public static void main(String[] args) {
        BufferedReader reader = null;
        List<String> coursesList = new ArrayList<>();
        String[] courses;

        try {
            File file = new File("courses.txt");
            reader = new BufferedReader(new FileReader(file));
            String readLine;

            do {
                readLine = reader.readLine();
                if(readLine == null)
                    break;

                coursesList.add(readLine);
            } while (true);
        } catch (IOException ex) {
            Logger.getLogger(ReadFile.class.getName()).log(Level.SEVERE, null, ex);
        } finally {
            try {
                reader.close();
            } catch (IOException ex) {
                Logger.getLogger(ReadFile.class.getName()).log(Level.SEVERE, null, ex);
            }
        }

        courses = coursesList.toArray(new String[coursesList.size()]);

        for(int i = 0; i < courses.length; i++) {
            System.out.println(courses[i]);
        }
    }
}
    InformationsquelleAutor Crazenezz