Reader

[Windows10]

Bei folgenden zwei Codezeilen taucht bei meinem Programm ein Fehler auf!
Die zwei Codezeilen:

 Student s = new Student(arr[0], arr[1]);
            l.add(s);

Der Fehler:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 1
at schuelerliste.Schuelerliste.main(Schuelerliste.java:37)
C:\Users\Philipp\AppData\Local\NetBeans\Cache\8.2\executor-snippets\run.xml:53: Java returned: 1
BUILD FAILED (total time: 5 seconds)

Was könnte da falsch sein?

Der ganze Code:

package schuelerliste;

import java.io.*;
import java.util.*;

public class Schuelerliste
{

    public static void main(String[] args) throws FileNotFoundException, IOException
    {
        Reader r = new FileReader("Schueler.txt");
        BufferedReader br = new BufferedReader(r);
        BufferedReader rb = new BufferedReader(new InputStreamReader(System.in));
        List<Student> l = new ArrayList<>();
        LineNumberReader lnr = new LineNumberReader(br);
        Writer out = new FileWriter("S.txt");
        while (true)
        {
            String line = br.readLine();
            if (line == null)
            {
                break;
            }
            String[] arr = line.split(";");
            Student s = new Student(arr[0], arr[1]);
            l.add(s);
            int linenumber = lnr.getLineNumber() + 1;
            System.out.println("Gebe Grupper ein!");
            String gruppe = rb.readLine();
            System.out.println("PC-Nummer eintragen!");
            String nr = rb.readLine();
            int pcNr = Integer.parseInt(nr);
            out.write(linenumber + " " + s.getNachname() + " " + s.getVorname() + " " + gruppe + " " + pcNr);
        }
        r.close();
        out.flush();
        out.close();
    }

}

Der Code der Klasse Schüler (für das angelegte Object oben):

package schuelerliste;

public class Student implements Comparable<Student>
{

  private String nachname;
  private String vorname;

  public Student(String nachname, String vorname)
  {
  this.nachname = nachname;
  this.vorname = vorname;
  }

  public String getNachname()
  {
  return nachname;
  }

  public void setNachname(String nachname)
  {
  this.nachname = nachname;
  }

  public String getVorname()
  {
  return vorname;
  }

  public void setVorname(String vorname)
  {
  this.vorname = vorname;
  }

  @Override
  public int compareTo(Student s)
  {
  if (nachname.charAt(0) == s.getNachname().charAt(0))
  {
  return 0;
  }
  if (nachname.charAt(0) > s.getNachname().charAt(0))
  {
  return 1;
  }
  return -1;
  }

}

 

[JStein52]

Ich würde mal sagen arr enthält nur ein einziges Element und du versuchst aber arr[1] anzusprechen

 

[InfectedBytes]

Im Grunde wieder das gleiche Problem wie du schon öfters hattest:
http://www.java-forum.org/thema/fehler-bei-integer-parseint.174989/
http://www.java-forum.org/thema/collection-problem.174889/
etc.

Und die Lösung ist immernoch die selbe...

 

[Windows10]

Danke, jetzt geht es wieder, ich hatte Leerzeilen in der Datei. Dafür hab ich jetzt ein anderes Problem: Die Linenumber ist immer 0!

Mein bearbeiteter Code:

package schuelerliste;

import java.io.*;
import java.util.*;

public class Schuelerliste
{

    public static void main(String[] args) throws FileNotFoundException, IOException
    {
        Reader r = new FileReader("Schueler.txt");
        BufferedReader br = new BufferedReader(r);
        BufferedReader rb = new BufferedReader(new InputStreamReader(System.in));
        List<Student> l = new ArrayList<>();
        LineNumberReader lnr = new LineNumberReader(br);
        Writer out = new FileWriter("S.txt");
        while (true)
        {
            String line = br.readLine();
            if (line == null)
            {
                break;
            }
            int linenumber = lnr.getLineNumber();
            String[] arr = line.split(";");
            Student s = new Student(arr[0], arr[1]);
            l.add(s);
            System.out.println("Gebe Gruppe ein!");
            String gruppe = rb.readLine();
            System.out.println("PC-Nummer eintragen!");
            String nr = rb.readLine();
            int pcNr = Integer.parseInt(nr);
            out.write(linenumber + " " + s.getNachname() + " " + s.getVorname() + " " + gruppe + " " + pcNr);
            out.write(System.getProperty("line.separator"));
        }
        r.close();
        out.flush();
        out.close();
    }

}

 

[InfectedBytes]

nutz readline von lnr

 

[mrBrown]

Du liest die Datei über den BufferedReader ein, da solltest du stattdessen den LineNumberReader benutzen

 

[Major_Sauce]

Das hier sollte in etwa passen. Ich habe den Linereader rausgenommen, und auch das "while(true)", das ist evil. Könnte sein dass noch was ausgebessert werden muss, habe das direkt hier in der Antwort umgebaut also sry wenn was an der Syntax nicht passt...

package schuelerliste;

import java.io.*;
import java.util.*;

public class Schuelerliste
{

    public static void main(String[] args) throws FileNotFoundException, IOException
    {
 
        BufferedReader br = new BufferedReader(new FileReader("Schueler.txt"));
        BufferedReader rb = new BufferedReader(new InputStreamReader(System.in));
        List<Student> l = new ArrayList<>();
        Writer out = new FileWriter("S.txt");
        String line = br.readLine();
        int linenumber = 0;
        while (line != null)
        {
            String[] arr = line.split(";");
            Student s = new Student(arr[0], arr[1]);
            l.add(s);
            System.out.println("Gebe Grupper ein!");
            String gruppe = rb.readLine();
            System.out.println("PC-Nummer eintragen!");
            String nr = rb.readLine();
            int pcNr = Integer.parseInt(nr);
            out.write(linenumber + " " + s.getNachname() + " " + s.getVorname() + " " + gruppe + " " + pcNr);
            line = br.readLine();
            linenumber++;
        }
        br.close();
        out.flush();
        out.close();
    }

}

Mfg Major

 

[mrBrown]

Wenn schon fertigen Code, dann doch auch vernünftigen

public static void main(String[] args) throws IOException {

        List<Student> students = new ArrayList<>();

        try (Writer out = new FileWriter("S.txt");
             LineNumberReader lnr = new LineNumberReader(new FileReader("Schueler.txt"))) {
            Scanner stdinReader = new Scanner(System.in)

            for (String line = lnr.readLine(); line != null; line = lnr.readLine()) {

                String[] arr = line.split(";");
                Student s = new Student(arr[0], arr[1]);
                students.add(s);

                System.out.println("Gebe Grupper ein!");
                String gruppe = stdinReader.nextLine();
                System.out.println("PC-Nummer eintragen!");
                int pcNr = stdinReader.nextInt();
                stdinReader.nextLine(); //Zeilenumbruch lesen

                out.write(lnr.getLineNumber() + " " + s.getNachname() + " " + s.getVorname() + " " + gruppe + " " + pcNr);
            }
        }
    }

Spoiler

Wie lange dauert es wohl diesmal, bis "Scanner ist doof!!11elf" kommt?

Edit: Scanner verschoben

 

[Flown]

@mrBrown Den Scanner bitte nicht in den try-resource Block legen, denn es wird auch damit System.in geschlossen und das muss unbedingt vom System und nicht vom User gemacht werden.

 

[mrBrown]

@mrBrown Den Scanner bitte nicht in den try-resource Block legen, denn es wird auch damit System.in geschlossen und das muss unbedingt vom System und nicht vom User gemacht werden.

Danke! Gar nicht drüber nachgedacht, ist mir bisher nie untergekommen