Merge pull request #2 from niravraval1/Emibranch1

Disallow negative value for all parameters

Author: niravraval1

src/emi.js

@@ -9,9 +9,10 @@
  */
 function Loan (amount, installmentsNumber, interestRate) {
   /** Checking params */
-  if (!amount ||
-     !installmentsNumber ||
-     !interestRate) {
+  // function Loan
+  if (!amount || amount <= 0 ||
+  !installmentsNumber || installmentsNumber <= 0 ||
+  !interestRate || interestRate <= 0) {
     throw new Error(`wrong parameters: ${amount} ${installmentsNumber} ${interestRate}`)
   }
 
@@ -39,7 +40,7 @@ function Loan (amount, installmentsNumber, interestRate) {
   }
 
   return {
-    installments: installments,
+    installments,
     amount: rnd(amount),
     interestSum: rnd(interestSum),
     principalSum: rnd(principalSum),
@@ -68,9 +69,9 @@ const getNextInstallment = (
   const principal = installment - interest
 
   return {
-    principal: principal,
-    interest: interest,
-    installment: installment,
+    principal,
+    interest,
+    installment,
     remain: amount - principalSum - principal,
     interestSum: interestSum + interest
   }
@@ -88,8 +89,8 @@ function emiToHtmlTable (loan, params) {
   params.formatMoney = params.formatMoney || function (num) {
     return num.toFixed(2)
   }
-  var fm = params.formatMoney
-  var html = [
+  const fm = params.formatMoney
+  const html = [
     '<table class="table table-striped">' +
       '<thead>' +
         '<tr>' +
@@ -107,9 +108,9 @@ function emiToHtmlTable (loan, params) {
     '</table>'
   ]
 
-  for (var i = 0; i < loan.installments.length; i++) {
-    var inst = loan.installments[i]
-    var instHtml =
+  for (let i = 0; i < loan.installments.length; i++) {
+    const inst = loan.installments[i]
+    const instHtml =
           '<tr>' +
             '<td>' + (i + 1) + '</td>' +
             '<td>' + fm(inst.principal) + '</td>' +
@@ -152,8 +153,8 @@ if (typeof module === 'undefined') {
 } else {
   // node or browserfy
   module.exports = {
-    Loan: Loan,
-    emiToHtmlTable: emiToHtmlTable,
-    rnd: rnd
+    Loan,
+    emiToHtmlTable,
+    rnd
   }
 }