Kommentare zu: VS-Tipps&Tricks: Einfache Debug-Ausgabe mit TRACE auch in der Release Version

Von: Martin Richter

Martin Richter — Wed, 12 Aug 2009 07:30:51 +0000

Man kann den Code optimieren, indem man eine zusätzliche statische Variable mit diesem Code lädt und die entsprechenden Ausabefunktionen nur zulässt wenn die entsprechende statische Variable true ist.

 
bool IsDebuggerViewWaiting()
{
  if (IsDebuggerPresent())
    return true;
  else
  {
    HANDLE hFileMap = ::OpenFileMapping(FILE_MAP_READ, FALSE, _T("DBWIN_BUFFER"));
    if (hFileMap)
    {
      ::CloseHandle(hFileMap);
      return true;
    }
    return false;
  }
}

Von: Torsten

Torsten — Wed, 12 Aug 2009 07:20:58 +0000

Ach wenn wir grad an kleinen Hilfsmakros sind

 
//---------------------------------------------------------------------------------------------
// FIXMEs / TODOs / NOTE macros
//---------------------------------------------------------------------------------------------
#define _QUOTE(x) # x
#define QUOTE(x) _QUOTE(x)
#define __FILE__LINE__ __FILE__ "(" QUOTE(__LINE__) ") : "
 
#define NOTE( x )  message( x )
#define FILE_LINE  message( __FILE__LINE__ )
 
#define TODO( x )  message( __FILE__LINE__ "\n"           \
  " ------------------------------------------------\n" \
  "|  TODO :   " #x "\n" \
  " -------------------------------------------------\n" )
#define FIXME( x )  message(  __FILE__LINE__ "\n"           \
  " ------------------------------------------------\n" \
  "|  FIXME :  " #x "\n" \
  " -------------------------------------------------\n" )
#define todo( x )  message( __FILE__LINE__ " TODO :   " #x "\n" ) 
#define fixme( x ) message( __FILE__LINE__ " FIXME:   " #x "\n" )

Beispiel:

#pragma todo(“sonderbehandlung ja/nein”)

ergibt
1>DlgFz.cpp
1>.\dlgs\DlgFz.cpp(604) : TODO : “sonderbehandlung ja/nein”

PS: falls ich code zeug falsch gemacht hab, bitte nacharbeiten

Von: Torsten

Torsten — Wed, 12 Aug 2009 05:58:13 +0000

Dein DBWinMutex Vorschlag fand ich auch nicht so schlecht, allerdings bin ich eher der Typ der das dann lieber selber schreibt als irgendwas fertiges zu nehmen.
Das auch der Grund weshalb mein Zeug nach 10 Jahren auch so stark angewachsen ist.

Von: edgar

edgar — Tue, 11 Aug 2009 20:24:15 +0000

Richtig Torsten. Dein Alternativvorschlag kommt der Intention meines ersten posts schon sehr nahe.
Ich meinte einfach: Es fehlt noch etwas Optimierung.

Von: Torsten Schröder

Torsten Schröder — Tue, 11 Aug 2009 17:52:18 +0000

Das seh ich nich so edgar, das liese sich recht schnell per ifdef komplett aus dem Programm entfernen, und ein leeres define auf Trace wird wegoptimiert. Alternativ gänge auch ein zur Laufzeit gesetztes Bool, wenn man es ein-/ausschaltbar haben möchte.

Von: edgar

edgar — Tue, 11 Aug 2009 13:20:19 +0000

Nun Torsten, es wird ja erst in OutputDebugString() festgestellt,
ob überhaupt ein Trace gewünscht wird.
Die Codesequenz davor wird also immer ausgeführt, ob es notwendig ist oder nicht.

Von: Torsten

Torsten — Tue, 11 Aug 2009 05:47:04 +0000

Hm, Kurze Frage: Was hat _vsnprintf mit dem Debug Output zu tun?

Also nach meiner Meinung läuft das nicht langsamer, egal was ein oder ausgeschaltet ist.

Von: edgar

edgar — Mon, 10 Aug 2009 15:12:17 +0000

Ja Martin, nettes Beispiel.
Das würde ich aber trotzdem so nicht benutzen wollen.
_vsnprintf() frisst zuviel(unnötige) performance bei abgeschaltetem debug output.

Da würde sich ein vorheriger Blick auf “DBWinMutex” lohnen.

Von: Torsten

Torsten — Mon, 10 Aug 2009 13:05:40 +0000

Jep. Vor Jahren schon geschrieben und läuft tadellos…
Zweiter Tipp, man kann es auch noch erweitern und gleich in ein Logfile ausgeben. Dazu nutze ich es dann auch noch, dazu dann noch verschlüsselt und in einem eigenen Format, damit ich es dann später mit einem Programm mir einfach anzeigen und auch nach Nachrichten filtern kann.

Das ganze läuft in einem extra Thread bei mir. Aus den eigentlichen Arbeitsthreads kommen alle Trace Sachen in meiner Log.dll an. Diese sammelt die Nachrichten und wenn Zeit ist, schreibt sie sie parallel mit raus. Damit verliert man recht wenig Zeit.

Im Laufe der Jahre hat mich allerdings auch ein was immer mehr genervt.
Nämlich das man es nicht wie einen Stream beschreiben kann und damit einfacher und C++ konformer in sein Programm mit einbauen kann.

Irgendwann muss ich mir die Arbeit wohl mal noch machen und das alles noch einmal dahin mit umschreiben, so das man beides kann.