From 93de04e9b6eb85f44317157d21fe945b2a6ca2c3 Mon Sep 17 00:00:00 2001
From: Fred Boniface <fred@fjla.uk>
Date: Sat, 22 Apr 2023 22:44:32 +0100
Subject: [PATCH] Remove repeated DOM inserts

---
 js/pis.js | 13 ++++++++-----
 pis.html  |  2 +-
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/js/pis.js b/js/pis.js
index dd7ffca..c539fdb 100644
--- a/js/pis.js
+++ b/js/pis.js
@@ -27,21 +27,24 @@ async function insertData(json) {
     // PIS codes.  Display the code and the stops with a method of scrolling between them.
     // Maybe as a table or a carousel?
     const div = document.getElementById('result-box')
-    const tableHtml = `<table id="result-table">
+    let tableData = `<table id="result-table">
         <tr>
         <th class="code">Code</th>
         <th class="stops">Stations</th>
-        </tr>
-        </table>`
+        </tr>`
     div.insertAdjacentHTML("beforeend", tableHtml)
     const table = document.getElementById('result-table')
     const data = JSON.parse(json)
+    let results = 0
     for(var i = 0; i < data.length; i++) {  // Break into separate function.  Need to
                                             // iterate over stops.
-        let row = `<tr><td class="code">${data[i][code]}</td>
+        tableData += `<tr><td class="code">${data[i][code]}</td>
             <td class="stops">${data[i][stops]}</td></tr>`
-        table.insertAdjacentHTML("beforeend", row)
+        results++
     }
+    tableData += "</table>"
+    table.insertAdjacentHTML("beforeend", tableData)
+    document.getElementById('result-count').textContent = results
 }
 
 async function noData() {
diff --git a/pis.html b/pis.html
index 2af8dbd..0b04c0b 100644
--- a/pis.html
+++ b/pis.html
@@ -55,7 +55,7 @@
     </div>
 
     <div id="result-box">
-        <h3>Results</h3>
+        <h3>Results: <span id="result-count">0</span></h3>
         <!-- display: none; by default, some kind of box which shows each code
              and destination individually with a previous and next arrow. -->
     </div>