Text und ArrayList

[Quick]

Hallo,
ich habe eine abstracte class Ticket eine sub class MusicE und eine class Elist.

Die Ticket class hat die Attribute :Titel,Datum ,Ort und Idcode.
Die sub class MusicE erbt alle diese Attribute und wird erweitert durch das Attribut Performer.
Die Elist class soll ein Txt file lesen das so aussieht.

Type: Music Event
Titel: Rock you
Datum : 15/02/10
Ort: Eifel
Performer: Loco
Idcode: 963A

Type: Cinema Event
Titel: Love you
Datum : 12/08/10
Ort: Bonn
Performer: Colo
Idcode: 185B

Nun versuche ich den Text zu lesen und zu ueberpruefen ob ein MusicE Objekt vorhanden it .Falls ja sollte es in eine Liste kommen.

Das Lesen vom ganzen Text klappt alles weitere klappt leider ueberhaupt nicht.

import java.util.ArrayList;
import java.io.*;
import java.util.Scanner;
public class EList 
{
private ArrayList<Ticket>list = new ArrayList<Ticket>();
 
    public EList() 
    {
        readFile();
        
    }
    
    public void readFile() 
    {
        
        try{
            Scanner scan = new Scanner(new FileInputStream("Info.txt"));
            String line;
            while(scan.hasNextLine())
            {
               line=scan.nextLine();
               System.out.println(line);
               

                
            }
        }
     catch(Exception e)
     {
            System.out.println("error");
     }
            }



             public ArrayList<Ticket> getList()
    {
        return list;
    }

     }

Bitte um Hilfe

 

[0x7F800000]

Nun versuche ich den Text zu lesen und zu ueberpruefen ob es sich um ein Cinema Event Objekt handelt.Falls ja sollte es in eine Liste kommen.

Die entsprechende Stelle konnte im im Code leider nicht entdecken. Für mich sieht's eher so aus, als ob der text einfach auf die Konsole geschrieben werden würde.

Ferner würde ich dir dringend empfehlen, deinen Code etwas sorgfältiger zu formatieren und vor allem auf Code Conventions zu achten: Reader() ist der denkbar schlechteste methodenname, den man sich einfallen lassen könnte. Zum einen klingt es nicht wie eine Aktion, sondern wie ein Objekt, zum anderen ist es sogar wie ein Klassenname hingeschrieben, und am allerschlimmsten: die Klasse gibt es tatsächlich, und zwar in den Kernpaketen der Standard-API. Methodennamen in camelCase und klein schreiben.

 

[Quick]

hallo,danke fuer deine Antwort.


"Die entsprechende Stelle konnte im im Code leider nicht entdecken.."

Ja ich hab sie mal weggelassen weil ich soviel herum probiert hatte ohne erfolg.

public void readFile() 
    {
        try{
     BufferedReader in  = new BufferedReader(new FileReader("Info.txt"));
     String line;


while((line = in.readLine()) != null) {
   list.add(line);
 
  
}
in.close();


  
    }
        catch(Exception e)
        {
            
            System.err.println("error");
        }
}

 

[0x7F800000]

nein, ich meinte eher die Stelle:

Falls ja sollten beide getrennt in eine Lsiste kommen.

wo soll die sein? In deinem code gibt es keinerlei fallunterscheidungen. Und worauf sich das Wort "beide" bezieht weiß ich auch nicht.

Ob du lieber Scanner oder BufferedReader verwendest ist hier einigermaßen egal. An deiner Stelle würde ich erstmal beim Scanner bleiben.

An den gruseligen Methodennamen hat sich seit dem ersten Post leider nicht viel verändert, daher kann ich das nur erneut empfehlen.

 

[Quick]

So hier noch meine IFs ohne grossen Erfolg

public void readFile()
    {
        MusicE music;
        try{
     
            Scanner scan = new Scanner(new FileInputStream("Info.txt"));
          
            String []line;
            while(scan.hasNextLine())
            {
               
             line=scan.nextLine().split("\n");
         
            if(line[0].startsWith("Type: Music Event"))
            {
            if(line[4].startsWith("Idcode"))
               list.add (new MusicE(line[1],line[2],line[3],line[4]));
                          
                          

            }
          }
        }

        catch(Exception e)
        {
            
            System.err.println("error");
  }
}
   

 public ArrayList<Ticket> getList()
    {
        return list;
    }
   
    }

 

[0x7F800000]

Eine einzelne Zeile zu scannen und diese dann an zeilenumbrüchen zu unterteilen macht irgendwie 0 Sinn, wenn es dadrin noch zeilenumbrüche gäbe, wäre es ja nicht eine Zeile... ???:L

 

[Quick]

So sollte er doch alle Zeilen Scannen .Macht aber immer noch nicht vielEine Idee?

public void readFile()
    {
        MusicE music;
        try{
     
            Scanner scan = new Scanner(new FileInputStream("Info.txt"));
          
            String []line;
            while(scan.hasNextLine())
            {
               while( scan.nextLine()!=null)
          {
             ////line=scan.nextLine().split("");
         
            if(line[0].startsWith("Type: Music Event"))
            {
            if(line[4].startsWith("Idcode:"))
               list.add (new MusicE(line[1],line[2],line[3],line[4]));
                          
                          
               }
            }
          }
        }
 
        catch(Exception e)
        {
            
            System.err.println("error");
  }
}

 

[TheChemist]

Hab hier ma schnell was geschrieben, was dir vllt weiterhilft. Is jetzt nicht getestet, hoffe es hilft dir bisschen weiter. Allerdings verstehe ich nicht wie sich die Events unterscheiden, in deinem Startpost haben beide sechs Attribute.

    public static final String CINEMA = "Cinema Event";
    public static final String MOVIE = "Music Event";
    
    private String[] info;

    public void readFileAndCreateEvent(String filePath)
    {
        try{     
            BufferedReader br = new BufferedReader(new FileReader(filePath));          
            String line = br.readLine();
            int i = 0;
            
            while(line != null) {       // solange Dokument nicht leer ist
                line = br.readLine();   // lese neue Zeile
                info[i] = line;         // und speichere Wert in das Info Array
                i++;
            }            
        } catch(Exception e) {
            e.printStackTrace();
            System.err.println("Fehler beim Einlesen");
        }
        
        if(info[0].contains(CINEMA)) {              // Überprüfe um welchen Typ es sich handelt
            MusicE musicEvent = new MusicE(info);   // und erzeuge das passende Objekt
        } else if(info[0].contains(MOVIE)) {
            CinemaE cinemaEvent = new CinemaE(info);
        }
    }

 

[ARadauer]

       String []line;
            while(scan.hasNextLine())
            {
               while( scan.nextLine()!=null)
          {
             ////line=scan.nextLine().split("");
         
            if(line[0].startsWith("Type: Music Event"))
            {
            if(line[4].startsWith("Idcode:"))
               list.add (new MusicE(line[1],line[2],line

räum das mal ein bisschen zusammen.... da kennt man sich nicht mehr aus...

du bist dir bewusst, wenn du zweimal nextLine aufrufst, dass dann zweimal gelesen wird? also es wird weitergelesen...

 

[Quick]

Hallo TheChemist ,danke fuer deine Zeit.Ich bin gerade drann den code nochmal neu zu schreiben und richte mich nach deinem code.

 

[Quick]

Ich schaffe es nicht eine Methode zu schreiben die volgendes macht.Wenn im Textfile oeffters ein Zeilenblock vorhanden ist der contains(Cinema) = true hat ,sollten auch so viele Objekte erzeugt werden.

Wenn er nur das Array[0] untersucht wirds wohl da auch enden und immer nur 1 Objekt erzeugen.

 

[Leroy42]

...die volgendes macht.Wenn im Textfile oeffters ein Zeilenblock vorhanden ist der contains(Cinema) = true hat ,sollten auch so viele Objekte erzeugt werden.

Mal ein kreativer Text!

Wenn im Textfile oeffters ein Zeilenblock vorhanden ist der contains(Cinema) = true hat ,sollten auch so viele Objekte erzeugt werden.

Was verstehst du unter

contains(Cinema)  = true

???:L

Variablen sollten mit einem Kleinbuchstaben begonner werden?

Und ist "contain" eine statische Variable?

 

[bygones]

Variablen sollten mit einem Kleinbuchstaben begonner werden?

Mal ein kreativer Text!

:lol:

 

[ARadauer]

Ich schaffe es nicht eine Methode zu schreiben die volgendes macht.Wenn im Textfile oeffters ein Zeilenblock vorhanden ist der contains(Cinema) = true hat ,sollten auch so viele Objekte erzeugt werden.

Wenn er nur das Array[0] untersucht wirds wohl da auch enden und immer nur 1 Objekt erzeugen.

nochmal bitte. Es ist heiß im Büro das schrenkt das Denken ein ... was willst du?
Deine Liste aufteilen?

 

[Quick]

Danke an TheChemist und ARadauer die wenigstens versucht haben seriös zu sein.
Über die anderen die nur mit Zitat antworten brauch nicht viel zu sagen. Besonders
Leroy 42 ist ein A............ und vertippt sich noch selber. Danke auch an bygones der die Kreativität von Leroy 42 zu schätzen weiß.


Super Forum :applaus::toll:

 

[0x7F800000]

Besonders Leroy 42 ist ein A............

Naja, wer gut gemeinte ratschläge aus der ersten Antwort befolgt, wird evtl. nach 20 Beiträgen nicht von irgendwelchen angeblichen A.... wegen demselben Grund genervt :noe:
Dass du hier nach 3 Tagen 15 Zeilen Code von TheChemist heldenhaft nachvollzogen hast ist zwar schön und gut, verleiht dir imho noch nicht ganz die nötige Erfahrung, um jemanden korrekterweise als A.... einzustufen.
Undeutlich formulierte Probleme führen nun mal dazu, dass die anderen Anfänger nicht verstehen was du willst, und die ganzen Fortgeschrittenen unterfordert sind und deinen Thread mit unpassenden Zitaten zumüllen... :autsch:

Bemühe dich bitte nächstes Mal einfach darum, kurz und klar zu schildern was deine Frage ist. :rtfm: Ohne den ganzen Prozess auf drei tage auszudehnen.

@all: Und 'n bisschen mehr Netiquette schadet auch nicht, hab ich neulich festgestellt (ja, das sagt ausgerechnet der scheiß Troll^^ )