#include <iostream>

using namespace std;

void main ()

{
double nArt;
double a = 180-20-0076.5;

cout << "Art.Nr. eingben" <<endl;
cin >> nArt;

if (nArt == 'a')
cout << "nummer: 11517" <<endl;

else
cout << "Artikelnummer nicht vorhanden" <<endl;

}

Es soll ein vereinfachungsprogramm werden: Es soll eine Artikelzahl wie zb. 150-45-4847.4 eingegeben werden. daraufhin soll dann die Lagernummer erscheinen.
Leider habe ich Probleme mit der deklaration der art.zahl. Ist es überhaupt möglich dass ich so eine Zahl mit Bindestrichen eingeben kann? Müsste ich die Zahl irgendwie als Wort deklarieren, weil im prinzip ist es ja keine zahl mit (bindestriche)

Wenn ich jetzt ca. 100 artikelnummern habe mit den dazugehörigen Lagernummern macht da eine if/else schleife überhaupt sinn? Geht das nicht irgendwie anders??

Wär um jede hilfe dankbar.

Gruss Kuse

wie funzt das mit dem map? gibts das auch auf deutsch? was ist das?

Google. He? Google.

Mach aus dem double ein string.
Dann meinst du, du hast 100 Artikelnummer und dazu passende Lagernummer, da würd ich wohl map empfehlen.
Und dann noch etwas ganz schlaggebendes
ES GIBT KEINE IF-SCHLEIFEN!!!

Meinst du nicht es würde auch eine einfaches Array reichen?

FreakY<3Cpp schrieb:

Mach aus dem double ein string.
Dann meinst du, du hast 100 Artikelnummer und dazu passende Lagernummer, da würd ich wohl map empfehlen.
Und dann noch etwas ganz schlaggebendes
ES GIBT KEINE IF-SCHLEIFEN!!!

Meinst du nicht es würde auch eine einfaches Array reichen?

Und wie soll das Array aussehen?

string Numbers[MAX_NUMBERS][1];

Oder wie ?
Zu jeder Artikelnummer gibt es eine passende Lagernummer, warum sollte map da nicht passen?

Hier ist ein komplettes Beispielprogramm:

#include <iostream>
#include <string>
using namespace std;

#define MAX 5

struct
{
string Artikelnummer;
string Lagernummer;
}Elemente[MAX];

int main()
{
	string ArtNum;
	bool found = false;
	cout << "Artikelnummer: ";cin >> ArtNum;
	cout << "Lagernummer: ";

	for(int loop=0;loop<MAX;loop++)
	{
		if(Elemente[loop].Artikelnummer==ArtNum)
		{
			cout << Elemente[loop].Lagernummer;
			found = true;
		}
	}
	if(found==false)cout << "(nicht gefunden)" << endl;
	cin.get();
	return 0;
}

MAX kann beliebig gewählt werden. Das Programm ist so noch nicht lauffähig, da Die Elemente keinen Inhalt haben. (Läuft zwar schon, erfüllt aber keinen Zweck). In jedes Element muss also noch die Artikelnummer und die zugehörige Lagernummer geschrieben werden. Also ich finde diese Lösung auch ohne Benutzung von map ganz schön.

MfG, Jochen

#include <iostream>
#include <vector>
#include <string>
#include <utility>

int main()
{
    std::vector<std::pair<std::string, std::string> > Numbers;
    std::string ArtikelNummer;
    std::string LagerNummer;
    while(ArtikelNummer != "0" && LagerNummer != "0")
    {
        std::cin >> ArtikelNummer;
        std::cin >> LagerNummer;
        Numbers.push_back(make_pair(ArtikelNummer, LagerNummer));
    }
    for(unsigned int i = 0; i < Numbers.size() - 1; i++)
    {
        std::cout << Numbers[i].first << " " << Numbers[i].second << std::endl;
    }
}

art.nr. 105-10-2467.5 lagernummer: 2117
art.nr. 108-48-4757.8 lagernummer: 3456
art.nr. 140-47-4564.2 lagernummer: 4567

gruss

Elemente[0].Artikelnummer="105-10-2467.5";Elemente[0].Lagernummer="2117";
Elemente[1].Artikelnummer="108-48-4757.8";Elemente[1].Lagernummer="3456";
Elemente[2].Artikelnummer="140-47-4564.2";Elemente[2].Lagernummer="4567";

Das ist natürlich keine Optimale Lösung. Besser wäre es, wenn man die Werte aus einer Textdatei einlesen könnte. Werd dazu gleich mal was Posten.
MfG, Jochen

#include <string>
#include <iostream>
#include <fstream>
using namespace std;

struct SElem
{
string Artikelnummer;
string Lagernummer;
}*Elemente;

int main()
{
	ifstream dat_ein;
	dat_ein.open("datei.txt",ios_base::in);
	int Anzahl;
	dat_ein >> Anzahl;
	Elemente = new SElem[Anzahl];

	for(int loop=0;loop<Anzahl;loop++)
	{
		dat_ein >> Elemente[loop].Artikelnummer;
		dat_ein >> Elemente[loop].Lagernummer;
	}

	dat_ein.close();

	string ArtNum;
	bool found = false;
	cout << "Artikelnummer: ";cin >> ArtNum;
	cout << "Lagernummer: ";

	for(int loop=0;loop<Anzahl;loop++)
	{
		if(Elemente[loop].Artikelnummer==ArtNum)
		{
			cout << Elemente[loop].Lagernummer;
			found = true;
			break;
		}
	}
	if(found==false)cout << "(nicht gefunden)" << endl;
	cin.get();
    delete[] Elemente;
	return 0;
}

Der Inhalt der Datei:

3
105-10-2467.5	2117
108-48-4757.8	3456
140-47-4564.2	4567

Die erste Zahl in der Textdatei zeigt dem Programm, wie viele Datensätze in der Datei enthalten sind. Danach kommen die Daten: Erst die Artikelnummer, dann die Lagernummer.

MfG, Jochen

Wie das genau geht, überlasse ich mal interessierten Lesern als Übung. Ist ganz einfach.

MFG

MfG, Jochen

#include <string>
#include <iostream>
#include <fstream>
#include <vector>
using namespace std;

struct SElem
{
string Artikelnummer;
string Lagernummer;
};

int main()
{
	ifstream dat_ein;
	dat_ein.open("datei.txt",ios_base::in);

	vector<SElem> Elemente;

	while(!dat_ein.eof())
	{
		SElem tmp;
		dat_ein >> tmp.Artikelnummer;
		dat_ein >> tmp.Lagernummer;
		Elemente.push_back(tmp);
	}

	int Anzahl = Elemente.size();

	dat_ein.close();

	string ArtNum;
	bool found = false;
	cout << "Artikelnummer: ";cin >> ArtNum;
	cout << "Lagernummer: ";

	for(int loop=0;loop<Anzahl;loop++)
	{
		if(Elemente[loop].Artikelnummer==ArtNum)
		{
			cout << Elemente[loop].Lagernummer;
			found = true;
			break;
		}
	}
	if(found==false)cout << "(nicht gefunden)" << endl;
	cin.get();
	for(int loop=0;loop<Anzahl;loop++)Elemente.erase(0);
	return 0;
}

Der Inhalt der Datei müsste jetzt so aussehen:
(So wie vorher, nur ohne die errste Zeile)

105-10-2467.5	2117
108-48-4757.8	3456
140-47-4564.2	4567

Stimmt, die Lösung ist schöner. Ich weiss zwar nicht, wie vector intern arbeitet, aber führt das nicht zu einer starken Heapfragmentierung, wenn immer Speicher für ein weiteres Element reserviert werden muss, wenn ein neues Element hinzugefügt wird?

MfG, Jochen

Ich weiss zwar nicht, wie vector intern arbeitet

http://www.sgi.com/tech/stl/Vector.html

[3] When it is necessary to increase capacity(), vector usually increases it by a factor of two. It is crucial that the amount of growth is proportional to the current capacity(), rather than a fixed constant: in the former case inserting a series of elements into a vector is a linear time operation, and in the latter case it is quadratic.

Und ueber Fragmentierung rede ich erst, wenn viel Speicher benoetigt wird. Desweiteren kann das Suchen auch beim Einlesen der Elemente stattfinden, dann spart man sich das Array. Auch das manuelle Loeschen der Elemente am Ende ist ueberfluessig, das passiert automatisch, wenn der Destruktor von std::vector aufgerufen wird. Ich weiss jetzt nicht genau, was erase macht, aber wenn von vorn geloescht wird, dann muss der restliche Inhalt staendig um eins nach vorne kopiert werden. Besser man beginnt hinten ...

Stimmt, das mit dem löschen ist nicht ganz so optimal. Noch einfacher ist es, wenn man gleich den Destruktor "manuell" aufruft. Oder sehe ich da etwas falsch?

Faustregel: Man ruft NIEMALS Destruktoren manuell auf. Destruktoren werden automatisch aufgerufen, wenn der Programmfluss den Gültigkeitsbereich eines Objekts verlässt.

Noch einfacher ist es, wenn man gleich den Destruktor "manuell" aufruft.

Das will ich sehen! :p

Jochen S. schrieb:

Noch einfacher ist es, wenn man gleich den Destruktor "manuell" aufruft.

Das will ich sehen! :p

Probiers aus. Es ruft den Destruktor auf, aber das Objekt ist weiterhin zugänglich.

class MyClass
{
public:
	MyClass() { std::cout << "Objekt erstellt" << std::endl ; }
	~MyClass() { std::cout << "Objekt gelöscht" << std::endl ; }
	void DoSomething() { std::cout << "Objekt ruft DoSomething() auf" << std::endl ; }
};

int main ( )
{
	MyClass Obj ;
	Obj.~MyClass();
	Obj.DoSomething() ;
}

knivil schrieb:

Auch das manuelle Loeschen der Elemente am Ende ist ueberfluessig, das passiert automatisch, wenn der Destruktor von std::vector aufgerufen wird.

Da dachte ich, dass man ja auch den Destruktor manuell aufrufen könne. In dem Fall würde also tatsächlich der Speicher freigegeben. Ein Weiterarbeiten mit dem Objekt wäre in dem Falle nicht mehr möglich.

Ein Weiterarbeiten mit dem Objekt wäre in dem Falle nicht mehr möglich.

Doch, aber mit undefiniertem Verhalten zur Laufzeit. Und das ist ja das Schlimme. pumuckl hat völlig Recht. Ausser bei seltenen Spezialfällen, wenn man auch Placement New verwendet, ruft man nie Destruktoren manuell auf. Entweder wird das Objekt über delete oder automatisch zerstört.

Falls man vorzeitig den Speicher freigeben möchte, was meistens keine gute Idee ist, kann man immer noch den swap() -Trick anwenden:

std::vector().swap(my_vector);

Übrigens: Äusserst kreativer Threadtitel.

ich muss zugeben ich bin sprachlos, und irgendwie ist das mir auch ein wenig zu hoch geworden ich kenn micht nicht mehr aus.
Würde dieses Programm gern lauffähig machen. beim ausführen bekomme ich immer diese Fehlermeldung:

Kompilierung läuft...
Lagersystem.cpp
C:\Dokumente und Einstellungen\Markus\Desktop\Lagersystem\Lagersystem.cpp(48) : error C2374: 'loop' : Neudefinition; Mehrfachinitialisierung
C:\Dokumente und Einstellungen\Markus\Desktop\Lagersystem\Lagersystem.cpp(37) : Siehe Deklaration von 'loop'
Fehler beim Ausführen von cl.exe.
EInlaufkurven.obj - 1 Fehler, 0 Warnung(en)

bin ratlos!!!!!!!!!!!!!
Hab ich aber das Prinzip schon richtig geschnallt, dass die 2 Lösung besser auf Änderungen in der Liste reagiert. dh. wenn zb. eine neue art.nummer + Lagernummer hinzukommt,könnte ich die einfach dahinter anreihen.
Beim nächsten ausführen wird diese Zahl mit ins prog integriert.!!!!!??????

Gruss

int main()
{
int loop;
// ... 
}

In den For-Schleifen müsstest du dann immer anstatt int loop = ... einfach nur loop = ... schreiben. Aber für konkrete Vorschläge müsste ich Einsicht in den Quelltext haben.

MfG, Jochen