MIA Rodrigo Flores#460

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

rflor012 wants to merge1 commit intoironhack-labs:masterfromrflor012:master

Closed

MIA Rodrigo Flores#460

rflor012 wants to merge1 commit intoironhack-labs:masterfromrflor012:master

Conversation

Copy link

rflor012 commentedJun 13, 2018

No description provided.

done

0de1ae9

MarcosPalaciosIII reviewed

Jun 13, 2018

View reviewed changes

starter-code/src/functions-and-arrays.js

		@@ -11,14 +15,101 @@ var words = [
		'crackpot'
		];

		function longest(words) {

Copy link

MarcosPalaciosIIIJun 13, 2018

Choose a reason for hiding this comment

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

your function seems to be a little off. It should look something more like this
function longest(words) { var longest = ""; for(i=0;i < words.length;i++) { if(words[i].length > longest.length) { longest = words[i]; } else { continue; } } return longest; }

Copy link

MarcosPalaciosIII commentedJun 13, 2018

good work on the lab. Just remember to look over what was given for examples in class so yo can compare errors that you may have had.
Keep up the good work.