Code optimieren

ThiefMaster

Hi, hat jemand eine Idee, ob ich diesen C-Code noch irgendwie optimieren kann?

code:

1:
2:
3:
4:

 if(metal>0) sprintf(string,"Metall: <b>%i</b> ",metal);
 if(crystal>0) sprintf(string,"%sKristall: <b>%i</b> ",string,crystal);
 if(deuterium>0) sprintf(string,"%sDeuterium: <b>%i</b> ",string,deuterium);
 if(gold>0) sprintf(string,"%sGold: <b>%i</b> ",string,gold);

Black Dove

Servus!

Idee:

"strncat"
<F1> (oder "man strncat", oder wie auch immer beliebt)

-->
Append characters of a string.
char *strncat(
char *strDest,
const char *strSource,
size_t count
);

Das sollte hier AFAIK Mittel der Wahl sein...

Geschwindigkeit habe ich nicht wirklich gemessen...
Allerdings ist die Verwendung von strncat sicherer (!) ...
Von wegen Buffer Overflows und so Augen rollen

Aber sonst... Näääähh.

cheers
BD.

Hanfling

Btw. gewöhn dir direkt an _immer_ bei if-Kram Klammern zu setzen.

MMB

Zitat:

Original von Hanfling
Btw. gewöhn dir direkt an _immer_ bei if-Kram Klammern zu setzen.

Warum das, man kann sie ja auch weglassen, das ist manchmal sogar viel besser.

Hanfling

Es geht um sauberen Code. Und es ist nicht besser. Und die 2 Sekunde für die Klammern ist auch nicht relevant, wenn man bedengt das ein Programmierer insgesamt wenige Zeilen produktiven Code pro Stunde schreibt. Überflüssig finde ich mitlerweile die Ungarische Notation oder die m_ Notation für Klassenvariablen. Sowas gibt einem schon zu denken:
"m_ppucField". Augenzwinkern

edit:
@Thiefmaster
Gib dir den Custom Titel B.O.F.H. smile

Forensoftware: Burning Board 2.3.1, entwickelt von WoltLab GmbH