C++ in der Uni

VDR-T · 25. Juli 2006

Ich muß dieses Semester eine Informatikklausur schreiben und wollte mal eure Meinung zu dieser Musterlösung hören.

Code

#include <iostream>
using namespace std;


class artikel
{
	private: 
   	int menge;
      char *name;
      static int anzahl_artikel;
      int art_nr;

   public: 
		void menge_setzen(int);
      int menge_aus();
      int verkaufen(int verkauf);
      void name_ein(char *);
      char *name_aus();
      int art_nr_aus();

      bool operator>(artikel&);     
      artikel& operator=(artikel&); 
      artikel();     
};


artikel::artikel()
{
	name=0x00;
   menge=0;
   anzahl_artikel++;
   art_nr=anzahl_artikel;
}


int artikel::anzahl_artikel=0;


int artikel::art_nr_aus() {return art_nr;};


char *artikel::name_aus()
{
	return name;
}


void artikel::name_ein(char *eingabe)
{
	int laenge=strlen(eingabe);
   if (name!=0x00) delete[] name;			// Falls name schon belegt ist	
   name=new char[laenge+1];
   for (int i=0; i<laenge;i++) name[i]=eingabe[i];
   name[laenge]=0x00;
}


void artikel::menge_setzen(int wert)
{
	if (wert > 0) menge=wert;
   else menge=0;
}


int artikel::menge_aus()
{
	return menge;
}


int artikel::verkaufen(int verkauf)
{
	int rueck;

   if (menge > verkauf)
   {
   	menge=menge-verkauf;
      rueck=verkauf;
      return rueck;
   }
   else if (menge > 0)
   {
   	rueck=menge;
      menge=0;
      return rueck;
   }
   else 
   {
   	return 0;
   }

}


bool artikel::operator>(artikel &vergleich)
{
     if (this->menge > vergleich.menge_aus()) return true;
     else return false;
}




artikel& artikel::operator=(artikel &arg2)
{
	if (this != &arg2)
   {
   	this->menge_setzen(arg2.menge_aus());
   	this->name_ein(arg2.name_aus());
   	this->art_nr=arg2.art_nr_aus();
   }
   return *this;

}






int main(void)
{


artikel *daten;
int n_artikel, menge;
char name[20];


	cout << "Bitte die Anzahl der Artikel eingeben: ";
   cin >>n_artikel;
   daten=new artikel[n_artikel];

   for (int i=0; i<n_artikel;i++)
   {
   	cout << "Bitte Namen eingeben: ";
      cin >> name;
      cout << "Bitte Menge eingeben: ";
      cin >> menge;
      daten[i].name_ein(name);
      daten[i].menge_setzen(menge);
   }


   artikel tausch;
   for (int i=0; i<n_artikel;i++)
   {
   	for (int j=i+1; j<n_artikel;j++)
      {
      	if (daten[j]>daten[i])
         {
         	tausch=daten[i];
            daten[i]=daten[j];
            daten[j]=tausch;
         }
      }
   }


	for (int i=0; i<n_artikel;i++)
   {
     cout << daten[i].art_nr_aus()<< " " << daten[i].name_aus() << " " << daten[i].menge_aus()<<endl;
   }
	return EXIT_SUCCESS;
}

Alles anzeigen

ich finde das ganze für eine Musterlösung doch etwas unübersichtlich, besonders da die Vorlesung nicht für Informatiker sondern für Maschinenbauer ist.

LordJaxom · 25. Juli 2006

Ist ja selbst mit Code-Tags nicht übersichtlicher

Und jetzt mein Senf:

MUSTERLÖSUNG??!?!?! OMFG!

1. Non-static und static Members unübersichtlich vermischt
2. Methodenprototypen inkonsequent mal mit Bezeichner in der Argumentliste, mal ohne
3. operator> ist nicht const-correct
4. operator= auch nicht
4. Statt char* nimmt man eigentlich strings
5. Die char* sind nicht const-correct (ich kann über name_aus den Namen manipulieren)
6. Ja was denn nun, nennen wir setter-Methoden "ein" oder "setzen"?
7. Wenn schon char*, dann wenigstens C-Stringfunktionen und nicht zu Fuß gehen (Methode name_ein)

Jetzt habe ich Kopfschmerzen

EDIT:
Ich hoffe ich beleidige damit niemanden, aber als Muster würde ich das niemandem anbieten!

VDR-T · 25. Juli 2006

Ich habe die Musterlösung 1zu 1 so aus der TXT-Datei übernommen die uns vom Institut zur Verfügung gestellt wird.
Das wir hier char* und nicht string benutzen liegt wohl daran, dass wir lernen sollen wie man mit Arrays umgeht.

Kopfschmerzen habe ich schon seit längeren, besonders wenn ich an die Klausur denke

LarsAC · 25. Juli 2006

Aber danach kannst Du Plugins für den VDR programmieren, das ist eh das einzig wichtige.

Lars

VDR-T · 25. Juli 2006

Das halte ich noch für ein Gerücht. Falls ich es dann doch ihnbekomme ist mein Code dann vermutlich so schlecht geschrieben das ich hier eins aufs Dach bekomme

LordJaxom · 25. Juli 2006

Falls Dir das irgendwie hilft, so hätte es bei mir ausgesehen, das string kann man ja immernoch wieder durch const char* ersetzen.

EDIT:
Habs mal nach http://phpfi.com/135110 gepostet, die haben im Gegensatz zu uns Syntax-Highlighting

EDIT2:
URL geändert, hab noch zwei Kleinigkeiten übersehen gehabt

VDR-T · 25. Juli 2006

Danke, vielleicht solltest Du lieber mal einen Kurs in C++ geben Ich werde mich dann mal weiter mit C++ auseinandersetzen und wenn ich was nicht verstehe weiß ich ja jetzt wen ich fragen kann

Gruß,
Vdr-t

Marcus · 25. Juli 2006

Kurse gibt es doch schon genug

schaue mal das findest bestimmt etwas: http://cplus.kompf.de/links.html

vg marcus

s_herzog · 25. Juli 2006

Zitat

Original von LordJaxom
Und jetzt mein Senf:

MUSTERLÖSUNG??!?!?! OMFG!

Musterlösung != BestmöglicheÜbersichtlichsteLösung

Im Zweifelsfall ist es einfach von einem Studenten, der Tutorien gehalten hat vom Jahr zuvor.

PS: Soo unübersichtlich finde ich das jetzt gerade nicht, das sind ja im emacs höchstens drei Seiten Code. Es geht ja vielleicht auch gerade drum, in der Lage zu sein, den Überblick zu gewinnen, egal, wie häßlich es aussieht.

Ich hab mich für die Diplomarbeit durch ein "gewachsenes" Multithreaded-Framework zur Implementierung von Netzwerkprotokollinstanzen gewühlt, ich habe schon schlimmeren Code-Wust gesehen aus der Hand von Dipl.-informs. und Dr.-ings.

Dieter · 30. Juli 2006

Hi,
ja ich habe schon unübersichtlicheren Code gesehen. Aber schön ist es nicht gerade. Und Regel Nr. 1: Niemals deutsche Bezeichner. Immer English!

skan! · 30. Juli 2006

Immer englisch? Nö, immer konsequent eine Sprache.

LarsAC · 30. Juli 2006

Zitat

Original von Dieter
Und Regel Nr. 1: Niemals deutsche Bezeichner. Immer English!

Seit ich letztens mal QUelltext mit tschechischen Bezeichnern gesehen habe weiss ich das umsomehr zu schätzen.

Da versteht man *gar nix* mehr.

Lars

Dieter · 31. Juli 2006

Hi,
was ist mit while, if and friends?

Die sind English, also immer eine Sprache. Somit auch Bezeichner und Kommentare in English.
Ist auch ne ganz gute Übung wenn mann nicht so gut in English ist.

Das habe ich schon immer so gehalten, auch als ich noch nicht so fit in English war.

ravemax · 31. Juli 2006

Jaxoms Kommentare sind eigentlich schon sehr ausführlich...

Code

if (menge > 0)

dürfe die Prüfung:

Code

if (menge_ > 0)

sein.
Abgesehen von stilistischen Fragen, vielleicht

Code

if (menge > 0)
	menge = menge_;
else
	menge = 0;

doch lieber:

Code

menge = (menge > 0) ? menge_ : 0;

oder - falls definiert:

Code

menge = MAX(menge_, 0)

LordJaxom · 31. Juli 2006

Zitat
Original von ravemax
Code
menge = MAX(menge_, 0)

Mit std::max garkein Problem.

Auf GARKEINEN Fall ein Makro verwenden bitte - std::max ist ein template, das ist typsicher

C++ in der Uni

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema