Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

Glasgow class 6- Siver Omar- JavaScript 2- week 1#265

Open
siveromar wants to merge 1 commit intoCodeYourFuture:mainfrom
siveromar:main
Open

Glasgow class 6- Siver Omar- JavaScript 2- week 1#265
siveromar wants to merge 1 commit intoCodeYourFuture:mainfrom
siveromar:main

Conversation

@siveromar
Copy link

  • Name: Siver
  • City: Glasgow
  • Slack Name:siveromar

Homework Details

  • Module:JavaScript 2
  • Week: 1

Copy link

@msimmdev msimmdev left a comment

Choose a reason for hiding this comment

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

Hi Siver, great work on completing the exercises, I've found very few problems and your work on the writing tests exercises is excellent.

One thing I've noticed throughout your code is in several places the formatting is inconsistent and difficult to read, I'd suggest checking your Visual Studio settings as Prettier should format the code you write for you when you save if enabled correctly. If you can't get this working let's have a look together in class.


let myCountry = "UnitedKingdom";
let myCapitalCity; // complete the code
let myCapitalCity ="london"; // complete the code

Choose a reason for hiding this comment

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

Issue: The goal here was to select the capital from the capitalCities object.

For example using capitalCities[myCountry] will access the member of the object based on the value stored in myCountry

- Use bracket notation to change the value of hasPassed
*/

if(student.attendance>=90 && student.examScore>60){student["hasPassed"]=true;}

Choose a reason for hiding this comment

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

Issue: The formatting of code here is not easily readable

// write code here
orderBurger:
function(balance)
{ if(balance >= MENU.burger){

Choose a reason for hiding this comment

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

Issue: The formatting and indentation of code here is not readable

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants