Skip to content

Paulina Kekkonen #51

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 4 additions & 29 deletions src/main/java/com/booleanuk/core/Article.java
Original file line number Diff line number Diff line change
@@ -1,35 +1,10 @@
package com.booleanuk.core;

public class Article {
String title;
public class Article extends Item {

boolean onLoan = false;

public Article(String title) {
this.title = title;
}

public boolean isOnLoan() {
return onLoan;
}

public String checkIn() {
if (!this.isOnLoan()) {
return "item is not currently on loan";
}

this.onLoan = false;

return "item has been checked in";
public Article(String title, Author author) {
super(title);
super.author = author;
}

public String checkOut() {
if (this.isOnLoan()) {
return "item is currently on loan";
}

this.onLoan = true;

return "item has been checked out";
}
}
31 changes: 31 additions & 0 deletions src/main/java/com/booleanuk/core/Author.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
package com.booleanuk.core;

public class Author {
private String name;
private String email;
private String phoneNumber;
private String website;

public Author(String name, String email, String phoneNumber, String website) {
this.name = name;
this.email = email;
this.phoneNumber = phoneNumber;
this.website = website;
}

public String getName() {
return name;
}

public String getEmail() {
return email;
}

public String getPhoneNumber() {
return phoneNumber;
}

public String getWebsite() {
return website;
}
}
34 changes: 5 additions & 29 deletions src/main/java/com/booleanuk/core/Book.java
Original file line number Diff line number Diff line change
@@ -1,35 +1,11 @@
package com.booleanuk.core;

public class Book {
String title;
public class Book extends Item {

boolean onLoan = false;

public Book(String title) {
this.title = title;
}

public boolean isOnLoan() {
return onLoan;
public Book(String title, Author author) {
super(title);
super.author = author;
}

public String checkIn() {
if (!this.isOnLoan()) {
return "item is not currently on loan";
}

this.onLoan = false;

return "item has been checked in";
}

public String checkOut() {
if (this.isOnLoan()) {
return "item is currently on loan";
}

this.onLoan = true;

return "item has been checked out";
}
}

43 changes: 43 additions & 0 deletions src/main/java/com/booleanuk/core/Item.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
package com.booleanuk.core;

public abstract class Item {
String title;
boolean onLoan = false;

Author author;

public Item(String title) {
this.title = title;
}
public boolean isOnLoan() {
return onLoan;
}

public String checkIn() {
if (!this.isOnLoan()) {
return "item is not currently on loan";
}

this.onLoan = false;

return "item has been checked in";
}

public String checkOut() {
if (this.isOnLoan()) {
return "item is currently on loan";
}

this.onLoan = true;

return "item has been checked out";
}

public String getAuthor() {
if(author == null) {
return "item does not have information about author(s)";
}
return "This item is written by " + author.getName() + " (e-mail: " + author.getEmail() + ", phone number: "
+ author.getPhoneNumber() +", website: " + author.getWebsite() + ")";
}
}
74 changes: 12 additions & 62 deletions src/main/java/com/booleanuk/core/Library.java
Original file line number Diff line number Diff line change
@@ -1,29 +1,25 @@
package com.booleanuk.core;

import java.util.ArrayList;
import java.util.List;

public class Library {
List<Article> articles;
List<Book> books;
List<Newspaper> newspapers;
List<Item> stock;

public void addToStock(Article item) {
this.articles.add(item);
public Library() {
stock = new ArrayList<>();
}

public void addToStock(Book item) {
this.books.add(item);
public void addToStock(Item item) {
this.stock.add(item);
}

public void addToStock(Newspaper item) {
this.newspapers.add(item);
}

// The following methods may contain code that you are unfamiliar with. The strange syntax of article -> something
// is called a lambda expression (https://www.w3schools.com/java/java_lambda.asp)
public String checkInArticle(String title) {
List<Article> filtered = this.articles.stream()
.filter(article -> article.title.equals(title))
public String checkInItem(String title) {
List<Item> filtered = this.stock.stream()
.filter(item -> item.title.equals(title))
.toList();

if (filtered.size() < 1) {
Expand All @@ -33,33 +29,10 @@ public String checkInArticle(String title) {
return filtered.get(0).checkIn();
}

public String checkOutArticle(String title) {
List<Article> filtered = this.articles.stream()
.filter(article -> article.title.equals(title))
.toList();

if (filtered.size() < 1) {
return "item is not part of the library's collection";
}

return filtered.get(0).checkOut();
}

public String checkInBook(String title) {
List<Book> filtered = this.books.stream()
.filter(book -> book.title.equals(title))
.toList();

if (filtered.size() < 1) {
return "item is not part of the library's collection";
}

return filtered.get(0).checkIn();
}

public String checkOutBook(String title) {
List<Book> filtered = this.books.stream()
.filter(book -> book.title.equals(title))
public String checkOutItem(String title) {
List<Item> filtered = this.stock.stream()
.filter(item -> item.title.equals(title))
.toList();

if (filtered.size() < 1) {
Expand All @@ -69,27 +42,4 @@ public String checkOutBook(String title) {
return filtered.get(0).checkOut();
}

public String checkInNewspaper(String title) {
List<Newspaper> filtered = this.newspapers.stream()
.filter(newspaper -> newspaper.title.equals(title))
.toList();

if (filtered.size() < 1) {
return "item is not part of the library's collection";
}

return filtered.get(0).checkIn();
}

public String checkOutNewspaper(String title) {
List<Newspaper> filtered = this.newspapers.stream()
.filter(newspaper -> newspaper.title.equals(title))
.toList();

if (filtered.size() < 1) {
return "item is not part of the library's collection";
}

return filtered.get(0).checkOut();
}
}
13 changes: 4 additions & 9 deletions src/main/java/com/booleanuk/core/Newspaper.java
Original file line number Diff line number Diff line change
@@ -1,22 +1,17 @@
package com.booleanuk.core;

public class Newspaper {
String title;

boolean onLoan = false;
public class Newspaper extends Item {

public Newspaper(String title) {
this.title = title;
}

public boolean isOnLoan() {
return onLoan;
super(title);
}

@Override
public String checkIn() {
return "newspapers are not available for loan";
}

@Override
public String checkOut() {
return "newspapers are not available for loan";
}
Expand Down
16 changes: 11 additions & 5 deletions src/test/java/com/booleanuk/core/ArticleTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -4,32 +4,38 @@
import org.junit.jupiter.api.Test;

class ArticleTest {

Author author = new Author("Bob", "[email protected]", "911", "www.bob.com");
Article article = new Article("JUnit Rocks", author);

@Test
public void shouldCheckOutIfAvailable() {
Article article = new Article("JUnit Rocks");
Assertions.assertEquals("item has been checked out", article.checkOut());
}

@Test
public void shouldDeclineIfNotAvailableToCheckout() {
Article article = new Article("JUnit Rocks");
article.checkOut();

Assertions.assertEquals("item is currently on loan", article.checkOut());
}

@Test
public void shouldCheckInIfOnLoan() {
Article article = new Article("JUnit Rocks");
article.checkOut();

Assertions.assertEquals("item has been checked in", article.checkIn());
}

@Test
public void shouldDeclineCheckInIfNotOnLoan() {
Article article = new Article("JUnit Rocks");

Assertions.assertEquals("item is not currently on loan", article.checkIn());
}

@Test
public void shouldReturnAuthorInfo() {
String expected = "This item is written by " + author.getName() + " (e-mail: " + author.getEmail() + ", phone number: "
+ author.getPhoneNumber() +", website: " + author.getWebsite() + ")";
Assertions.assertEquals(expected, article.getAuthor());
}
}
16 changes: 11 additions & 5 deletions src/test/java/com/booleanuk/core/BookTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -4,32 +4,38 @@
import org.junit.jupiter.api.Test;

public class BookTest {

Author author = new Author("Bob", "[email protected]", "911", "www.bob.com");
Book book = new Book("JUnit Rocks", author);

@Test
public void shouldCheckOutIfAvailable() {
Book book = new Book("JUnit Rocks");
Assertions.assertEquals("item has been checked out", book.checkOut());
}

@Test
public void shouldDeclineIfNotAvailableToCheckout() {
Book book = new Book("JUnit Rocks");
book.checkOut();

Assertions.assertEquals("item is currently on loan", book.checkOut());
}

@Test
public void shouldCheckInIfOnLoan() {
Book book = new Book("JUnit Rocks");
book.checkOut();

Assertions.assertEquals("item has been checked in", book.checkIn());
}

@Test
public void shouldDeclineCheckInIfNotOnLoan() {
Book book = new Book("JUnit Rocks");

Assertions.assertEquals("item is not currently on loan", book.checkIn());
}

@Test
public void shouldReturnAuthorInfo() {
String expected = "This item is written by " + author.getName() + " (e-mail: " + author.getEmail() + ", phone number: "
+ author.getPhoneNumber() +", website: " + author.getWebsite() + ")";
Assertions.assertEquals(expected, book.getAuthor());
}
}
Loading