Closed Muaj17 closed 10 months ago
I will give a detailed check. In fact main don't need import except menu view I believe. I will get java doc done for use case rn. TODO: javadocs (Muaj Entity, Yehyun all use cases, Tyseer Run game) interface (replace usecase) test cases api slides
Love the improvement with CA for GameView with replacing data_access as interface.
I think we need to replace the entity and use case as well.
I will right now try and improve CA on this aspect.
We need to make sure the inner layer isn't modified by out layer
This has been fixed and improved with CA. I improved game view CA dependency rule in our latest commit!
Added java doctests to all the entities. Unfortunately, I couldn't figure out how to make GameView.java/TestGameView.java not import from use_case and entity. I did make it adhere to CA a little more than it did before though as I edited a few files to make it so that GameView and TestGameView doesn't need to import data access objects. Perhaps someone else can help?
A side note, there were imports made in Main.java. I checked the week 5 coding homework and they did import a data access object so I assumed this is okay to do.