Jquery dynamische Paginierung für Kommentare Expertenmeinung

Question

Ich versuche, ein dynamisches Paginierungssystem für meine Website zu erstellen, und ich habe beschlossen, mit Ajax Callback-Funktion dynamisch die Kommentare anzuzeigen, wenn der Benutzer auf verschiedene Seitenelemente klickt. Also ich Ich gebe den folgenden Code als Beispiel dafür an, wie ich es mache ... Ich würde Sie gerne fragen, ob ich es richtig mache oder ob es ein besserer Weg ist, um es zu tun. Bitte weisen Sie mich in die richtige Richtung :

javascript:

$(document).ready(function(){ 
$(".pages").click(function(){ 
var pageno=$(this).attr("id").slice(4,8);//slicing out numbers from the id i.e "1" from "page1" 
$.get("news.php", { pageno: pageno}, 
    function(data){ 
    $("#comments").html(data); 
    }); 
}); 
}); 

html:

<div> 
<span id="page1" class="pages" >1</span> 
<span id="page2" class="pages" >2</span> 
<span id="page3" class="pages" >3</span> 
</div > 
<div id="comments"> 
</div> 

php:

<div><?php echo $_REQUEST['pageno'];?></div> 

Answer 1

Ich glaube nicht, das ist nicht vertretbar ist, obwohl ich zwei Gedanken erwähnen würde.

Erstens, ich denke, es ist eine gute Sache, die Click-Handler dynamisch zuweisen, anstatt tatsächlich "click =" in den HTML-Code (das ist das Herz der "progressive Enhancement"). Hover, möchten Sie vielleicht eine andere Art von Selektor verwenden. Das Auffinden einer Gruppe von Elementen nach Klassen ist eine der langsameren Möglichkeiten zum Definieren von Elementen, insbesondere wenn es viele Seiten gibt. Du könntest sie alle in ein Div setzen und $ ("# myDiv span") machen. Das sollte viel schneller sein.

Zweitens scheint die Annahme, dass die Seitennummer immer am vierten Zeichen der Span-ID beginnt, etwas fragil zu sein. Es ist vielleicht besser, die ID selbst als Seitennummer zu definieren.

Answer 2

$(document).ready(function(){ 
    var isWorking = false; 
    $(".pages").click(function(){ 
     if (isWorking) 
     { 
      alert('Please wait for your preceeding request to finish.'); 
      return false; 
     } 
     isWorking = true; 
     var pageno = $(this).text(); 
     $.get("news.php", { pageno: pageno}, 
      function(data){ 
       $("#comments").html(data); 
       isWorking = false; 
      }); 
    }); 
}); 

1. Es sieht aus wie Sie einfach den Textinhalt der Spanne für die Seitenzahl (siehe .text() ändern) nutzen könnten.

2. Sie möchten möglicherweise verhindern, dass Benutzer versuchen, zu viele Seiten gleichzeitig zu laden (siehe die Variable isWorking, die verhindern soll, dass Benutzer mehr als 1 Seite gleichzeitig laden).