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

[T2A6][T01-T04] Lui Sheng Jie #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[T2A6][T01-T04] Lui Sheng Jie #87

wants to merge 1 commit into from

Conversation

luishengjie
Copy link

Ready for review

…onent of address to be read with the getBlock(), getPostalCode, getStreet and getUnit()


String[] splitTrimmedAddress = trimmedAddress.split("\\s*,\\s*");
// a/BLOCK, STREET, UNIT, POSTAL_CODE

Choose a reason for hiding this comment

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

This comment is unnecessary.

}
if (splitTrimmedAddress.length >= 4){
addressPostalCode = new PostalCode(splitTrimmedAddress[3]);
}

Choose a reason for hiding this comment

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

Better remove the magic number.

@jeffryhartanto
Copy link

@luishengjie Some comments added. Please close the PR after reading comments. I think you messed up the PR because I cannot the newly created classes.

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