-
Notifications
You must be signed in to change notification settings - Fork 0
Assignment8 #6
base: master
Are you sure you want to change the base?
Assignment8 #6
Conversation
b891cd3
to
3b9b210
Compare
51b3df0
to
115b532
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please incorporate the review comments. Also, make your code modular. Move controllers, routes into respective folders.
const text_tax = require('./sales_imported_text'); | ||
app.use("/", bodyParser.json()); | ||
app.use("/", bodyParser.text()) | ||
app.get("/", (req, res) =>{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting?
}) | ||
let restime=new Date().getTime(); | ||
res.end(); | ||
} if (req.headers["content-type"] == "text/plain") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same line?
regex = new RegExp('(\\d+)'); | ||
temp = sentences[i].split(regex); | ||
quantity.push(temp[1]); | ||
lastind=temp[2].lastIndexOf("at"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh ugh
sales=0 | ||
for(let i=1;i<sentences.length;i++){ | ||
sales+=text_tax.sales_tax(quantity[i-1],body[i-1],price[i-1])+text_tax.imported_tax(quantity[i-1],body[i-1],price[i-1]); | ||
aftertax=parseFloat(price[i-1])+text_tax.sales_tax(quantity[i-1],body[i-1],price[i-1])+text_tax.imported_tax(quantity[i-1],body[i-1],price[i-1]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not just a murder. This is genocide!
down: (queryInterface, Sequelize) => { | ||
return queryInterface.dropTable('orders'); | ||
} | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EOF?
No description provided.