Pair Programming • 12 min read

Sep 7, 2023

Draft

Nitro Type - a game where the user is given a string of random words and they have to correctly type them as fast as possible. Based on the time it takes them, we then calculate how many words per minute they’d be able to type and maybe save their own high scores locally in a leaderboard(?)

Feedback (ChatGPT)

Your code appears to be a web page for a typing game. I’ll review it for you:

HTML Structure:
- The HTML structure is clean and well-organized.
- You have included the necessary elements like head, body, and various div elements with appropriate IDs and classes.
CSS Styling:
- The CSS styles are defined in the head section. This is good for keeping the styles separate from the HTML content.
- The styles are appropriately named and organized. Each rule is easy to understand.
- The use of IDs and classes for styling is clear and makes the code easy to maintain.
External Dependencies:
- You’re using external resources like jQuery and DataTables. This is fine, but be aware that relying on external resources can introduce dependencies and potential points of failure.
JavaScript:
- The JavaScript code is well-structured and organized.
- You have used async/await correctly for fetching random words.
- Functions are appropriately named and follow a logical flow.
- You’ve handled potential errors when fetching a new word.
Variable Declarations:
- You’ve declared variables using var, which is perfectly fine. However, in modern JavaScript, you might use const and let instead of var for better scoping.
Error Handling:
- You’ve added error handling for failed fetch requests, which is a good practice.
Event Handling:
- You’ve attached event listeners for user input. The logic for updating the timer, highlighting text, and checking for accuracy seems correct.
Comments:
- You have added a front-matter section at the beginning, which is typical for Jekyll or similar static site generators. This helps in specifying metadata for the page.
- You haven’t included any comments within the JavaScript code. Adding comments can help others (and yourself in the future) understand the code’s functionality.
Security Concerns:
- Be cautious about exposing API keys directly in the code, especially if the code will be publicly accessible. It’s better to handle API keys server-side or through environment variables.
Overall:
- The code is well-written, organized, and appears to work as intended.

Remember, this review is based solely on the provided code snippet, and I don’t have context on how this code interacts with other parts of your project. Always ensure you’re following best practices and security measures, especially when deploying code on the web.

Issues we are having

Need an API that can give random sentences (currently requesting 5 words from a random word API)
Need to fix problem where if you type in incorrect letter, it replaces the actual letter you have to type
Need to fix the max accuracy score being 93%

What we could add/improve on

Want to research YamlDB to add a leaderboard
Want to highlight incorrectly typed letters in another color
Takes a long time for the words to show up

CSA Code, Code, Coding assignment.

Review. Teacher posts for JS Itunes API, JS Input, JS Output w/jquery, JS Output w/API.

Option 1. Add to the lessons and making code cells and hacks for better breakdown and understanding Option 2. Combine all the concepts into something unique and you feel is awesome.