Question

我有这个PHP：

<?php

$data = file_get_contents('http://newsrss.bbc.co.uk/rss/sportonline_uk_edition/football/rss.xml');

$xml = simplexml_load_string($data);

$data1 = file_get_contents('http://www.skysports.com/rss/0,20514,11661,00.xml');

$xml1 = simplexml_load_string($data1);

$master[0] = $xml;
$master[1] = $xml1;

echo json_encode($master);

?>

这个jQuery：

    var myRSS =[];

function rssReader() {

    console.log('ran');

    $.getJSON('bbc.php', function(data){
        console.log(data);
        $.each(data[0].channel.item, function(index, item){
            // check if item title is stored in the array   
            if (jQuery.inArray(item.title, myRSS) != -1) { 
                //do nothing
            } else {
                // save item title in the array
                myRSS.push(item.title);

                // publish item  
                $('.container').prepend("<a target='_BLANK' href='" + item.link + "' class='title' data-date='" + item.pubDate + "'>" + item.title + "</a>");
                $("title").text('EMG: ' + item.title);
                $('#loader').remove();
            }
        });

        $.each(data[1].channel.item, function(index, item){
            // check if item title is stored in the array   
            if (jQuery.inArray(item.title, myRSS) != -1) { 
                //do nothing
            } else {
                // save item title in the array
                myRSS.push(item.title);

                // publish item  
                $('.container').prepend("<a target='_BLANK' href='" + item.link + "' class='title' data-date='" + item.pubDate + "'>" + item.title + "</a>");
                $("title").text('EMG: ' + item.title);
            }
        });

    });

}

rssReader();

setInterval(rssReader, 10000);

似乎有很多重复的代码，因此不是很干的编程。返回的JSON实际上具有相同的结构，来自BBC和Sky Sports，所以必须有一种更有效的方式来编写它。

由于

Answer 1

你可以将你的jquery削减到这个（没有测试它可能有一个错字）：

  var myRSS =[];

    function rssReader() {

        console.log('ran');

        $.getJSON('bbc.php', function(data){
            console.log(data);
            $.each(data[0].channel.item, function(index, item){
                linkhandler(item)
                if (jQuery.inArray(item.title, myRSS) == -1) { 
                    $('#loader').remove();
                }
            });

            $.each(data[1].channel.item, function(index, item){
               linkhandler(item)
            });

        });

    }
    function linkhandler(item)
    { // check if item title is stored in the array   
       if (jQuery.inArray(item.title, myRSS) == -1) { 
             // save item title in the array
             myRSS.push(item.title);
             $('.container').prepend("<a target='_BLANK' href='" + item.link + "' class='title' data-date='" + item.pubDate + "'>" + item.title + "</a>");
             $("title").text('EMG: ' + item.title);
       }
    }
    rssReader();

    setInterval(rssReader, 10000);

Answer 2

为了让事情变得更清洁，你可以这样做：

var myRSS =[];

var handleData = function(index, item) {
    // check if item title is stored in the array   
    if (jQuery.inArray(item.title, myRSS) != -1) { 
        //do nothing
    } else {
        // save item title in the array
        myRSS.push(item.title);

        // publish item  
        $('.container').prepend("<a target='_BLANK' href='" + item.link + "' class='title' data-date='" + item.pubDate + "'>" + item.title + "</a>");
        $("title").text('EMG: ' + item.title);
        $('#loader').remove();
    }
}


function rssReader() {
    console.log('ran');

    $.getJSON('bbc.php', function(data){
        console.log(data);
        $.each(data[0].channel.item, handleData);
        $.each(data[1].channel.item, handleData);
    });

}

rssReader();

setInterval(rssReader, 10000);

但是，就效率而言，我并不认为这是不同的。但是，它至少更清洁。

Answer 3

只需注意一点，尝试对DOM进行较少的连续更改。在一些答案中，无论如何它都是最小的，所以不是太多的问题。但是，与构建空白对象并将其大量插入或对列表等进行大量更改相比，对DOM的每次更改都会产生巨大的开销。

有没有更有效的方法来编写这个PHP / jQuery

3 个答案: