Frage zu overload + reintroduce

[Der schöne Günther]

Eine Kindklasse möchte eine abstrakte Methode der Oberklasse implementieren und überladen. Beispiel:

Delphi-Quellcode:

   TBaseClass = class abstract
      procedure someProc(); virtual; abstract;
   end;

   TChildClass = class(TBaseClass)
      procedure someProc(); overload; override;
      procedure someProc(const someInput: Integer); overload; virtual;
   end;

Der Compiler meint:

Code:

Warnung: W1010 Methode 'someProc' verbirgt virtuelle Methode vom Basistyp 'TBaseClass'

Ich sehe nicht, wie da irgendetwas verborgen wird :gruebel:

Die Compilerwarnung geht weg wenn man entweder

1. TBaseClass::someProc() die Direktive 'overload' hinzufügt
2. TChildClass::someProc(Integer) die Direktive 'reintroduce' hinzufügt

Ersteres finde ich Quatsch. In der Oberklasse interessiert es mich herzlich wenig, ob in einer Kindklasse mal überladen wird. Außerdem würde ich mir so wahrscheinlich angewöhnen, grundsätzlich an jede Methode ein 'overload' anzuhängen.

Zweiteres ist logisch, laut Hilfe macht 'reintroduce' nichts anderes, als nur Compilerwarnungen zu unterdrücken. Aber wie gesagt: Ich sehe nicht, wie hier überhaupt etwas verdeckt wird...

[baumina]

In der Hilfe von meinem Delphi XE ist diese Compiler-Warnung ganz gut erklärt und 3 Dinge zum Vermeiden der Warnung angegeben.

[himitsu]

Nja, im Prinzip wird da vermutlich nur was falsch getestet (beim kompilieren des zweiten someProc wird das Erste nicht beachtet), aber das ist schon ewig so und läßt sich via reintroduce gut unterdrücken.

[Der schöne Günther]

Zitat:

Zitat von baumina (Beitrag 1237297)

In der Hilfe von meinem Delphi XE ist diese Compiler-Warnung ganz gut erklärt und 3 Dinge zum Vermeiden der Warnung angegeben.

Die drei Punkte sind

1. [Passt nicht, da keine Überladung]
2. "Dann Überladen sie halt nicht"
3. "Verwenden Sie reintroduce um die Warnung zu unterdrücken"

Mein Punkt ist nur, dass hier nichts verdeckt wird und der Compiler mit seiner Warnung nicht Recht hat.

[stahli]

Mach Dir doch mal den QC-Spaß...

[Sherlock]

Zitat:

Zitat von stahli (Beitrag 1237319)

Mach Dir doch mal den QC-Spaß...

Ich bitte sogar darum!

Sherlock

[Der schöne Günther]

Kann ich gerne tun, ich wollte nur sicherstellen, dass hier kein Denkfehler drin ist und der Compiler mit seiner Warnung wirklich Unrecht hat.

Falls also noch jemand seine qualifizierte Meinung abgeben möchte ;-)

[baumina]

Gibt's wohl schon öfters, wie hier z.B. http://qc.embarcadero.com/wc/qcmain.aspx?d=106026

[Der schöne Günther]

"and u can see it in tests"

Etwas ungeschickt beschrieben aber ja, genau das ist es. Danke.

[stahli]

Vielleicht ist es schon zu spät heute ... oder verstehe ich die Fehlermeldung zu Recht nicht?

Create ist normal überschrieben.
Create(xxx) will ich virtuell erzeugen und überladen, damit ich es später bei Bedarf überschreiben kann.

Nach "overload; virtual;" wird "reintroduce;" jedoch nicht akzeptiert.
Wenn ich "reintroduce;" weg lasse funktioniert das Geraffel so.
Die unsinnige Fehlermeldung kann ich also nicht mal unterdrücken.


Oder liegt hier ein Denkfehler meinerseits vor?

Delphi-Quellcode:

  TTest = class(TBase)
  ...
  public
    RTState: Boolean;
    constructor Create; overload; override;
    constructor Create(aClassName: String); overload; virtual; reintroduce;
    destructor Destroy; override;

Zitat:

[dcc32 Warnung] soTypes.pas(104): W1010 Methode 'Create' verbirgt virtuelle Methode vom Basistyp 'TBase'
[dcc32 Fehler] soTypes.pas(104): E2169 Felddefinition nicht erlaubt nach Methoden oder Eigenschaften
[dcc32 Fehler] soTypes.pas(104): E2029 ',' oder ':' erwartet, aber ';' gefunden
[dcc32 Fehler] soTypes.pas(105): E2029 Typ erwartet, aber 'DESTRUCTOR' gefunden

[himitsu]

Zitat:

Nach

Schreib mal das vor's virtual oder so. :zwinker:

Die Reihenfolge dieser Angaben ist nicht beliebig. (sinnloser Weise)

[Sir Rufo]

Die Deklaration von

Delphi-Quellcode:

TBase

wäre jetzt hilfreich. Am Besten sogar beide Deklarationen aus dem interface-Teil komplett.

Zitat:

Zitat von himitsu (Beitrag 1260028)

Zitat:

Nach

Schreib mal das vor's virtual oder so. :zwinker:

Die Reihenfolge dieser Angaben ist nicht beliebig.

Ich glaube ja, das

Delphi-Quellcode:

reintroduce

ist hier völlig überflüssig ;)

[stahli]

@himi
Hatte ich schon versucht, aber so geht es jetzt: "reintroduce; overload; virtual;"

@Sir
Da war nix relevantes drin.

[himitsu]

Zitat:

W1010 Methode 'Create' verbirgt virtuelle Methode vom Basistyp 'TBase'

Joar, ein Blick kann nicht schaden, aber so wie das klingt, dann meint der Compiler, daß es doch nötig ist. :angle:

[edit]
Die RedBox schläft wohl schon. :roll:

[Sir Rufo]

Zitat:

Zitat von stahli (Beitrag 1260030)

@Sir
Da war nix relevantes drin.

Bekommst du denn eine Warnung, wenn du das

Delphi-Quellcode:

reintroduce

weglässt?

[stahli]

Ja, die um die es in dem Thread hier ging:

Zitat:

[dcc32 Warnung] soTypes.pas(104): W1010 Methode 'Create' verbirgt virtuelle Methode vom Basistyp 'TBase'

Wie gesagt, mit der Reihenfolge "reintroduce; overload; virtual;" funktioniert es.
Emba hätte es wohl wenig Mühe gekostet, das für die Anwender mal flexibler zu gestalten.

[Sir Rufo]

Zitat:

Zitat von stahli (Beitrag 1260033)

Ja, die um die es in dem Thread hier ging:

Zitat:

[dcc32 Warnung] soTypes.pas(104): W1010 Methode 'Create' verbirgt virtuelle Methode vom Basistyp 'TBase'

Wie gesagt, mit der Reihenfolge "reintroduce; overload; virtual;" funktioniert es.
Emba hätte es wohl wenig Mühe gekostet, das für die Anwender mal flexibler zu gestalten.

Da

Delphi-Quellcode:

reintroduce

nichts weiter macht als eine Warnung zu unterdrücken, ist es durchaus legitim den Entwickler insoweit zu gängeln, dass er sich wirklich Gedanken darüber macht, ob das was er macht auch das macht was es soll ;)