diff options
| author | Flavio Lerda <flerda@google.com> | 2013-09-08 18:04:58 +0100 |
|---|---|---|
| committer | Flavio Lerda <flerda@google.com> | 2013-09-08 22:29:44 +0100 |
| commit | 5771302ca13c31cb85f17bc58da8f6f8227c9b85 (patch) | |
| tree | ef459de53c9d138b4996f2063de215a70c3394c4 /core/java/android/util/SparseBooleanArray.java | |
| parent | d57de6afb8981dc1663d41a842ab7cdae93427a1 (diff) | |
Document the order of values returned by keyAt().
The values returned by keyAt() are currently guaranteed to be in
ascending order. This is helpful to users of the API to be able to make
assumptions about the keys and values when iterating over one of the
sparse array implementations.
This commit adds documentation about this.
Change-Id: I3d7eb78e115ce174f1167b83904b44bf5120b223
Diffstat (limited to 'core/java/android/util/SparseBooleanArray.java')
| -rw-r--r-- | core/java/android/util/SparseBooleanArray.java | 23 |
1 files changed, 20 insertions, 3 deletions
diff --git a/core/java/android/util/SparseBooleanArray.java b/core/java/android/util/SparseBooleanArray.java index da196d78161d..68487e391710 100644 --- a/core/java/android/util/SparseBooleanArray.java +++ b/core/java/android/util/SparseBooleanArray.java @@ -33,6 +33,12 @@ import com.android.internal.util.ArrayUtils; * HashMap, since lookups require a binary search and adds and removes require inserting * and deleting entries in the array. For containers holding up to hundreds of items, * the performance difference is not significant, less than 50%.</p> + * + * <p>It is possible to iterate over the items in this container using + * {@link #keyAt(int)} and {@link #valueAt(int)}. Iterating over the keys using + * <code>keyAt(int)</code> with ascending values of the index will return the + * keys in ascending order, or the values corresponding to the keys in ascending + * order in the case of <code>valueAt(int)<code>.</p> */ public class SparseBooleanArray implements Cloneable { /** @@ -159,16 +165,27 @@ public class SparseBooleanArray implements Cloneable { /** * Given an index in the range <code>0...size()-1</code>, returns * the key from the <code>index</code>th key-value mapping that this - * SparseBooleanArray stores. + * SparseBooleanArray stores. + * + * <p>The keys corresponding to indices in ascending order are guaranteed to + * be in ascending order, e.g., <code>keyAt(0)</code> will return the + * smallest key and <code>keyAt(size()-1)</code> will return the largest + * key.</p> */ public int keyAt(int index) { return mKeys[index]; } - + /** * Given an index in the range <code>0...size()-1</code>, returns * the value from the <code>index</code>th key-value mapping that this - * SparseBooleanArray stores. + * SparseBooleanArray stores. + * + * <p>The values corresponding to indices in ascending order are guaranteed + * to be associated with keys in ascending order, e.g., + * <code>valueAt(0)</code> will return the value associated with the + * smallest key and <code>valueAt(size()-1)</code> will return the value + * associated with the largest key.</p> */ public boolean valueAt(int index) { return mValues[index]; |
