Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added dfs.java #138

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

added dfs.java #138

wants to merge 5 commits into from

Conversation

nvinayvarma189
Copy link
Contributor

I added Depth First Search Algorithm in java . I think this solves issue #111 .

@nvinayvarma189
Copy link
Contributor Author

Please review this PR and kindly notify me if any changes are requested.

Copy link
Owner

@khanchi97 khanchi97 left a comment

Choose a reason for hiding this comment

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

Please add a README file describing about what it does and how is this implemented.

Added visualizing feature for the algorithm
@nvinayvarma189
Copy link
Contributor Author

I added the README file and also gave a link to visualize the code for a better understanding for beginners. CAn you please review this now @khanchi97

Copy link
Owner

@khanchi97 khanchi97 left a comment

Choose a reason for hiding this comment

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

It should prompt the user to enter values.

@@ -0,0 +1,49 @@
import java.io.*;
Copy link
Owner

Choose a reason for hiding this comment

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

Not in this way.Make it ask to users to enter values for this.

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