#include <string>
#include <algortihm>
#include <clocale>

bool is_integer(std::string const & str)
{
    return std::count_if(str.begin(), str.end(), &isdigit) == str.size();
}

?

Erfahrungsgemäß muss ich aber noch anmerken, dass dies so klingt, als wolltest du etwas ganz einfaches auf umständliche Weise erreichen. Nutzereingaben prüfen geht auch direkter. Also: Was willst du überhaupt erreichen? Beschreib das Ziel, nicht nur wie du dir die Umsetzung vorstellst.

Wieso muss das nicht std::isdigit heißen?

bool is_integer(std::string const & str)
{
    return std::count_if(str.begin(), str.end(), &isdigit) == str.size();
}

Wieso muss das nicht std::isdigit heißen?

Naja, eigentlich wäre das besser. Da brauchst Du dann aber noch ein std::locale-Objekt, und das bekommt man dann nicht mehr so schön mit einem Einzeiler hin. So ist es die C-Version von isdigit im globalen Namensraum.

out schrieb:

Wieso muss das nicht std::isdigit heißen?

Naja, eigentlich wäre das besser. Da brauchst Du dann aber noch ein std::locale-Objekt, und das bekommt man dann nicht mehr so schön mit einem Einzeiler hin.

aber sicher doch:

std::string wort="12345";
    if( std::all_of( begin(wort), end(wort), []( char c ) { return std::isdigit( c, std::locale("") ); } ))
        std::cout << "sind alles Ziffern" << std::endl;

Tachyon schrieb:

out schrieb:

Wieso muss das nicht std::isdigit heißen?

Naja, eigentlich wäre das besser. Da brauchst Du dann aber noch ein std::locale-Objekt, und das bekommt man dann nicht mehr so schön mit einem Einzeiler hin.

aber sicher doch:

std::string wort="12345";
    if( std::all_of( begin(wort), end(wort), []( char c ) { return std::isdigit( c, std::locale("") ); } ))
        std::cout << "sind alles Ziffern" << std::endl;

Geht bei mir nicht. Ich hab' kein C++11.

[]( char c ) { return std::isdigit( c, std::locale("") ); }

Seit es Lambdas gibt, scheint man bind vergessen zu haben.

std::bind(std::isdigit, _1, std::locale())

Wahlweise natürlich auch boost::bind .

Seit es Lambdas gibt, scheint man bind vergessen zu haben.

bind gibt es so richtig offiziell ja auch erst, seit es lambdas gibt . Die Erinnerung an bind1st, not2, const_mem_fun1_ref_t & Co. hat aber wohl jeder schon aus gutem Grund verdrängt.

"465   876  783"

ist doch auch ein String, der nur ganze Zahlen enthält ... ?!

Ähhhm ...
das hier:

"465   876  783"

ist doch auch ein String, der nur ganze Zahlen enthält ... ?!

Nope. Streng genommen enthält der neben ganzen Zahlen auch Leerzeichen/Whitespaces.

ipsec schrieb:

Seit es Lambdas gibt, scheint man bind vergessen zu haben.

Ich persoenlich finde Lambdas einfach wesentlich lesbarer als bind. Liegt vielleicht auch an meiner Erfahrung mit anderen Sprachen, aber die Lambdas in C++11 sind einer der Gruende, warum ich jetzt doch manchmal wieder mehr C++ als unbedingt noetig verwende.

Belli schrieb:

Ähhhm ...
das hier:

"465   876  783"

ist doch auch ein String, der nur ganze Zahlen enthält ... ?!

Nope. Streng genommen enthält der neben ganzen Zahlen auch Leerzeichen/Whitespaces.

Naja, aber streng genommen ist

"456"

ein String, der nur eine ganze Zahl enthält ...

std::string str("6846");
if(str.find_first_not_of("0123456789") == std::string::npos)
     std::cout << "Nur ganze Zahlen!";

Das Problem mit den Leerzeichen lässt sich auch ganz einfach einfügen (durch ein zusätzliches Leerzeichen im Parameter von find_... ).

Und was nman meint, da kann man nur noch vom Stream einlesen und diesen nach Fehlern abfragen.
BTW: Ganze Zahlen bestehen auch aus den negativen - ihr meint wohl die Menge $N_0$ (Das spezielle N \N geht nicht...).

Wieso hat noch keiner

std::string str("6846");
if(str.find_first_not_of("0123456789") == std::string::npos)
     std::cout << "Nur ganze Zahlen!";

Weil das wieder eine ineffiziente Unsinnslösung ist. Siehe mein Kommentar gestern zu deinem Vorschlag für den armen Kerl mit der Laufschrift. Immerhin hast du dieses Mal wenigstens eine höhere Abstraktionsebene erreicht, da der Code auch zur Überprüfung anderer Kriterien geeignet ist. Dafür hast du nun aber Laufzeitkosten auf dich genommen. Tachyons Lösung ist in dem gleichen Stil wie deine, ist aber effektiv das gleiche wie eine naive Prüfschleife elegant geschrieben.

Sone schrieb:

Wieso hat noch keiner

std::string str("6846");
if(str.find_first_not_of("0123456789") == std::string::npos)
     std::cout << "Nur ganze Zahlen!";

Weil das wieder eine ineffiziente Unsinnslösung ist. Siehe mein Kommentar gestern zu deinem Vorschlag für den armen Kerl mit der Laufschrift. Immerhin hast du dieses Mal wenigstens eine höhere Abstraktionsebene erreicht, da der Code auch zur Überprüfung anderer Kriterien geeignet ist. Dafür hast du nun aber Laufzeitkosten auf dich genommen. Tachyons Lösung ist in dem gleichen Stil wie deine, ist aber effektiv das gleiche wie eine naive Prüfschleife elegant geschrieben.

Ja, hast Recht. Meine ist ca. 1,5 mal langsamer als seine...

Laufzeitkosten

Wenn es danach geht wuerde ich ja das so machen:

bool number_string1(char const* p, int length)
{
    return (length==0) ? true : (isdigit(*p) || isspace(*p)) && number_string1(++p, --length);
}

Leider schafft es Visual Studio nicht zu optimieren, deswegen muss ihm etwas unter die Arme gegriffen werden:

bool number_string2(char const* p, int length)
{
    if (length==0) return true;
    if (isdigit(*p) || isspace(*p)) return number_string2(++p, --length);
    return false;
}

SeppJ schrieb:

Laufzeitkosten

Wenn es danach geht wuerde ich ja das so machen:

Na bitte! So sieht eine interessante neue Lösung aus. Hast du das mal mit Tachyons Variante (mit isdigit und isspace mittels Lambda oder Funktor kombiniert) verglichen?

Um mal eine Lanze fuer Sone zu brechen: Seine Loesung finde ich okay.

ich schreibe ein elektronisches adressbuch und befasse mich gerade mit der fehlermeldung...um das problem zu lösen das bei der plz genau 5 zahlen eingegeben werden sollen habe ich das so gelößt

struct Adresse
    string postleitzahl
.........................
char a;
Adresse a;
cout << "postleitzahl:";
cin >> a.postleitzahl;
     while(a.postleitzahl.length() != 5)
     { 
      cout << "falsch\n";
      cout << "postleitzahl:";
      cin >> a.postleitzahl;
      }
..............
datenbank.push_back(a);

ich frage solange die plz ab bis der string 5 zeichen enthält klappt auch alles sehr gut ich will das selbe jetzt noch einbauen das wenn man statt einer zahl, ein buchstaben eingibt das er auch ne fehlermeldung raus schmeißt und so lange in der while schleife bleibt....