-
Notifications
You must be signed in to change notification settings - Fork 0
[feat] @everyone 이 첫줄 이어도 title로 사용하지 않습니다.
#442
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
Conversation
| import java.util.Arrays; | ||
|
|
||
| @Getter | ||
| public class TitleAndContentParser { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확실히 객체 분리 해놓으니까 유용하게 쓰이네요 굳!!!
| public class TitleAndContentParser { | ||
| private String title; | ||
| private String content; | ||
| private static final String EVERYONE_MENTION = "@everyone"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
static final 상수 설정 좋습니다!
| return new TitleAndContentParser(title,content); | ||
| } | ||
|
|
||
| private static boolean validateTitle(String line){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메서드 분리 굳!
| return true; | ||
| } | ||
|
|
||
| private static String deleteMention(String input){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 메서드는 no use 인데 뭐죠??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정해서 올렸습니다
📝 요약
@everyone은 제목으로 설정하지 않음 #439🔖 변경 사항
✅ 리뷰 요구사항
📸 확인 방법 (선택)
📌 PR 진행 시 이러한 점들을 참고해 주세요