Befehl kürzen

[jensw_2000]

Hier ist ein interessanter Artikel zum Thema "IF THEN ELSE Knoten auflösen"
Branching on multiple conditions made readable

Schöne Grüße,
Jens

[Nuclear-Ping]

Eigentlich sollte man sich schämen, so einen Code erst zu verbrechen und dann auch noch der Öffentlichkeit anzubieten. Grenzt schon fast an Frechheit ... Ganz nach dem Motto: "Kann mir das hier mal jemand entstricken? Ich blick da nicht mehr durch."

[R34DM3]

ja also es geht mir eigentl darum den prog-code was übersichtlicher zu machen

durchblicken tuh ich da schon
falls jmd eine erklärung des progs will kann er das ja äußern dann mach
ich mir die mühe
ich wusste nur nich ob es möglich ist dieses monster zu kürzen
da ich ca 20 buttons mit der gleichen abfrage habe und ich aber
nicht weiß wie man zb im namen von button per for schleife oder sowas
ne zahl erhöht...ihr versteht was ich meine?!

und ja in der tat dieser teil ist häßlich - aber hey er funzt

ahja und wegen der klammer, ja die hab ich iwie net mitkopiert ^^

aber so wies aussieht kann man nich wirklich ne kürzere
sache draus zu machen!?

- danke für die antworten!!

*EDIT*: Und ja es wird ein BOS-Tool - zumindeste ein einsatztagebuchtool ohne funkanbindung

[mkinzler]

Zitat:

da ich ca 20 buttons mit der gleichen abfrage habe

Warum lagerst du das dann nicht in ein Funktion/Methode aus?

Zitat:

und ich aber nicht weiß wie man zb im namen von button per for schleife oder sowas
ne zahl erhöht...ihr versteht was ich meine?!

.FindComponent()

[R34DM3]

Zitat:

Warum lagerst du das dann nicht in ein Funktion/Methode aus?

was soll ich auslagern?!
steh gerade aufm schlauch sory ^^

und was liefert mir dann .fincomponent()?!
müsste ja prinzipiell sowas geben:

Delphi-Quellcode:

procedure form1.blub(komponente : "buttonX.caption")
begin
    for X := 1 to 20 do begin
       blablabla buttonX.Caption blabla
    end;
end;

nur das prob is halt das 'X'

[marabu]

Moin,

das Hauptproblem an dem gezeigten Code ist, dass Benutzeroberfläche und Regelwerk eng miteinander verwoben sind. Man "rechnet" doch nicht mit Captions, sondern entwirft geeignete Datenstrukturen. Insofern gehen für mich alle Hilfestellungen hier in eine Richtung, die mir nicht gefallen kann. Eine weitere Sünde sind die unübersichtlichen Ausdrücke, die man unter Anwendung der Moore'schen Gesetze vereinfachen kann. Ausdrücke wie (falscherstatus = false) formuliert man wartungsfreundlicher als (not falscherstatus). Momentan werden scheinbar Fahrzeug-Eigenschaften in UI-Controls festgehalten, die Zuordnung zu einem konkreten Fahrzeug geschieht über eine GruppenBox-Caption. Dieses Programm wird mehr Kummer als Freude machen - versprochen.

Ich würde zuerst mal eine Datenanalyse vorschlagen: Entitäten (Fahrzeug, Fahrer, ...) und Attribute (Fahrzeugkennung, Fahrername, ...) ermitteln und geeignet zusammenfassen.

Grüße vom marabu

[mkinzler]

Zitat:

was soll ich auslagern?!

Die Abfrage und dann den Button als Parameter

Zitat:

und was liefert mir dann .fincomponent()?!

eine Referenz auf eine Komponente anhand des Namens:

but := Form.FindComponent('Button'+IntTostr( i));

[hitzi]

Zitat von DeddyH:

Klingt für mich nach FemaleBool: Yes, No, MayBe

Sorry für das OT ... aber der Spruch ist geil

[R34DM3]

Zitat:

Moin,

das Hauptproblem an dem gezeigten Code ist, dass Benutzeroberfläche und Regelwerk eng miteinander verwoben sind. Man "rechnet"
(...)
geschieht über eine GruppenBox-Caption. Dieses Programm wird mehr Kummer als Freude machen - versprochen.

Ich würde zuerst mal eine Datenanalyse vorschlagen: Entitäten (Fahrzeug, Fahrer, ...) und Attribute (Fahrzeugkennung, Fahrername, ...) ermitteln und geeignet zusammenfassen.

Grüße vom marabu

^^ ja wie gesagt so der pro bin ich net
und viele der worte aus deinem text machen mir ein wenig angst xD

nee spaß bei seite...es wäre sicher günstiger hier mit klassen und
ähnlichem zu handtieren aber da ich nicht vor hatte mir den kopf
zu zermatern sondern eigentl so simpel wie möglich schreiben wollte
(also für mich is das simpel ^^)
habe ich versucht es so zu machen...

ich danke euch recht herzlich und viellt versuch ichs mal mit dem vorschlag
von mkinzler

will jmd viellt das prog so haben?!

[WS1976]

Sorry,
aber dann auch noch:

Zitat:

durchblicken tuh ich da schon
falls jmd eine erklärung des progs will kann er das ja äußern dann mach
ich mir die mühe
ich wusste nur nich ob es möglich ist dieses monster zu kürzen
da ich ca 20 buttons mit der gleichen abfrage habe und ich aber
nicht weiß wie man zb im namen von button per for schleife oder sowas
ne zahl erhöht...ihr versteht was ich meine?!

Istich däutsche Spraache schwäre Spraaache!!!
Hast du in der Schule nicht aufgepasst oder willst du uns ärgern?

Grüsse
Rainer