Discuz.gtimg.cn不明代码的解决办法

在网站开发和维护过程中，我们经常会遇到一些棘手的问题，例如JavaScript代码冲突。最近，在排查一个频道页面的问题时，我发现了一个由底部代码引起的冲突问题。以下是详细的解决过程和分析，希望对您有所帮助。 ### 问题定位最初，我通过排除法确定问题来源于页面底部的某些脚本代码。既然问题出现在底部，那么我们可以直接检查模版文件中的`footer.htm`部分。经过初步分析，发现该文件中包含了一段之前不存在的新代码。进一步调查后，我发现这段代码是在应用0339补丁后新增的。为了更精确地找到问题根源，我在0339补丁中搜索了相关代码，最终定位到`source/function/function_cloud.php`文件中的一段关键代码：


var clearTips;
DiscuzCloudNameSpace = new Object();
DiscuzCloudNameSpace.register = function(fullName) {
    try {
        var nsArray = fullName.split(".");
        var strNS = "";
        var strEval = "";
        for (var i = 0; i < nsArray.length; i++) {
            if (strNS.length > 0) strNS += ".";
            strNS += nsArray[i];
            strEval += " if (typeof(" + strNS + ") == 'undefined') " + strNS + " = new Object(); ";
        };
        if (strEval != "") eval(strEval)
    } catch(e) {
        alert(e.message)
    }
};
DiscuzCloudNameSpace.register('DiscuzCloud');

### 问题分析从上述代码可以看出，它尝试注册一个名为`DiscuzCloud`的对象，并使用了`eval`函数动态生成代码。这种做法虽然灵活，但也带来了潜在的风险，比如可能影响页面上其他JavaScript代码的正常运行。此外，该代码还加载了一个来自`gtimg.cn`的外部JS文件。需要注意的是，这个域名与官方并无直接关联，且下载的JS文件使用了`eval(function(p,a,c,k,e,d)`加密方式，这增加了调试难度。 ### 解密后的代码解密后得到的代码如下所示：


DiscuzCloud.JSONP = (function() {
    var counter = 0,
    head, query, key, window = this;
    function load(url) {
        script = document.createElement('script'),
        done = false;
        script.src = url;
        script.charset = 'UTF-8';
        script.async = true;
        script.onload = script.onreadystatechange = function() {
            if (!done && (!this.readyState || this.readyState === "loaded" || this.readyState === "complete")) {
                done = true;
                script.onload = script.onreadystatechange = null;
                if (script && script.parentNode) {
                    script.parentNode.removeChild(script)
                }
            }
        };
        if (!head) {
            head = document.getElementsByTagName('head')[0]
        };
        head.appendChild(script)
    };
    function jsonp(url, params, callback) {
        if (url.indexOf('?') > -1) {
            query = '&'
        } else {
            query = '?'
        };
        params = params || {};
        for (key in params) {
            if (params.hasOwnProperty(key)) {
                query += encodeURIComponent(key) + "=" + encodeURIComponent(params[key]) + "&"
            }
        };
        var jsonp = 'discuzTipsCallback';
        window[jsonp] = function(data) {
            callback(data);
            try {
                delete window[jsonp]
            } catch(e) {}
            window[jsonp] = null
        };
        load(url + query + "callback=" + jsonp);
        return jsonp
    };
    return {
        get: jsonp
    }
} ());

### 总结通过以上分析，我们可以得出以下结论： 1. **安全性问题**：引入的外部JS文件来源不明，可能存在安全隐患。 2. **性能问题**：使用`eval`函数可能会降低代码执行效率，并且增加了代码维护难度。 3. **功能冲突**：新引入的代码可能与其他已有JS代码产生冲突，导致页面功能异常。为了解决这些问题，建议采取以下措施： - 审查所有外部JS文件的来源，确保其安全性。 - 替换或移除不必要的`eval`调用，改用更安全的代码实现方式。 - 对现有代码进行充分测试，确保修改不会影响其他功能模块。希望这篇文章能帮助您更好地理解和解决类似问题。如果您有更多疑问或需要进一步的帮助，请随时联系技术支持团队。
注意：本文内容仅供参考，请根据实际情况调整解决方案。

推荐阅读