ich hab hier grad ein Problem: Ich habe hier eine Split-Funktion, die jedoch leider nicht ganz will und ich komm noch nicht dahinter, wie ich das ändern kann.

std::vector<std::string> splittolines(std::string text)
{
  std::vector<std::string> erg;
  const char* newline = "\n";

  for (size_t i = 0; i < text.length(); i++)
  {
    std::string s;
    if (text.at(i) == newline)
    {
      erg.push_back(s);
    }
    else
    {
      s.append(text.at(i));
    }
  }

  return erg;
}

Fehlermeldung:

Functions.cpp: In function ‘std::vector<std::basic_string<char> > splittolines(std::string)’:
Functions.cpp:71:23: error: ISO C++ forbids comparison between pointer and integer
Functions.cpp:77:26: error: invalid conversion from ‘char’ to ‘const char*’

So, ich hab gelesen, dass ich dieses at() nicht mit einem chat vergleichen kann. Was kann ich aber machen, damit ich so einen Vergleich machen kann?

const char newline = '\n';

Functions.cpp: In function ‘std::vector<std::basic_string<char> > splittolines(std::string)’:
Functions.cpp:77:26: error: invalid conversion from ‘char’ to ‘const char*’

Zeig den gesamten Code der den Fehler verursacht - wer weiss was Du geändert hast...

std::vector<std::string> split(std::string text, char delim)
{
    std::stringstream ss(std::move(text));
    std::vector<std::string> result;

    while(std::getline(ss, text, delim))
        result.push_back(std::move(text));

    return result;
}

Ich brauch die Lösung für dieses Problem noch in einer anderen Funktion. Deshlab wäre es gut, wenn ich die Lösung irgendwie so hinbekomme.
Im schlimmsten Fall müsste ich zuviel neu schreiben.

std::vector<std::string> splittolines(std::string text)
{
  std::vector<std::string> erg;
  char newline = '\n';

  for (size_t i = 0; i < text.length(); i++)
  {
    std::string s;
    if (text.at(i) == newline)
    {
      erg.push_back(s);
    }
    else
    {
      s.append(text.at(i));
    }
  }

  return erg;
}

s += text[i];

daraus.

@314159265358979:

Ich brauch die Lösung für dieses Problem noch in einer anderen Funktion. Deshlab wäre es gut, wenn ich die Lösung irgendwie so hinbekomme.
Im schlimmsten Fall müsste ich zuviel neu schreiben.

What?

Mach doch ein

s += text[i];

daraus.

Ändert doch nichts an der Tatsache, dass ich hier

if (text.at(i) == newline)

ein Problem hab ...

Und nimm 314159265358979s Vorschlag, der ist in jeder Hinsicht besser. Dein Argument verstehe ich überhaupt nicht (sowohl sprachlich als auch das was ich mir inhaltlich zusammen reime, was du wohl meinst):

Ich brauch die Lösung für dieses Problem noch in einer anderen Funktion. Deshlab wäre es gut, wenn ich die Lösung irgendwie so hinbekomme.
Im schlimmsten Fall müsste ich zuviel neu schreiben.

s += text[i];

template <typename C, typename T, typename A>
std::vector<std::basic_string<C, T, A>> split(std::basic_string<C, T, A> text, C delim)
{
    std::basic_stringstream<C, T, A> ss(std::move(text));
    std::vector<std::basic_string<C, T, A>> result;

    while(std::getline(ss, text, delim))
        result.push_back(std::move(text));

    return result;
}

Functions.cpp: In function ‘std::vector<std::basic_string<char> > split(std::string, char)’:
Functions.cpp:67:26: error: ‘move’ is not a member of ‘std’
Functions.cpp:71:26: error: ‘move’ is not a member of ‘std’
make: *** [Functions.o] Error 1

C++0x musst du auch aktivieren, ansonsten die moves weglassen. (Dadurch wird der Code aber ineffizienter.)

const std::string&

ändern.

Edit: Unter Visual Studio ist C++0x übrigens von Haus aus aktiviert, wenn du die 10er Version hast.

std::vector<std::string> split(const std::string& text, char delim) 
{ 
    std::stringstream ss(text); 
    std::vector<std::string> result; 

    while(std::getline(ss, text, delim)) 
        result.push_back(text); 

    return result; 
}

Functions.cpp: In function ‘std::vector<std::basic_string<char> > split(const std::string&, char)’:
Functions.cpp:69:39: error: no matching function for call to ‘getline(std::stringstream&, const std::string&, char&)’
make: *** [Functions.o] Error 1

std::vector<std::string> split(const std::string& text, char delim)
{
    std::stringstream ss(text);
    std::vector<std::string> result;

    std::string substr;
    while(std::getline(ss, substr, delim))
        result.push_back(substr);

    return result;
}

Um nochmal meinen Edit zu wiederholen, falls du ihn überliest: Wenn du MSVC 10 verwendest, ist C++0x von Haus aus aktiviert.

Um nochmal meinen Edit zu wiederholen, falls du ihn überliest: Wenn du MSVC 10 verwendest, ist C++0x von Haus aus aktiviert.

Ich hab hier ein Projekt, dass ich mit Linux machen soll...

314159265358979 schrieb:

Um nochmal meinen Edit zu wiederholen, falls du ihn überliest: Wenn du MSVC 10 verwendest, ist C++0x von Haus aus aktiviert.

Ich hab hier ein Projekt, dass ich mit Linux machen soll...

Alle gängigen Linuxcompiler kennen auch sehr viel von C++11 (tendenziell sogar viel mehr als MSVC). Gib mal -std=c++0x beim compilieren mit an.