mathias handeland

tdd-oop-inheritance.CSharp.Main/Article.cs

@@ -6,37 +6,14 @@
 
 namespace tdd_oop_inheritance.CSharp.Main
 {
-    public class Article {
-        public string title;
+    public class Article : LibraryItem
+    {
+        public Author Author { get; }
 
-        bool onLoan = false;
-
-        public Article(string title) {
-            this.title = title;
-        }
-
-        public bool 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 Article(string title, Author author) : base(title) 
+        { 
+            Author = author;
         }
+
     }
 }

tdd-oop-inheritance.CSharp.Main/Author.cs

@@ -0,0 +1,16 @@
+namespace tdd_oop_inheritance.CSharp.Main
+{
+    public class Author
+    {
+        public string Name { get; }
+        public string ContactInfo { get; }
+        public string Website { get; }
+
+        public Author(string name, string contactInfo, string website)
+        {
+            Name = name;
+            ContactInfo = contactInfo;
+            Website = website;
+        }
+    }
+}

tdd-oop-inheritance.CSharp.Main/Book.cs

@@ -6,37 +6,14 @@
 
 namespace tdd_oop_inheritance.CSharp.Main
 {
-    public class Book {
-        public string title;
+    public class Book : LibraryItem
+    {
+        public Author Author {  get; }
 
-        bool onLoan = false;
-
-        public Book(string title) {
-            this.title = title;
-        }
-
-        public bool 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 Book(string title, Author author) : base(title) 
+        {   
+            Author = author;        
         }
+
     }
 }

tdd-oop-inheritance.CSharp.Main/Library.cs

@@ -6,48 +6,19 @@
 
 namespace tdd_oop_inheritance.CSharp.Main
 {
-    public class Library {
-        List<Article> articles = new List<Article>();
-        List<Book> books = new List<Book>();
-        List<Newspaper> newspapers = new List<Newspaper>();
-
-        public void addToStock(Article item) {
-            this.articles.Add(item);
-        }
-
-        public void addToStock(Book item) {
-            this.books.Add(item);
-        }
-
-        public void addToStock(Newspaper item) {
-            this.newspapers.Add(item);
+    public class Library 
+    {
+        List<LibraryItem> items = new List<LibraryItem>();
+
+        public void addToStock(LibraryItem item) {
+            this.items.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 = (List<Article>)this.articles.Where(article => article.title.Equals(title));
-
-            if (filtered.Count() < 1) {
-                return "item is not part of the library's collection";
-            }
-
-            return filtered[0].checkIn();
-        }
-
-        public string checkOutArticle(string title) {
-            List<Article> filtered = (List<Article>)this.articles.Where(article => article.title.Equals(title));
-
-            if (filtered.Count() < 1) {
-                return "item is not part of the library's collection";
-            }
+        public string checkInItem(string title) {
 
-            return filtered[0].checkOut();
-        }
-
-        public string checkInBook(string title) {
-            List<Book> filtered = (List<Book>)this.books.Where(book => book.Equals(title));
+            List<LibraryItem> filtered = (List<LibraryItem>)this.items.Where(item => item.title.Equals(title)).ToList();
 
             if (filtered.Count() < 1) {
                 return "item is not part of the library's collection";
@@ -56,8 +27,8 @@ public string checkInBook(string title) {
             return filtered[0].checkIn();
         }
 
-        public string checkOutBook(string title) {
-            List<Book> filtered = (List<Book>)this.books.Where(book => book.Equals(title));
+        public string checkOutItem(string title) {
+            List<LibraryItem> filtered = (List<LibraryItem>)this.items.Where(item => item.title.Equals(title)).ToList();
 
             if (filtered.Count() < 1) {
                 return "item is not part of the library's collection";
@@ -66,24 +37,5 @@ public string checkOutBook(string title) {
             return filtered[0].checkOut();
         }
 
-        public string checkInNewspaper(string title) {
-            List<Newspaper> filtered = (List<Newspaper>)this.newspapers.Where(newspaper => newspaper.title.Equals(title));
-
-            if (filtered.Count() < 1) {
-                return "item is not part of the library's collection";
-            }
-
-            return filtered[0].checkIn();
-        }
-
-        public string checkOutNewspaper(string title) {
-            List<Newspaper> filtered = (List<Newspaper>)this.newspapers.Where(newspaper => newspaper.title.Equals(title));
-
-            if (filtered.Count() < 1) {
-                return "item is not part of the library's collection";
-            }
-
-            return filtered[0].checkOut();
-        }
     }
 }

tdd-oop-inheritance.CSharp.Main/LibraryItem.cs

@@ -0,0 +1,48 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace tdd_oop_inheritance.CSharp.Main
+{
+    public class LibraryItem
+    {
+        bool onLoan = false;
+
+        public string title;
+
+        public LibraryItem(string title)
+        {
+            this.title = title;
+        }
+        public bool isOnLoan()
+        {
+            return onLoan;
+        }
+
+        public virtual string checkIn()
+        {
+            if (!this.isOnLoan())
+            {
+                return "item is not currently on loan";
+            }
+
+            this.onLoan = false;
+
+            return "item has been checked in";
+        }
+
+        public virtual string checkOut()
+        {
+            if (this.isOnLoan())
+            {
+                return "item is currently on loan";
+            }
+
+            this.onLoan = true;
+
+            return "item has been checked out";
+        }
+    }
+}

tdd-oop-inheritance.CSharp.Main/Newspaper.cs

@@ -6,25 +6,16 @@
 
 namespace tdd_oop_inheritance.CSharp.Main
 {
-    public class Newspaper
+    public class Newspaper : LibraryItem
     {
-        public string title;
 
-        Boolean onLoan = false;
-
-        public Newspaper(string title) {
-            this.title = title;
-        }
-
-        public bool isOnLoan() {
-            return onLoan;
-        }
-
-        public string checkIn() {
+        public Newspaper(string title) : base(title) { }
+
+        public override string checkIn() {
             return "newspapers are not available for loan";
         }
 
-        public string checkOut() {
+        public override string checkOut() {
             return "newspapers are not available for loan";
         }
     }

tdd-oop-inheritance.CSharp.Test/ArticleTest.cs

@@ -8,14 +8,16 @@ class ArticleTest
         [Test]
         public void shouldCheckOutIfAvailable()
             {
-                Article article = new Article("JUnit Rocks");
+                var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+                Article article = new Article("JUnit Rocks", author);
                 Assert.AreEqual("item has been checked out", article.checkOut());
             }
 
         [Test]
         public void shouldDeclineIfNotAvailableToCheckout()
             {
-                Article article = new Article("JUnit Rocks");
+                var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+                Article article = new Article("JUnit Rocks", author);
                 article.checkOut();
 
                 Assert.AreEqual("item is currently on loan", article.checkOut());
@@ -24,7 +26,8 @@ public void shouldDeclineIfNotAvailableToCheckout()
         [Test]
         public void shouldCheckInIfOnLoan()
             {
-                Article article = new Article("JUnit Rocks");
+                var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+                Article article = new Article("JUnit Rocks", author);
                 article.checkOut();
 
                 Assert.AreEqual("item has been checked in", article.checkIn());
@@ -33,7 +36,8 @@ public void shouldCheckInIfOnLoan()
         [Test]
         public void shouldDeclineCheckInIfNotOnLoan()
             {
-                Article article = new Article("JUnit Rocks");
+                var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+                Article article = new Article("JUnit Rocks", author);
 
                 Assert.AreEqual("item is not currently on loan", article.checkIn());
             }

tdd-oop-inheritance.CSharp.Test/BookTest.cs

@@ -8,14 +8,16 @@ public class BookTest
         [Test]
         public void shouldCheckOutIfAvailable()
         {
-            Book book = new Book("JUnit Rocks");
+            var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+            Book book = new Book("JUnit Rocks", author);
             Assert.AreEqual("item has been checked out", book.checkOut());
         }
 
         [Test]
         public void shouldDeclineIfNotAvailableToCheckout()
         {
-            Book book = new Book("JUnit Rocks");
+            var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+            Book book = new Book("JUnit Rocks", author);
             book.checkOut();
 
             Assert.AreEqual("item is currently on loan", book.checkOut());
@@ -24,7 +26,8 @@ public void shouldDeclineIfNotAvailableToCheckout()
         [Test]
         public void shouldCheckInIfOnLoan()
         {
-            Book book = new Book("JUnit Rocks");
+            var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+            Book book = new Book("JUnit Rocks", author);
             book.checkOut();
 
             Assert.AreEqual("item has been checked in", book.checkIn());
@@ -33,7 +36,8 @@ public void shouldCheckInIfOnLoan()
         [Test]
         public void shouldDeclineCheckInIfNotOnLoan()
         {
-            Book book = new Book("JUnit Rocks");
+            var author = new Author("John Smith", "john@example.com", "www.johnsmith.com");
+            Book book = new Book("JUnit Rocks", author);
 
             Assert.AreEqual("item is not currently on loan", book.checkIn());
         }