0

我正在为 Jquery 获取 .getJSON 函数的 JSON 数据。我正在考虑使用 .text 来确保数据安全(我相信这是正确的做法)。我已经测试过了,JSON 是正确的。

这是我正在处理的脚本:

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

^ 全局变量

        $.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 ......

此外,我正在尝试获取 data['New'] 对象/数组中的最后一个值。.length 似乎不起作用。

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

这是我的第一个 javascript/Jquery 程序,所以如果您在代码中发现有问题,请告诉我。

4

1 回答 1

1

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

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

注意:Google、jquery 等都遵循这个实践系统!

于 2013-03-19T04:38:45.153 回答