使用Samsung Health SDK读取心率触发IllegalArgumentException问题求助

你遇到的java.lang.IllegalArgumentException: heart_rate (READ) is invalid错误，核心原因是错误地使用了心率字段名代替数据类型标识符，同时还有几处逻辑和代码的小问题，下面一步步帮你修复：

1. 修正权限请求中的数据类型参数 #

在MainActivity的onCreate方法中，你创建PermissionKey时用了HealthConstants.HeartRate.HEART_RATE（这是心率数据里的字段名），但正确的参数应该是心率数据类型的全局标识符HealthConstants.HeartRate.HEALTH_DATA_TYPE：

错误代码： #

mKeySet.add(new HealthPermissionManager.PermissionKey(HealthConstants.HeartRate.HEART_RATE, HealthPermissionManager.PermissionType.READ));

修正后： #

mKeySet.add(new HealthPermissionManager.PermissionKey(HealthConstants.HeartRate.HEALTH_DATA_TYPE, HealthPermissionManager.PermissionType.READ));
// 别忘了把步数的权限加回来，之前注释掉了
mKeySet.add(new HealthPermissionManager.PermissionKey(HealthConstants.StepCount.HEALTH_DATA_TYPE, HealthPermissionManager.PermissionType.READ));

2. 修正HeartRateReporter中的数据类型配置 #

在HeartRateReporter的start方法和readHeartRate方法中，你同样错误地用了HealthConstants.HeartRate.HEART_RATE作为数据类型，需要替换为HealthConstants.HeartRate.HEALTH_DATA_TYPE：

错误代码（start方法）： #

HealthDataObserver.addObserver(mStore, HealthConstants.HeartRate.HEART_RATE, mObserver);

修正后： #

HealthDataObserver.addObserver(mStore, HealthConstants.HeartRate.HEALTH_DATA_TYPE, mObserver);

错误代码（readHeartRate方法）： #

HealthDataResolver.ReadRequest request = new HealthDataResolver.ReadRequest.Builder()
        .setDataType(HealthConstants.HeartRate.HEART_RATE)
        .setProperties(new String[] {HealthConstants.HeartRate.HEART_RATE})
        .setLocalTimeRange(HealthConstants.HeartRate.START_TIME, HealthConstants.HeartRate.TIME_OFFSET, startTime, endTime)
        .build();

修正后： #

HealthDataResolver.ReadRequest request = new HealthDataResolver.ReadRequest.Builder()
        .setDataType(HealthConstants.HeartRate.HEALTH_DATA_TYPE) // 这里修正
        .setProperties(new String[] {HealthConstants.HeartRate.HEART_RATE})
        .setLocalTimeRange(HealthConstants.HeartRate.START_TIME, HealthConstants.HeartRate.TIME_OFFSET, startTime, endTime)
        .build();

3. 修复心率读取的逻辑错误 #

心率数据是瞬时测量值，不是步数那样的累加计数，你当前用count += data.getInt(...)会把所有历史心率值累加，这完全不符合需求。应该取最新的一条心率数据，或者计算平均值，这里以获取最新值为例：

修正HeartRateReporter中的mListener： #

private final HealthResultHolder.ResultListener<HealthDataResolver.ReadResult> mListener = result -> {
    int latestHeartRate = 0;
    try {
        // 遍历结果，取最后一条（最新的）心率数据
        for (HealthData data : result) {
            latestHeartRate = data.getInt(HealthConstants.HeartRate.HEART_RATE);
        }
    } finally {
        result.close();
    }
    if (mHeartRateObserver != null) {
        mHeartRateObserver.onChanged(latestHeartRate);
    }
};

4. 修复updateHeartRateView的递归调用问题 #

在MainActivity的updateHeartRateView方法中，你错误地递归调用了自己，会导致栈溢出：

错误代码： #

private void updateHeartRateView(String valueOf) {
    Log.d(APP_TAG, "heart rate reported : " + valueOf);
    updateHeartRateView(String.valueOf(valueOf));
}

修正后： #

private void updateHeartRateView(String heartRateValue) {
    Log.d(APP_TAG, "heart rate reported : " + heartRateValue);
    heartRate.setText(heartRateValue);
}

5. 确认Manifest权限配置正确 #

你的Manifest里的权限配置是正确的，无需修改：

<meta-data android:name="com.samsung.android.health.permission.read" 
          android:value="com.samsung.health.step_count;com.samsung.health.heart_rate"/>

完成以上修正后，重新编译运行，应该就能正常读取心率数据了。另外注意测试时需要确保三星健康App已经安装并授权你的应用访问心率数据。

内容的提问来源于stack exchange，提问作者Muneerah Rashed