#include <iostream>
#include <string>
#include <sstream>

using namespace std;

int main()
{
    string s = "LOL";
    stringstream ss(s);

    int value = 0;

    if(!ss >> value)
        return 1;

    cout << value;
}

Er gibt true zurück und value ist immer noch 0.

So geht es...

if(ss >> value)
        cout << value;
    else
        return 1;

Das macht einen bitwise right shift und zwar auf 0.

Vorschlag:

std::string const stringval = "12345";
int intval;

if(std::istringstream(stringval) >> intval)
    // yay
else
    // oh noes

if(!(ss >> value))

Bei dem vorherigen

if(!ss >> value)

wird zuerst !ss ausgewertet, was false ergibt, da der Stream zu dem Zeitpunkt nicht in einem Fehlerzustand ist. false als Integer hat den Wert 0, dieser wird dann um 0 nach rechts geshiftet, was immer noch 0 ist und entsprechend schlägt die if-Bedingung nicht an. Ebenso wurde an value nichts geändert, es wurde bloß sein Wert benutzt.

@realKoala: Das ist totaler Schwachsinn.

Weil..

• In dem String gar keine Zahl drin ist und
• Vergessen wurde '0' abzuziehen

Zur Erläuterung: In der ASCII-Tabelle lässt sich erkennen, dass die Ziffern 0 bis 9 direkt hintereinder liegen. Mit

string s = "123";
cout << int(s[0]);

erreicht man den ASCII-Tabellenwert von '1', also 49. Zieht man nun den Wert von '0' ab, also 48, erhält man 49-48, also 1. Das gilt dann logischerweise auch für die anderen Ziffern.
So erhält man also die Werte, blöd nur, das man die nicht einfach so in einen int reinschreiben kann (1+2+3+4 = 10, nicht 1234). Man macht also
10 * 0 + 1 = 1
10 * 1 + 2 = 12
10 * 12 + 3 = 123
...

Das ganze dann noch in eine Funktion verpacken, und so siehts dann aus:

#include <iostream>

int stringZuInt(const std::string& s) //als Referenz um nicht kopieren zu müssen, const um zu sagen,
{                                   //dass nichts verändert wird
    int zahl = 0;
    for(unsigned i = 0; i < s.length(); ++i)
        zahl = (zahl * 10) + (s[i] - '0'); //wie besprochen
    return zahl;
}

int main()
{
    std::cout << stringZuInt("321");
}

Dann weiß man wenigstens, was hinter den Kulissen passiert.

Um mal Butter bei die Fische zu tun und das ganze Halbwissen im Thread besser zu erklären:

[...]

Aber am Ergebnis würde das doch nichts ändern, weil in dem String keine Zahl steht?!

SeppJ schrieb:

Um mal Butter bei die Fische zu tun und das ganze Halbwissen im Thread besser zu erklären:

[...]

Aber am Ergebnis würde das doch nichts ändern, weil in dem String keine Zahl steht?!

Doch! Das eine Mal bekommt man 0, also false, das andere Mal !0, also true! Also gegenteiliges Ergebnis und das ist auch, was den Threadersteller überraschte. Außerdem passiert etwas gänzlich anderes als erwartet, da überhaupt keine Leseaktion versucht wird, sondern stattdessen ein bisschen das Integerrechenwerk belastet wird!

Der Unterschied ist also ganz fatal!

er hat absichtlich ASCII genommen fällt dir das nicht irgendwie auf??????? omfg

deine func macht absolut keinen sinn

In C++11 gibt es übrigens std::stoi dafür. Das wirft in dem Fall einfach eine Exception.

Die trauen sich jetzt sogar einfach so Exceptions zu werfen.