Skip to content

lab1 updates #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
konjet123 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

lab1 updates #1

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 26 additions & 25 deletions class-01/lab-a/starter-code/js/app.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,20 +1,20 @@
'use strict';

var names = ['bag', 'banana', 'bathroom', 'boots', 'breakfast', 'bubblegum', 'chair', 'cthulhu', 'dog-duck', 'dragon', 'pen', 'pet-sweep', 'scissors', 'shark', 'sweep', 'tauntaun', 'unicorn', 'usb', 'water-can', 'wine-glass'];
const names = ['bag', 'banana', 'bathroom', 'boots', 'breakfast', 'bubblegum', 'chair', 'cthulhu', 'dog-duck', 'dragon', 'pen', 'pet-sweep', 'scissors', 'shark', 'sweep', 'tauntaun', 'unicorn', 'usb', 'water-can', 'wine-glass'];

var leftImage = document.getElementById('left');
var centerImage = document.getElementById('center');
var rightImage = document.getElementById('right');
const leftImage = document.getElementById('left');
const centerImage = document.getElementById('center');
const rightImage = document.getElementById('right');

var allProducts = [];
var container = document.getElementById('image_container');
var viewed = [];
var labels = [];
var pics = [leftImage, centerImage, rightImage];
var list = document.getElementById('productlist');
var totalClicks = 0;
var views = [];
var votes = [];
let allProducts = [];
const container = document.getElementById('image_container');
const viewed = [];
const labels = [];
const pics = [leftImage, centerImage, rightImage];
const list = document.getElementById('productlist');
let totalClicks = 0;
const views = [];
const votes = [];

function Product(name) {
this.name = name;
Expand All @@ -30,18 +30,18 @@ function makeRandom() {

function displayPics(){
while(viewed.length < 6){
var rando = makeRandom();
const rando = makeRandom();
while(!viewed.includes(rando)){
viewed.push(rando);
}
}
console.log(rando);
// TODO: In a sentence or two, explain why the previous line of code threw an error when we changed the variable declaration from `var to `let`.
// PUT YOUR RESPONSE IN THIS COMMENT

// TODO: In a sentence or two, explain why the previous line of code threw an error when we changed the letiable declaration from `var to `let`.
//Answer:rando is the block variable and context for rando expires after the block and hence not accessible.
console.log(viewed);

for (var i = 0; i < 3; i++){
var temp = viewed.shift();
for (let i = 0; i < 3; i++){
const temp = viewed.shift();
pics[i].src = allProducts[temp].path;
pics[i].id = allProducts[temp].name;
allProducts[temp].views += 1;
Expand All @@ -59,10 +59,11 @@ function handleClick(event) {
showList();
makeChart();
}
for(var i = 0; i < names.length; i++){
for(let i = 0; i < names.length; i++){
if(event.target.id === allProducts[i].name) {
allProducts[i].votes += 1;
console.log(event.target.id + ' has ' + allProducts[i].votes + ' votes in ' + allProducts[i].views + ' views');

console.log(`${event.target.id} has ${allProducts[i].votes} votes in ${allProducts[i].views} views`);
}
}
localStorage.busmall = JSON.stringify(allProducts);
Expand All @@ -71,8 +72,8 @@ function handleClick(event) {
}

function showList() {
for(var i = 0; i < allProducts.length; i++) {
var liEl = document.createElement('li');
for(let i = 0; i < allProducts.length; i++) {
const liEl = document.createElement('li');
liEl.textContent = allProducts[i].name + ' has ' + allProducts[i].votes + ' votes in ' + allProducts[i].views + ' views';
list.appendChild(liEl);
}
Expand All @@ -88,7 +89,7 @@ function makeChartData(){

function makeChart(){
makeChartData();
var ctx = document.getElementById('chartypants').getContext('2d');
const ctx = document.getElementById('chartypants').getContext('2d');
new Chart(ctx, { //eslint-disable-line
type: 'bar',
data: {
Expand Down Expand Up @@ -128,7 +129,7 @@ if(localStorage.busmall){
allProducts = JSON.parse(localStorage.busmall);
} else {
console.log('There is no local storage data; initialize app by creating instances');
for(var i = 0; i < names.length; i++) {
for(let i = 0; i < names.length; i++) {
new Product(names[i]);
}
}
Expand Down