fixed changes suggested by jshint

Author: jyotiska

prettytable.js

@@ -17,40 +17,40 @@ exports.create = function(headers, rows) {
     for (var i=0; i < rows.length; i++) {
         addTableRow(rows[i]);
     }
-}
+};
 
 // Define list of columns for the table
 exports.fieldNames = function(names) {
     addTableHeader(names);
-}
+};
 
 // Add a single row to the table
 exports.addRow = function(row) {
     addTableRow(row);
-}
+};
 
 // Convert the table to string
 exports.toString = function() {
     return tableToString();
-}
+};
 
 // Write the table string to the console
 exports.print = function() {
     console.log(tableToString());
-}
+};
 
 // Write the table string to the console as HTML table formats
 exports.html = function(attributes) {
     return tableToHTML(attributes);
-}
+};
 
 // Create the table from a CSV file
 exports.csv = function(filename) {
     var stream = fs.createReadStream(filename);
     line_counter = 0;
     var csvStream = csv()
         .on("data", function(data){
-            if (line_counter == 0) {
+            if (line_counter === 0) {
                 addTableHeader(data);
                 line_counter += 1;
             } else {
@@ -62,67 +62,67 @@ exports.csv = function(filename) {
             console.log(tableToString());
         });
     stream.pipe(csvStream);
-}
+};
 
 // Create the table from a JSON file
 exports.json = function(filename) {
     var jsondata = JSON.parse(fs.readFileSync(filename, 'utf8'));
     for (var i=0; i < jsondata.length; i++) {
         rowKeys = Object.keys(jsondata[i]);
-        rowVals = []
+        rowVals = [];
         for (var k=0; k < rowKeys.length; k++) {
             rowVals.push(jsondata[i][rowKeys[k]]);
         }
-        if (table.columnNames.length == 0) {
+        if (table.columnNames.length === 0) {
             addTableHeader(rowKeys);
         }
         addTableRow(rowVals);
     }
     console.log(tableToString());
-}
+};
 
 // Sort the table given a column in ascending or descending order
 exports.sortTable = function(colname, reverse) {
     var reverseSort = false;
-    if (typeof(reverse) == "boolean" && reverse == true) {
+    if (typeof(reverse) === "boolean" && reverse === true) {
         reverseSort = true;
     }
     sortTableByColumn(colname, reverseSort);
-}
+};
 
 // Delete a single row from the table given row number
 exports.deleteRow = function(rownum) {
     if (rownum <= table.rows.length && rownum > 0) {
         table.rows.splice(rownum-1, 1);
     }
-}
+};
 
 // Clear the contents from the table, but keep columns and structure
 exports.clearTable = function() {
     table.rows = [];
-}
+};
 
 // Delete the entire table
 exports.deleteTable = function() {
     table = {"columnNames": [], "rows": [], "maxWidth": []};
-}
+};
 
 // Draw a line based on the max width of each column and return
 var drawLine = function() {
     arrayLength = 0;
     for (var i=0; i < table.maxWidth.length; i++) {
         arrayLength += table.maxWidth[i];
     }
-    return '+' + Array(arrayLength + table.maxWidth.length * 3).join('-') + '+'
-}
+    return '+' + Array(arrayLength + table.maxWidth.length * 3).join('-') + '+';
+};
 
 // Helper method to add column names of the table and add maxwidth for each column element
 var addTableHeader = function(names) {
     table.columnNames = names;
     for (var i=0; i < names.length; i++) {
         table.maxWidth.push(names[i].length);
     }
-}
+};
 
 // Helper method to add a row to the table and re-adjust max width of each row element
 var addTableRow = function(row) {
@@ -132,15 +132,15 @@ var addTableRow = function(row) {
             table.maxWidth[i] = row[i].toString().length;
         }
     }
-}
+};
 
 // Convert the table to string and return
 var tableToString = function() {
     // Define final table string as empty string
     finalTable = "";
 
     // If no columns present, return empty string
-    if (table.columnNames.length == 0) {
+    if (table.columnNames.length === 0) {
         return finalTable;
     }
 
@@ -152,7 +152,7 @@ var tableToString = function() {
         // Adjust for max width of the column and pad spaces
         if (table.columnNames[i].length < table.maxWidth[i]) {
             lengthDifference = table.maxWidth[i] - table.columnNames[i].length;
-            columnString += Array(lengthDifference + 1).join(' ')
+            columnString += Array(lengthDifference + 1).join(' ');
         }
         columnString += " | ";
     }
@@ -161,14 +161,14 @@ var tableToString = function() {
     finalTable += drawLine() + "\n";
 
     // Construct the table body
-    for (var i=0; i < table.rows.length; i++) {
+    for (i=0; i < table.rows.length; i++) {
         var tempRowString = "| ";
         for (var k=0; k < table.rows[i].length; k++) {
             tempRowString += table.rows[i][k];
             // Adjust max width of each cell and pad spaces as necessary
             if (table.rows[i][k].toString().length < table.maxWidth[k]) {
                 lengthDifference = table.maxWidth[k] - table.rows[i][k].toString().length;
-                tempRowString += Array(lengthDifference + 1).join(' ')
+                tempRowString += Array(lengthDifference + 1).join(' ');
             }
             tempRowString += " | ";
         }
@@ -186,15 +186,16 @@ var tableToString = function() {
 // Convert the table to HTML table
 var tableToHTML = function(attributes) {
     // If attributes provided, add them as inline properties, else create default table tag
+    var htmlTable = "";
     if (typeof attributes == "undefined") {
-        var htmlTable = "<table>";
+        htmlTable = "<table>";
     } else {
         var attributeList = [];
         for (var key in attributes) {
             attributeList.push(key + '="' + attributes[key] + '"');
         }
         var attributeString = attributeList.join(" ");
-        var htmlTable = "<table " + attributeString + ">";
+        htmlTable = "<table " + attributeString + ">";
     }
 
     // Define the table headers in <thead> from table column list
@@ -208,7 +209,7 @@ var tableToHTML = function(attributes) {
 
     // Construct the table body from the array of rows
     var tableBody = "<tbody>";
-    for (var i=0; i < table.rows.length; i++) {
+    for (i=0; i < table.rows.length; i++) {
         var rowData = "<tr>";
         for (var k=0; k < table.rows[i].length; k++) {
             var cellData = "<td>" + table.rows[i][k] + "</td>";
@@ -223,7 +224,7 @@ var tableToHTML = function(attributes) {
     htmlTable += "</table>";
 
     return htmlTable;
-}
+};
 
 // Helper method to sort table given column name
 var sortTableByColumn = function(colname, reverse) {
@@ -232,7 +233,7 @@ var sortTableByColumn = function(colname, reverse) {
 
     // Comparator method which takes the column index and sort direction
     function Comparator(a,b){
-        if (reverse == true) {
+        if (reverse === true) {
             if (a[colindex] < b[colindex]) return 1;
             if (a[colindex] > b[colindex]) return -1;
             return 0;
@@ -244,6 +245,6 @@ var sortTableByColumn = function(colname, reverse) {
     }
     // Sort array of table rows
     table.rows = table.rows.sort(Comparator);
-}
+};
 
 exports.version = "0.3.0";