Ampel_OOP

remo.struss

Hallo Leute, wäre sehr froh wenn mich jemand ein bisschen unterstützen könnte oder wenigstens einen guten ratschlag geben könnte wie ich beim folgenden programm code weiter komme.
besten dank für eure hilfe.

//Ampel.cpp - Projekt_Ampel.cpp

#include <time.h> // Include der Header-Datei time.h
#include <stdio.h> // Include der Header-Datei stdio.h
#include <stdlib.h> // Include der Header-Datei stdlib.h
#include <string.h> // Include der Header-Datei string.h
#include <iostream.h> // Include der Header-Datei iostream.h
#include <malloc.h> // Include der Header-Datei malloc.h
#include <windows.h>

class CAmpel
{
public:
int rotlicht;
int gruenlicht;

void rotlicht_ein_aus(); // Deklaration der Methoden
void gruenlicht_ein_aus();

};

class CStrassenampel //:: public CAmpel
{
public:
CStrassenampel() {};

int gelblicht;

// CStrassenampel();
~CStrassenampel();

void gelblicht_ein_aus();

};

CStrassenampel::CStrassenampel()
{
rotlicht = 1;
gruenlicht = 0;
gelblicht = 0;
}

CStrassenampel::~CStrassenampel()
{
}

void Strassenampel::rotlicht_ein_aus()
{
if(rotlicht == 0) {
rotlicht = 1;
}
else {
rotlicht = 0;
}

class CFussgaengerampel
{
public:
char * buzzer;

Fussgaengerampel();
~Fussgaengerampel();

void buzzer_ein_aus();
}

void main (void) // Hauptprogramm
{

CStrassenampel Strassenampel();

system("CLS");
char ein_aus, j;

int Auswahl;
cout << "Ich begruesse Sie ganz herzlich" << endl;
cout << "zu diesem Programm der Lagerverwaltung" << endl;

while (ein_aus == j)
{
cout << endl;

cout << "Was wuenschen Sie zu tun?" << endl;

cout << endl;
cout << "Strassenampel" << endl;
cout << "[1] Rotlicht" << Strassenampel.rotlicht << endl;
cout << "[2] Gelblicht" << Strassenampel.gelblicht << endl;
cout << "[3] Gruenlicht" << Strassenampel.gruenlicht << endl;
cout << endl;
cout << "Fussgaengerampel" << endl;
cout << "[4] Rotlicht" << Fussgaengerampel.rotlicht << endl;
cout << "[5] Gruenlicht" << Fussgaengerampel.gruenlicht << endl;
cout << "[6] Buzzer" << Fussgaengerampel.buzzer << endl;

cout << "[7] Programm beenden" << endl;
cout << endl;
cout << "Zahl in Klammer eingeben und mit Enter bestaetigen!" << endl;
cin >> Auswahl;

switch (Auswahl)
{
case 1: Strassenampel.rotlicht_ein_aus();
break;
case 2: Strassenampel.gelblicht_ein_aus();
break;
case 3: Strassenampel.gruenlicht_ein_aus();
break;
case 4: Fussgaengerampel.rotlicht_ein_aus();
break;
case 5: Fussgaengerampel.gruenlicht_ein_aus();
break;
case 6: Fussgaengerampel.buzzer_ein_aus();
break;
case 7: Programm beenden();
cout << "Wir danken Ihnen fuer die Benutzung unseres Programms" << endl;
cout << "und wuenschen Ihnen einen schoenen Tag!" << endl;
exit(0);
break;
default:
cout << "Falsche Eingabe! Bitte wiederholen! " << endl; // Textausgabe wen falsche Eingabe
break;
}
}
}

frosty

Frage:

Was willst Du wissen?

Aber schonmal: Du solltest int main() verwenden statt void main().

volkard

frosty schrieb:

Frage:
Was willst Du wissen?

bitte ankreuzen

was eine ampel ist (das ding hängt) und was eine straßenverkehrslichtzeichenanlage ist (das ding regelt den straßenverkehr, inzwischen eher selten als ampel realisiert).

warum man klassen mit C am anfang schreibt

warum exit(0) besser als return ist

wie er sich die improved console saugt und farbe ins spiel bringt

otze

wieso man cpp tags benutzen sollte

phlox81

[] wieso iostream.h wirklich böse ist

groovemaster

[] wieso man keine aktuellen Headerdateien verwenden sollte
[] wieso ein Semikolon ans Ende einer Funktionsdefinition hingehört
[] wieso man system("CLS") zum Leeren der Konsole verwenden sollte
[] wieso man unerreichbaren Code in sein Programm einbauen sollte

Walli

[] wieso man überflüssigerweise die Includes kommentiert, aber sonst nichts.

golden_jubilee

[] wieso rotlicht,gruenlicht und gelblicht vom typ int sind, oder können die noch mehr als an oder aus sein?

Walli

steff3 schrieb:

[] wieso rotlicht,gruenlicht und gelblicht vom typ int sind, oder können die noch mehr als an oder aus sein?

kaputt

golden_jubilee

Walli schrieb:

steff3 schrieb:

[] wieso rotlicht,gruenlicht und gelblicht vom typ int sind, oder können die noch mehr als an oder aus sein?

kaputt

ist aber nicht vorgesehen im Programm

jetzt weis er fast alles was er anders ausführen sollte.
aber das solltet ihr doch wissen: c-class und 'include'.h == der quellcode
wurde für einen älteren microsoft-compiler geschrieben.
@volkard wieso console, wenn windows.h?

@r.s schau mal in windows(.h) was dein file schon alles mitincludet!
mach mal eine tabelle, etwa so einmündung 1 = rot, einm... 2 = grün u.s.w.
dann überleg wenn 'ampel.usa' nicht kaputt und einfachkreuzung:
einm... 1 = einm... gegenüber. vergiss nicht für kurze zeit sollte meist
alles rot oder gelb sonst crash.

mfg stuss

Vorden

würde das problem eigentlich einfach mit nem enum angehen, dann hat er keine problem so, wie:

enum ampel{rot,gelb,gruen};

damit würde er zwar nur jeweils eine ampelform abdecken, finds aba schoener, als alles einzeln zu schreiben

volkard

stuss schrieb:

@volkard wieso console, wenn windows.h?

konsole, weil er noch anfänger ist.
<windows.h> für Sleep.