Skip to content

Refactor Contacts0#1

Open
okkhoy wants to merge 2 commits intonus-cs2113-AY2425S2:mainfrom
okkhoy:main
Open

Refactor Contacts0#1
okkhoy wants to merge 2 commits intonus-cs2113-AY2425S2:mainfrom
okkhoy:main

Conversation

@okkhoy
Copy link
Copy Markdown
Contributor

@okkhoy okkhoy commented Jan 31, 2025

No description provided.

Copy link
Copy Markdown
Contributor Author

@okkhoy okkhoy left a comment

Choose a reason for hiding this comment

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

keep up the time!


public class Contacts0 {

public static final int MAX_ENTRIES = 100;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good job extracting a constant

public class Contacts0 {

public static final int MAX_ENTRIES = 100;
public static final String LINE_SEPARATOR = "===================================================";
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Why so??

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.

1 participant