android - 这是如何正确地将照片设置为地址簿上的联系人？

Question

背景

假设我通过使用地址簿上的一些查询找到了一个联系人。正如我所写的那样，通过执行查询以从特定帐户(如 WhatsApp)获取联系信息来找到它 here .

现在我有一张图片，我想用它来更新联系人的照片。

问题

我根据我在 StackOverflow 上找到的内容创建了代码，用于更新联系人照片。

问题是，一些用户声称它没有做任何事情。我不确定是什么原因造成的。访问联系人的方式可能不好(id/lookup-key？)。也许我需要一个额外的地址簿字段。也许查询本身是错误的...

我无法找出此问题的原因，因为我无法重现它，大多数用户也是如此。

代码

这是我做的:

        final ArrayList<ContentProviderOperation> ops = new ArrayList<>();
        final String lookupKey=...;
        try {
            FileInputStream fileInputStream = new FileInputStream(file);
            final byte[] photoByteArray = new byte[(int) file.length()];
            fileInputStream.read(photoByteArray);
            fileInputStream.close();
            final Builder builder = ContentProviderOperation.newUpdate(Data.CONTENT_URI);
            builder.withSelection(Data.LOOKUP_KEY + "=?" + " AND " + Data.MIMETYPE + "=?", new String[]{lookupKey, Photo.CONTENT_ITEM_TYPE});
            builder.withValue(Photo.PHOTO, photoByteArray);
            ops.add(builder.build());
            file.delete();
        } catch (IOException e) {
            e.printStackTrace();
        }
        context.getContentResolver().applyBatch(ContactsContract.AUTHORITY, ops )

问题

代码有问题吗？

我应该以不同方式更新联系人吗？

为什么有些用户更新失败？查找键还不够吗？它可以更新错误的数据吗？

我在某处读到我可能需要使用 RAW-contact-ids，但找不到具体的操作方法。查询然后更新/插入？

有人建议开发者应该考虑使用他们应用程序的帐户来保存照片，而不是更新，因为这可能会导致问题。是真的吗？

Answer 1

我在代码中看到了一些可能的问题:

1. 您假设为该联系人存储了一张现有照片(您正在使用 newUpdate)，如果当前没有照片，您应该执行 newInsert相反。
2. 您应该在特定的 RawContact 上更新/插入，您编写的内容会更新联系人的所有 RawContacts 照片数据原始数据，一些 RawContacts 可能具有只读状态，并且不允许您的应用修改它。
3. 处理照片，尤其是将照片存储到联系人数据库时，是一项非常占用内存的操作，对于某些设备，它可能会因 OutOfMemoryError 而崩溃，因为它们的 RAM 非常有限，或者因为当前内存不足被其他东西大量使用。您应该使用 try/catch 包装包括 applyBatch 在内的整个代码，并捕获所有异常/错误并将它们报告到日志中，最好也报告到您的错误报告库(Firebase/crashlytics/等)
4. 您只更新原始数据的 PHOTO 字段，您还应该更新 PHOTO_FILE_ID 字段，因为某些应用可能会选择读取 PHOTO_FILE_ID 并从中获取照片。
5. 您没有将新照片设置为其联系人的 SUPER_PRIMARY，因此如果同一联系人中的其他 RawContact 也有照片，它可能会“获胜”并成为主照片对于那个联系人。

下面是应该有用的代码:

// To simplify the code i didn't worry about try/catch, freeing resources, 
// or running slow queries on the main thread, make sure your code does!
public ContentProviderOperation setPhoto(int contactId, byte[] photoByteArray) {
    long rawId = getRawId(contactId);

    ContentProviderOperation.Builder builder;

    String selection = Data.RAW_CONTACT_ID + " = '" + rawId + "' AND " + Data.MIMETYPE + "= '" + Photo.CONTENT_ITEM_TYPE + "'";
    Cursor cur = contentResolver.query(Data.CONTENT_URI, new String[]{ Data._ID }, selection, null, null);
    if (cur.moveToFirst()) {
        // this RawContact already has a photo!
        Long photoDataId = cur.getLong(0);
        Uri uri = ContentUris.withAppendedId(Data.CONTENT_URI, photoDataId);
        builder = ContentProviderOperation.newUpdate(uri);
    } else {
        // this RawContact has no photo.
        builder = ContentProviderOperation.newInsert(Data.CONTENT_URI);
        builder.withValue(Data.RAW_CONTACT_ID, rawId);
    }
    builder.withValue(Data.MIMETYPE, Photo.CONTENT_ITEM_TYPE);
    builder.withValue(Data.IS_SUPER_PRIMARY, 1);
    builder.withValue(Data.IS_PRIMARY, 1);

    builder.withValue(Photo.PHOTO, photoByteArray);
    builder.withValue(Photo.PHOTO_FILE_ID, null);

    return builder.build();
}

private long getRawId(long contactId) {
    String selection = RawContacts.CONTACT_ID + "='" + contactId + "'";
    Cursor cur = contentResolver.query(RawContacts.CONTENT_URI, new String[]{ RawContacts._ID }, selection, null, null);
    try {
        if (cur.moveToNext()) {
            return cur.getLong(0);
        }
    } finally {
        cur.close();
    }
    return 0;
}