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

Day1 #6

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

Day1 #6

wants to merge 36 commits into from

Conversation

jenarp
Copy link

@jenarp jenarp commented Sep 24, 2021

No description provided.

jenarp and others added 30 commits July 21, 2021 17:00
…am doing. ;)

In this commit I updated Day01.java with the puzzle's answers methods and I wrote a test script for puzzle day 1.1..
…ally, I wrote several test scripts (Day02Test.java).
…ally, I wrote several test scripts (Day02Test.java).
Co-authored-by: Michelle Fernandez Bieber <[email protected]>
- I wrote several test scripts (Day03Test.java).
- I resolved the comments from Michelle for Day02.java
- Raw Data is added in resources
…ally, I wrote several test scripts (Day04Test.java).
…ally, I wrote several test scripts (Day04Test.java).
…ally, I wrote several test scripts (Day02Test.java).
…Change of for loop to stream() in Day05.java.
…format scripts, change modifiers to private and final whenever possible.
… Data (DataLoader.java), to create problemStatus Maps (Problemstatus.java) and to throw illegal Argument Exception if Strings are empty (CheckStringIsEmpty.java).

 Additionally, some script formatting, modifiers changing to private and final whenever possible and variable renaming.
… the application.

-> day's constructors do not need "String filename" as argument anymore, as data is loaded in DataLoader from now on.
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