spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Also bevor du so rumbasteln gehst, würde ich doch einfach die Warnings auf ein niedrigeres Level schalten, wenn sie dich stören.

Zitat von »"n0_0ne"«

	C-/C++-Quelltext
1	#define strcpy(a, b) (strcpy_s((a), sizeof((a)), (b)))

soweit so gut ich denke mal, es ist nicht grade die perfekte lösung, aber für den arbeitsaufwand doch schonmal nich schlecht und macht es auch schon etwas sicherer

:shock:

sicherer? ich würde mal sagen du hast extremes glück wenn das nicht abstürzt. schonmal überlegt was sizeof((a)) bedeutet!? wenn a ein zeiger is (und das ist es wohl meistens, weil die funktion immerhin an der stelle einen zeiger erwartet), dann wird das mächtig in die hose gehen.

_CRT_SECURE_NO_DEPRECATE definieren und gut ist.

warnings haben nen sinn. man sollte nicht einfach alles daran setzen sie wegzubekommen, sondern erstmal besser den ursachen nachgehen...

Zitat von »"n0_0ne"«

warning C4002: too many actual parameters for macro 'sprintf'

was genau wird denn hier dann gemacht? wird das macro einfach ignoriert, oder die 3 parameter ersetzt, und der rest halt so wie es da steht auch? (würde das problem auch lösen)?

sprintf is ne funktion mit ner variablen anzahl an parametern. mit dem makro wirst du da nicht weit kommen. wenn dann wirst du da auf variadic macros setzen müssen (ist zwar nonstandard, aber da du eh die _s funktionen verwendest schätze ich mal dass das egal ist).

anyway. wenn du meinen rat willst: ich würd den ganzen #define kram möglichst schnell wieder loswerden...

Zitat von »"drakon"«

[...] aber i.d.R ist es nicht sonderlich gut die Grösse eines Arrays mittels sizeof zu ermitteln. Vor allem funktioniert das mit dynamisch angefordertem Speicher nicht mehr.

es is kein problem mit sizeof die größe eines arrays zu bestimmen, da spricht nichts dagegen.
das problem ist, dass viele denken das arrays und zeiger das gleiche sind. sind sie aber nicht. auch wenn man den namen eines arrays wie einen zeiger verwenden kann. und das führt dann genau zu sowas wie dem define da oben...
aber ja, du hast recht. bei dynamisch angefordertem speicher funzt das nicht, eben weil da kein array zurückkommt

Zitat

es is kein problem mit sizeof die größe eines arrays zu bestimmen, da spricht nichts dagegen.

Joa,habe auch nicht gesagt, dass es richtig angewendet ein Problem ist.
Aber wegen dem, was du noch geschrieben hast, ist es keine gute Idee. Im übrigen ist es ja eh Sinnfrei das für ein statisches Array zu machen, da dort die Grösse zur Kompilierzeit bekannt ist und man somit das sizeof auch mit einer konstanten erstzen kann.

EDIT:

Zitat

auch wenn man den namen eines arrays wie einen zeiger verwenden kann.

Auch nicht ganz.

da brauchts gar kein größeres projekt:



das reicht schon...

@No_One
dot hat Recht mit seinem Beispiel zu Deinem Macro.

Beispiel :

char buf[256];
strcpy_s(buf, sizeof(buf), "irgendeintext"); // sizeof(buf)=256

Stattdessen ist folgendes FALSCH :

char* buf = new char[256]; // sizeof(buf)=4



Schau Dir den Befehl tbGetD3DFormatString() in tbUtils.cpp an.
Dort findest Du

case D3DFMT_R8G8B8: strcpy(pcOut, "R8G8B8"); break;

Dein Macro gibt für die Länge immer 4 zurück, weil pcOut ein Zeiger ist.
Das stürzt dann zwar im ersten Moment nicht ab, weil anstatt der 6 Buchstaben nur 3 kopiert werden (+ Nullbyte am Ende ergibt die max. erlaubten 4).
Aber mit dem Ergebnis dürftest Du nicht zufrieden sein.

Eine Lösung wäre :

tbGetD3DFormatString(D3DFORMAT Format, char* pcOut, int laengeOut)

case D3DFMT_R8G8B8: strcpy_s(pcOut, laengeOut, "R8G8B8"); break;

Nach Durchsicht der Tribase-Texte scheint tbGetD3DFormatString() aber das einzige Mal zu sein, wo David einen Zeiger benutzt.
Die anderen Male hat er immer ein array geschrieben.

almidi