AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Zurück Delphi-PRAXiS Projekte ADBShoutcastQuery - Suchen nach Internetz-Radiosendern
Thema durchsuchen
Ansicht
Themen-Optionen

ADBShoutcastQuery - Suchen nach Internetz-Radiosendern

Ein Thema von Dunkel · begonnen am 21. Apr 2007 · letzter Beitrag vom 21. Sep 2008
 
alzaimar
(Moderator)

Registriert seit: 6. Mai 2005
Ort: Berlin
4.956 Beiträge
 
Delphi 2007 Enterprise
 
#2

Re: ADBShoutcastQuery - Suchen nach Internetz-Radiosendern

  Alt 21. Apr 2007, 07:11
Kosmetik und refactoring:
Da Du englische Beschreibungen und englische Variablennamen verwendest, solltest Du bezüglich der Rechtschreibung nochmal nachbessern (InProgres, Kommaregeln, Im Englischen werden '-' selten verwendet. 'query result', statt 'query-result')

Weiterhin solltest Du dich an die Delphi-Konvention zur Bennung von Events und Recordmembers halten. Events verschicken in ihrem ersten Parameter i.A. sich selbst, also statt:
TMyEvent = Procedure (Parameter : TType) Of Object; lieber
TMyEvent = Procedure (Sender : TObject; Parameter : TType) Of Object; Und dann beim Abfeuern des Events;
fMyEvent (Self, MyParameter); Anstatt
Delphi-Quellcode:
If Assigned (fEvent) Then
  fEvent(Self)
Solltest Du jedem Event ein
Procedure MyClass.DoEvent;
Delphi-Quellcode:
Begin
  If Assigned (fEvent) Then
    fEvent(Self)
End;
spendieren. Im Code rufst Du dann 'DoEvent' auf, das erhöht die Übersichtlichkeit. Für jeden Event 'fXXXXX' schreibst Du eine private Methode 'DoXXXXX'.

Statt Records kannst Du vielleicht noch Klassen deklarieren, falls die Ergebnisse (TPlayList, TPlayListItem) eine Funktionalität besitzen oder besitzen sollen.

Das ist aber kein Muss: Nur ist es dann irgendwie konformer.
"Wenn ist das Nunstruck git und Slotermeyer? Ja! Beiherhund das Oder die Flipperwaldt gersput!"
(Monty Python "Joke Warefare")
  Mit Zitat antworten Zitat
 


Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 06:03 Uhr.
Powered by vBulletin® Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024-2025 by Thomas Breitkreuz