Classes name refactoring, added ModelException, update on TimeFrame update

app/src/main/java/net/pokeranalytics/android/calculus/Calculator.kt

@@ -1,7 +1,7 @@
 package net.pokeranalytics.android.calculus
 
 import net.pokeranalytics.android.calculus.Stat.*
-import net.pokeranalytics.android.model.realm.TimeFrameGroup
+import net.pokeranalytics.android.model.realm.SessionGroup
 
 
 class Calculator {
@@ -45,7 +45,7 @@ class Calculator {
         }
 
         // Computes all stats for list of Session group
-        fun computeGroups(groups: List<SessionGroup>, options: Options): List<ComputedGroup> {
+        fun computeGroups(groups: List<ComputableSessions>, options: Options): List<ComputedGroup> {
 
             var computedGroups: MutableList<ComputedGroup> = mutableListOf()
             groups.forEach { group ->
@@ -69,10 +69,10 @@ class Calculator {
         }
 
         // Computes stats for a SessionGroup
-        fun compute(group: SessionGroup, options: Options) : ComputedResults {
+        fun compute(group: ComputableSessions, options: Options) : ComputedResults {
 
             val sessions: List<SessionInterface> = group.sessionGroup
-            val sessionGroups: Set<TimeFrameGroup> = setOf() // @todo get unique list of serie
+            val sessionGroups: Set<SessionGroup> = setOf() // @todo get unique list of serie
             var results: ComputedResults = ComputedResults()
 
             var sum: Double = 0.0

app/src/main/java/net/pokeranalytics/android/calculus/Computable.kt

@@ -1,15 +1,19 @@
 package net.pokeranalytics.android.calculus
 
-import net.pokeranalytics.android.model.realm.TimeFrameGroup
+import net.pokeranalytics.android.model.realm.SessionGroup
 
-// An interface to describe objects that can be summed
+/**
+ * An interface to describe objects that can be summed
+ */
 interface Summable {
     var value: Double
 }
 
-// An interface describing some class that can be computed
+/**
+ * An interface describing some class that can be computed
+ */
 interface SessionInterface : Summable {
-    var serie: TimeFrameGroup
+    var serie: SessionGroup
     var estimatedHands: Double
     var bbNetResult: Double
     var bigBlindSessionCount: Int // 0 or 1
@@ -19,22 +23,22 @@ interface SessionInterface : Summable {
 /**
  * A group of computable items identified by a name
  */
-class SessionGroup(name: String, sessions: List<SessionInterface>) {
+class ComputableSessions(name: String, sessions: List<SessionInterface>) {
 
     var name: String = name
     var sessionGroup: List<SessionInterface> = sessions
 
     // A subgroup used to compute stat variation
-    var comparedGroup: SessionGroup? = null
+    var comparedGroup: ComputableSessions? = null
 
     // The computed stats of the comparable group
     var comparedComputedGroup: ComputedGroup? = null
 
 }
 
-class ComputedGroup(group: SessionGroup, computedResults: ComputedResults) {
+class ComputedGroup(group: ComputableSessions, computedResults: ComputedResults) {
     // A computable group
-    var group: SessionGroup = group
+    var group: ComputableSessions = group
 
     // The computed stats of the group
     var computedResults: ComputedResults = computedResults

app/src/main/java/net/pokeranalytics/android/exceptions/ModelException.kt

@@ -0,0 +1,5 @@
+package net.pokeranalytics.android.exceptions
+
+class ModelException(string: String) : Exception(string) {
+
+}

app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt

@@ -23,7 +23,7 @@ open class Session(comment: String = "") : RealmObject(), DynamicRowDelegate {
     var timeFrame: TimeFrame? = null
 
     // The time frame group, which can contain multiple sessions
-    var timeFrameGroup: TimeFrameGroup? = null
+    var sessionGroup: SessionGroup? = null
 
     // the date of creation of the app
     var creationDate: Date = Date()
@@ -107,7 +107,6 @@ open class Session(comment: String = "") : RealmObject(), DynamicRowDelegate {
 
     @Ignore
     var bigBlindSessionCount: Int = if (this.cgBigBlind != null) 1 else 0
-//        get() = if (this.cgBigBlind != null) 1 else 0
 
     @Ignore
     var buyin: Double = 0.0

app/src/main/java/net/pokeranalytics/android/model/realm/SessionGroup.kt

@@ -5,7 +5,7 @@ import io.realm.RealmObject
 import io.realm.annotations.Ignore
 
 
-open class TimeFrameGroup() : RealmObject() {
+open class SessionGroup() : RealmObject() {
 
     // The timeframe of the group, i.e. its start & end date
     var timeFrame: TimeFrame? = null
@@ -27,5 +27,17 @@ open class TimeFrameGroup() : RealmObject() {
 
     @Ignore
     var bbNetResult: Double = 0.0
+
+    companion object {
+
+        fun newInstance() : SessionGroup {
+            val sessionGroup: SessionGroup = SessionGroup()
+            sessionGroup.timeFrame = TimeFrame()
+            return sessionGroup
+        }
+
+    }
+
+
 }
 

app/src/main/java/net/pokeranalytics/android/model/realm/TimeFrame.kt

@@ -1,12 +1,14 @@
 package net.pokeranalytics.android.model.realm
 
+import io.realm.Realm
 import io.realm.RealmObject
+import io.realm.RealmQuery
 import io.realm.RealmResults
 import io.realm.annotations.LinkingObjects
+import net.pokeranalytics.android.exceptions.ModelException
 import java.util.*
 
 
-
 open class TimeFrame : RealmObject() {
 
     // A start date
@@ -15,7 +17,7 @@ open class TimeFrame : RealmObject() {
             field = value
             this.computeDuration()
             if (this.session != null) {
-                this.notifyDateChange()
+                this.notifySessionDateChange()
             }
         }
 
@@ -25,7 +27,7 @@ open class TimeFrame : RealmObject() {
             field = value
             this.computeDuration()
             if (this.session != null) {
-                this.notifyDateChange()
+                this.notifySessionDateChange()
             }
         }
 
@@ -47,7 +49,7 @@ open class TimeFrame : RealmObject() {
     private val session: RealmResults<Session>? = null
 
     @LinkingObjects("timeFrame")
-    private val group: RealmResults<TimeFrameGroup>? = null
+    private val group: RealmResults<SessionGroup>? = null
 
     private fun computeDuration() {
 
@@ -60,20 +62,106 @@ open class TimeFrame : RealmObject() {
         this.duration = endDate.time - startDate.time - this.breakDuration
     }
 
-    private fun notifyDateChange() {
-//        val realm = Realm.getDefaultInstance()
-//
-//        var query: RealmQuery<TimeFrame> = realm.where(TimeFrame::class.java)
-//        query.greaterThan("startDate", this.startDate.time)
-//
-//        this.endDate?.let { endDate ->
-//            query.or()
-//                .greaterThan("startDate", endDate.time)
-//                .lessThan("endDate", endDate.time)
-//        }
-//
-//
-//        realm.close()
+    private fun notifySessionDateChange() {
+        val realm = Realm.getDefaultInstance()
+        var query: RealmQuery<SessionGroup> = realm.where(SessionGroup::class.java)
+        query.isNotNull("timeFrame")
+
+        if (this.endDate == null) {
+            query.greaterThan("timeFrame.startDate", this.startDate.time).or().greaterThan("timeFrame.endDate", this.startDate.time)
+        } else {
+            val endDate = this.endDate!!
+            query
+                .greaterThan("timeFrame.startDate", this.startDate.time)
+                .lessThan("timeFrame.endDate", this.startDate.time)
+                .or()
+                .greaterThan("timeFrame.startDate", endDate)
+                .lessThan("timeFrame.endDate", endDate)
+                .or()
+                .lessThan("timeFrame.startDate", this.startDate.time)
+                .greaterThan("timeFrame.endDate", endDate)
+        }
+
+        val sessionGroups = query.findAll()
+
+        this.updateTimeFrames(sessionGroups)
+
+        realm.close()
+    }
+
+    /**
+     * Update Time frames from groups
+     */
+    private fun updateTimeFrames(sessionGroups: RealmResults<SessionGroup>) {
+
+        // update
+        if (sessionGroups.size == 1) {
+            // we are sure that the timeframe is not null from the realm query
+            val groupTimeFrame: TimeFrame = sessionGroups.first()?.timeFrame!!
+            if (this.startDate.before(groupTimeFrame.startDate)) {
+                groupTimeFrame.startDate = this.startDate
+            }
+            val endDate = this.endDate
+            if (endDate != null && groupTimeFrame.endDate != null && endDate.after(groupTimeFrame.endDate)) {
+                groupTimeFrame.endDate = endDate
+            } else if (endDate == null) {
+                groupTimeFrame.endDate = null
+            }
+
+        } else if (sessionGroups.size > 1) {
+
+            var startDate: Date = this.startDate
+            var endDate: Date? = this.endDate
+
+            val timeFrames = sessionGroups.mapNotNull { it.timeFrame }
+            timeFrames.forEach { tf ->
+                if (tf.startDate.before(startDate)) {
+                    startDate = tf.startDate
+                }
+
+                endDate?.let { ed ->
+                    tf.endDate?.let { tfed ->
+                        if (tfed.after(ed)) {
+                            endDate = tfed
+                        }
+                    }
+                } ?: run {
+                    endDate = tf.endDate
+                }
+
+            }
+
+            // get all sessions from groups
+            val sessions = sessionGroups.flatMap { it.sessions }
+
+            // delete all groups
+            val realm = Realm.getDefaultInstance()
+            realm.executeTransaction {
+                sessionGroups.deleteAllFromRealm()
+            }
+
+            val group: SessionGroup = SessionGroup.newInstance()
+            group.timeFrame?.let {
+                it.startDate = startDate
+                it.endDate = endDate
+            } ?: run {
+                throw ModelException("TimeFrame should never be null here")
+            }
+
+            this.session?.first()?.let {
+                group.sessions.add(it)
+            } ?: run {
+                throw ModelException("TimeFrame should never be null here")
+            }
+
+
+
+            //@todo delete all timeframes, create a new one
+
+
+
+        }
+
     }
 
 }

app/src/test/java/net/pokeranalytics/android/ExampleUnitTest.kt

@@ -1,7 +1,7 @@
 package net.pokeranalytics.android
 
 import net.pokeranalytics.android.calculus.*
-import net.pokeranalytics.android.model.realm.TimeFrameGroup
+import net.pokeranalytics.android.model.realm.SessionGroup
 import org.junit.Assert.fail
 import org.junit.Test
 
@@ -17,7 +17,7 @@ class ExampleUnitTest {
 //        override var serie: Serie = Serie(TimeFrame())
         override var value: Double = someValue
 
-        override var serie: TimeFrameGroup = TimeFrameGroup()
+        override var serie: SessionGroup = SessionGroup()
         override var estimatedHands: Double = 0.0
         override var bbNetResult: Double = 0.0
         override var bigBlindSessionCount: Int = 0 // 0 or 1