AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Zurück Delphi-PRAXiS Projekte "DoubleA!Zeitgeber" Zeit Erinnerung
Thema durchsuchen
Ansicht
Themen-Optionen

"DoubleA!Zeitgeber" Zeit Erinnerung

Ein Thema von Darkface · begonnen am 20. Sep 2008 · letzter Beitrag vom 21. Sep 2008
 
Benutzerbild von sx2008
sx2008

Registriert seit: 15. Feb 2008
Ort: Baden-Württemberg
2.332 Beiträge
 
Delphi 2007 Professional
 
#4

Re: "DoubleA!Zeitgeber" Zeit Erinnerung

  Alt 20. Sep 2008, 14:22
Ich hätte da ein paar Kritikpunkte...
1.)
Delphi-Quellcode:
var
  Form1: TForm1; zeit,minuten,stunden,s5,soundpfad,installpfad:string; m,posi,zg1,zg2,zg3,zg4,zg5:integer;
Die ganzen Variablen zeit,minuten,... gehören eigentlich unter den Abschnitt private der Form-Klasse.
Globale Variablen sollte man wann immer möglich vermeinden.
Nicht, dass es irgendeinen Unterschied an der Funktionalität machen würde, aber wenn du ein Programm mit sagen wir mal 100 Formularen hast und überall liegen diese globalen Variablen rum, wirst du daran verzweifeln.
2.)
du hast die Namen deiner Steuerelement unverändert gelassen.
Hier ein Beispiel aus deinem Code:
Delphi-Quellcode:
procedure TForm1.CheckBox5Click(Sender: TObject);
var Reg:TRegistry;
begin
If CheckBox5.Checked=True Then
Frag' dich mal kritisch: worum geht's in dem Code?
Und hier der verbesserte Code:
Delphi-Quellcode:
procedure TForm1.ChkAutoStartClick(Sender: TObject);
var Reg:TRegistry;
begin
If ChkAutoStart.Checked Then // beachte: Vergleich mit True ist unnötig
So wird es viel klarer: es geht um die AutoStart-Funktionalität.
  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 19:34 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