Schreibstil bitte bewerten:

stimmt leider... das problem ist das ich früher bissel c++ geproggt hab und jetzt in der ausbildung fangen wir im ersten jahr mit C an und next jahr c# (ka ob die c++ nehmen)

naja egal... weiterbewerten und tips geben bitte

hi,

zwischen Variablendeklaration und eigentliche Codeanfang wurde ich ne Leerzeile machen. Dann sieht man sofort wo was deklariert wird, und wo der Code anfängt.

Ist ne Kleinigkeit, aber immerhin

Am sonsten ist der Code gut lesbar und verständlich auch

cu

flammenvogel

warum schreibtst du nicht

char test[] = "Das ist ein Test";

Das sollte eigentlich gehen.

Mal ein paar C++ Vorschläge:

#include <iostream>
#include <string>

using namespace std;

void main()
{
  //C++ Strings sind einfacher zu handhaben als Char Arrays
  string xy,buf;
  int zahl;
  xy = "Test";
  cout << xy << endl;  //oder cout << xy << "\n"
  cout << "Bitte was eingeben: " << endl;
  cin >> buf;
  cout << "Bitte eine Zahl eingeben: " << endl;
  cin >> zahl;
  cout << "Die Zahl war: " << zahl << endl;
  return;
}

1.) Es handelt sich eindeutig um C. Von ++ ist hier nichts zu sehen.

2.) Man includiert für gewöhnlich nur Header und keine CPP-Files, warum auch.

3.) Wenn du schon (fast) alle Variablen initialisiert hast. Warum dann nicht auch:

char cWahl;

4.) Du solltest nur includieren was notwendig ist. Das heisst, ich sehe keinen
Bedarf für

#include <math.h>

5.) Was spricht gegen

if (iWahl == 1)// Fortfahren
{
}
else if (iWahl ==2) // Neue Simulation starten
{
}
else if (iWahl == 3)  //Simulation beenden
{
}

6.) Einige passende Leerzeilen würden sicherlich die Übersichtlichkeit erhöhen.

7.) Was machts du bei Fehlern. Zum Beispiel

int res = scanf("%d", &iWahl);

wäre ein Verbesserungsvorschlag.

Mehr ist mir auf die Schnelle nicht eingefallen.

Redhead schrieb:

5.) Was spricht gegen

if (iWahl == 1)// Fortfahren
{
}
else if (iWahl ==2) // Neue Simulation starten
{
}
else if (iWahl == 3)  //Simulation beenden
{
}

Hier würde auch ein switch gut passen, insbesondere wenn sich die Zahl der Alternativen
noch erhöhen würde.

pumuckl

Redhead schrieb:

2.) Man includiert für gewöhnlich nur Header und keine CPP-Files, warum auch.

Mach ich z.B. mit der Implementierung von Templates. Als eigene UE gehen sie bei den meisten COmpilern nicht, aber ich brauch mir damit auch nicht unbedingt die Header vollzustopfen... Ist aber sicherlich Geschmackssache..

2.) Man includiert für gewöhnlich nur Header und keine CPP-Files, warum auch.

Wird es sich hierbei

#include "classes.cpp"

wohl um Templates handeln ??

darein schreib ich alle klassen (ich finds übersichtlicher wenn ich net alles in die hauptdatei mit reinstopf)

flammenvogel

die definition der Klassen musst du in eine eigene UE schreiben. Und die dann mitkompelieren (in deiner IDE zum Projekt hinzufügen). Man darf cpp Datein nicht mit #include einbinden. Es mag da sicherlich einige ausnamhmen geben, aber die Regel ist es nicht zu machen.

HumeSikkins

TravisG schrieb:

(ich finds übersichtlicher wenn ich net alles in die hauptdatei mit reinstopf)

Das ist auch nicht die Alternative. Richtig wäre es, die Klassendefinitionen zu inkludieren und die Definitionen der Methoden seperat zu übersetzen.

Zu dem Code. Imo solltest du deinen Code lieber besser strukturieren (Funktionen die eine und nur eine Aufgabe erfüllen) statt ihn übermäßig zu kommentieren.