Ich habe eine Menge console.log() Aufrufe in meinem JavaScript.

Sollte ich sie auskommentieren, bevor ich sie in der Produktion bereitstelle?

Ich möchte sie einfach dort belassen, damit ich mir später nicht die Mühe machen muss, die Kommentare erneut hinzuzufügen, wenn ich noch mehr debuggen muss. Ist das eine schlechte Idee?

Es verursacht Javascript-Fehler und beendet die Ausführung des Javascript-Blocks, der den Fehler enthält.

Sie könnten jedoch eine Dummy-Funktion definieren, die keine Operation ist, wenn Firebug nicht aktiv ist:

if(typeof console === "undefined") {
    console = { log: function() { } };
}

Wenn Sie andere Methoden als verwenden logmüssten Sie diese auch ausdrücken.

Wie andere bereits darauf hingewiesen haben, führt das Belassen zu Fehlern in einigen Browsern, aber diese Fehler können umgangen werden, indem einige Stubs eingefügt werden.

Ich würde sie jedoch nicht nur auskommentieren, sondern diese Zeilen direkt entfernen. Es scheint einfach schlampig zu sein, etwas anderes zu tun. Vielleicht bin ich pedantisch, aber ich denke nicht, dass “Produktionscode” überhaupt “Debug”-Code enthalten sollte, nicht einmal in kommentierter Form. Wenn Sie überhaupt Kommentare hinterlassen, sollten diese Kommentare beschreiben, was der Code macht, oder die Gründe dafür – nicht Blöcke von deaktiviertem Code. (Obwohl die meisten Kommentare automatisch durch Ihren Minimierungsprozess entfernt werden sollten. Sie minimieren doch, oder?)

Außerdem kann ich mich in mehreren Jahren der Arbeit mit JavaScript nicht erinnern, jemals zu einer Funktion zurückgekehrt zu sein und gesagt zu haben: “Mensch, ich wünschte, ich hätte diese console.logs hier gelassen!” Wenn ich mit der Arbeit an einer Funktion “fertig” bin und später darauf zurückkommen muss, komme ich im Allgemeinen zurück, um ein anderes Problem zu beheben. Was auch immer dieses neue Problem ist, wenn die console.logs aus einer früheren Arbeitsrunde hilfreich gewesen wären, dann hätte ich das Problem beim ersten Mal entdeckt. Mit anderen Worten, wenn ich auf etwas zurückkomme, benötige ich wahrscheinlich nicht genau dieselben Debug-Informationen wie bei früheren Gelegenheiten.

Nur meine zwei Cent… Viel Glück!

Wenn Sie über ein Bereitstellungsskript verfügen, können Sie damit die Aufrufe von console.log entfernen (und die Datei minimieren).

Wenn Sie schon dabei sind, können Sie Ihr JS durch JSLint werfen und die Verstöße zur Überprüfung protokollieren (oder die Bereitstellung verhindern).

Dies ist ein hervorragendes Beispiel dafür, warum Sie Ihre Bereitstellung automatisieren möchten. Wenn Ihr Prozess es Ihnen erlaubt, eine js-Datei mit console.logs darin zu veröffentlichen, müssen Sie irgendwann Wille Tu es.

Meines Wissens nach gibt es keine kürzere Methode zum Stubben console.log als die folgenden 45 Zeichen:

window.console||(console={log:function(){}});

Das ist die erste von 3 verschiedenen Versionen, je nachdem, welche Konsolenmethoden Sie verwenden möchten, alle sind winzig und alle wurden in IE6+ und modernen Browsern getestet.

Die anderen beiden Versionen decken verschiedene andere Konsolenmethoden ab. Eine behandelt die vier Grundlagen und die andere alle bekannten Konsolenmethoden für Firebug und Webkit. Auch hier in den kleinstmöglichen Dateigrößen.

Dieses Projekt ist auf github: https://github.com/andyet/ConsoleDummy.js

Wenn Ihnen eine Möglichkeit einfällt, den Code weiter zu minimieren, sind Beiträge willkommen.

— BEARBEITEN — 16. Mai 2012

Ich habe seitdem diesen Code verbessert. Es ist immer noch winzig, fügt aber die Möglichkeit hinzu, die Konsolenausgabe ein- und auszuschalten: https://github.com/HenrikJoreteg/andlog

Es war in der Changelog Show vorgestellt

Sie sollten zumindest einen Dummy erstellen console.log Wenn das Objekt nicht vorhanden ist, wird Ihr Code keine Fehler auf den Computern der Benutzer ausgeben, ohne dass Firebug installiert ist.

Eine andere Möglichkeit wäre, das Logging nur im ‘Debug Mode’ auszulösen, also wenn ein bestimmtes Flag gesetzt ist:

if(_debug) console.log('foo');
_debug && console.log('foo');

Hoffe, es hilft jemandem – ich habe vor einiger Zeit einen Wrapper dafür geschrieben, der etwas flexibler ist als die akzeptierte Lösung.

Wenn Sie andere Methoden wie console.info usw. verwenden, können Sie den Effekt natürlich replizieren. Wenn Sie mit Ihrer Staging-Umgebung fertig sind, ändern Sie einfach den Standard-C.debug für die Produktion auf „false“ und Sie müssen keinen anderen Code ändern / Zeilen entfernen usw. Sehr einfach, später darauf zurückzukommen und zu debuggen.

var C = {
    // console wrapper
    debug: true, // global debug on|off
    quietDismiss: false, // may want to just drop, or alert instead
    log: function() {
        if (!C.debug) return false;

        if (typeof console == 'object' && typeof console.log != "undefined") {
            console.log.apply(this, arguments); 
        }
        else {
            if (!C.quietDismiss) {
                var result = "";
                for (var i = 0, l = arguments.length; i < l; i++)
                    result += arguments[i] + " ("+typeof arguments[i]+") ";

                alert(result);
            }
        }
    }
}; // end console wrapper.

// example data and object
var foo = "foo", bar = document.getElementById("divImage");
C.log(foo, bar);

// to surpress alerts on IE w/o a console:
C.quietDismiss = true;
C.log("this won't show if no console");

// to disable console completely everywhere:
C.debug = false;
C.log("this won't show ever");

das scheint bei mir zu funktionieren…

if (!window.console) {
    window.console = {
        log: function () {},
        group: function () {},
        error: function () {},
        warn: function () {},
        groupEnd: function () {}
    };
}

Ich habe eine Menge console.log() Aufrufe in meinem JavaScript.

Sollte ich sie auskommentieren, bevor ich sie in der Produktion bereitstelle?

Ich möchte sie einfach dort belassen, damit ich mir später nicht die Mühe machen muss, die Kommentare erneut hinzuzufügen, wenn ich noch mehr debuggen muss. Ist das eine schlechte Idee?

Es verursacht Javascript-Fehler und beendet die Ausführung des Javascript-Blocks, der den Fehler enthält.

Sie könnten jedoch eine Dummy-Funktion definieren, die keine Operation ist, wenn Firebug nicht aktiv ist:

if(typeof console === "undefined") {
    console = { log: function() { } };
}

Wenn Sie andere Methoden als verwenden logmüssten Sie diese auch ausdrücken.

Wie andere bereits darauf hingewiesen haben, führt das Belassen zu Fehlern in einigen Browsern, aber diese Fehler können umgangen werden, indem einige Stubs eingefügt werden.

Ich würde sie jedoch nicht nur auskommentieren, sondern diese Zeilen direkt entfernen. Es scheint einfach schlampig zu sein, etwas anderes zu tun. Vielleicht bin ich pedantisch, aber ich denke nicht, dass “Produktionscode” überhaupt “Debug”-Code enthalten sollte, nicht einmal in kommentierter Form. Wenn Sie überhaupt Kommentare hinterlassen, sollten diese Kommentare beschreiben, was der Code macht, oder die Gründe dafür – nicht Blöcke von deaktiviertem Code. (Obwohl die meisten Kommentare automatisch durch Ihren Minimierungsprozess entfernt werden sollten. Sie minimieren doch, oder?)

Außerdem kann ich mich in mehreren Jahren der Arbeit mit JavaScript nicht erinnern, jemals zu einer Funktion zurückgekehrt zu sein und gesagt zu haben: “Mensch, ich wünschte, ich hätte diese console.logs hier gelassen!” Wenn ich mit der Arbeit an einer Funktion “fertig” bin und später darauf zurückkommen muss, komme ich im Allgemeinen zurück, um ein anderes Problem zu beheben. Was auch immer dieses neue Problem ist, wenn die console.logs aus einer früheren Arbeitsrunde hilfreich gewesen wären, dann hätte ich das Problem beim ersten Mal entdeckt. Mit anderen Worten, wenn ich auf etwas zurückkomme, benötige ich wahrscheinlich nicht genau dieselben Debug-Informationen wie bei früheren Gelegenheiten.

Nur meine zwei Cent… Viel Glück!

Wenn Sie über ein Bereitstellungsskript verfügen, können Sie damit die Aufrufe von console.log entfernen (und die Datei minimieren).

Wenn Sie schon dabei sind, können Sie Ihr JS durch JSLint werfen und die Verstöße zur Überprüfung protokollieren (oder die Bereitstellung verhindern).

Dies ist ein hervorragendes Beispiel dafür, warum Sie Ihre Bereitstellung automatisieren möchten. Wenn Ihr Prozess es Ihnen erlaubt, eine js-Datei mit console.logs darin zu veröffentlichen, müssen Sie irgendwann Wille Tu es.

Meines Wissens nach gibt es keine kürzere Methode zum Stubben console.log als die folgenden 45 Zeichen:

window.console||(console={log:function(){}});

Das ist die erste von 3 verschiedenen Versionen, je nachdem, welche Konsolenmethoden Sie verwenden möchten, alle sind winzig und alle wurden in IE6+ und modernen Browsern getestet.

Die anderen beiden Versionen decken verschiedene andere Konsolenmethoden ab. Eine behandelt die vier Grundlagen und die andere alle bekannten Konsolenmethoden für Firebug und Webkit. Auch hier in den kleinstmöglichen Dateigrößen.

Dieses Projekt ist auf github: https://github.com/andyet/ConsoleDummy.js

Wenn Ihnen eine Möglichkeit einfällt, den Code weiter zu minimieren, sind Beiträge willkommen.

— BEARBEITEN — 16. Mai 2012

Ich habe seitdem diesen Code verbessert. Es ist immer noch winzig, fügt aber die Möglichkeit hinzu, die Konsolenausgabe ein- und auszuschalten: https://github.com/HenrikJoreteg/andlog

Es war in der Changelog Show vorgestellt

Sie sollten zumindest einen Dummy erstellen console.log Wenn das Objekt nicht vorhanden ist, wird Ihr Code keine Fehler auf den Computern der Benutzer ausgeben, ohne dass Firebug installiert ist.

Eine andere Möglichkeit wäre, das Logging nur im ‘Debug Mode’ auszulösen, also wenn ein bestimmtes Flag gesetzt ist:

if(_debug) console.log('foo');
_debug && console.log('foo');

Hoffe, es hilft jemandem – ich habe vor einiger Zeit einen Wrapper dafür geschrieben, der etwas flexibler ist als die akzeptierte Lösung.

Wenn Sie andere Methoden wie console.info usw. verwenden, können Sie den Effekt natürlich replizieren. Wenn Sie mit Ihrer Staging-Umgebung fertig sind, ändern Sie einfach den Standard-C.debug für die Produktion auf „false“ und Sie müssen keinen anderen Code ändern / Zeilen entfernen usw. Sehr einfach, später darauf zurückzukommen und zu debuggen.

var C = {
    // console wrapper
    debug: true, // global debug on|off
    quietDismiss: false, // may want to just drop, or alert instead
    log: function() {
        if (!C.debug) return false;

        if (typeof console == 'object' && typeof console.log != "undefined") {
            console.log.apply(this, arguments); 
        }
        else {
            if (!C.quietDismiss) {
                var result = "";
                for (var i = 0, l = arguments.length; i < l; i++)
                    result += arguments[i] + " ("+typeof arguments[i]+") ";

                alert(result);
            }
        }
    }
}; // end console wrapper.

// example data and object
var foo = "foo", bar = document.getElementById("divImage");
C.log(foo, bar);

// to surpress alerts on IE w/o a console:
C.quietDismiss = true;
C.log("this won't show if no console");

// to disable console completely everywhere:
C.debug = false;
C.log("this won't show ever");

das scheint bei mir zu funktionieren…

if (!window.console) {
    window.console = {
        log: function () {},
        group: function () {},
        error: function () {},
        warn: function () {},
        groupEnd: function () {}
    };
}