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.

Sign up for GitHub

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

Jump to bottom

check and fix all Main-Classes from masymos-cli #1

Open
7 tasks
BradleyScrim opened this issue May 27, 2020 · 0 comments
Open
7 tasks

check and fix all Main-Classes from masymos-cli #1

BradleyScrim opened this issue May 27, 2020 · 0 comments
Labels
cleancode related to clean code guidelines refactoring problems, making the code hard to understand or to run ineffective

Comments

@BradleyScrim
Copy link
Member

overview

  • bad things happen here, i.e. s.equals("exit") … switch (Integer.valueOf(s)) without catch

small guide

  • check input and output
  • implement shutdown hook instead of `while(xyz)
  • implement exit codes
  • use ENUMs
  • return meaningful error messages

ToDo

  • MainAnalyzer.java
  • MainDelete.java
  • MainExtractor.java
  • MainIndex.java
  • MainQuery.java
  • MainStats.java
  • MainWeb.java
@BradleyScrim BradleyScrim added bug Something isn't working cleancode related to clean code guidelines refactoring problems, making the code hard to understand or to run ineffective labels May 27, 2020
@BradleyScrim BradleyScrim removed the bug Something isn't working label Jun 3, 2020
BradleyScrim pushed a commit that referenced this issue Jun 4, 2020
#1 allow eclipse project preferences
dc4b45f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cleancode related to clean code guidelines refactoring problems, making the code hard to understand or to run ineffective
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@BradleyScrim