ich versuche mich gerade an einem Programm, um Text aus einer Datei aus zu lesen und ggf. zu manipulieren. Dabei habe ich eine Funktion geschrieben, die pruefen soll, ob eine Zeile mit Leerzeichen beginnt und es/sie, soweit vorhanden, loescht.

Hier erstmal die Funktion:

void removespace(string *fstr)
{
     string space = " ";
     int pos1;
     while(pos1 != string::npos)
     {
     string::size_type pos1 = (*fstr).find(space);
     (*fstr).erase(pos1,1);
     }
}

Ich bin Neuling und bekomme daher laufend Fehler beim compilen. Aber diesmal beschwert sich DEV C++ nicht, dafuer bekomme ich aber von Windows eine Fehlermeldung und das Programm rauscht ab. Das passiert nur wenn ich diese Funktion im Laufe des Programms aufrufe, muss also an ihr liegen.

Ich hatte in einem Tutorial zu Pointern gelesen, dass es eine bessere Moeglichkeit gibt als Pointer bzw. Adressen an Funktionen zu uebergeben. Dort stand dann das ist erstmal nicht wichtig nur kam dann auch nichts mehr dazu. Hat evtl. jemand einen Link zu einem Tutorial fuer eine bessere Methode?

Waere fuer jede Hilfe dankbar.

void removespace(string &fstr) 
{
//Dann kann fstr wie ein normaler string benutzt werden, zB: fstr.length()
}

Du sagst, dass nur Leerzeichen am Anfang entfernt werden sollen, du suchst aber im ganzen string.
pos1 gibt es zweimal!?
Der Rückgabewert von find wird nicht korrekt geprüft.

@rocco123:
Hast du mal einen Code angeschaut? Erstmal wird npos nicht initialisiert und der Wert gleich mal fleissig gelesen. Dann wird npos ein Wert zugewiesen und ohne Sicherheitsabfrage, was npos ueberhaupt ist, einfach dort geloescht. Klar das es crashed, sobald npos nicht auf den String zeigt... f'`8k

Gruß, TGGC (Was Gamestar sagt...)

Mit Referenzen hat das nichts zu tun!

Die "bessere Methode" für die Übergabe an die Funktion hat sehr wohl mit Referenzen zu tun.

@rocco123:
Hast du mal einen Code angeschaut? Erstmal wird npos nicht initialisiert und der Wert gleich mal fleissig gelesen. Dann wird npos ein Wert zugewiesen und ohne Sicherheitsabfrage, was npos ueberhaupt ist, einfach dort geloescht. Klar das es crashed, sobald npos nicht auf den String zeigt...

Ersetze "npos" jeweils durch "pos1" und lass den sinnfreien Rest, der nichts mit dem Thema zu tun hat weg, dann ergibt das ne passende Aussage.

#include <algorithm>
#include <string>

void remove_chars(std::string &these, char to_remove)
{
  these.erase( std::remove(these.begin(), these.end(), to_remove), these.end() );
}

Verwendung:

#include <string>

int main()
{
  std::string value = "asdfghjklkjhgfdsdfghjklkjhgfdfghjk";
  remove_chars(value, 'j');
}

bb

void trim_left(std::string& s)
{
    s.erase(0, s.find_first_not_of(' '));
}

Danke euch allen.

void removespace(string *fstr)
{
     string space = " ";
     int pos1;
     while(pos1 != string::npos)

Du benutzt pos1 ohne pos1 initialisiert zu haben.