FileUtilsOpenHarmony::isFileExistInternal 有问题

1774358245 · 2024年08月20日 08:59

Creator 版本：3.8.4-oh分支

DevEco Studio NEXT Developer Beta2

构建版本：5.0.3.502, built on July 28, 2024

"compatibleSdkVersion": "4.1.0(11)"

目标平台：纯血鸿蒙 HarmonyOS Next 0.0.36
重现方式：

// 先拿一个可读写路径
const path = native.fileUtils.getWritablePath() + "test.txt"

// 然后进行写入
native.fileUtils.writeStringToFile("test", path);

// 然后判断文件是否存在
const isFileExist = native.fileUtils.isFileExist(path);
// 预期应该是要存在 但是结果输出 false
console.log("isFileExist", isFileExist)

手机型号：华为 Mate 60 Pro

我看了下源码发现问题出现在这

github.com

cocos/cocos-engine/blob/v3.8.4-oh/native/cocos/platform/openharmony/FileUtils-OpenHarmony.cpp#L186



std::string FileUtilsOpenHarmony::getWritablePath() const {
    return _ohWritablePath;
}

bool FileUtilsOpenHarmony::isFileExistInternal(const std::string &strFilePath) const {
    if (strFilePath.empty()) {
        return false;
    }
    std::string strPath = strFilePath;
    if (!isAbsolutePath(strPath)) { // Not absolute path, add the default root path at the beginning.
        strPath.insert(0, _defaultResRootPath);
    }

    if (nullptr == _nativeResourceManager) {
        CC_LOG_ERROR("nativeResourceManager is nullptr");
        return false;
    }

    RawFile* rawFile = OH_ResourceManager_OpenRawFile(_nativeResourceManager, strPath.c_str());
    if(rawFile) {

然后我参考了FileUtils-ohos的实现, 发现之前是有区分绝对路径和相对路径的

github.com

cocos/cocos-engine/blob/v3.8.4-oh/native/cocos/platform/ohos/FileUtils-ohos.cpp#L180


        return "./";
    }
    return tmp.append("/");
}

bool FileUtilsOHOS::isFileExistInternal(const ccstd::string &strFilePath) const {
    if (strFilePath.empty()) return false;
    auto filePath = strFilePath;
    auto fileFound = false;

    if (strFilePath[0] == '/') { // absolute path
        struct stat info;
        return ::stat(filePath.c_str(), &info) == 0;
    }

    // relative path
    if (strFilePath.find(_defaultResRootPath) == 0) {
        filePath = rawfilePrefix + filePath.substr(_defaultResRootPath.length());
    }

    auto rawFile = OpenRawFile(ohosResourceMgr, filePath.c_str());

然后我照着改, 发现这样确实就正常了

1774358245 · 2024年08月20日 09:01

@king668

1774358245 · 2024年08月20日 09:07

再说多一个问题原本的 native.fileUtils.getWritablePath() 获取到的值最后不带 /

这个行为和Android和iOS都不同, 我自己把 NapiHelper.cpp 的 napiWritablePathInit 方法改了下, 最后应该要加上 /

FileUtilsOpenHarmony::_ohWritablePath = info[0].As<Napi::String>().Utf8Value().append("/");

而且我看 FileUtils-ohos 的 getWritablePath 实现也是最后带 / 的

github.com

cocos/cocos-engine/blob/v3.8.4-oh/native/cocos/platform/ohos/FileUtils-ohos.cpp#L172



bool FileUtilsOHOS::isAbsolutePath(const ccstd::string &strPath) const {
    return !strPath.empty() && (strPath[0] == '/' || strPath.find(ASSETS_FOLDER_NAME) == 0);
}

ccstd::string FileUtilsOHOS::getWritablePath() const {
    auto tmp = cc::JniHelper::callStaticStringMethod(JCLS_HELPER, "getWritablePath");
    if (tmp.empty()) {
        return "./";
    }
    return tmp.append("/");
}

bool FileUtilsOHOS::isFileExistInternal(const ccstd::string &strFilePath) const {
    if (strFilePath.empty()) return false;
    auto filePath = strFilePath;
    auto fileFound = false;

    if (strFilePath[0] == '/') { // absolute path
        struct stat info;
        return ::stat(filePath.c_str(), &info) == 0;

king668 · 2024年08月20日 12:37

不是一样的么？

king668 · 2024年08月20日 12:41

可写路径比较特殊，这个路径是从主线程传下来的，是一个沙盒的路径。

1774358245 · 2024年08月21日 01:29

OH_ResourceManager_OpenRawFile 打不开沙盒环境的绝对路径

king668 · 2024年08月21日 01:34

OH_ResourceManager_OpenRawFile这个接口是只能加载资源里面的路径
沙盒里的绝对路径，是用C的标准接口来读取的

1774358245 · 2024年08月21日 01:35

这个我知道我是指 native.fileUtils.getWritablePath() 拿到的值最后没有带 /

但是android和iOS最后都是带/的

king668 · 2024年08月21日 01:36

好的，我看看

1774358245 · 2024年08月21日 01:36

所以 native.fileUtils.isFileExist 传沙盒的绝对路径就有问题了

king668 · 2024年08月21日 01:37

嗯，确实有问题，我修改下。感谢感谢

1774358245 · 2024年08月21日 01:37

1774358245 · 2024年08月21日 08:53

@king668 我还发现了一个新问题这里勾选加密脚本不会生效是否跟现在运行方式是js都丢在了entry/src/main/ets/cocos/assets下运行有关

经过一番查阅是因为下面这个方法被删掉了

github.com

cocos/cocos-engine/blob/v3.8.4-oh/scripts/native-pack-tool/source/platforms/ohos.ts#L107


        if (!appNameItem) {
            appNameItem = { name: 'app_name', value: 'CocosGame' };
            stringList.push(appNameItem);
        }
        appNameItem.value = this.params.projectName || 'CocosGame';
        await fs.outputJSON(stringJson, stringJsonObj, { spaces: 2 });
    } catch (e) {
        console.error(e);
    }

    await this.encrypteScripts();
    return true;
}

async make() {
    cchelper.checkJavaHome();

    const projectDir = this.paths.platformTemplateDirInPrj;
    let gradle = 'gradlew';
    if (process.platform === 'win32') {
        gradle += '.bat';

king668 · 2024年08月21日 13:37

不支持加密，openharmony不支持动态加载

1774358245 · 2024年08月22日 01:22

1774358245 · 2024年08月28日 03:32

@king668

github.com

cocos/cocos-engine/blob/v3.8.4-oh/platforms/native/engine/jsb-videoplayer-openharmony.js#L107


setVisible (visible) {
    window.oh.postMessage('setVideoVisible', { tag: this._videoIndex, visible });
}
resume () {
    window.oh.postMessage('resumeVideo', this._videoIndex);
}
currentTime () {
    window.oh.postSyncMessage('currentTime', this._videoIndex).then((result) => {
        this._currentTime = result;
    });
    return this._currentTime;
}
stop () {
    window.oh.postMessage('stopVideo', this._videoIndex);
}
seekTo (val) {
    window.oh.postMessage('seekVideoTo', { tag: this._videoIndex, time: val });
}
duration () {
    window.oh.postSyncMessage('getVideoDuration', this._videoIndex).then((result) => {
        this._duration = result;

这个currentTime方法是不是有问题, 他明明是异步的, 但是却直接返回了

然后我现在正在找cocos的ts层和鸿蒙的交互, 发现这样是能传值和回调, 但是只能是异步的, 没看到有同步的

1774358245 · 2024年08月28日 06:31

不会是要写cpp代码才能同步回调值吧…