lib: document abstract_collection + nitunit tests
authorJean Privat <jean@pryen.org>
Thu, 20 Mar 2014 01:23:18 +0000 (21:23 -0400)
committerJean Privat <jean@pryen.org>
Thu, 20 Mar 2014 20:30:03 +0000 (16:30 -0400)
Signed-off-by: Jean Privat <jean@pryen.org>

lib/standard/collection/abstract_collection.nit

index 18bac96..8b72d85 100644 (file)
@@ -4,13 +4,15 @@
 #
 # This file is free software, which comes along with NIT.  This software is
 # distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY;
-# without  even  the implied warranty of  MERCHANTABILITY or  FITNESS FOR A 
+# without  even  the implied warranty of  MERCHANTABILITY or  FITNESS FOR A
 # PARTICULAR PURPOSE.  You can modify it is you want,  provided this header
 # is kept unaltered, and a notification of the changes is added.
 # You  are  allowed  to  redistribute it and sell it, alone or is a part of
 # another product.
 
-# This module define several abstract collection classes.
+# Abstract collection classes and services.
+#
+# TODO specify the behavior on iterators when collections are modified.
 module abstract_collection
 
 import kernel
@@ -71,7 +73,6 @@ interface Collection[E]
                return nb
        end
 
-
        # Is `item` in the collection ?
        # Comparisons are done with ==
        #
@@ -151,7 +152,11 @@ interface Iterator[E]
 end
 
 # A collection that contains only one item.
-# Used to pass arguments by reference
+#
+# Used to pass arguments by reference.
+#
+# Also used when one want to give asingle element when a full
+# collection is expected
 class Container[E]
        super Collection[E]
 
@@ -200,30 +205,57 @@ end
 # Items can be removed from this collection
 interface RemovableCollection[E]
        super Collection[E]
+
        # Remove all items
+       #
+       #     var a = [1,2,3]
+       #     a.clear
+       #     assert a.length == 0
+       #
+       # ENSURE `is_empty`
        fun clear is abstract
 
        # Remove an occucence of `item`
+       #
+       #     var a = [1,2,3,1,2,3]
+       #     a.remove 2
+       #     assert a == [1,3,1,2,3]
        fun remove(item: E) is abstract
 
        # Remove all occurences of `item`
+       #
+       #     var a = [1,2,3,1,2,3]
+       #     a.remove_all 2
+       #     assert a == [1,3,1,3]
        fun remove_all(item: E) do while has(item) do remove(item)
 end
 
 # Items can be added to these collections.
 interface SimpleCollection[E]
        super RemovableCollection[E]
+
        # Add an item in a collection.
+       #
+       #     var a = [1,2]
+       #     a.add 3
+       #     assert a.has(3)  == true
+       #     assert a.has(10) == false
+       #
        # Ensure col.has(item)
        fun add(item: E) is abstract
 
        # Add each item of `coll`.
+       #     var a = [1,2]
+       #     a.add_all [3..5]
+       #     assert a.has(4)  == true
+       #     assert a.has(10) == false
        fun add_all(coll: Collection[E]) do for i in coll do add(i)
 end
 
 # Abstract sets.
 #
-# Set is a collection without ducplicates (according to ==)
+# Set is a collection without duplicates (according to `==`)
+#
 #      var s: Set[String] = new ArraySet[String]
 #      var a = "Hello"
 #      var b = "Hel" + "lo"
@@ -266,7 +298,7 @@ interface Set[E: Object]
                return has_all(other)
        end
 
-       # because of the law between `==` and `hash`, hash is redefined to be the sum of the hash of the elements
+       # Because of the law between `==` and `hash`, `hash` is redefined to be the sum of the hash of the elements
        redef fun hash
        do
                var res = 0
@@ -277,7 +309,15 @@ end
 
 # MapRead are abstract associative collections: `key` -> `item`.
 interface MapRead[K: Object, E]
-       # Get the item at `key`.
+       # Get the item at `key`
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     assert x["four"] == 4
+       #     # assert x["five"] #=> abort
+       #
+       # If the key is not in the map, `provide_default_value` is called (that aborts by default)
+       # See `get_or_null` and `get_or_default` for safe variations.
        fun [](key: K): E is abstract
 
        # Get the item at `key` or null if `key` is not in the map.
@@ -295,6 +335,12 @@ interface MapRead[K: Object, E]
        end
 
        # Get the item at `key` or return `default` if not in map
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     assert x.get_or_default("four", 40) == 4
+       #     assert x.get_or_default("five", 50) == 50
+       #
        fun get_or_default(key: K, default: E): E
        do
                if has_key(key) then return self[key]
@@ -310,17 +356,39 @@ interface MapRead[K: Object, E]
        # Return the point of view of self on the values only.
        # Note that `self` and `values` are views on the same data;
        # therefore any modification of one is visible on the other.
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     assert x.values.has(4) == true
+       #     assert x.values.has(5) == false
        fun values: Collection[E] is abstract
 
        # Return the point of view of self on the keys only.
        # Note that `self` and `keys` are views on the same data;
        # therefore any modification of one is visible on the other.
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     assert x.keys.has("four") == true
+       #     assert x.keys.has("five") == false
        fun keys: Collection[K] is abstract
 
        # Is there no item in the collection?
+       #
+       #     var x = new HashMap[String, Int]
+       #     assert x.is_empty  == true
+       #     x["four"] = 4
+       #     assert x.is_empty  == false
        fun is_empty: Bool is abstract
 
        # Number of items in the collection.
+       #
+       #     var x = new HashMap[String, Int]
+       #     assert x.length  == 0
+       #     x["four"] = 4
+       #     assert x.length  == 1
+       #     x["five"] = 5
+       #     assert x.length  == 2
        fun length: Int is abstract
 
        # Called by the underling implementation of `[]` to provide a default value when a `key` has no value
@@ -357,11 +425,39 @@ end
 #
 interface Map[K: Object, E]
        super MapRead[K, E]
-       # Set the`item` at `key`.
-       fun []=(key: K, item: E) is abstract
+
+       # Set the `value` at `key`.
+       #
+       # Values can then get retrieved with `[]`.
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     assert x["four"]   == 4
+       #
+       # If the key was associated with a value, this old value is discarted
+       # and replaced with the new one.
+       #
+       #     x["four"] = 40
+       #     assert x["four"]         == 40
+       #     assert x.values.has(4)   == false
+       #
+       fun []=(key: K, value: E) is abstract
 
        # Add each (key,value) of `map` into `self`.
        # If a same key exists in `map` and `self`, then the value in self is discarded.
+       #
+       # It is the analogous of `SimpleCollection::add_all`
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     x["five"] = 5
+       #     var y = new HashMap[String, Int]
+       #     y["four"] = 40
+       #     y["nine"] = 90
+       #     x.recover_with y
+       #     assert x["four"]  == 40
+       #     assert x["five"]  == 5
+       #     assert x["nine"]  == 90
        fun recover_with(map: Map[K, E])
        do
                var i = map.iterator
@@ -372,6 +468,13 @@ interface Map[K: Object, E]
        end
 
        # Remove all items
+       #
+       #     var x = new HashMap[String, Int]
+       #     x["four"] = 4
+       #     x.clear
+       #     x.keys.has("four") == false
+       #
+       # ENSURE `is_empty`
        fun clear is abstract
 
        redef fun values: RemovableCollection[E] is abstract
@@ -424,23 +527,44 @@ end
 
 # Sequences are indexed collections.
 # The first item is 0. The last is `length-1`.
+#
+# The order is the main caracteristic of sequence
+# and all concrete implementation of sequences are basically interchangeable.
 interface SequenceRead[E]
        super Collection[E]
+
        # Get the first item.
        # Is equivalent with `self[0]`.
+       #
+       #     var a = [1,2,3]
+       #     assert a.first   == 1
+       #
+       # REQUIRE `not is_empty`
        redef fun first
        do
                assert not_empty: not is_empty
                return self[0]
        end
 
-       # Return the index=th element of the sequence.
-       # The first element is 0 and the last if `length-1`
+       # Return the index-th element of the sequence.
+       # The first element is 0 and the last is `length-1`
        # If index is invalid, the program aborts
+       #
+       #     var a = [10,20,30]
+       #     assert a[0]   == 10
+       #     assert a[1]   == 20
+       #     assert a[2]   == 30
+       #
+       # REQUIRE `index >= 0 and index < length`
        fun [](index: Int): E is abstract
 
        # Get the last item.
        # Is equivalent with `self[length-1]`.
+       #
+       #     var a = [1,2,3]
+       #     assert a.last   == 3
+       #
+       # REQUIRE `not is_empty`
        fun last: E
        do
                assert not_empty: not is_empty
@@ -450,6 +574,10 @@ interface SequenceRead[E]
        # Return the index of the first occurrence of `item`.
        # Return -1 if `item` is not found
        # Comparison is done with ==
+       #
+       #     var a = [10,20,30,10,20,30]
+       #     assert a.index_of(20)   == 1
+       #     assert a.index_of(40)   == -1
        fun index_of(item: E): Int
        do
                var i = iterator
@@ -463,6 +591,13 @@ interface SequenceRead[E]
        redef fun iterator: IndexedIterator[E] is abstract
 
        # Two sequences are equals if they have the same items in the same order.
+       #
+       #     var a = new List[Int]
+       #     a.add(1)
+       #     a.add(2)
+       #     a.add(3)
+       #     assert a == [1,2,3]
+       #     assert a != [1,3,2]
        redef fun ==(o)
        do
                if not o isa SequenceRead[nullable Object] then return false
@@ -476,7 +611,7 @@ interface SequenceRead[E]
                return true
        end
 
-       # because of the law between `==` and `hash`, hash is redefined to be the sum of the hash of the elements
+       # Because of the law between `==` and `hash`, `hash` is redefined to be the sum of the hash of the elements
        redef fun hash
        do
                var res = 0
@@ -493,13 +628,27 @@ interface Sequence[E]
 
        # Set the first item.
        # Is equivalent with `self[0] = item`.
+       #
+       #     var a = [1,2,3]
+       #     a.first = 10
+       #     assert a == [10,2,3]
        fun first=(item: E)
        do self[0] = item end
 
        # Set the last item.
        # Is equivalent with `self[length-1] = item`.
-       fun last=(item: E) 
-       do 
+       #
+       #     var a = [1,2,3]
+       #     a.last = 10
+       #     assert a == [1,2,10]
+       #
+       # If the sequence is empty, `last=` is equivalent with `self[0]=` (thus with `first=`)
+       #
+       #     var b = new Array[Int]
+       #     b.last = 10
+       #     assert b == [10]
+       fun last=(item: E)
+       do
                var l = length
                if l > 0 then
                        self[l-1] = item
@@ -511,26 +660,72 @@ interface Sequence[E]
        # A synonym of `push`
        redef fun add(e) do push(e)
 
-       # Add an item after the last.
+       # Add an item after the last one.
+       #
+       #     var a = [1,2,3]
+       #     a.push(10)
+       #     a.push(20)
+       #     assert a  == [1,2,3,10,20]
        fun push(e: E) is abstract
 
        # Add each item of `coll` after the last.
+       #
+       #     var a = [1,2,3]
+       #     a.append([7..9])
+       #     assert a  == [1,2,3,7,8,9]
        fun append(coll: Collection[E]) do for i in coll do push(i)
 
        # Remove the last item.
+       #
+       #     var a = [1,2,3]
+       #     assert a.pop  == 3
+       #     assert a.pop  == 2
+       #     assert a == [1]
+       #
+       # REQUIRE `not is_empty`
        fun pop: E is abstract
 
-       # Add an item before the last.
+       # Add an item before the first one.
+       #
+       #     var a = [1,2,3]
+       #     a.unshift(10)
+       #     a.unshift(20)
+       #     assert a  == [20,10,1,2,3]
        fun unshift(e: E) is abstract
 
        # Remove the first item.
-       # The second item become the first.
+       # The second item thus become the first.
+       #
+       #     var a = [1,2,3]
+       #     assert a.shift  == 1
+       #     assert a.shift  == 2
+       #     assert a == [3]
+       #
+       # REQUIRE `not is_empty`
        fun shift: E is abstract
 
        # Set the `item` at `index`.
+       #
+       #     var a = [10,20,30]
+       #     a[1] = 200
+       #     assert a  == [10,200,30]
+       #
+       # like with `[]`, index should be between `0` and `length-1`
+       # However, if `index==length`, `[]=` works like `push`.
+       #
+       #     a[3] = 400
+       #     assert a  == [10,200,30,400]
+       #
+       # REQUIRE `index >= 0 and index <= length`
        fun []=(index: Int, item: E) is abstract
 
        # Remove the item at `index` and shift all following elements
+       #
+       #     var a = [10,20,30]
+       #     a.remove_at(1)
+       #     assert a  == [10,30]
+       #
+       # REQUIRE `index >= 0 and index < length`
        fun remove_at(index: Int) is abstract
 end
 
@@ -573,7 +768,7 @@ class CoupleMapIterator[K: Object, E]
        redef fun is_ok do return _iter.is_ok
 
        redef fun next
-       do 
+       do
                _iter.next
        end