Skip to content

[SAO FT5 - PP - W1D3] Pedro & Millene - #478

Closed
Ptdavid0 wants to merge 2 commits into
ironhack-labs:masterfrom
Ptdavid0:master
Closed

[SAO FT5 - PP - W1D3] Pedro & Millene#478
Ptdavid0 wants to merge 2 commits into
ironhack-labs:masterfrom
Ptdavid0:master

Conversation

@Ptdavid0

@Ptdavid0 Ptdavid0 commented Sep 4, 2019

Copy link
Copy Markdown

No description provided.

Comment thread basic.js
function reverseName(name) {
let str = name.split("");
let str2 = str.reverse();
let str3 = str2.join("");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

É possível fazer em uma linha (name.split("").reverse().join("").
Mas cuidado com excesso de loops, cuide da performace do código.

@heitorgodau

Copy link
Copy Markdown

Boa! Aí sim!

@stale

stale Bot commented Dec 17, 2019

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the stale label Dec 17, 2019
@stale

stale Bot commented Dec 19, 2019

Copy link
Copy Markdown

This pull request is closed. Thank you.

@stale stale Bot closed this Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants