Jaki jest najlepszy sposób przycinania std :: string?

Obecnie używam następującego kodu do przycięcia wszystkich std::stringsprogramów w moich programach:

std::string s;
s.erase(s.find_last_not_of(" \n\r\t")+1);

Działa dobrze, ale zastanawiam się, czy są jakieś przypadki końcowe, w których może się nie powieść?

Oczywiście mile widziane są odpowiedzi z eleganckimi alternatywami, a także lewe wykończenia.

EDYCJA Od wersji c ++ 17 niektóre części standardowej biblioteki zostały usunięte. Na szczęście, począwszy od c ++ 11, mamy lambdy, które są doskonałym rozwiązaniem.

#include <algorithm> 
#include <cctype>
#include <locale>

// trim from start (in place)
static inline void ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](int ch) {
        return !std::isspace(ch);
    }));
}

// trim from end (in place)
static inline void rtrim(std::string &s) {
    s.erase(std::find_if(s.rbegin(), s.rend(), [](int ch) {
        return !std::isspace(ch);
    }).base(), s.end());
}

// trim from both ends (in place)
static inline void trim(std::string &s) {
    ltrim(s);
    rtrim(s);
}

// trim from start (copying)
static inline std::string ltrim_copy(std::string s) {
    ltrim(s);
    return s;
}

// trim from end (copying)
static inline std::string rtrim_copy(std::string s) {
    rtrim(s);
    return s;
}

// trim from both ends (copying)
static inline std::string trim_copy(std::string s) {
    trim(s);
    return s;
}

Dzięki https://stackoverflow.com/a/44973498/524503 za wprowadzenie nowoczesnego rozwiązania.

Oryginalna odpowiedź:

Zwykle używam jednego z tych 3 do moich potrzeb przycinania:

#include <algorithm> 
#include <functional> 
#include <cctype>
#include <locale>

// trim from start
static inline std::string &ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(),
            std::not1(std::ptr_fun<int, int>(std::isspace))));
    return s;
}

// trim from end
static inline std::string &rtrim(std::string &s) {
    s.erase(std::find_if(s.rbegin(), s.rend(),
            std::not1(std::ptr_fun<int, int>(std::isspace))).base(), s.end());
    return s;
}

// trim from both ends
static inline std::string &trim(std::string &s) {
    return ltrim(rtrim(s));
}

Są dość zrozumiałe i działają bardzo dobrze.

EDYCJA : BTW, mam std::ptr_funtam pomóc std::isspacew jednoznacznym ustaleniu, ponieważ tak naprawdę istnieje druga definicja, która obsługuje ustawienia narodowe. To mogła być obsada tak samo, ale ja lubię to bardziej.

EDYCJA : Aby odnieść się do niektórych komentarzy na temat akceptowania parametru przez odniesienie, modyfikowanie i zwracanie go. Zgadzam się. Implementacją, którą prawdopodobnie wolałbym, byłyby dwa zestawy funkcji, jeden na miejscu i jeden, który tworzy kopię. Lepszym zestawem przykładów byłoby:

#include <algorithm> 
#include <functional> 
#include <cctype>
#include <locale>

// trim from start (in place)
static inline void ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(),
            std::not1(std::ptr_fun<int, int>(std::isspace))));
}

// trim from end (in place)
static inline void rtrim(std::string &s) {
    s.erase(std::find_if(s.rbegin(), s.rend(),
            std::not1(std::ptr_fun<int, int>(std::isspace))).base(), s.end());
}

// trim from both ends (in place)
static inline void trim(std::string &s) {
    ltrim(s);
    rtrim(s);
}

// trim from start (copying)
static inline std::string ltrim_copy(std::string s) {
    ltrim(s);
    return s;
}

// trim from end (copying)
static inline std::string rtrim_copy(std::string s) {
    rtrim(s);
    return s;
}

// trim from both ends (copying)
static inline std::string trim_copy(std::string s) {
    trim(s);
    return s;
}

Zachowuję jednak pierwotną odpowiedź powyżej w kontekście i w celu utrzymania wysokiej głosowanej odpowiedzi nadal dostępnej.

Użycie algorytmów ciągu Boost byłoby najłatwiejsze:

#include <boost/algorithm/string.hpp>

std::string str("hello world! ");
boost::trim_right(str);

strjest teraz "hello world!". Jest też trim_lefti trim, która przycina obie strony.

Jeśli dodasz _copysufiks do dowolnej z powyższych nazw funkcji, np trim_copy. Funkcja zwróci przyciętą kopię ciągu zamiast modyfikować ją przez odwołanie.

Jeśli dodasz _ifsufiks do dowolnej z powyższych nazw funkcji, np. trim_copy_ifMożesz przyciąć wszystkie znaki spełniające Twój predykat, w przeciwieństwie do samych białych znaków.

Użyj następującego kodu, aby wyrównać (końcowe) spacje i znaki tabulacji w std::strings( ideone ):

// trim trailing spaces
size_t endpos = str.find_last_not_of(" \t");
size_t startpos = str.find_first_not_of(" \t");
if( std::string::npos != endpos )
{
    str = str.substr( 0, endpos+1 );
    str = str.substr( startpos );
}
else {
    str.erase(std::remove(std::begin(str), std::end(str), ' '), std::end(str));
}

I żeby to zrównoważyć, dołączę również lewy kod wykończenia ( ideone ):

// trim leading spaces
size_t startpos = str.find_first_not_of(" \t");
if( string::npos != startpos )
{
    str = str.substr( startpos );
}

To, co robisz, jest dobre i solidne. Długo używałem tej samej metody i jeszcze nie znalazłem szybszej metody:

const char* ws = " \t\n\r\f\v";

// trim from end of string (right)
inline std::string& rtrim(std::string& s, const char* t = ws)
{
    s.erase(s.find_last_not_of(t) + 1);
    return s;
}

// trim from beginning of string (left)
inline std::string& ltrim(std::string& s, const char* t = ws)
{
    s.erase(0, s.find_first_not_of(t));
    return s;
}

// trim from both ends of string (right then left)
inline std::string& trim(std::string& s, const char* t = ws)
{
    return ltrim(rtrim(s, t), t);
}

Dostarczając znaki, które mają zostać przycięte, masz elastyczność przycinania znaków spoza białych i wydajność przycinania tylko tych znaków, które chcesz przyciąć.

Trochę późno na imprezę, ale nieważne. Teraz jest już C ++ 11, mamy lambdy i zmienne automatyczne. Tak więc moja wersja, która obsługuje również białe znaki i puste ciągi, to:

#include <cctype>
#include <string>
#include <algorithm>

inline std::string trim(const std::string &s)
{
   auto wsfront=std::find_if_not(s.begin(),s.end(),[](int c){return std::isspace(c);});
   auto wsback=std::find_if_not(s.rbegin(),s.rend(),[](int c){return std::isspace(c);}).base();
   return (wsback<=wsfront ? std::string() : std::string(wsfront,wsback));
}

Moglibyśmy zrobić iterator do tyłu wsfronti użyć go jako warunku zakończenia w drugim, find_if_notale jest to przydatne tylko w przypadku łańcucha całkowicie białych znaków, a gcc 4.8 przynajmniej nie jest wystarczająco inteligentny, aby wywnioskować rodzaj iteratora do tyłu ( std::string::const_reverse_iterator) z auto. Nie wiem, jak drogie jest tworzenie iteratora wstecznego, więc tutaj YMMV. Po tej zmianie kod wygląda następująco:

inline std::string trim(const std::string &s)
{
   auto  wsfront=std::find_if_not(s.begin(),s.end(),[](int c){return std::isspace(c);});
   return std::string(wsfront,std::find_if_not(s.rbegin(),std::string::const_reverse_iterator(wsfront),[](int c){return std::isspace(c);}).base());
}

Wypróbuj to, działa dla mnie.

inline std::string trim(std::string& str)
{
    str.erase(0, str.find_first_not_of(' '));       //prefixing spaces
    str.erase(str.find_last_not_of(' ')+1);         //surfixing spaces
    return str;
}

Podoba mi się rozwiązanie tzamana, jedynym problemem jest to, że nie przycina ciągu zawierającego tylko spacje.

Aby poprawić tę 1 wadę, dodaj str.clear () pomiędzy 2 liniami trymera

std::stringstream trimmer;
trimmer << str;
str.clear();
trimmer >> str;

http://ideone.com/nFVtEo

std::string trim(const std::string &s)
{
    std::string::const_iterator it = s.begin();
    while (it != s.end() && isspace(*it))
        it++;

    std::string::const_reverse_iterator rit = s.rbegin();
    while (rit.base() != it && isspace(*rit))
        rit++;

    return std::string(it, rit.base());
}

W przypadku pustego ciągu kod zakłada, że ​​dodanie 1 do wartości string::npos0. string::nposjest typu string::size_type, który nie jest podpisany. Dlatego polegasz na przepełnieniu dodawania.

Zhakowany z Cplusplus.com

std::string choppa(const std::string &t, const std::string &ws)
{
    std::string str = t;
    size_t found;
    found = str.find_last_not_of(ws);
    if (found != std::string::npos)
        str.erase(found+1);
    else
        str.clear();            // str is all whitespace

    return str;
}

Działa to również w przypadku wartości zerowej. :-)

W C ++ 17 możesz używać basic_string_view :: remove_prefix i basic_string_view :: remove_suffix :

std::string_view trim(std::string_view s)
{
    s.remove_prefix(std::min(s.find_first_not_of(" \t\r\v\n"), s.size()));
    s.remove_suffix(std::min(s.size() - s.find_last_not_of(" \t\r\v\n") - 1, s.size()));

    return s;
}

Ładna alternatywa:

std::string_view ltrim(std::string_view s)
{
    s.remove_prefix(std::distance(s.cbegin(), std::find_if(s.cbegin(), s.cend(),
         [](int c) {return !std::isspace(c);})));

    return s;
}

std::string_view rtrim(std::string_view s)
{
    s.remove_suffix(std::distance(s.crbegin(), std::find_if(s.crbegin(), s.crend(),
        [](int c) {return !std::isspace(c);})));

    return s;
}

std::string_view trim(std::string_view s)
{
    return ltrim(rtrim(s));
}

Moje rozwiązanie oparte na odpowiedzi @Bill the Lizard .

Zauważ, że te funkcje zwrócą pusty ciąg, jeśli ciąg wejściowy nie zawiera nic oprócz białych znaków.

const std::string StringUtils::WHITESPACE = " \n\r\t";

std::string StringUtils::Trim(const std::string& s)
{
    return TrimRight(TrimLeft(s));
}

std::string StringUtils::TrimLeft(const std::string& s)
{
    size_t startpos = s.find_first_not_of(StringUtils::WHITESPACE);
    return (startpos == std::string::npos) ? "" : s.substr(startpos);
}

std::string StringUtils::TrimRight(const std::string& s)
{
    size_t endpos = s.find_last_not_of(StringUtils::WHITESPACE);
    return (endpos == std::string::npos) ? "" : s.substr(0, endpos+1);
}

Moja odpowiedź jest poprawą w stosunku do górnej odpowiedzi dla tego postu, która przycina znaki kontrolne oraz spacje (0-32 i 127 w tabeli ASCII ).

std::isgraphokreśla, czy znak ma reprezentację graficzną, więc możesz użyć tego do zmiany odpowiedzi Evana, aby usunąć dowolny znak, który nie ma reprezentacji graficznej z żadnej strony łańcucha. Rezultatem jest znacznie bardziej eleganckie rozwiązanie:

#include <algorithm>
#include <functional>
#include <string>

/**
 * @brief Left Trim
 *
 * Trims whitespace from the left end of the provided std::string
 *
 * @param[out] s The std::string to trim
 *
 * @return The modified std::string&
 */
std::string& ltrim(std::string& s) {
  s.erase(s.begin(), std::find_if(s.begin(), s.end(),
    std::ptr_fun<int, int>(std::isgraph)));
  return s;
}

/**
 * @brief Right Trim
 *
 * Trims whitespace from the right end of the provided std::string
 *
 * @param[out] s The std::string to trim
 *
 * @return The modified std::string&
 */
std::string& rtrim(std::string& s) {
  s.erase(std::find_if(s.rbegin(), s.rend(),
    std::ptr_fun<int, int>(std::isgraph)).base(), s.end());
  return s;
}

/**
 * @brief Trim
 *
 * Trims whitespace from both ends of the provided std::string
 *
 * @param[out] s The std::string to trim
 *
 * @return The modified std::string&
 */
std::string& trim(std::string& s) {
  return ltrim(rtrim(s));
}

Uwaga: Alternatywnie powinieneś być w stanie użyć, std::iswgraphjeśli potrzebujesz obsługi szerokich znaków, ale będziesz musiał również edytować ten kod, aby umożliwić std::wstringmanipulację, co jest czymś, czego nie przetestowałem (zobacz stronę referencyjną, std::basic_stringaby zbadać tę opcję) .

Wraz z C ++ 11 pojawiło się również wyrażenie regularne moduł , który oczywiście może być użyty do przycinania spacji wiodących lub końcowych.

Może coś takiego:

std::string ltrim(const std::string& s)
{
    static const std::regex lws{"^[[:space:]]*", std::regex_constants::extended};
    return std::regex_replace(s, lws, "");
}

std::string rtrim(const std::string& s)
{
    static const std::regex tws{"[[:space:]]*$", std::regex_constants::extended};
    return std::regex_replace(s, tws, "");
}

std::string trim(const std::string& s)
{
    return ltrim(rtrim(s));
}

Tego używam. Po prostu usuwaj przestrzeń z przodu, a następnie, jeśli coś pozostało, zrób to samo z tyłu.

void trim(string& s) {
    while(s.compare(0,1," ")==0)
        s.erase(s.begin()); // remove leading whitespaces
    while(s.size()>0 && s.compare(s.size()-1,1," ")==0)
        s.erase(s.end()-1); // remove trailing whitespaces
}

s.erase(0, s.find_first_not_of(" \n\r\t"));                                                                                               
s.erase(s.find_last_not_of(" \n\r\t")+1);   

Dla tego, co jest warte, oto implementacja wykończenia z myślą o wydajności. Jest znacznie szybszy niż wiele innych procedur przycinania, które widziałem w pobliżu. Zamiast używać iteratorów i std :: find, używa surowych ciągów c i indeksów. Optymalizuje następujące przypadki specjalne: łańcuch o rozmiarze 0 (nic nie rób), łańcuch bez białych znaków do przycinania (nic nie rób), łańcuch z tylko końcowymi białymi znakami do przycinania (wystarczy zmienić rozmiar łańcucha), łańcuch całkowicie białą spacją (wystarczy wyczyścić łańcuch) . I wreszcie, w najgorszym przypadku (łańcuch z wiodącymi białymi znakami), robi wszystko, aby wykonać wydajną konstrukcję kopii, wykonując tylko 1 kopię, a następnie przenosząc tę ​​kopię w miejsce oryginalnego ciągu.

void TrimString(std::string & str)
{ 
    if(str.empty())
        return;

    const auto pStr = str.c_str();

    size_t front = 0;
    while(front < str.length() && std::isspace(int(pStr[front]))) {++front;}

    size_t back = str.length();
    while(back > front && std::isspace(int(pStr[back-1]))) {--back;}

    if(0 == front)
    {
        if(back < str.length())
        {
            str.resize(back - front);
        }
    }
    else if(back <= front)
    {
        str.clear();
    }
    else
    {
        str = std::move(std::string(str.begin()+front, str.begin()+back));
    }
}

Może to być elegancki sposób na zrobienie tego

std::string & trim(std::string & str)
{
   return ltrim(rtrim(str));
}

Funkcje pomocnicze są realizowane jako:

std::string & ltrim(std::string & str)
{
  auto it =  std::find_if( str.begin() , str.end() , [](char ch){ return !std::isspace<char>(ch , std::locale::classic() ) ; } );
  str.erase( str.begin() , it);
  return str;   
}

std::string & rtrim(std::string & str)
{
  auto it =  std::find_if( str.rbegin() , str.rend() , [](char ch){ return !std::isspace<char>(ch , std::locale::classic() ) ; } );
  str.erase( it.base() , str.end() );
  return str;   
}

A kiedy już to wszystko umieścisz, możesz również napisać:

std::string trim_copy(std::string const & str)
{
   auto s = str;
   return ltrim(rtrim(s));
}

Implementacja Trim C ++ 11:

static void trim(std::string &s) {
     s.erase(s.begin(), std::find_if_not(s.begin(), s.end(), [](char c){ return std::isspace(c); }));
     s.erase(std::find_if_not(s.rbegin(), s.rend(), [](char c){ return std::isspace(c); }).base(), s.end());
}

Myślę, że jeśli zaczniesz pytać o „najlepszy sposób” przycinania łańcucha, powiedziałbym, że dobrym rozwiązaniem byłoby:

1. Nie przydziela tymczasowych ciągów
2. Ma przeciążenia do przycinania na miejscu i kopiowania przycinania
3. Może być łatwo dostosowany do akceptowania różnych sekwencji / logiki sprawdzania poprawności

Oczywiście istnieje zbyt wiele różnych sposobów podejścia do tego i na pewno zależy to od tego, czego naprawdę potrzebujesz. Jednak biblioteka standardowa C nadal ma bardzo przydatne funkcje w <string.h>, takie jak memchr. Jest powód, dla którego C jest nadal uważany za najlepszy język dla IO - jego stdlib to czysta wydajność.

inline const char* trim_start(const char* str)
{
    while (memchr(" \t\n\r", *str, 4))  ++str;
    return str;
}
inline const char* trim_end(const char* end)
{
    while (memchr(" \t\n\r", end[-1], 4)) --end;
    return end;
}
inline std::string trim(const char* buffer, int len) // trim a buffer (input?)
{
    return std::string(trim_start(buffer), trim_end(buffer + len));
}
inline void trim_inplace(std::string& str)
{
    str.assign(trim_start(str.c_str()),
        trim_end(str.c_str() + str.length()));
}

int main()
{
    char str [] = "\t \nhello\r \t \n";

    string trimmed = trim(str, strlen(str));
    cout << "'" << trimmed << "'" << endl;

    system("pause");
    return 0;
}

Nie jestem pewien, czy twoje środowisko jest takie samo, ale w moim przypadku pusty ciąg znaków spowoduje przerwanie programu. Chciałbym owinąć to połączenie kasowania poleceniem if (! S.empty ()) lub użyć Boosta, jak już wspomniano.

Oto, co wymyśliłem:

std::stringstream trimmer;
trimmer << str;
trimmer >> str;

Ekstrakcja strumienia automatycznie eliminuje białe spacje, więc działa to jak urok.
Całkiem czysty i elegancki, jeśli sam to powiem. ;)

Przyczyniając się do rozwiązania mojego problemu z hałasem. trimdomyślnie tworzy nowy ciąg i zwraca zmodyfikowany trim_in_placeciąg, jednocześnie modyfikując przekazany ciąg. Ta trimfunkcja obsługuje semantykę przenoszenia c ++ 11.

#include <string>

// modifies input string, returns input

std::string& trim_left_in_place(std::string& str) {
    size_t i = 0;
    while(i < str.size() && isspace(str[i])) { ++i; };
    return str.erase(0, i);
}

std::string& trim_right_in_place(std::string& str) {
    size_t i = str.size();
    while(i > 0 && isspace(str[i - 1])) { --i; };
    return str.erase(i, str.size());
}

std::string& trim_in_place(std::string& str) {
    return trim_left_in_place(trim_right_in_place(str));
}

// returns newly created strings

std::string trim_right(std::string str) {
    return trim_right_in_place(str);
}

std::string trim_left(std::string str) {
    return trim_left_in_place(str);
}

std::string trim(std::string str) {
    return trim_left_in_place(trim_right_in_place(str));
}

#include <cassert>

int main() {

    std::string s1(" \t\r\n  ");
    std::string s2("  \r\nc");
    std::string s3("c \t");
    std::string s4("  \rc ");

    assert(trim(s1) == "");
    assert(trim(s2) == "c");
    assert(trim(s3) == "c");
    assert(trim(s4) == "c");

    assert(s1 == " \t\r\n  ");
    assert(s2 == "  \r\nc");
    assert(s3 == "c \t");
    assert(s4 == "  \rc ");

    assert(trim_in_place(s1) == "");
    assert(trim_in_place(s2) == "c");
    assert(trim_in_place(s3) == "c");
    assert(trim_in_place(s4) == "c");

    assert(s1 == "");
    assert(s2 == "c");
    assert(s3 == "c");
    assert(s4 == "c");  
}

Można to zrobić prościej w C ++ 11 dzięki dodaniu back()i pop_back().

while ( !s.empty() && isspace(s.back()) ) s.pop_back();

Oto moja wersja:

size_t beg = s.find_first_not_of(" \r\n");
return (beg == string::npos) ? "" : in.substr(beg, s.find_last_not_of(" \r\n") - beg);

Powyższe metody są świetne, ale czasami chcesz użyć kombinacji funkcji do tego, co rutyna uważa za spacje. W takim przypadku używanie funktorów do łączenia operacji może być nieporządne, dlatego wolę prostą pętlę, którą mogę zmodyfikować dla wykończenia. Oto nieco zmodyfikowana funkcja przycinania skopiowana z wersji C tutaj na SO. W tym przykładzie przycinam znaki nie alfanumeryczne.

string trim(char const *str)
{
  // Trim leading non-letters
  while(!isalnum(*str)) str++;

  // Trim trailing non-letters
  end = str + strlen(str) - 1;
  while(end > str && !isalnum(*end)) end--;

  return string(str, end+1);
}

Oto prosta implementacja. Do tak prostej operacji prawdopodobnie nie powinieneś używać żadnych specjalnych konstrukcji. Wbudowana funkcja isspace () zajmuje się różnymi formami białych znaków, dlatego powinniśmy z niej skorzystać. Musisz również wziąć pod uwagę specjalne przypadki, w których łańcuch jest pusty lub po prostu kilka spacji. Przycinanie w lewo lub w prawo można uzyskać z następującego kodu.

string trimSpace(const string &str) {
   if (str.empty()) return str;
   string::size_type i,j;
   i=0;
   while (i<str.size() && isspace(str[i])) ++i;
   if (i == str.size())
      return string(); // empty string
   j = str.size() - 1;
   //while (j>0 && isspace(str[j])) --j; // the j>0 check is not needed
   while (isspace(str[j])) --j
   return str.substr(i, j-i+1);
}

Oto rozwiązanie łatwe do zrozumienia dla początkujących, którzy nie używają pisma std::wszędzie i jeszcze nie znają constpoprawności, iterators, STL algorithmitp.

#include <string>
#include <cctype> // for isspace
using namespace std;


// Left trim the given string ("  hello!  " --> "hello!  ")
string left_trim(string str) {
    int numStartSpaces = 0;
    for (int i = 0; i < str.length(); i++) {
        if (!isspace(str[i])) break;
        numStartSpaces++;
    }
    return str.substr(numStartSpaces);
}

// Right trim the given string ("  hello!  " --> "  hello!")
string right_trim(string str) {
    int numEndSpaces = 0;
    for (int i = str.length() - 1; i >= 0; i--) {
        if (!isspace(str[i])) break;
        numEndSpaces++;
    }
    return str.substr(0, str.length() - numEndSpaces);
}

// Left and right trim the given string ("  hello!  " --> "hello!")
string trim(string str) {
    return right_trim(left_trim(str));
}

Mam nadzieję, że to pomoże...

Ta wersja przycina wewnętrzne białe znaki i znaki niealfanumeryczne:

static inline std::string &trimAll(std::string &s)
{   
    if(s.size() == 0)
    {
        return s;
    }

    int val = 0;
    for (int cur = 0; cur < s.size(); cur++)
    {
        if(s[cur] != ' ' && std::isalnum(s[cur]))
        {
            s[val] = s[cur];
            val++;
        }
    }
    s.resize(val);
    return s;
}

Jeszcze inna opcja - usuwa jedną lub więcej postaci z obu końców.

string strip(const string& s, const string& chars=" ") {
    size_t begin = 0;
    size_t end = s.size()-1;
    for(; begin < s.size(); begin++)
        if(chars.find_first_of(s[begin]) == string::npos)
            break;
    for(; end > begin; end--)
        if(chars.find_first_of(s[end]) == string::npos)
            break;
    return s.substr(begin, end-begin+1);
}