search

Tribler / trustchain-android

An example android app for using trustchain
GNU Lesser General Public License v3.0
3 stars 2 forks source link

Split overviewconnectionactivity into two classes #20

Closed wkmeijer closed 6 years ago

wkmeijer commented 6 years ago

OverviewConnectionsActivity is kind of a god class at the moment since it is responsible for handling all the received messages. This is due to it being a networkcommunicationlistener. However, this system is not needed since we only have one listener for the network. It will be better to let overviewconnectionsactivity mostly be responsible for the UI and handle the further processes of messages in a separate class.

So the networkcommunicationlistener can be removed and replaced by simply calling methods in two different classes: overviewConnectionsActivity for all the UI information and a new class for handling the messages.

wkmeijer commented 6 years ago

Moved the message handling methods to a separate class: MessageHandler. Fixed in branch 12_improve_overlay