阿里Android开发规范：文件与数据库

以下内容摘自阿里巴巴Android开发手册

我们的目标是：

防患未然，提升质量意识，降低故障率和维护成本；
标准统一，提升协作效率；
追求卓越的工匠精神，打磨精品代码。
【强制】必须遵守，违反本约定或将会引起严重的后果；
【推荐】尽量遵守，长期遵守有助于系统稳定性和合作效率的提升；
【参考】充分理解，技术意识的引导，是个人学习、团队沟通、项目合作的方向。

阿里Android开发规范：资源文件命名与使用规范
阿里Android开发规范：四大基本组件
阿里Android开发规范：UI 与布局
阿里Android开发规范：进程、线程与消息通信
阿里Android开发规范：文件与数据库
阿里Android开发规范：Bitmap、Drawable 与动画
阿里Android开发规范：安全与其他

1、【强制】任何时候不要硬编码文件路径，请使用 Android 文件系统 API 访问。
说明：
Android 应用提供内部和外部存储，分别用于存放应用自身数据以及应用产生的用户数据。可以通过相关 API 接口获取对应的目录，进行文件操作。

android.os.Environment#getExternalStorageDirectory()android.os.Environment#getExternalStoragePublicDirectory()android.content.Context#getFilesDir()android.content.Context#getCacheDir

正例：

public File getDir(String alName) {    File file = new File(Environment.getExternalStoragePublicDirectory(Environment.    DIRECTORY_PICTURES), alName);    if (!file.mkdirs()) {        Log.e(LOG_TAG, "Directory not created");    }    return file;}

反例：

public File getDir(String alName) {    // 任何时候都不要硬编码文件路径，这不仅存在安全隐患，也让 app 更容易出现适配问题    File file = new File("/mnt/sdcard/Download/Album", alName);    if (!file.mkdirs()) {    Log.e(LOG_TAG, "Directory not created");    }    return file;}

扩展参考：
1) https://developer.android.com/training/data-storage/files.html
2) https://developer.android.com/reference/android/os/Environment.html#getExternalStorageDirectory()
2、【强制】当使用外部存储时，必须检查外部存储的可用性。
正例：

// 读/写检查public boolean isExternalStorageWritable() {    String state = Environment.getExternalStorageState();    if (Environment.MEDIA_MOUNTED.equals(state)) {        return true;    }    return false;}// 只读检查public boolean isExternalStorageReadable() {    String state = Environment.getExternalStorageState();    if (Environment.MEDIA_MOUNTED.equals(state) ||        Environment.MEDIA_MOUNTED_READ_ONLY.equals(state)) {        return true;    }    return false;}

3、【强制】应用间共享文件时，不要通过放宽文件系统权限的方式去实现，而应使用FileProvider。
正例：

    ...        ...    "android.support.v4.content.FileProvider"        android:authorities="com.example.fileprovider"        android:exported="false"        android:grantUriPermissions="true">        "android.support.FILE_PROVIDER_PATHS"            android:resource="@xml/provider_paths" />        ...          "album/" name="myimages" />void getAlbumImage(String imagePath) {    File image = new File(imagePath);    Intent getAlbumImageIntent = new Intent(MediaStore.ACTION_IMAGE_CAPTURE);    Uri imageUri = FileProvider.getUriForFile(    this,    "com.example.provider",    image);    getAlbumImageIntent.putExtra(MediaStore.EXTRA_OUTPUT, imageUri);    startActivityForResult(takePhotoIntent, REQUEST_GET_ALBUMIMAGE);}

反例：

void getAlbumImage(String imagePath) {    File image = new File(imagePath);    Intent getAlbumImageIntent = new Intent(MediaStore.ACTION_IMAGE_CAPTURE);    //不要使用 file://的 URI 分享文件给别的应用，包括但不限于 Intent    getAlbumImageIntent.putExtra(MediaStore.EXTRA_OUTPUT, Uri.fromFile(image));    startActivityForResult(takePhotoIntent, REQUEST_GET_ALBUMIMAGE);}

4、【推荐】SharedPreference 中只能存储简单数据类型（int、boolean、String 等），复杂数据类型建议使用文件、数据库等其他方式存储。
正例：

public void updateSettings() {    SharedPreferences mySharedPreferences = getSharedPreferences("settings",Activity.MODE_PRIVATE);    SharedPreferences.Editor editor = mySharedPreferences.edit();    editor.putString("id", "foo");    editor.putString("nick", "bar");    //不要把复杂数据类型转成 String 存储    editor.apply();}

5、【推荐】SharedPreference 提交数据时，尽量使用 Editor#apply() ，而非Editor#commit()。一般来讲，仅当需要确定提交结果，并据此有后续操作时，才使用 Editor#commit()。
说明：
SharedPreference 相关修改使用 apply 方法进行提交会先写入内存，然后异步写入磁盘，commit 方法是直接写入磁盘。如果频繁操作的话 apply 的性能会优于 commit，apply 会将最后修改内容写入磁盘。但是如果希望立刻获取存储操作的结果，并据此做相应的其他操作，应当使用 commit。
正例：

public void updateSettingsAsync() {    SharedPreferences mySharedPreferences = getSharedPreferences("settings",Activity.MODE_PRIVATE);    SharedPreferences.Editor editor = mySharedPreferences.edit();    editor.putString("id", "foo");    editor.apply();}public void updateSettings() {    SharedPreferences mySharedPreferences = getSharedPreferences("settings",Activity.MODE_PRIVATE);    SharedPreferences.Editor editor = mySharedPreferences.edit();    editor.putString("id", "foo");    if (!editor.commit()) {        Log.e(LOG_TAG, "Failed to commit setting changes");    }}

反例：

editor.putLong("key_name", "long value");editor.commit();

扩展参考：
https://developer.android.com/reference/android/content/SharedPreferences.Editor.html#apply()
6、【强制】数据库 Cursor 必须确保使用完后关闭，以免内存泄漏。
说明：
Cursor 是对数据库查询结果集管理的一个类，当查询的结果集较小时，消耗内存不易察觉。但是当结果集较大，长时间重复操作会导致内存消耗过大，需要开发者在操作完成后手动关闭 Cursor。数据库 Cursor 在创建及使用时，可能发生各种异常，无论程序是否正常结束，必须在最后确保 Cursor 正确关闭，以避免内存泄漏。同时，如果 Cursor 的使用还牵涉多线程场景，那么需要自行保证操作同步。
正例：

public void handlePhotos(SQLiteDatabase db, String userId) {    Cursor cursor;    try {        cursor = db.query(TUserPhoto, new String[] { "userId", "content" }, "userId=?", new        String[] { userId }, null, null, null);        while (cursor.moveToNext()) {            // TODO        }    } catch (Exception e) {        // TODO    } finally {        if (cursor != null) {        cursor.close();        }    }}

反例：

public void handlePhotos(SQLiteDatabase db, String userId) {    Cursor cursor = db.query(TUserPhoto, new String[] { "userId", "content" }, "userId=?", new    String[] { userId }, null, null, null);    while (cursor.moveToNext()) {        // TODO    }    // 不能放任 cursor 不关闭}

7、【强制】多线程操作写入数据库时，需要使用事务，以免出现同步问题。
说明：
Android 的通过 SQLiteOpenHelper 获取数据库 SQLiteDatabase 实例，Helper 中会自动缓存已经打开的 SQLiteDatabase 实例，单个 App 中应使用 SQLiteOpenHelper的单例模式确保数据库连接唯一。由于 SQLite 自身是数据库级锁，单个数据库操作是保证线程安全的（不能同时写入），transaction 时一次原子操作，因此处于事务中的操作是线程安全的。
若同时打开多个数据库连接，并通过多线程写入数据库，会导致数据库异常，提示数据库已被锁住。
正例：

public void insertUserPhoto(SQLiteDatabase db, String userId, String content) {    ContentValues cv = new ContentValues();    cv.put("userId", userId);    cv.put("content", content);    db.beginTransaction();    try {        db.insert(TUserPhoto, null, cv);        // 其他操作        db.setTransactionSuccessful();    } catch (Exception e) {        // TODO    } finally {        db.endTransaction();    }}

反例：

public void insertUserPhoto(SQLiteDatabase db, String userId, String content) {    ContentValues cv = new ContentValues();    cv.put("userId", userId);    cv.put("content", content);    db.insert(TUserPhoto, null, cv);}

扩展参考：
1) https://nfrolov.wordpress.com/2014/08/16/android-sqlitedatabase-locking-and-multi-threading/
2) https://developer.android.com/reference/android/database/sqlite/SQLiteDatabase.html#beginTransaction())
3) https://www.androiddesignpatterns.com/2012/05/correctly-managing-your-sqlite-database.html
4) https://www.jianshu.com/p/57eb08fe071d
8、【推荐】大数据写入数据库时，请使用事务或其他能够提高 I/O 效率的机制，保证执行速度。
正例：

public void insertBulk(SQLiteDatabase db, ArrayList users) {    db.beginTransaction();    try {        for (int i = 0; i < users.size; i++) {        ContentValues cv = new ContentValues();        cv.put("userId", users[i].userId);        cv.put("content", users[i].content);        db.insert(TUserPhoto, null, cv);        }        // 其他操作        db.setTransactionSuccessful();    } catch (Exception e) {        // TODO    } finally {        db.endTransaction();    }}

9、【强制】执行 SQL 语句时，应使用 SQLiteDatabase#insert()、update()、delete()，不要使用 SQLiteDatabase#execSQL()，以免 SQL 注入风险。
正例：

public int updateUserPhoto(SQLiteDatabase db, String userId, String content) {    ContentValues cv = new ContentValues();    cv.put("content", content);    String[] args = {String.valueOf(userId)};    return db.update(TUserPhoto, cv, "userId=?", args);}

反例：

public void updateUserPhoto(SQLiteDatabase db, String userId, String content) {    String sqlStmt = String.format("UPDATE %s SET content=%s WHERE userId=%s",    TUserPhoto, userId, content);    //请提高安全意识，不要直接执行字符串作为 SQL 语句    db.execSQL(sqlStmt);}

10、【强制】如果 ContentProvider 管理的数据存储在 SQL 数据库中，应该避免将不受信任的外部数据直接拼接在原始 SQL 语句中，可使用一个用于将 ? 作为可替换参数的选择子句以及一个单独的选择参数数组，会避免 SQL 注入。
正例：

// 使用一个可替换参数String mSelectionClause = "var = ?";String[] selectionArgs = {""};selectionArgs[0] = mUserInput;

反例：

// 拼接用户输入内容和列名String mSelectionClause = "var = " + mUserInput;

更多相关文章

随机推荐