Skip to content

BCN Will Javier Jose#153

Closed
will-ks wants to merge 1 commit into
ironhack-labs:masterfrom
will-ks:master
Closed

BCN Will Javier Jose#153
will-ks wants to merge 1 commit into
ironhack-labs:masterfrom
will-ks:master

Conversation

@will-ks
Copy link
Copy Markdown

@will-ks will-ks commented Jun 11, 2018

Copy link
Copy Markdown
Contributor

@tawebbcn tawebbcn left a comment

Choose a reason for hiding this comment

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

Good job! Keep working hard! 🚀


for (var i = 0; i < hacker2.length; i++) {
spacedString = spacedString + hacker2[i].toUpperCase() + " ";
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice!

console.log(
"wow, you both got equally long names, " + hacker1.length + " characters!"
);
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍


for (var i = hacker1.length - 1; i >= 0; i--) {
backwardsString = backwardsString + hacker1[i];
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice. Also try with:

 backwardsString += hacker1[i];

var backwardsString = "";

for (var i = hacker1.length - 1; i >= 0; i--) {
backwardsString = backwardsString + hacker1[i];
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You also can do:

backwardsString += hacker1[i];

backwardsString = backwardsString + hacker1[i];
}

if (hacker1 < hacker2) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@tawebbcn tawebbcn closed this Jun 11, 2018
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.

2 participants