Skip to content
This repository has been archived by the owner on Apr 1, 2021. It is now read-only.

Create Java-Loops.md #560

Merged
merged 1 commit into from
Mar 11, 2016
Merged

Conversation

madhav-datt
Copy link
Contributor

Java Loops. In response to issue #534

@raisedadead
Copy link
Member

@koustuvsinha Please have a look.

@koustuvsinha
Copy link
Member

Looks good to me, @alayek please have a look too.

@madhav-datt great work! thanks a lot for contributing! however, please keep in mind that before starting to work on any article notify us at the respective issue, or someone else's work will go to waste. @scrumuser1 please work on a different Article now, as Loops is already done in this PR. Apologies for the inconvenience!

@madhav-datt
Copy link
Contributor Author

@koustuvsinha I completely understand and will discuss before I start next time.
@scrumuser1 Sorry about that - I should have discussed this.


Whenever you need to execute a block of code multiple times, a loop will often come in handy. In general, a loop looks something like this:

![Loop Schematic Diagram](http://cse.iitkgp.ac.in/pds/notes/img/loop2.gif)
Copy link
Member

Choose a reason for hiding this comment

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

I never thought someone would link to a PDS handout 💯

@madhav-datt
Copy link
Contributor Author

@alayek Fixed the issues

@koustuvsinha
Copy link
Member

@madhav-datt please squash your commits too :)

@alayek
Copy link
Member

alayek commented Mar 11, 2016

I am not sure...I don't see all suggestions implemented.

Update Java-Loops.md

Update Java-Loops.md

Update Java-Loops.md
@madhav-datt
Copy link
Contributor Author

@koustuvsinha Commits squashed :)
@alayek I think I got everything. In case you are looking for the part about breakpoints, I have covered that at the beginning of loops instead of adding the same thing for each type of loop - what do you think hasn't been implemented?

@koustuvsinha
Copy link
Member

wow Github now supports reactions too! :) @alayek

@koustuvsinha
Copy link
Member

@madhav-datt i checked you have indeed mentioned the breakpoints at the beginning. I think @alayek might have missed that. No issues!

@Rafase282 @raisedadead consider merging this!

raisedadead added a commit that referenced this pull request Mar 11, 2016
@raisedadead raisedadead merged commit 57ab7cb into freeCodeCamp:master Mar 11, 2016
@BerkeleyTrue BerkeleyTrue removed the QA label Mar 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants