Neue Memberfunktion für String

Die erinnerung

Ich glaube das Dürfte funktionieren:

#include <string>
#include <iostream>
using namespace std;

string trim(string& str)
{
    while((str[0] == '\n') || (str[0] == ' ') || (str[0] == '\t'))
        str = str.substr(1);

    return str;
}

string trim(string str)
{
    while((str[0] == '\n') || (str[0] == ' ') || (str[0] == '\t'))
        str = str.substr(1);

    return str;
}

int main()
{
    string str = "   lol";

    cout << trim("   lol") << endl << trim(&str) << endl << str << endl;

    return 0;
}

Oder was meint ihr?

Athar

Dass es nicht geht, hast du doch in schon selbst festgestellt.
Der Compiler wird dir das gerne noch einmal bestätigen.

Nexus

string trim(string& str)

hat eine merkwürdige Semantik. Willst du nicht ein Inplace-Trim und ein Kopier-Trim bereitstellen?

Das würde ich so tun:

void trim(std::string& str)
{
    ...
}

std::string trim_copy(std::string str)
{
    trim(str);
    return str;    
}

l'abra d'or

Die erinnerung schrieb:

Oder was meint ihr?

Wir meinen: Nimm NUR die zweite Variante, die erste macht dir dein Programm kaputt...

Die erinnerung

l'abra d'or schrieb:

Die erinnerung schrieb:

Oder was meint ihr?

Wir meinen: Nimm NUR die zweite Variante, die erste macht dir dein Programm kaputt...

0k. War nur ne Frage. Ich verwende im Moment eh nur die 2. Version im Programm.

l'abra d'or

Dann noch ein paar Tips wie du den Code optimieren kannst:
Kopien sind teuer. Deshalb solltest du das Argument auch per const-Referenz übergeben.
Auch substr() erzeugt eine Kopie -> teuer!
Speicher dir stattdessen einen "size_t currentIndex", den du immer hochzählst, solange str[currentIndex] ein Space ist. Gibt isspace false zurück, hast du deinen Index für substr. Dann hast du
Übergabe per Referenz (billig) Index inkrementieren (billig) und ein substr() - die teuerste Operation in der Funktion.

Die erinnerung

Und das ganze als Funktion für nichtFreaks, bitte?!?!?

Athar

l'abra d'or sagte, dass du alle Whitespacezeichen gleichzeitig mit einem einzigen Aufruf von substr wegmachen sollst und dass der Parameter eine konstante Referenz sein sollte, damit keine unnötige Kopie angefertigt wird.
War doch verständlich genug.

Beispielsweise

std::string trim_copy(std::string const &s) {
  std::string::size_type vorn   = s.find_first_not_of(" \t\n");

  if(vorn == std::string::npos) return "";

  std::string::size_type hinten = s.find_last_not_of (" \t\n");

  return s.substr(vorn, hinten - vorn + 1);
}

Allerdings gibt's das Ganze von boost schon hübscher (und mit Locale-Unterstützung): http://www.boost.org/doc/libs/1_45_0/doc/html/string_algo.html

EOP

std::string Trim( std::string & str )
{ 
size_t trim;

	if( (trim = str.find_first_not_of( " \t" )) != string::npos )
		str.erase( 0, trim );
	if( (trim = str.find_last_not_of( " \t" )) != string::npos )
		str.resize( trim+1 );
	return( str );
}