Source unnötig lang?

[kasten23]

Hallo, wollte eigentlich nur fragen, ob man meine Sourcen irgendwie verkürzen könnte oder so etwas in der Art
Also, das habe ich jetzt ein wenig zum üben geschrieben.
main.cpp

#include <iostream>
#include <Windows.h>
#include "kampf.h"
#include "kopfzahl.h"

int main(){

kopfoderzahl();
kampf();

getchar();
return 0;
}

kampf_ele.h

bool auswahl;
int Trank = 20;
int WinLose;
int sleep = 2500;
int HeroTrankanzahl=1;
int GegTrankanzahl=1;
int GegHP = 100;
int HeroHP = 100;
int GegATK;
int HeroATK;

kampf.cpp

#include <iostream>
#include <Windows.h>
#include <time.h>
#include "kampf.h"
#include "kampf_ele.h"
#include "kopfzahl.h"
//-----KAMPF-----\\


void kampf(){
do{
Hero();
Gegner();
}while(WinLose==0);
}


void Hero(){
srand(time(NULL));
HeroATK =  rand() % (13 - 9 + 1) + 9;

std::cout << "Deine Hp: " << HeroHP <<"	Gegner HP: "  <<GegHP<<std::endl;
std::cout << " 0 = Angreifen	1 = Trank nehmen" <<"("<<HeroTrankanzahl<<")"<<std::endl;
std::cin >> auswahl;

//Hero's Zug
if (auswahl==true){
	if(HeroTrankanzahl==1){
HeroHP += Trank;
HeroTrankanzahl--;
std::cout << "Du hast einen Trank genommen und hast jetzt " << HeroHP <<"	(HP + "<<Trank<<")"<<std::endl;
Sleep (sleep);
	}else{
		std::cout << "Du hast keinen Trank mehr "<<std::endl;
Sleep (sleep);
 kampf();
	}

}

if (auswahl==false){
GegHP -= HeroATK;
if (GegHP>=0){
std::cout << "Du hast deinen Gegner angegriffen und der hat noch " << GegHP << " HP"<<"	( - "<<HeroATK<<")"<<std::endl;
Sleep (sleep);
}
if (GegHP<=0){
std::cout <<"Du hast deinen Gegner angegriffen und der hat keine HP mehr."<< std::endl;
Sleep (sleep);
}
}

//Gewonnen!
if (GegHP<=0){
	std::cout << "Du hast gewonnen!" << std::endl;
	WinLose=1;
}
}
void Gegner(){
srand(time(NULL));
GegATK =  rand() % (13 - 9 + 1) + 9;
//Gegnerischer Zug
if (GegHP <=10&&WinLose==0&&GegTrankanzahl==1){
GegHP+=Trank;
GegTrankanzahl--;
std::cout << " >>Der Gegner hat einen Trank genommen und hat wieder "<< GegHP << " HP."<<"	(HP + "<<Trank<<")"<< std::endl <<std::endl;
Sleep (sleep);
}
else if (WinLose==0){
HeroHP -= GegATK;
if (HeroHP>=0){
std::cout << " >>Der Gegner hat dich angegriffen und du hast noch "<<HeroHP << " HP." <<"	( - "<<GegATK<<")"<< std::endl<<std::endl;
Sleep (sleep);
}
if (HeroHP<=0){
std::cout << " >>Der Gegner hat dich angegriffen und du hast keine HP mehr."<< std::endl<<std::endl;
Sleep (sleep);
}
}
//Verloren!
if (HeroHP<=0){
	std::cout << "Du hast verloren!" << std::endl;
	WinLose=1;	
}


}

kampf.h

#ifndef _kampf_h_
#define _kampf_h_

void kampf();
void Hero();
void Gegner();


#endif

kopfzahl.cpp

#include <iostream>
#include <time.h>
#include <Windows.h>
#include "kopfzahl.h"
#include "kampf.h"

bool bMünze;
bool kopfzahl;

void kopfoderzahl(){
srand(time(NULL));
bMünze =  rand() % (1 - 0 + 1) + 0;
std::cout << "			Kopf oder Zahl?\n" 
	      << "Kopf = 0	Zahl = 1" <<std::endl;
std::cin >>kopfzahl;

if (kopfzahl==0){
std::cout << "Du hast \"Kopf\" ausgewaehlt!" << std::endl;
Sleep(1000);
}
else {
std::cout << "Du hast \"Zahl\" ausgewaehlt!" << std::endl;
Sleep(1000);
}

switch (bMünze){
case 0:std::cout << "Die Muenze liegt bei \"Kopf\"!" <<std::endl;Sleep(2000);break;
case 1:std::cout << "Die Muenze liegt bei \"Zahl\"!" <<std::endl;Sleep(2000);break;
}

if(bMünze==kopfzahl){
std::cout << "Du faengst an!\n";
Hero();
}
else{
std::cout << "Der Gegner faengt an!\n";
Gegner();
}
}

kopfzahl.h

void kopfoderzahl();

Danke im vorraus

[Schlüsselbein]

Source unnötig häßlich wäre der passendere Titel.
Um auf Einzelheiten einzugehen, fehlt mir jetzt die Lust, vielleicht übernimmts ja jemand anders.

Lern die Sprache ordentlich, denk an C++11 und eine einheitliche naming convention - dann schreibe das Ding neu.

[Padmak]

Was. ist. das. denn.
"bMünze = rand() % (1 - 0 + 1) + 0;" (Ich wills gar nicht in Code-Tags setzen)

Also:
- stell dich auf eine Sprache ein, nicht Gegner, Hero, Kampf
- schreib Klassen
- ...
Ich sag's mal so: das einzig positive, das mir aufgefallen ist: kein globales using namespace std;
Alles andere ist ehrlich gesagt Schrott, lern lieber erst mal die grundsätzlichen Strukturen (genau daran haperts nämlich)
Auch ich muss auf andere verweisen, alle Einzelheiten aufzuzählen.. glaube aber nicht dass sich wer genug Zeit nimmt
Also wie Schlüsselbein: C++ lernen, C++11 anwenden wo geht und bitte keine deutschen Variablennamen mehr..

Padmak

[Shadow992]

Originally Posted by Schlüsselbein

Source unnötig häßlich wäre der passendere Titel.
Um auf Einzelheiten einzugehen, fehlt mir jetzt die Lust, vielleicht übernimmts ja jemand anders.

Lern die Sprache ordentlich, denk an C++11 und eine einheitliche naming convention - dann schreibe das Ding neu.

Ich bin mir nicht sicher worauf du hässlich beziehst, aber ich finde diese Art von Einrückungs-Stil einfach nur unübersichtlich und hässlich. Wer bitte hat dir das so beigebracht? So viel wie ich weiß, ist das der Stil, der hauptsächlich in Java empfohlen/verwendet wird. Mir jedenfalls missfällt dieser Stil total.

Ich kann nur diese Art von Stil empfehlen (Geschmäcker sind verschieden):

PHP Code:

if (...)
{

}
...

int main()
{
      if (...)
      {
            FuncAufruf();
      }
      return 1;
} 


Ein paar Verkürzungen sehe ich direkt auf Anhieb ich schreib es dir mal so kurz und anfängerfreundlich wie möglich (es geht jetzt nur um mögliche Verkürzungen, nicht um besseren Code-Stil!):

PHP Code:

void Gegner()
{

    GegATK =  rand() % 5  + 9;
//Gegnerischer Zug
    if (GegHP <=10 && WinLose==0 && GegTrankanzahl==1)
    {
        GegHP+=Trank;
        GegTrankanzahl--;
        std::cout << " >>Der Gegner hat einen Trank genommen und hat wieder "<< GegHP << " HP."<<"    (HP + "<<Trank<<")"<< std::endl <<std::endl;
    }
    else if (WinLose==0)
    {
        HeroHP -= GegATK;
        if (HeroHP>0)
            std::cout << " >>Der Gegner hat dich angegriffen und du hast noch "<<HeroHP << " HP." <<"    ( - "<<GegATK<<")"<< std::endl<<std::endl;
        else
            std::cout << " >>Der Gegner hat dich angegriffen und du hast keine HP mehr."<< std::endl<<std::endl;
    }
    Sleep (sleep);
//Verloren!
    if (HeroHP<=0)
    {
        std::cout << "Du hast verloren!" << std::endl;
        WinLose=1;
    }
} 


PHP Code:

void Hero()
{

    HeroATK =  rand() % 5 + 9;

    std::cout << "Deine Hp: " << HeroHP <<"    Gegner HP: "  <<GegHP<<std::endl;
    std::cout << " 0 = Angreifen    1 = Trank nehmen" <<"("<<HeroTrankanzahl<<")"<<std::endl;
    std::cin >> auswahl;

//Hero's Zug
    if (auswahl==true)
    {
        if(HeroTrankanzahl==1)
        {
            HeroHP += Trank;
            HeroTrankanzahl--;
            std::cout << "Du hast einen Trank genommen und hast jetzt " << HeroHP <<"    (HP + "<<Trank<<")"<<std::endl;
            Sleep (sleep);
        }
        else
        {
            std::cout << "Du hast keinen Trank mehr "<<std::endl;
            Sleep (sleep);
            kampf();
        }

    }
    else
    {
        GegHP -= HeroATK;
        if (GegHP>0)
            std::cout << "Du hast deinen Gegner angegriffen und der hat noch " << GegHP << " HP"<<"    ( - "<<HeroATK<<")"<<std::endl;
        else
            std::cout <<"Du hast deinen Gegner angegriffen und der hat keine HP mehr."<< std::endl;
        Sleep (sleep);
    }

//Gewonnen!
    if (GegHP<=0)
    {
        std::cout << "Du hast gewonnen!" << std::endl;
        WinLose=1;
    }
} 


PHP Code:

void kampf()
{
    do
    {
        Hero();
        Gegner();
    }
    while(WinLose==0);
} 


PHP Code:

void kopfoderzahl()
{
    bMünze =  rand() % 2;
    std::cout << "            Kopf oder Zahl?\n"
              << "Kopf = 0    Zahl = 1" <<std::endl;
    std::cin >>kopfzahl;

    if (kopfzahl==0)
        std::cout << "Du hast \"Kopf\" ausgewaehlt!" << std::endl;
    else
        std::cout << "Du hast \"Zahl\" ausgewaehlt!" << std::endl;

    Sleep(1000);

    switch (bMünze)
    {
    case 0:
        std::cout << "Die Muenze liegt bei \"Kopf\"!" <<std::endl;
        break;
    case 1:
        std::cout << "Die Muenze liegt bei \"Zahl\"!" <<std::endl;
        break;
    }

    Sleep(2000);

    if(bMünze==kopfzahl)
    {
        std::cout << "Du faengst an!\n";
        Hero();
    }
    else
    {
        std::cout << "Der Gegner faengt an!\n";
        Gegner();
    }
} 


Das wars soweit.
Zusätzlich sind diese ganzen Includes, die du teilweise 2-3x einbinden lässt überflüssig, einmal reicht doch vollkommen.

Ansonsten am Stil gibt es einiges zu bemängeln, nur mal stichartig aufgeführt (Liste ist sicherlich unvollständig):

- Globale Variablen sind möglichst immer zu vermeiden und in deinem Fall sogar 100% vermeidbar.
- Rekursion kann zwar in manchen Situationen gut sein, hier ists aber total überflüssig.
- Sachen, die konstant sind sollte man auch mit const deklarieren (z.B. dein sleep).
- Einheitlicher Funktionsnamenstil ist wichtig (nicht mal den ersten Buchstaben groß und dann bei einer anderen Funktion ihn wieder klein)
- Unnötige Rechnungen selbstwegrechnen (sowas wie 1-0+1 ist total überflüssig, wird vom Compiler zwar eh wegoptimiert, aber macht den Code nicht sonderlich lesbarer)
- Die Funktionen und Variablen sollten klare Bezeichnungen haben, mit Hero() kann kein Mensch etwas anfangen, zumindest nicht auf den ersten Blick.
- Manche meinen reine Nomen sind als Funktionen ungeeignet, liegt aber im Auge des Betrachters
- Code-Teile, die öfter in ähnlicher Art und Weise benötigt werden, sollten zu Funktionen zusammengefasst werden (wenn nötig mit Parametern).
- Klassen wären, wenn man wirklich reines C++ programmieren will auch empfehlenswert, bei C/C++ denke ich sind Klassen nicht das was du zuerst ausbessern solltest, besser wärs dennoch
...

Mehr ist sicher auszustzen, mach jetzt aber erstmal Schluss.

[Delinquenz]

Ich kann nur diese Art von Stil empfehlen (Geschmäcker sind verschieden):

Sofern er wenigstens einen einheitlichen Stil verfolgen würde, der mit Einrücken funktioniert, wäre das ja noch ok. Aber selbst das kommt nicht zustande. Achja, afaik schreibt man Makros groß.

So viel wie ich weiß, ist das der Stil, der hauptsächlich in Java empfohlen/verwendet wird. Mir jedenfalls missfällt dieser Stil total.

Jo, große Klassen/Funktionsnamen und Camelcase kommt auch aus Java. Sofern man einheitlich codet, soll man es eben nutzen.

[Master674b]

Originally Posted by Delinquenz

Sofern er wenigstens einen einheitlichen Stil verfolgen würde, der mit Einrücken funktioniert, wäre das ja noch ok. Aber selbst das kommt nicht zustande. Achja, afaik schreibt man Makros groß.


Jo, große Klassen/Funktionsnamen und Camelcase kommt auch aus Java. Sofern man einheitlich codet, soll man es eben nutzen.

camelCase ist meiner Meinung nach auch in C und C++ sehr verbreitet, allerdings meist nur bei Variablen.
Ich finde auch das sieht viel besser aus als sowas: camel_case.

Was genau meinst du mit große Klassen- und Funktionsnamen? Funktionen und Klassen fangen bei mir und wohl auch den meisten anderen ausschließlich mit einem Großbuchstaben an (natürlich STL und Boost ausgenommen).

Interface Klassen heißen bei mir immer so ISomeClass, C und S-Klassenprefix mach ich nicht, genauso wenig wie die hässliche ungarische Notation.

Globale Variablen fangen bei mir immer mit // an, Klassenvariablen so wie m_someValue.

#defines solltest du immer in GROSS_BUCHSTABEN schreiben und außerdem vermeiden.

[MrSm!th]

Hä, ein Variablenname kann mit // beginnen? Ich nutze da g_globVar, aber das ist ja Geschmackssache.
Konstanten (die man Defines ohnehin vorziehen sollte) gehören natürlich ebenso gecaps'ed ;O

[Master674b]

Originally Posted by MrSm!th

Hä, ein Variablenname kann mit // beginnen? Ich nutze da g_globVar, aber das ist ja Geschmackssache.
Konstanten (die man Defines ohnehin vorziehen sollte) gehören natürlich ebenso gecaps'ed ;O

jo aber vordem musst du noch // oder /* machen:

The names of global variables should start with //.

[.SkyneT.]

Originally Posted by MrSm!th

Hä, ein Variablenname kann mit // beginnen?

Ich vermute(hoffe) das soll eine Anspielung darauf sein, keine globalen
Variablen zu verwenden

[Delinquenz]

The names of global variables should start with //.

Musste lachen

Was genau meinst du mit große Klassen- und Funktionsnamen? Funktionen und Klassen fangen bei mir und wohl auch den meisten anderen ausschließlich mit einem Großbuchstaben an (natürlich STL und Boost ausgenommen).

Die STL sollte aber den Coding-Stil nach außen hin repräsentieren, oder sehe ich das falsch?

[Master674b]

Originally Posted by Delinquenz

Musste lachen


Die STL sollte aber den Coding-Stil nach außen hin repräsentieren, oder sehe ich das falsch?

Das hängt halt alles davon ab mit was man bisher so alles gearbeitet hat, das schwappt dann irgendwann einfach so auf einen über. Ich hab früher auch die ungarische Notation benutzt und mich nun aber davon verabschiedet.

Der Grund war recht simpel: Früher viel mit Windows Zeug zu tun gehabt und dann aber mal eingesehen das es nur meinen Lesefluss stört und das es viel besser aussieht ohne. Außerdem mal mit der Maus über nen Variablennamen fahren reicht schon um zu wissen was es fürn Typ ist. Intellisense ftw.

Das wichtigste ist: arbeitet man mit mehreren Leuten an einem Projekt, sollte man Coding-Guidelines aufstellen an die man sich zu halten hat. Einheitlich muss es immer sein, ganz wurscht wie das nun aussieht. Ich wette bei den MS Entwicklern passt auch nicht jedem die Naming Convention, dennoch ist das immer schön gleichmäßig, das wäre n totales Chaos wenn nicht.

[_Roman_]

Und was wäre eigentlich na andere Möglichkeit als globale Variablen. Sagen wir mal, z. B. bei der WINAPI hat man die HWND Variable für das Hauptfenster in der WinMain definiert und man braucht die dann bei irgendeiner Funktion (nicht in der WndProc Funktion, da hwnd ja als Parameter gegeben ist). Ich kenne da nur die Möglichkeit, eine globale Variable zu machen.

[Master674b]

Originally Posted by _Roman_

Und was wäre eigentlich na andere Möglichkeit als globale Variablen. Sagen wir mal, z. B. bei der WINAPI hat man die HWND Variable für das Hauptfenster in der WinMain definiert und man braucht die dann bei irgendeiner Funktion (nicht in der WndProc Funktion, da hwnd ja als Parameter gegeben ist). Ich kenne da nur die Möglichkeit, eine globale Variable zu machen.

Dafür hast du dann eine Fensterklasse, die die Variable m_hWnd definiert. Dann fügst du noch eine GetHWND methode hinzu und fertig. Die Fensterklasse kann z.B. außerdem im Konstruktor das Fenster erstellen etc.

[Padmak]

Singletons!
Du kannst ja trotzdem teils Klassenbasiert programmieren, globale Variablen braucht man für sowas eigentlich nich^^

€: ****. Too late^^

Padmak

[MrSm!th]

Wtf, Singletons? Man gibt den hwnd den Funktionen einfach als Parameter oO

@globale Variablen
Und ich war mal wieder zu inkompetent, es zu verstehen :<

Originally Posted by Delinquenz

Musste lachen


Die STL sollte aber den Coding-Stil nach außen hin repräsentieren, oder sehe ich das falsch?

Was nach außen? Jeder kann ja seinen Stil selbst festlegen.