Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Make default implementation of fillWindow typesafe. Bug: 5218310"

Browse Source
This commit is contained in:
Jeff Brown
2011-10-14 21:06:08 -07:00
committed by Android (Google) Code Review
parent a39a055304 80e7b80fa6
commit d00851082f
3 changed files with 103 additions and 79 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
core/java/android/database/AbstractCursor.java
View File

@@ -53,7 +53,10 @@ public abstract class AbstractCursor implements CrossProcessCursor {
abstract public boolean isNull(int column); abstract public boolean isNull(int column);
public int getType(int column) { public int getType(int column) {
throw new UnsupportedOperationException(); // Reflects the assumption that all commonly used field types (meaning everything
// but blobs) are convertible to strings so it should be safe to call
// getString to retrieve them.
return FIELD_TYPE_STRING;
} }
// TODO implement getBlob in all cursor types // TODO implement getBlob in all cursor types
@@ -185,46 +188,9 @@ public abstract class AbstractCursor implements CrossProcessCursor {
return result; return result;
} }
/** @Override
* Copy data from cursor to CursorWindow
* @param position start position of data
* @param window
*/
public void fillWindow(int position, CursorWindow window) { public void fillWindow(int position, CursorWindow window) {
if (position < 0 || position >= getCount()) { DatabaseUtils.cursorFillWindow(this, position, window);
return;
}
window.acquireReference();
try {
int oldpos = mPos;
mPos = position - 1;
window.clear();
window.setStartPosition(position);
int columnNum = getColumnCount();
window.setNumColumns(columnNum);
while (moveToNext() && window.allocRow()) {
for (int i = 0; i < columnNum; i++) {
String field = getString(i);
if (field != null) {
if (!window.putString(field, mPos, i)) {
window.freeLastRow();
break;
}
} else {
if (!window.putNull(mPos, i)) {
window.freeLastRow();
break;
}
}
}
}
mPos = oldpos;
} catch (IllegalStateException e){
// simply ignore it
} finally {
window.releaseReference();
}
} }
public final boolean move(int offset) { public final boolean move(int offset) {

57
core/java/android/database/CursorWindow.java
View File

@@ -271,8 +271,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Returns true if the field at the specified row and column index * Returns true if the field at the specified row and column index
* has type {@link Cursor#FIELD_TYPE_NULL}. * has type {@link Cursor#FIELD_TYPE_NULL}.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if the field has type {@link Cursor#FIELD_TYPE_NULL}. * @return True if the field has type {@link Cursor#FIELD_TYPE_NULL}.
* @deprecated Use {@link #getType(int, int)} instead. * @deprecated Use {@link #getType(int, int)} instead.
@@ -286,8 +285,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Returns true if the field at the specified row and column index * Returns true if the field at the specified row and column index
* has type {@link Cursor#FIELD_TYPE_BLOB} or {@link Cursor#FIELD_TYPE_NULL}. * has type {@link Cursor#FIELD_TYPE_BLOB} or {@link Cursor#FIELD_TYPE_NULL}.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if the field has type {@link Cursor#FIELD_TYPE_BLOB} or * @return True if the field has type {@link Cursor#FIELD_TYPE_BLOB} or
* {@link Cursor#FIELD_TYPE_NULL}. * {@link Cursor#FIELD_TYPE_NULL}.
@@ -303,8 +301,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Returns true if the field at the specified row and column index * Returns true if the field at the specified row and column index
* has type {@link Cursor#FIELD_TYPE_INTEGER}. * has type {@link Cursor#FIELD_TYPE_INTEGER}.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if the field has type {@link Cursor#FIELD_TYPE_INTEGER}. * @return True if the field has type {@link Cursor#FIELD_TYPE_INTEGER}.
* @deprecated Use {@link #getType(int, int)} instead. * @deprecated Use {@link #getType(int, int)} instead.
@@ -318,8 +315,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Returns true if the field at the specified row and column index * Returns true if the field at the specified row and column index
* has type {@link Cursor#FIELD_TYPE_FLOAT}. * has type {@link Cursor#FIELD_TYPE_FLOAT}.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if the field has type {@link Cursor#FIELD_TYPE_FLOAT}. * @return True if the field has type {@link Cursor#FIELD_TYPE_FLOAT}.
* @deprecated Use {@link #getType(int, int)} instead. * @deprecated Use {@link #getType(int, int)} instead.
@@ -333,8 +329,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Returns true if the field at the specified row and column index * Returns true if the field at the specified row and column index
* has type {@link Cursor#FIELD_TYPE_STRING} or {@link Cursor#FIELD_TYPE_NULL}. * has type {@link Cursor#FIELD_TYPE_STRING} or {@link Cursor#FIELD_TYPE_NULL}.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if the field has type {@link Cursor#FIELD_TYPE_STRING} * @return True if the field has type {@link Cursor#FIELD_TYPE_STRING}
* or {@link Cursor#FIELD_TYPE_NULL}. * or {@link Cursor#FIELD_TYPE_NULL}.
@@ -359,8 +354,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The field type. * @return The field type.
*/ */
@@ -390,8 +384,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as a byte array. * @return The value of the field as a byte array.
*/ */
@@ -426,8 +419,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as a string. * @return The value of the field as a string.
*/ */
@@ -465,8 +457,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @param buffer The {@link CharArrayBuffer} to hold the string. It is automatically * @param buffer The {@link CharArrayBuffer} to hold the string. It is automatically
* resized if the requested string is larger than the buffer's current capacity. * resized if the requested string is larger than the buffer's current capacity.
@@ -501,8 +492,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as a <code>long</code>. * @return The value of the field as a <code>long</code>.
*/ */
@@ -534,8 +524,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* </ul> * </ul>
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as a <code>double</code>. * @return The value of the field as a <code>double</code>.
*/ */
@@ -556,8 +545,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* result to <code>short</code>. * result to <code>short</code>.
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as a <code>short</code>. * @return The value of the field as a <code>short</code>.
*/ */
@@ -573,8 +561,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* result to <code>int</code>. * result to <code>int</code>.
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as an <code>int</code>. * @return The value of the field as an <code>int</code>.
*/ */
@@ -590,8 +577,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* result to <code>float</code>. * result to <code>float</code>.
* </p> * </p>
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return The value of the field as an <code>float</code>. * @return The value of the field as an <code>float</code>.
*/ */
@@ -603,8 +589,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Copies a byte array into the field at the specified row and column index. * Copies a byte array into the field at the specified row and column index.
* *
* @param value The value to store. * @param value The value to store.
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if successful. * @return True if successful.
*/ */
@@ -621,8 +606,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Copies a string into the field at the specified row and column index. * Copies a string into the field at the specified row and column index.
* *
* @param value The value to store. * @param value The value to store.
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if successful. * @return True if successful.
*/ */
@@ -639,8 +623,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* Puts a long integer into the field at the specified row and column index. * Puts a long integer into the field at the specified row and column index.
* *
* @param value The value to store. * @param value The value to store.
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if successful. * @return True if successful.
*/ */
@@ -658,8 +641,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
* specified row and column index. * specified row and column index.
* *
* @param value The value to store. * @param value The value to store.
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if successful. * @return True if successful.
*/ */
@@ -675,8 +657,7 @@ public class CursorWindow extends SQLiteClosable implements Parcelable {
/** /**
* Puts a null value into the field at the specified row and column index. * Puts a null value into the field at the specified row and column index.
* *
* @param row The zero-based row index, relative to the cursor window's * @param row The zero-based row index.
* start position ({@link #getStartPosition()}).
* @param column The zero-based column index. * @param column The zero-based column index.
* @return True if successful. * @return True if successful.
*/ */

79
core/java/android/database/DatabaseUtils.java
View File

@@ -237,13 +237,90 @@ public class DatabaseUtils {
return Cursor.FIELD_TYPE_BLOB; return Cursor.FIELD_TYPE_BLOB;
} else if (obj instanceof Float || obj instanceof Double) { } else if (obj instanceof Float || obj instanceof Double) {
return Cursor.FIELD_TYPE_FLOAT; return Cursor.FIELD_TYPE_FLOAT;
} else if (obj instanceof Long || obj instanceof Integer) { } else if (obj instanceof Long || obj instanceof Integer
|| obj instanceof Short || obj instanceof Byte) {
return Cursor.FIELD_TYPE_INTEGER; return Cursor.FIELD_TYPE_INTEGER;
} else { } else {
return Cursor.FIELD_TYPE_STRING; return Cursor.FIELD_TYPE_STRING;
} }
} }
/**
* Fills the specified cursor window by iterating over the contents of the cursor.
* The window is filled until the cursor is exhausted or the window runs out
* of space.
*
* The original position of the cursor is left unchanged by this operation.
*
* @param cursor The cursor that contains the data to put in the window.
* @param position The start position for filling the window.
* @param window The window to fill.
* @hide
*/
public static void cursorFillWindow(final Cursor cursor,
int position, final CursorWindow window) {
if (position < 0 || position >= cursor.getCount()) {
return;
}
window.acquireReference();
try {
final int oldPos = cursor.getPosition();
final int numColumns = cursor.getColumnCount();
window.clear();
window.setStartPosition(position);
window.setNumColumns(numColumns);
if (cursor.moveToPosition(position)) {
do {
if (!window.allocRow()) {
break;
}
for (int i = 0; i < numColumns; i++) {
final int type = cursor.getType(i);
final boolean success;
switch (type) {
case Cursor.FIELD_TYPE_NULL:
success = window.putNull(position, i);
break;
case Cursor.FIELD_TYPE_INTEGER:
success = window.putLong(cursor.getLong(i), position, i);
break;
case Cursor.FIELD_TYPE_FLOAT:
success = window.putDouble(cursor.getDouble(i), position, i);
break;
case Cursor.FIELD_TYPE_BLOB: {
final byte[] value = cursor.getBlob(i);
success = value != null ? window.putBlob(value, position, i)
: window.putNull(position, i);
break;
}
default: // assume value is convertible to String
case Cursor.FIELD_TYPE_STRING: {
final String value = cursor.getString(i);
success = value != null ? window.putString(value, position, i)
: window.putNull(position, i);
break;
}
}
if (!success) {
window.freeLastRow();
break;
}
}
position += 1;
} while (cursor.moveToNext());
}
cursor.moveToPosition(oldPos);
} catch (IllegalStateException e){
// simply ignore it
} finally {
window.releaseReference();
}
}
/** /**
* Appends an SQL string to the given StringBuilder, including the opening * Appends an SQL string to the given StringBuilder, including the opening
* and closing single quotes. Any single quotes internal to sqlString will * and closing single quotes. Any single quotes internal to sqlString will