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

Java Assignment3 upload by HyunseokOh #17

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

brotherstone97
Copy link

No description provided.

Copy link

@Jungdahee Jungdahee left a comment

Choose a reason for hiding this comment

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

안녕하세요 형석님~
먼저 과제 수행하시느라 고생하셨습니다.

로직이 들어가는 부분들을 메소드화 시킨 부분들, 예외 처리 모두 잘하셨습니다.
다만, String을 연산할 때 성능 측면에서 유리한 부분들을 공부하시면 좋을 것 같아요!
'+' 로 하게 되면 코드가 짧아지고 보기 쉬운 장점은 있으나, 조금 더 복잡한 연산이 들어가게 되면, 속도에서 불리한 측면이 있어요...! 한번 공부해볼까요?

수고하셨습니다:)

src/main/java/practice1/AuthMethod.java Outdated Show resolved Hide resolved
src/main/java/practice1/Company.java Outdated Show resolved Hide resolved
src/main/java/practice1/Electronic.java Outdated Show resolved Hide resolved
src/main/java/practice1/Electronic.java Outdated Show resolved Hide resolved
src/main/java/practice1/Electronic.java Outdated Show resolved Hide resolved
src/main/java/practice1/User.java Show resolved Hide resolved
src/main/java/practice1/Users.java Outdated Show resolved Hide resolved
src/main/java/practice1/Users.java Outdated Show resolved Hide resolved
src/main/java/practice1/Users.java Outdated Show resolved Hide resolved
src/main/java/practice1/Users.java Show resolved Hide resolved
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