Kigge kode igennem for fejl og mangler

Du kan ikke uddele mere end 200 point ialt for et spørgsmål
per Ekspertens regler.

Men jeg kan da godt prøv at kigge på det (i aften).

Vil du poste koden her eller smide et link eller emaile ?

mailer det...

Jeg skal jo også kun af med 100 p indtil videre... :-)

Har du min email-adresse ?

nej

arne_v@mail.danbbs.dk

er der inden en time!

soreno @ stofanet dk

Kan nok ikke komme med ongen guldkorn, men vil gerne prøve at se hvordan andre strukturerer deres kode

thomasnj85@hotmail.com

Jeg kan da også gerne kigge på det, men det bliver ikke før i morgen.
driis@mobilixnet.dk

imorgen er desværre for sent... skal aflevere imorgen!

Kode kan hentes her: www.tms.adsl.dk/exam.zip

1)

Du bruger ikke ÆØÅ.

2)

Jeg kan ikke lide stilen med at sætte break til højre for en anden statement.

3)

Din navngivning er meget C'sk - ingen kapitaliseringer.

Det er ikke nødvendig skidt - jeg nævner det bare.

4)

Jeg ville lade samling opret have en bog som argument i.s.f. alle
argumenterne til bog constructor.

5)

Jeg ville bruge STL vector i.s.f. array til at indeholde bøgerne i samling.

6)

De int konstanter som diverse metoder i samling returnerer burde have
symbolske konstanter.

7)

bog har også numeriske retur værdier.

8)

Du kunne have laver operator overload på << i bog og brugt den
i samling.

9)

Din get metoder i bog ser meget java'ske ud - du kunne måske bare
have lavet dem public

10)

bog bruger også array fremfor STL vector

11)

copy constructoren bør normalt lave det samme som = operatoren

og den er nødvendig i mange tilfælde

12)

Jeg mener at din = operator mangler at få reservationer med

13)

reservation kunne efter min overbevisning være lavet som struct
uden metoder.

Bemærk at ovenstående er ikke 13 fejl - det er 13 kommentarer til nogle valg
du har lavet og nogle forslag til hvordan nogle ting kunne være lavet
anderledes

[jeg har iøvrigt ikke engang kørt programmet]

Forskellen på copy constructor og = operator kan ses i følgende program:

#include <iostream>

using namespace std;

class Foobar
{
  public:
      Foobar()
      {
          // nothing
      }
      Foobar(const Foobar& x)
      {
          cout << "copy constructor called" << endl;
      }
      void operator=(const Foobar& x)
      {
          cout << "= operator called" << endl;
      }
};

int main()
{
    Foobar A;
    Foobar B = A;
    A = B;
    return 0;
}

Tak for det arne_v! Jeg har brugt lidt af, men det er jo lidt svært når man skal opfylde nogle specifikke programmering krav fra undervisningen.

Da arne_v er den eneste der har responderet får arne_v 100 point.

Tak igen!

svarer du lige arne_v

svar