atoi() Frage

Kóyaánasqatsi

Was soll'n das werden? oO (Habe nicht den kompletten Thread durchgelesen)

n00b42

Kóyaánasqatsi schrieb:

Was soll'n das werden? oO (Habe nicht den kompletten Thread durchgelesen)

Naja, an den Leerzeichen wird der String in Teile geschnitten und dann in int's übersetzt und ausgegeben.
Versuchs nachzuvollziehen.

SeppJ

n00b42 schrieb:

Naja, an den Leerzeichen wird der String in Teile geschnitten und dann in int's übersetzt und ausgegeben.
Versuchs nachzuvollziehen.

Und was war jetzt nochmal der tiefere Grund, warum du das so semi-manuell machen willst, anstatt stringstream zu benutzen, das optimal für diesen Zweck geeignet ist?

Was passt dir an adatcas Ansatz nicht???

volkard

n00b42 schrieb:

Geht es etwa gar nich mit atoi(), ich habs jetzt so geändert:

Doch, klar. Aber wenn Du mehrere einliest, sind streams doch angemessener, oder?

n00b42

ja sicher aber was stimmt dadran nich???

#include <iostream>
#include <string>
#include <cstdio>
#include <stdlib.h>

using namespace std;
int main ()
{
    string text;
    string zeichen[100];
    int posi[100];
    int lustig[100];
    posi[1]=0;
    int loop;
    getline(cin,text);
for (loop=2;loop<10;++loop)
{
    posi[loop]=text.find(" ",posi[(loop-1)]);                    
    zeichen[loop]=text.substr(posi[(loop-1)],posi[loop]);        
    cout<<zeichen[loop]<<endl;                               
    lustig[loop]=atoi(zeichen[loop].c_str());
    cout<<lustig[loop];
}
return 0;
}

n00b42 schrieb:

ja sicher aber was stimmt dadran nich???

#include <iostream>
#include <string>
#include <cstdio>
#include <stdlib.h>

using namespace std;
int main ()
{
    string text;
    string zeichen[100];
    int posi[100];
    int lustig[100];
    posi[1]=0;
    int loop;
    getline(cin,text);
for (loop=2;loop<10;++loop)
{
    posi[loop]=text.find(" ",posi[(loop-1)]);                    
    zeichen[loop]=text.substr(posi[(loop-1)],posi[loop]);        
    cout<<zeichen[loop]<<endl;                               
    lustig[loop]=atoi(zeichen[loop].c_str());
    cout<<lustig[loop];
}
return 0;
}

Hi,

dein Code ist total unlesbar. Auch die Einrückung ist dir nicht wirklich gut gelungen
Außerdem ist dein Ansatz zur Lösung des Problems nicht sonderlich schön, da du es dir viel zu kompliziert machst.

So könnte man es machen, aber das empfinde ich als total umständlich:

#include <iostream>
#include <string>

int main()
{
	std::string text; // Der eigentliche Text

	std::getline(std::cin, text); // einlesen

	std::string temp;

	for (unsigned int i = 0; i < text.length() + 1; ++i)
	{
		if (text[i] == ' ' || text.length() == i)
		{
			std::cout << atoi(temp.c_str()) << std::endl;
			temp = "";
		}
		else
		{
			temp += text[i];
		}
	}

	return 0;
}

Folgender Code macht das gleiche, aber viel besser:

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

int main()
{
	std::string text; // Der eigentliche Text

	std::getline(std::cin, text); // einlesen

	int number;

	std::stringstream strm(text);

	while (strm >> number)
	{
		std::cout << number << std::endl;
	}

	return 0;
}

Und dazu ergänzend ein Beispiel aus der STL Doku von sgi, das den istream_iterator nutzt (http://www.sgi.com/tech/stl/istream_iterator.html) und dann man hat die Werte auch in einem vector:

vector<int> V;
copy(istream_iterator<int>(strm), istream_iterator<int>(), back_inserter(V));

n00b42

langsam hab ich das Gefühl ich solte von Dev-C++ 4.9.9.2 auf was andres umsteigen.
Kann mir da jemand einen Tipp geben?

volkard

n00b42 schrieb:

langsam hab ich das Gefühl ich solte von Dev-C++ 4.9.9.2 auf was andres umsteigen.
Kann mir da jemand einen Tipp geben?

Code::Blocks natürlich. Oder falls Du einen brutal schnellen Rechner hast eine Java-IDE wie Netbeans oder Eclipse.

Und hier noch ein Beispiel ohne streams mit einem formschönen C Style Cast:

void example ( char* s )
{
	while (*s)
		printf("%ld\n", strtoul ( s, &s, 10 ) );
}

int main() 
{ 
    string text("1 12 123"); 
	example ( (char*)text.c_str() );
    return 0; 
}