sprintf vs. selber parsen, performance?

Da ich printf meist sowieso drin habe, verwende ich dann auch sprintf in 
solchen Fällen. Allerdings nicht für's komplette Template, weil sonst 
die Grösse des Zielpuffers eine hässliche Rolle spielen kann.

Also verwende ich eigene Templates mit einem einfachen Parser, der 
wenn's passt selber auch sprintf verwendet.

Wenn dir das Tempo eines Webservers Sorgen macht, dann bist du bei einem 
8bitter mit zuwenig Speicher für ein wirklich sauberes TCP-Protokoll (*) 
sowieso falsch. Da ist sprintf nicht das Problem.

*: Beispiel: Wenn der Sendepuffer der Wiznet TCP/IP-Controller zuwenig 
Speicher für 2 Frames hat, dann sendet das Teil jeden Frame zweimal 
über's Netz um die Performance zu verbessern(!). Sonst wird's aufgrund 
von Timeouts nämlich viel langsamer. Soviel zum Thema Performance.

Wenn du schon solche String-Funktionen baust, dann bau die 
Buffer-Overflows nicht gleich mit ein. Also bau dir welche, die den 
fälligen Range-Check gleich mit enthalten.

Denk dran, dass im Netzpuffer vielleicht nicht genug Platz für den 
kompletten Ersatzstring ist. Wenn du dann mitten in einem solchen String 
neu aufsetzen musst wird's krass. Lieber vorher kontrollieren und ggf 
den alten Frame kürzer raussenden damit die Ersetzung garantiert in 
einem Stück passt.

die usage ist so gedacht: pBuffer+=stradd(pBuffer,g_CfgHtmlHeader);

Klar, aber besser wär's du machst ebendies mitsamt Überlauftest in der 
Funktion. Entweder mit globalen Variablen weil es sowieso nur einen 
Puffer gibt, oder ein Pufferdescriptor als Parameter für die Funktion. 
Erspart dir die Notwendigkeit, das überall in den Aufrufen 
rumzuschleppen.

jo das kan ich mir sparen und darunter target[Length] = 0 anstadt 
target[0] = 0 machen
stpcpy ist bei meinem WinAVR nicht definiert in string.h
aber strlcpy ist evl das richtige

hängt

Trax Xavier wrote:
> die usage ist so gedacht: pBuffer+=stradd(pBuffer,g_CfgHtmlHeader);


nicht besonders gut gedacht.
Du machst viel zu viele unnötige Operationen


Usage

1

    pBuffer = stradd( pBuffer, g_CfgHtmlHeader );

wobei stradd so definiert ist

1

char* stradd( char* Target, const char* Source )

2

{

3

  while( *source )

4

    *target++ = *source++;

5

  *target = '\0';

6

7

  return target;

8

}

Wenn du schon optimierst, dann machs richtig :-)

Das noch fehlende Argument für die target Länge musst du noch einbauen