Question

我正在为Jquery获取.getJSON函数的JSON数据。我正在考虑使用.text来保证数据安全（我相信这是正确的事情）。我已经测试过，JSON是正确的。

以下是我正在处理的脚本：

var firstpost = 0;
var firstrun = 0;
var lastpost = 0;

^ Global vars

        $.getJSON('chatget.php', {
    'chatroomid' : '<?php echo $chatroomid; ?>',
    'firstpost': firstpost,
    'lastpost': '1'},
    function(data) {
        var template = '<div id="_ID_" class="chatpost"> <div><b>_NAME_ </b> <a href="_URL_"> _USERNAME_ </a> _DATETIME_</div> <div><em>_TARGETS_</em></div> <div>_TEXT_</div> </div>';
        var appendhtml ='';

        var datarows = data['New'].length;
        lastpost = data['New'][datarows]['CPid'];

        // Each row processor               
        $.each(data['New'], function(index, col){

        // Get initial data
        if (firstrun == 0){
        firstpost = col.CPid;
        firstrun = 1;}

            // process targets
            if(col.Targets !== null){
                var target = col.Targets.split(',');
                var trow = target.length;
                var targets = '';
                for (var i=0, len=target.length; i<len; i++){

                    targets = targets + '@' + target[i] + ' ';}     
            }else {var targets = '';};

            // Append data to chatroom
            var cpid = $.text(col.CPid);
            var name = $.text(col.Name);
            var username = $.text(col.Username);
            var url = $.text(col.Url);
            var text = $.text(col.Text);
            var datetime = $.text(col.Datetime);
            var targets = $.text(targets);

            appendhtml = template.replace('_ID_',cpid).replace('_NAME_',name).replace('_USERNAME_',username).replace('_URL_',url).replace('_TEXT_',text).replace('_DATETIME_',date).replace('_TARGETS_', targets);

        $('#chatroom').append(appendhtml);
        });






        } // End Data function
    ) // End Get Json

出于某种原因，因为我在这段代码中更改了一些东西，它正在崩溃firebug，所以要么我发现了firebug中的错误，要么我在编码中做了一些非常错误的事情。我认为我没有正确使用$ .text ...

另外，我试图获取数据['New']对象/数组中的最后一个值。 .length似乎没有起作用。

            var datarows = data['New'].length;
        lastpost = data['New'][datarows]['CPid'];

这是我的第一个javascript / Jquery程序，所以如果你在代码中看到错误请告诉我。

Answer 1

为了获得最佳实践，请将您的代码封装在anonymouse包装函数中，以便您创建/使用的任何函数或变量都无法被外部环境访问。

(function(){
    //your code
}())

注意：Google，jquery等都遵循这个练习系统！

Javascript / Jquery防止执行代码的注入

1 个答案: