Code-Stil Frage

[Tommy1988]

Hallo, ich habe diesen Code hier:

Delphi-Quellcode:

dif := 'BlaBla';
.
.
.
Label1.Caption := Format('Text BlaBla %s',[dif]);

Jetzt stelle ich mir die Frage, ob der folgende Code "gerechter" ist:

Delphi-Quellcode:

dif := 'BlaBla';
.
.
.
Label1.Caption := 'Text BlaBla '+dif;

[Meflin]

Wegen der besseren Lesbarkeit würde ich die zweite Version bevorzugen. Dürfte sich aber nicht viel schenken ;)

[Tommy1988]

Delphi-Quellcode:

dif  := 'BlaBla';
dif2 := 'Blaaa2';

Label1.Caption := 'Text BlaBla '+dif+' meeeehr Text und wieder'+dif2;

Wäre jetzt es so aber besser oder ?

Delphi-Quellcode:

dif  := 'BlaBla';
dif2 := 'Blaaa2';
.
.
.
Label1.Caption := Format('Text BlaBla %s meeeehr Text und wieder %s',[dif,dif2];

[DeddyH]

Nö, denn %d ist nicht kompatibel zu einer String-Variablen :mrgreen:

[Meflin]

Weiterhin finde ich die Version ohne Format besser. Man sieht einfach schneller was der Output nun wirklich ist ;)

Wenn dann aber so :stupid:

Delphi-Quellcode:

Label1.Caption := 'Text BlaBla ' + dif + ' meeeehr Text und wieder ' + dif2;

[Der_Unwissende]

Hi,
es wäre schon ein schlechter Style-Guide, der Label1 als Bezeichner zulässt :wink:

An sich gibt es ja keinen sinnvollen Grund Ersteres zu verbieten, es handelt sich um eine Funktion, deren Rückgabe ein String ist. Dass es zufällig eine Formatierungsfunktion ist, deren Funktion Du nachbilden kannst ist eigentlich egal. Immerhin dürfte das für viele Funktionen gelten und trotzdem wird man eher auf Bibliotheken zurückgreifen. Hier wäre also die Frage, ab wann man da eine Ausnahme macht. Dürfte kaum eine "scharfe" Grenze geben, deswegen sollte man es also für alle Funktionen erlauben und damit auch für Format. Kann ja auch Vorteile haben (sobald eben nicht nur ein String angehangen wird).

Gruß Der Unwissende

[edit]
Wo kommen denn die Beiträge alle her?! Roter Kasten wo bist DU?!

Dann ist die Quizfrage doch was ist schöner:

Delphi-Quellcode:

Label1.Caption := 'Irgendein Text ' + var1 + 'noch mehr Text ' + IntToStr(var2);

// oder 

Label1.Caption := Format('Irgendein Text %s noch mehr Text %d', 
                         [var1, var2]);

[/edit]

[DeddyH]

Wenn Du viele Daten unterschiedlicher Typen darstellen willst, finde ich die Format-Lösung einfach besser als mit 20 IntToStr herumzuhantieren. Bei reinem Text kann man ja die einfache Konkatenation wählen.

[Sidorion]

Na in jedem Fall letzeres. Sobald das was im Moment noch fester String ist aus einer Ressource kommt und ggf. in verschiedene Sprachen übersetzt wird, wirds beim zweiten Fall braun.
Ein Beispiel:

Delphi-Quellcode:

deutsch
   Format('Die datei wurde am %s1. %s2 zum letzen mal geändert.',['19','Mai']);
englisch
   Format('The file was last modified on %s2 %s1.',['19','Mai']);

[Der_Unwissende]

Ich finde man sollte (wie gesagt) nicht für bestimmte Typen unterscheiden, ob der eine oder der andere Weg genommen wird. Ein Codestil sollte vorallem konsequent sein und das wäre es (imho) nicht, wenn man für Strings die einfache Konkatentaion wählt und sonst Format.
Was man nimmt sollte nie eine reine Frage des Stils sein. Ohne zu wissen wie Format im Detail arbeitet, es könnte ggf. etwas perfomanter sein, wenn die Funktion intern nicht mit einfacher Konkatenation arbeitet (Strings sind ja in der Länge statisch), sondern wenn gleich Speicher in der richtigen Größe alloziert und dann entsprechend gefüllt wird.

Auch das Beispiel von Sidorion (endlich funktioniert der rote Kasten auch bei mir!) zeigt, dass man es eben von anderen Dingen als den Stil abhängig machen sollte.

[edit]
falschen Namen getippt und korrigiert
[/edit]

[dfried]

Da kann ich Sidorio nur zustimmen. Sobald du mit dem Gedanken spielst in MultiLanguage einzusteigen ist die Variante mit dem "Format" fast zwingend notwendig.
Ich persönlich finde diese Version generell auch etwas übersichtlicher (vor allem bei mehreren Varaiblen anteilen).