Nun jedoch mein Problem:
Das Parsen einzelner Textdateien funktioniert, also gehe ich davon aus, dass die parsen-Funktion korrekt ist. Was allerdings nicht funktioniert ist, dass er eine Liste bekommt, mit den Dateinamen, welche er einlesen soll und eben jene als Parameter an die parsen-Funktion übergibt. Dadurch bleibt die Index-Datei (also jene, in welcher die Wörter Zeile für Zeile gespeichert werden) leer.

Eine Fehlermeldung bekomme ich nicht.

Das Problem sollte in diesem Code liegen. Ich vermute an der getline.
Falls ihr mehr von dem Code benötigt, könnt ihr ja einfach Bescheid sagen

3 Parser::Parser(string Liste)
  4 {
  5   //parsen("test.txt"); //funktioniert
  6 
  7   ifstream l;
  8   l.open(Liste.c_str());
  9   if(l.is_open())
 10   {
 11     char* a = new char[256];
 12     while(!l.eof())
 13     {
 14      l.getline(a, 256); //delim = \n
 15      parsen(string(a)); //funktioniert nicht
 16      cout << a << " "; //das druckt er, also "test.txt test2.txt "
 17     }
 18     l.close();
 19   }
 20   else
 21   {
 22     cout << "Datei nicht lesbar!!" << endl;
 23   }
 24 
 25 }

Die Liste sieht einfach so aus:

test.txt
test2.txt

Ich bin übrigens noch absoluter Anfänger und für Verbesserungsvorschläge absolut offen

Ich bin übrigens noch absoluter Anfänger und für Verbesserungsvorschläge absolut offen

Was mir sofort ins Auge sticht ist

char* a = new char[256];

Zumal du vergisst, den Speicher wieder mit delete[] freizugeben.

Hier kannst du einfach

char a[256];

nehmen oder noch besser gleich einen std::string (dafür gibts auch eine eigene getline()-Funktion, da hast du keine Probleme mehr, wenn die Strngs doch einmal länger als 256 Zeichen werden.)

Also, das ist parsen(string Dateiname)

Wenn ich den Dateiname direkt einfüge, also z.b.: parsen(test.txt) funktioniert alles tadellos (abgesehen von etlichen Ausnahmefällen die ich noch nicht drinne habe, aber das stört erstmal nicht) und in der index steht dann z.B.

Ich test.txt
weiß test.txt
nicht test.txt
mehr test.txt
weiter test.txt

Wenn ich genau das allerdings mithilfe de getline-Funktion probiere und die Dateinamen aus einer Liste ziehe, ist die index vollkommen leer. Warum?

Ich empfehle aber auch std::string mit der zugehörigen freien Funktion 'getline(string, istream)' zu benutzen.

Außerdem kannst du das Öffnen der Datei gleich beim Konstruktor vornehmen:

ifstream f(Dateiname.c_str());

Ich gehe ja auch richtig in der Annahme, dass er in der Index nur anhängt und diese nicht jedes Mal neu erstellt oder gegebenenfalls irgendetwas löscht, oder?

Mache ich in die parsen eine Ausgabe für den Dateinamen rein, so gibt er diesen auch korrekt aus.

Danke auf jeden Fall schoneinmal für die Tipps.

EDIT: Ich habe gerade einmal einen else-Zweig reingemacht, um zu schauen, ob er die Dateien öffnet, ... und siehe da, er tut es nicht.
Woran könnte das liegen?

int Parser::parsen(string Dateiname)

doch einfach mal:

int Parser::parsen(const char *Dateiname)

Ich weiß nämlich nicht ob es da Datenverluste/-veränderungen geben kann. Einfach mal ausprobieren und debuggen...

EDIT: Ich habe es nun vollständig mit const char* probiert, doch bleibt das Problem dasselbe: Er kann die Dateien nicht öffnen, obwohl er den richtigen Dateinamen überliefert bekommt und die Dateinamen innerhalb der parsen-Funktion auch korrekt ausgeben kann.

EDIT: Ich habe es nun vollständig mit const char* probiert, doch bleibt das Problem dasselbe: Er kann die Dateien nicht öffnen, obwohl er den richtigen Dateinamen überliefert bekommt und die Dateinamen innerhalb der parsen-Funktion auch korrekt ausgeben kann.

EDIT2: Ich seh gerade, dass er ungewöhnlicherweise, trotz 2er Dateien, nur einmal "Kann Datei nicht öffnen!" ausgibt.

Nocheinmal der vollständige Code, wie er jetzt ist:

#include "Parser.h" 
  Parser::Parser(string Liste)
  {
    //parsen("test.txt"); //funktioniert
    ifstream l(Liste.c_str());
    if(l.is_open())
    {
      string a;
      while(!l.eof())
      {
       getline(l, a); //delim = \n
       parsen(a); //funktioniert nicht
       cout << a << " ";
      }
      l.close();
    }
    else
    {
      cout << "Datei nicht lesbar!!" << endl;
    }

  }

  int Parser::parsen(string Dateiname)
  {
    ifstream f(Dateiname.c_str());
    ofstream index; //ifstream = Lesen, ofstream = Schreiben, fstream = flexibel 
    index.open("index");
    if(f.is_open()) //testet ob Datei wirklich geöffnet ist
    {
      char c;
      char zeichendavor = ' ';

  /* Problematik: 
   * - Problem bei fehlenden Leerzeichen */

      while(!f.eof()) //testet ob Datei zuende ist
      {
        f.get(c);

          switch(c)
        {
          case ' ': case '\n':
           switch(zeichendavor)
            {
              case ' ': case '\n': break;
              default:
              index << "\t\t" << Dateiname << "\n";
              break;
            }
            break;
          case ',': case '!': case '?': case '.': case '"': break;
          default:
            switch(zeichendavor)
            {
              case ',': case '!': case '?': case '.': case '"':
                  index << "\t\t" << Dateiname << "\n" << c; break;
              default:  index << c;
              break;
            }
            break;
        }
       zeichendavor = c;
      }
      f.close();
       index.close();
    }
     else
      cout << "Kann Datei nicht öffnen!" << endl;
     return 0;
  }

l.open(Liste.c_str());

mal

l.open(Liste.c_str(), std::ios::in);

machen...

Ausgabe:

test.txt test2.txt Kann Datei nicht öffnen!

Ändere ich den else-Zweig auf:

else
     cout << "Kann " << Dateiname << " nicht öffnen!" << endl;

Bekomme ich:

test.txt test2.txt Kann nicht öffnen!

Er kennt also den Dateinamen nicht mehr. Hat das einen Grund?

Setze ich jedoch eine Ausgabe des Dateinames an die erste Stelle in parsen, so kann er es noch.

Mach aus:

int Parser::parsen(string Dateiname)

doch einfach mal:

int Parser::parsen(const char *Dateiname)

Ich weiß nämlich nicht ob es da Datenverluste/-veränderungen geben kann. Einfach mal ausprobieren und debuggen...

Kóyaánasqatsi schrieb:

Aus

l.open(Liste.c_str());

mal

l.open(Liste.c_str(), std::ios::in);

machen...

Hauptsache wild im Zeug rumraten...
Der erste Vorschlag ist nicht wirklich ratsam, wenn die.ina mit std::getline() flexibel sein will. Wie kommst du eigentlich darauf, dass man durch den std::string Daten verlieren könnte? Allerdings könnte sich eine Const-Referenz, also const std::string& Dateiname aus Performancegründen empfehlen.

Dein zweiter Vorschlag bringt überhaupt nichts.

die.ina schrieb:

index.open("index");

Bist du sicher, dass die Datei "index" heisst (ohne Endung)?

Was willst du eigentlich genau erreichen? In einer Datei die Namen anderer Dateien lesen, die du dann einzeln parst? Übrigens rate ich dir, so schnell wie möglich den Umgang mit dem Debugger zu lernen. Das ist eines der hilfreichsten und mächtigsten Werkzeuge eines Programmierers.

Momentan hänge ich zwischen Schritt 1 und 2, wobei 1 eigentlich noch vollkommen fehlt.

Index ohne Endung ist kein Problem, ... Linux.

Also klappt das browsen durch die Ordner nicht? Respektive hast du Probleme mit Pfaden?