Hallo,
habe für meinen Betrieb einen Urlaubsplaner machen dürfen. Habe den kompletten Quelllcode im Anhang mitgeschickt.

Die Funktionsweise ist grob im Hilfe Menü erkärt.

Ziel ist es noch, dass man einzelne Zellen im StringGrid1 selektieren kann. Und nicht nur zusammenhängende (sprich mit Hilfe der STRG-Taste).

Wird noch folgen, falls dies überhaupt möglich ist. Wenn jemand eine Idee hat, dann informiert mich.

P.S. der Schichtplaner wurde aber noch nicht in der Praxis getestet, deshalb könnnen noch Bugs auftreten. Er wird bei uns in der Firma aber im Januar 2007 eingeführt.

Falls euch irgendwelche Fehler auffallen, dann gibt mir bescheid.

Noch zur Info das Programm wurde mit Delphi 6 erstellt

Du solltest noch ein paar Dinge dazu erläutern:

1. Welche Datenbank wird eingesetzt?
2. Einstellungen für die INI-Datei.

Übrigens musst Du nicht die Dateien mit *.~* mitschicken. Selbst auf *.DCU kann man verzichten in diesem Fall. Das macht die ZIP-Datei etwas schlanker.

Erster Eindruck:
- Übersichtlich
- Die Fortschrittsanzeige wirkt irgendwie sinnlos
- Eingabe ungültiger Jahre möglich
- Feiertagsverwaltung nicht gefunden
- Was macht Button4?
- Schichteintrag ausgewählt: Listenindex überschreitet Maximum
- Abwesend: das selbe
- Bei Auswahl des Monats ist der Aufbau des unteren Grids langsam
- Auch beim Neuzeichnen (Programmwechsel mit alt-tab) wirkt das träge
- Wenn ich im oberen Grid seitlich scrolle (Tastatur), läuft das untere nicht mit.

Die Datenbank ist eine Access DB.

Zu der INI-Datei, es wird doch alles erklärt bei den Fehlermeldungen die kommen, wenn die DB nicht gefunden wurde.

Bei mir funktioniert das mit dem Schichteintrag und Abwesend. Man darf nur keine Zeile auswählen, bei der kein Name vorne steht.

Das mit den Jahren mache ich noch so, dass man nur Jahre auswählen kann, sprich nicht in die CombBocx schreiben darf.

Was meinst du mit Feiertagsverwaltung. Die Feierttage werden vom Programm berechnet. Muss man halt für jedes Bundesland anpassen. Zur Zeit werden die Feierttage für BW berechnet.

Button4 war nur zum Test da, wollte ich noch entfernen.

Das mit dem Scrollen möchte ich auch noch ändern, dass man nur mit Hilfe der Scrollbar Scrollen kann. Ich weiß nur noch nicht wie ich das machen soll

Das mit der Geschwindigkeit nehme ich in Kauf, das dauert halt, da er sämtliche Daten aus der DB holt und diese berechnet.

INI-Datei:
Warum muss denn erst eine Fehlermeldung kommen? Außerdem ist nicht anwenderfreundlich, wenn man die INI-Datei manuell korrigieren muss. Ein kleines Konfigurationsmodul (Auswahl der Datenbank) ist nicht wirklich kompliziert.

Schichteintrag und Abwesend:
Bei mir stehen mehrere Gridzeilen, ohne einen Namen auf dem Bildschirm. Personaldaten sind aber vorhanden. Ist der Fehler vielleicht woanders?

Feiertagsverwaltung:
Es gibt auch Feiertage, die nicht berechenbar sind. Außerdem willst Du wohl nicht für jedes Bundesland eine eigene Unit schreiben, oder?

Scrollen:
Nutz doch das Ereignis TopLeftChange des Grids. Keine Ahnung, ob das wirklich so heißt, aber ich weiß, das es etwas in der Art gibt. Habe ich selbst in einer alten Anwendung für genau so einen Fall genutzt.

Das mit den Feiertagen lasse ich so, der wo den Planer in einem anderen Bundesland nutzen will, muss die Feiertagsberechnung anpassen.

Schichteintrag und Abwesend:
Ja es stehen einige leere Zeilen darin. du darfst nur keine Zeile markieren bei der kein Name drin steht und dann in der Combobox eine Schicht auswählen. Wenn du eine Zelle markierst bei der auch ein Name in der ersten Spalte steht, erscheint kein Fehler. Werde diesen Bug aber noch beheben.

Hab mal einen kurzen Blick drauf geworfen:

(Entpackt und INI an lokales Verzeichnis angepaßt, so daß DB gefunden wird)

1) Masken schauen sehr schräg aus, wenn auf dem Rechner große Schriftarten aktiviert sind.
2) Die Combos für Schichteintrag und Abwesend liefern bei mir nur 'Listenindex überschreitet Maximum (0)'
3) Da anscheinend bei Start für den aktuellen Systembenutzer ein frischer Eintrag in der INI-gemacht wird (wenn es ihn nicht gibt), warum setzt Du das DB Verzeichnis dann nicht gleich richtig?
4) Masken haben keinen einheitlichen Hintergrund: z.B. 'Urlaub' grau aber 'Abteilung hinzufügen'türkis
5) Sinnvolle Namensgebung für die Units und Objekte! Wenn Du mal nicht mehr in der Firma arbeiten solltest, dann muß dein Nachfolger sich den Code vieleicht anschauen.

HTH,
Ulrich

Hallo,

hab mir das Programm mal kurz angeschaut.

Dabei ist mir auch das langsame Zeichnen des zweiten Stringgrids aufgefallen.
In der OnDrawCell-Prozedur führst Du bei jedem Ereignis( ! eine Zelle wird gezeichnet) die Prozedur Ini_Schreiben aus.
Das heist z.B. bei jedem horizontalen Scrollen um eine Bildschirmbreite ca. 60*9= 540-mal TiniFile > Create, WriteInteger und Free ausgeführt.

Da ist soooooviel Platz zwischen den Grids, setz doch besser einen Splitter zwischen die Grids, dann kann man auch mit einer halbwegs normalen Auflösung damit arbeiten.

Im unteren Grid zeigt er in allen Zellen '0' an?

Hallo, habe jetzt einige Bugs entfernt und zusätzliche Dinge eingefügt. Habe jetzt nicht die aktuellste Version, da ich diese im Geshäft habe. Bei dieser Version ist aber schon einiges neu. Wenn ich im Geschäft mal die Gelegenheit habe, werde ich auch diese Version Online stellen. Hier mal die vorab Version.

Falls euch igrendwelche Bugs auffallen oder ihr habt irgenwelche Verbesserungsvorschläge, dann postet dies bitte.

Hallo,

also so einen Planer wollte ich auch mal programmieren, daher habe ich ihn mir mal (flüchtig) angesehen.
Muss dazu sagen, dass ich nur die EXE probiert habe und mit den Quellcode nicht angesehen habe.

Leider ist das mit dem Pfad der DB in der INI Datei immer noch so, dass der Benutzer das von Hand eingeben muss. Wenn die DB sowieso im selben Pfad ist, wie die EXE, dann verwende doch extractfilepath(paramstr(0)).

Viele Fenster (wie zum Beispiel Personen hinzufügen) werden zu kleine dargestellt (mit Scrollbar, was bedeutet, dass ich diese größer ziehen muss. (arbeite mit 1024*768).
Lass doch das Hauptfenster nur dann aktualisieren, wenn auch wirklich Änderungen vorgenommen wurden. So wird das ganze sehr ausgebremst.
Das Fenster Jahreswechsel ließ sich nicht schließen. ?!?

Verstehe ich das richtig, dass schon immer 382 Personen angelegt sind ?!?

Das mal fürs erste

Aber bei allen Fehlern - nicht die Freude am Projekt verlieren!!!

Gruß vodasu

Moin ...,
Uff, mich verwundert das du bei dem Codestyle die Übersicht behältst. Du solltest dir angewöhnen, Code den du öfter benötigst in eigene Units, Prozeduren oder Functionen auszulagern. Ich würde zb. alles was mit Inis zu tun hat in eine eigene Unit unterbringen. Das verbessert die Übersichtlichkeit und Fehlersuche ungemein. Und bitte gib den Controls\Komponenten aussagekräftige Namen, man sucht sich echt den Wolf weil man laufend schauen muss was zb ComboBox1 ist
Vom Code her sind mir auf den ersten Blick nur Kleinigkeiten aufgefallen:

kann durch

ersetzt werden und das flackern des Grids sollte sich mit

lösen lassen.
Was ich aber überhaupt net kapier, warum gehst du 2 unterschiedliche Wege beim lesen und schreiben der Ini-Datei?

Aber alles in allem, ein gutes Programm...

LG Mario

Hallo Bronstein!

Bronstein hat folgendes geschrieben:

Hallo, habe jetzt einige Bugs entfernt und zusätzliche Dinge eingefügt. Habe jetzt nicht die aktuellste Version, da ich diese im Geshäft habe. Bei dieser Version ist aber schon einiges neu. Wenn ich im Geschäft mal die Gelegenheit habe, werde ich auch diese Version Online stellen. Hier mal die vorab Version.

Würdest Du bitte immer den Anhang in Deinem ersten Posting aktualisieren? Siehe dazu auch die Regel. Dann wissen unsere User immer wo sie die aktuelle Version finden und müssen sich nicht erst durch einen langen Topic wühlen.

Danke!