Refactor the ReadOnly interfaces

[w-yeehong]

There are currently three interfaces that serve a similar purpose: ReadOnlyPatientRecords, ReadOnlyRoomList, and ReadOnlyTaskList.

We can refactor these interfaces into one using generics ReadOnlyList<T> to avoid code duplication.

Other changes:

• Add authorship tags
• Move UniqueRoomList to the room package

[chiamyunqing]

LGTM! Good learning point for using Java Generics, definitely makes it neater!

[codecov-io]

Codecov Report

Merging #154 into master will not change coverage.
The diff coverage is 90.00%.

@@            Coverage Diff            @@
##             master     #154   +/-   ##
=========================================
  Coverage     67.42%   67.42%           
  Complexity      752      752           
=========================================
  Files           117      117           
  Lines          2646     2646           
  Branches        328      328           
=========================================
  Hits           1784     1784           
  Misses          743      743           
  Partials        119      119           

Impacted Files	Coverage Δ	Complexity Δ
src/main/java/seedu/address/MainApp.java	0.00% <ø> (ø)	0.00 <0.00> (ø)
.../java/seedu/address/commons/util/DateTimeUtil.java	96.42% <ø> (ø)	4.00 <0.00> (ø)
...rc/main/java/seedu/address/logic/LogicManager.java	61.29% <ø> (ø)	3.00 <0.00> (ø)
...du/address/logic/commands/task/AddTaskCommand.java	38.09% <ø> (ø)	8.00 <0.00> (ø)
...address/logic/commands/task/DeleteTaskCommand.java	32.00% <ø> (ø)	7.00 <0.00> (ø)
...u/address/logic/commands/task/EditTaskCommand.java	46.00% <ø> (ø)	8.00 <0.00> (ø)
...ddress/logic/parser/task/AddTaskCommandParser.java	100.00% <ø> (ø)	4.00 <0.00> (ø)
...ess/logic/parser/task/DeleteTaskCommandParser.java	100.00% <ø> (ø)	4.00 <0.00> (ø)
...dress/logic/parser/task/EditTaskCommandParser.java	90.00% <ø> (ø)	7.00 <0.00> (ø)
src/main/java/seedu/address/model/Model.java	100.00% <ø> (ø)	3.00 <0.00> (ø)
... and 16 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89ca9c2...ada0cdf. Read the comment docs.