Странный код JavaScript, который не может жить без console.log () - PullRequest
0 голосов
/ 30 ноября 2011

Я получил очень странный код: Без отладки, без работы! Я почти без ума от этого.

Код для отображения уведомления в Chrome.В коде много комментариев.

Посмотрите на !! ВАЖНО && WEIRD !!, следующая строка - "console.log (_notification);"это не может быть опущено, если это так, никакие события не могут работать.

Хотя теперь код работает нормально, но мне просто интересно, почему я не могу удалить "console.log (_notification);"?

    /**
     *  Notification
     *  @author:    ijse
     *  @require:   Chrome10+
     *  @params:    same as webkitNotifications.create[HTML]Notification()
     *  @usage:
     *      new Notify("http://www.baidu.com").onshow(function() {
     *              alert("show");
     *          }).onclose(function() {
     *              alert("close");
     *          }).show();
     */
    var Notify = function() {
        var _params = arguments;
        // Validate arguments
        if(_params.length == 0) {
            console.error("Notify need at least one argument");
            return ;
        }
        // Check browser support
        if(!window.webkitNotifications) {
            console.error("Your browser does not support webkitNotifications feature!!");
            return ;
        }

        var _onclose, _onclick, _onerror, _onshow;
        var _notification, _replaceId, _showFlag = false;

        function bindEvents() {
            // Add event listeners
            // In W3C, display event is called show
            _notification.addEventListener("display", _onshow, false);
            _notification.addEventListener("click", _onclick, false);
            _notification.addEventListener("error", _onerror, false);
            _notification.addEventListener("close", _onclose, false);

            if(_replaceId)
                _notification.replaceId = _replaceId;
            // !!IMPORTANT&&WEIRD!! remove next line no events will work
            console.log(_notification);
        }
        function createfn(permission) {
            // About permission on Chrome:
            //      PERMISSION_ALLOWED (0) indicates that the user has granted permission to scripts with this origin to show notifications.
            //      PERMISSION_NOT_ALLOWED (1) indicates that the user has not taken an action regarding notifications for scripts from this origin.
            //      PERMISSION_DENIED (2) indicates that the user has explicitly blocked scripts with this origin from showing notifications.
            if(permission == 0) {
                // If permission is allowed
                // Create notification
                if(_params.length == 1)
                    _notification = window.webkitNotifications.createHTMLNotification(_params[0]);
                else
                    _notification = window.webkitNotifications.createNotification(_params[0],_params[1],_params[2]);

                // Bind events
                console.log("bind event in createfn");
                bindEvents();

                // Show, if yes flag
                !!_showFlag && _notification.show();
            } else {
                if(_onerror)
                    _onerror.call(this);
                console.error("Notification permission is denied!!");
            }
        }

        // If permission already allowed, do not require again
        if(window.webkitNotifications.checkPermission() != 0) {
            // Require permission from user
            window.webkitNotifications.requestPermission(function() {
                createfn.call(this, window.webkitNotifications.checkPermission());
            });
        } else {
            createfn.call(this, window.webkitNotifications.checkPermission());
        }

        // Return handler methods
        return {
            onclose: function(fn) { _onclose = fn; console.log(1); return this; },
            onclick: function(fn) { _onclick = fn; console.log(2); return this; },
            onerror: function(fn) { _onerror = fn; console.log(3); return this; },
            onshow : function(fn) { _onshow  = fn; console.log(4); return this; },

            show: function(replaceId) {
                console.log("method show");
                _replaceId = replaceId;
                if(_notification) {
                    // Notification already been created
                    bindEvents();
                    _notification.show();
                } else {
                    // Flag yes to show
                    _showFlag = true;
                }
                return _notification;
            },
            cancel: function() {
                _notification.cancel();
            }
        } // return handler
    }


    new Notify("","Success!!", "Welcome to use empcy!!").onshow(function() {
        var that = this;
        window.setTimeout(function() { that.cancel(); }, 3000);
    }).onclose(function() {
        alert("close");
    }).onclick(function() {
        alert("clicked");
    }).show("Welcome");

Ответы [ 3 ]

2 голосов
/ 30 декабря 2011

Я думаю, что это может быть проблема с брекетингом.Вот соответствующий раздел кода, который у вас есть:

    function bindEvents() {
        // Add event listeners
        // In W3C, display event is called show
        _notification.addEventListener("display", _onshow, false);
        _notification.addEventListener("click", _onclick, false);
        _notification.addEventListener("error", _onerror, false);
        _notification.addEventListener("close", _onclose, false);

        if(_replaceId)
            _notification.replaceId = _replaceId;
        // !!IMPORTANT&&WEIRD!! remove next line no events will work
        console.log(_notification);
    }

Если вы удалите строку console.log, она станет такой:иметь оператор if без скобок, но за ним следует закрывающая скобка.Вы должны попытаться добавить скобки к вашему оператору if, чтобы он выглядел так:

    function bindEvents() {
        // Add event listeners
        // In W3C, display event is called show
        _notification.addEventListener("display", _onshow, false);
        _notification.addEventListener("click", _onclick, false);
        _notification.addEventListener("error", _onerror, false);
        _notification.addEventListener("close", _onclose, false);

        if(_replaceId) {
            _notification.replaceId = _replaceId;
        }
    }
0 голосов
/ 15 декабря 2011

В конце концов, я не разобрался, и не нашел ответов на это.Я просто поместил этот код, и он работает хорошо, за исключением печати информации журнала.

0 голосов
/ 30 ноября 2011

может быть, вы могли бы включить следующий код

https://github.com/h5bp/html5-boilerplate/blob/master/js/plugins.js

Это взято из проекта html5-шаблон, проверка, если можно вызвать console.log.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...