Skip to content
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.

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

Edited readme.md and uploaded Flow Chart #24

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

subutai1175
Copy link

No description provided.


function randomColor () {
var col = colorArr[randomNum()]
return col

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may do this : return colorArr[randomNum()]

scoreObj.score += 10
scoreObj.scoreBoard.text(`Score : ${scoreObj.score}`)
// TODO: No Click? click = false
} else if (recArr[count - 1] != recArr[count - n]) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

!==

audioCorrect()
scoreObj.score += 10
scoreObj.scoreBoard.text(`Score : ${scoreObj.score}`)
} else if (recCol[count - 1] != recCol[count - n]) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

!==

// Random Number Generator
function randomNum () {
var index = Math.floor(Math.random() * 8)
return index

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may just return the value

}

function audioCorrect () {
return document.getElementById('audioCorrect').play()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can we do this with jQuery?

var recCol = []
var count = 0
var index = 0
var on = 0

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe it is a confusing name with .on(). You can try gameOn , gameStart .

<div class="container">
<h1>Dual N Back Game Instructions</h1>

<h3>Each turn a random location and color will flash on the gameboard. The goal is to memorize the location and color 'n' turns back. Currently n = 1. If the location or color of current turn is the same as that of the previous turn, we should trigger the location or color button</h3><br>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why everything is h3 tags here?

<body>
<audio id="audioCorrect" type="audio/mpeg" src="./audio/correctAnswer.mp3"></audio>
<audio id="audioWrong" type="audio/mpeg" src="./audio/wrongAnswer.mp3"></audio>
<div class="container">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation issues here

var index = 0
var on = 0
var refreshIntervalId
var n = 2 // Difficulty Setting
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why dont name this variable difficultyLevel instead?

gameOverDiv.css('display', 'none')
})
$colorBtn.on('click', compareColor)
$(document).on('keydown', function (e) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you give some spaces in between these codes? group them logically so it's easier to be read

@alexkimin
Copy link

alexkimin commented Oct 15, 2017

Project Workflow: 3 / 5
Technical Requirement: 3 / 5
Creativity: 3 / 5
Code Quality: 3 / 5
Problem Solving: 3 / 5

Glow

Scaling down on the idea.
Step by step approach for additional functionality.
Active on questioning.

Grow

Need more detailed commit message, not like 'second try'.
Naming convention for variables.

Things to look for

I know it is hard to be familiar with many methods. Just keep practicing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants