Can this AJAX + PHP code be improved in terms of security, performance, etc.? The Next CEO of Stack OverflowMySQLi_Recordset: blending SPL and Statement/Query resultsCan my router code be improved?How can this layered PHP architecture be improved?Can this code be improved upon?Can this PHP code to settle up payments be improved?Update database, refresh the page to show changes, show a success messageHow can this autoloader be improved?PHP & AJAX CAESAR CIPHER ENCRYPTED chat script - Slow in performancePHP function which filters content from Wordpress database (wp_posts table)PHP MySQLi wrapper class

Find non-case sensitive string in a mixed list of elements?

How to avoid supervisors with prejudiced views?

Is it professional to write unrelated content in an almost-empty email?

Is French Guiana a (hard) EU border?

Why the difference in type-inference over the as-pattern in two similar function definitions?

Writing differences on a blackboard

Why do airplanes bank sharply to the right after air-to-air refueling?

How to invert MapIndexed on a ragged structure? How to construct a tree from rules?

Running a General Election and the European Elections together

A Man With a Stainless Steel Endoskeleton (like The Terminator) Fighting Cloaked Aliens Only He Can See

Method for adding error messages to a dictionary given a key

Do I need to write [sic] when a number is less than 10 but isn't written out?

If Nick Fury and Coulson already knew about aliens (Kree and Skrull) why did they wait until Thor's appearance to start making weapons?

What steps are necessary to read a Modern SSD in Medieval Europe?

Why didn't Khan get resurrected in the Genesis Explosion?

Why is information "lost" when it got into a black hole?

Why isn't the Mueller report being released completely and unredacted?

What happened in Rome, when the western empire "fell"?

Is it possible to replace duplicates of a character with one character using tr

Axiom Schema vs Axiom

Does increasing your ability score affect your main stat?

How many extra stops do monopods offer for tele photographs?

Should I tutor a student who I know has cheated on their homework?

Can MTA send mail via a relay without being told so?



Can this AJAX + PHP code be improved in terms of security, performance, etc.?



The Next CEO of Stack OverflowMySQLi_Recordset: blending SPL and Statement/Query resultsCan my router code be improved?How can this layered PHP architecture be improved?Can this code be improved upon?Can this PHP code to settle up payments be improved?Update database, refresh the page to show changes, show a success messageHow can this autoloader be improved?PHP & AJAX CAESAR CIPHER ENCRYPTED chat script - Slow in performancePHP function which filters content from Wordpress database (wp_posts table)PHP MySQLi wrapper class










0












$begingroup$


I would like to know if this code follows recommended practices and standards for AJAX/PHP calls.



ajax.js:



function getTableData() 
const xmlhttp = new XMLHttpRequest();
xmlhttp.onreadystatechange = function()
if (this.readyState === 4 && this.status === 200)
document.querySelector('#test-table tbody').innerHTML = this.responseText;


xmlhttp.open('GET', 'ajax.php', true);
xmlhttp.send();


$(document).ready(function ()
getTableData();
);


ajax.php:



<?php
$data = prepareDataForTable();
$data_length = count($data);
$columns = 4;

for ($row = 0; $row < $data_length; $row += $columns)
echo '<tr>';
for ($col = 0; $col < $columns && ($col + $row) < $data_length; $col++)
echo '<td>' . $data[$col + $row] . '</td>';

echo '</tr>';

?>


ajax.html:



<table id="test-table">
<tbody>
</tbody>
</table>


I specifically want to know if this is a good way to send information back to the client - by running loops and echoing output.









share







New contributor




tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$
















    0












    $begingroup$


    I would like to know if this code follows recommended practices and standards for AJAX/PHP calls.



    ajax.js:



    function getTableData() 
    const xmlhttp = new XMLHttpRequest();
    xmlhttp.onreadystatechange = function()
    if (this.readyState === 4 && this.status === 200)
    document.querySelector('#test-table tbody').innerHTML = this.responseText;


    xmlhttp.open('GET', 'ajax.php', true);
    xmlhttp.send();


    $(document).ready(function ()
    getTableData();
    );


    ajax.php:



    <?php
    $data = prepareDataForTable();
    $data_length = count($data);
    $columns = 4;

    for ($row = 0; $row < $data_length; $row += $columns)
    echo '<tr>';
    for ($col = 0; $col < $columns && ($col + $row) < $data_length; $col++)
    echo '<td>' . $data[$col + $row] . '</td>';

    echo '</tr>';

    ?>


    ajax.html:



    <table id="test-table">
    <tbody>
    </tbody>
    </table>


    I specifically want to know if this is a good way to send information back to the client - by running loops and echoing output.









    share







    New contributor




    tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.







    $endgroup$














      0












      0








      0





      $begingroup$


      I would like to know if this code follows recommended practices and standards for AJAX/PHP calls.



      ajax.js:



      function getTableData() 
      const xmlhttp = new XMLHttpRequest();
      xmlhttp.onreadystatechange = function()
      if (this.readyState === 4 && this.status === 200)
      document.querySelector('#test-table tbody').innerHTML = this.responseText;


      xmlhttp.open('GET', 'ajax.php', true);
      xmlhttp.send();


      $(document).ready(function ()
      getTableData();
      );


      ajax.php:



      <?php
      $data = prepareDataForTable();
      $data_length = count($data);
      $columns = 4;

      for ($row = 0; $row < $data_length; $row += $columns)
      echo '<tr>';
      for ($col = 0; $col < $columns && ($col + $row) < $data_length; $col++)
      echo '<td>' . $data[$col + $row] . '</td>';

      echo '</tr>';

      ?>


      ajax.html:



      <table id="test-table">
      <tbody>
      </tbody>
      </table>


      I specifically want to know if this is a good way to send information back to the client - by running loops and echoing output.









      share







      New contributor




      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.







      $endgroup$




      I would like to know if this code follows recommended practices and standards for AJAX/PHP calls.



      ajax.js:



      function getTableData() 
      const xmlhttp = new XMLHttpRequest();
      xmlhttp.onreadystatechange = function()
      if (this.readyState === 4 && this.status === 200)
      document.querySelector('#test-table tbody').innerHTML = this.responseText;


      xmlhttp.open('GET', 'ajax.php', true);
      xmlhttp.send();


      $(document).ready(function ()
      getTableData();
      );


      ajax.php:



      <?php
      $data = prepareDataForTable();
      $data_length = count($data);
      $columns = 4;

      for ($row = 0; $row < $data_length; $row += $columns)
      echo '<tr>';
      for ($col = 0; $col < $columns && ($col + $row) < $data_length; $col++)
      echo '<td>' . $data[$col + $row] . '</td>';

      echo '</tr>';

      ?>


      ajax.html:



      <table id="test-table">
      <tbody>
      </tbody>
      </table>


      I specifically want to know if this is a good way to send information back to the client - by running loops and echoing output.







      php ajax





      share







      New contributor




      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.










      share







      New contributor




      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.








      share



      share






      New contributor




      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 7 mins ago









      tera_789tera_789

      1011




      1011




      New contributor




      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      tera_789 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.




















          0






          active

          oldest

          votes












          Your Answer





          StackExchange.ifUsing("editor", function ()
          return StackExchange.using("mathjaxEditing", function ()
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          );
          );
          , "mathjax-editing");

          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "196"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );






          tera_789 is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216572%2fcan-this-ajax-php-code-be-improved-in-terms-of-security-performance-etc%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          tera_789 is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          tera_789 is a new contributor. Be nice, and check out our Code of Conduct.












          tera_789 is a new contributor. Be nice, and check out our Code of Conduct.











          tera_789 is a new contributor. Be nice, and check out our Code of Conduct.














          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216572%2fcan-this-ajax-php-code-be-improved-in-terms-of-security-performance-etc%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          कुँवर स्रोत दिक्चालन सूची"कुँवर""राणा कुँवरके वंशावली"

          शेव्रोले वोल्ट अनुक्रम इतिहास इन्हे भी देखें चित्र दीर्घा संदर्भ दिक्चालन सूची

          चैत्य भूमि चित्र दीर्घा सन्दर्भ बाहरी कडियाँ दिक्चालन सूची"Chaitya Bhoomi""Chaitya Bhoomi: Statue of Equality in India""Dadar Chaitya Bhoomi: Statue of Equality in India""Ambedkar memorial: Centre okays transfer of Indu Mill land"चैत्यभमि