diff --git a/interfaces/kits/js/src/file_manager_napi.cpp b/interfaces/kits/js/src/file_manager_napi.cpp index 8d9d0add4876fdfe5333461e5d8a62a786066602..bf6732d660c9f85215dc30f6a8fe976280bdff6e 100644 --- a/interfaces/kits/js/src/file_manager_napi.cpp +++ b/interfaces/kits/js/src/file_manager_napi.cpp @@ -31,7 +31,7 @@ using namespace std; using namespace DistributedFS; struct AsyncFileInfoArg { NRef ref_; - vector> fileRes_; + vector> fileRes_; explicit AsyncFileInfoArg(NVal ref) : ref_(ref), fileRes_() {}; ~AsyncFileInfoArg() = default; }; @@ -181,7 +181,7 @@ void CreateFileArray(napi_env env, shared_ptr arg) { for (unsigned int i = 0; i < arg->fileRes_.size(); i++) { NVal obj = NVal::CreateObject(env); - unique_ptr &res = arg->fileRes_[i]; + shared_ptr &res = arg->fileRes_[i]; obj.AddProp("name", NVal::CreateUTF8String(env, res->GetName()).val_); obj.AddProp("path", NVal::CreateUTF8String(env, res->GetPath()).val_); obj.AddProp("type", NVal::CreateUTF8String(env, res->GetType()).val_); diff --git a/services/src/client/file_manager_proxy.cpp b/services/src/client/file_manager_proxy.cpp index b3a120ba2ed6315afd6c1358f14f0d842f9e57d8..605a4c84d93771dbd65fc4aff2f88a281e5567be 100644 --- a/services/src/client/file_manager_proxy.cpp +++ b/services/src/client/file_manager_proxy.cpp @@ -14,6 +14,7 @@ */ #include "file_manager_proxy.h" +#include "cmd_response.h" #include "file_info.h" #include "file_manager_service_def.h" #include "file_manager_service_errno.h" @@ -27,7 +28,7 @@ namespace OHOS { namespace FileManagerService { FileManagerProxy::FileManagerProxy(const sptr &impl) : IRemoteProxy(impl) {} -int FileManagerProxy::GetRoot(const CmdOptions &option, vector> &fileRes) +int FileManagerProxy::GetRoot(const CmdOptions &option, vector> &fileRes) { CmdOptions op(option); if (op.GetDevInfo().GetName() == "external_storage") { @@ -104,7 +105,7 @@ IFmsClient *IFmsClient::GetFmsInstance() } int FileManagerProxy::ListFile(const std::string &type, const std::string &path, const CmdOptions &option, - std::vector> &fileRes) + std::vector> &fileRes) { MessageParcel data; CmdOptions op(option); @@ -130,14 +131,12 @@ int FileManagerProxy::ListFile(const std::string &type, const std::string &path, ERR_LOG("inner error send request fail %{public}d", err); return FAIL; } - int fileInfoNum = 0; - reply.ReadInt32(fileInfoNum); - while (fileInfoNum) { - unique_ptr file(reply.ReadParcelable()); - fileRes.emplace_back(move(file)); - fileInfoNum--; + std::unique_ptr cmdResponse {CmdResponse::Unmarshalling(reply)}; + if (cmdResponse == nullptr) { + ERR_LOG("Unmarshalling cmdResponse fail %{public}d", err); + return FAIL; } - reply.ReadInt32(err); + fileRes = cmdResponse->GetFileInfoList(); return err; } diff --git a/services/src/client/file_manager_proxy.h b/services/src/client/file_manager_proxy.h index 085d32361989e70bc933156aab91ba787f78c6ec..494648ed26703f008083ec301e84e68a30329f74 100644 --- a/services/src/client/file_manager_proxy.h +++ b/services/src/client/file_manager_proxy.h @@ -29,10 +29,10 @@ public: virtual ~FileManagerProxy() = default; int Mkdir(const std::string &name, const std::string &path) override; int ListFile(const std::string &type, const std::string &path, const CmdOptions &option, - std::vector> &fileRes) override; + std::vector> &fileRes) override; int CreateFile(const std::string &path, const std::string &fileName, const CmdOptions &option, std::string &uri) override; - int GetRoot(const CmdOptions &option, std::vector> &fileRes) override; + int GetRoot(const CmdOptions &option, std::vector> &fileRes) override; private: static inline BrokerDelegator delegator_; }; diff --git a/services/src/client/ifms_client.h b/services/src/client/ifms_client.h index 386c4febe5c168cdf0a62da52b9acb289fcb65b5..ba0ce14abfe063493042b82988963f0a168dee63 100644 --- a/services/src/client/ifms_client.h +++ b/services/src/client/ifms_client.h @@ -24,8 +24,8 @@ public: static IFmsClient *GetFmsInstance(); virtual int Mkdir(const std::string &name, const std::string &path) = 0; virtual int ListFile(const std::string &type, const std::string &path, const CmdOptions &option, - std::vector> &fileRes) = 0; - virtual int GetRoot(const CmdOptions &option, std::vector> &fileRes) = 0; + std::vector> &fileRes) = 0; + virtual int GetRoot(const CmdOptions &option, std::vector> &fileRes) = 0; virtual int CreateFile(const std::string &path, const std::string &fileName, const CmdOptions &option, std::string &uri) = 0; }; diff --git a/services/src/fileoper/cmd_response.h b/services/src/fileoper/cmd_response.h new file mode 100644 index 0000000000000000000000000000000000000000..eb65bac408d0c51add40144f857d02b37098586d --- /dev/null +++ b/services/src/fileoper/cmd_response.h @@ -0,0 +1,98 @@ +/* + * Copyright (C) 2021 Huawei Device Co., Ltd. + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +#ifndef STORAGE_SERVICES_CMD_RESPONSE_H +#define STORAGE_SERVICES_CMD_RESPONSE_H + +#include +#include +#include + +#include "file_info.h" +#include "parcel.h" +namespace OHOS { +namespace FileManagerService { +class CmdResponse : public Parcelable { +public: + CmdResponse() = default; + CmdResponse(int &err, std::string &uri, std::vector> &fileInfoList) + : err_(err), uri_(uri), vecFileInfo_(fileInfoList) + {} + ~CmdResponse() = default; + + void SetErr(const int err) + { + err_ = err; + } + + int GetErr() const + { + return err_; + } + + void SetUri(const std::string &uri) + { + uri_ = uri; + } + + std::string GetUri() const + { + return uri_; + } + + void SetFileInfoList(std::vector> &fileInfoList) + { + vecFileInfo_ = fileInfoList; + } + + std::vector> GetFileInfoList() + { + return vecFileInfo_; + } + + virtual bool Marshalling(Parcel &parcel) const override + { + parcel.WriteInt32(err_); + parcel.WriteString(uri_); + int fileCount = vecFileInfo_.size(); + parcel.WriteInt32(fileCount); + for (int i = 0; i < fileCount; i++) { + parcel.WriteParcelable(vecFileInfo_[i].get()); + } + return true; + } + + static CmdResponse* Unmarshalling(Parcel &parcel) + { + auto *obj = new (std::nothrow) CmdResponse(); + if (obj == nullptr) { + return nullptr; + } + obj->err_ = parcel.ReadInt32(); + obj->uri_ = parcel.ReadString(); + int fileCount = parcel.ReadInt32(); + for (int i = 0; i < fileCount; i++) { + std::shared_ptr file(parcel.ReadParcelable()); + obj->vecFileInfo_.emplace_back(move(file)); + } + return obj; + } +private: + int err_; + std::string uri_; + std::vector> vecFileInfo_; +}; +} // FileManagerService +} // namespace OHOS +#endif // STORAGE_SERVICES_CMD_RESPONSE_H \ No newline at end of file diff --git a/services/src/fileoper/external_storage_oper.cpp b/services/src/fileoper/external_storage_oper.cpp index a23e5b92ad1d1ba7d64335f06eb919302539c2db..d4ec73dda8a77604af97f7d7fff13413bbaf78ff 100644 --- a/services/src/fileoper/external_storage_oper.cpp +++ b/services/src/fileoper/external_storage_oper.cpp @@ -17,6 +17,7 @@ #include +#include "cmd_response.h" #include "external_storage_utils.h" #include "file_info.h" #include "file_manager_service_def.h" @@ -77,7 +78,14 @@ int ExternalStorageOper::CreateFile(const std::string &uri, const std::string &n int ExternalStorageOper::ListFile(const std::string &type, const std::string &uri, const CmdOptions &option, MessageParcel &reply) const { - return ExternalStorageUtils::DoListFile(type, uri, reply); + std::vector> fileList; + int ret = ExternalStorageUtils::DoListFile(type, uri, fileList); + CmdResponse cmdResponse; + cmdResponse.SetErr(ret); + cmdResponse.SetUri(""); + cmdResponse.SetFileInfoList(fileList); + cmdResponse.Marshalling(reply); + return ret; } } // namespace FileManagerService } // namespace OHOS \ No newline at end of file diff --git a/services/src/fileoper/external_storage_utils.cpp b/services/src/fileoper/external_storage_utils.cpp index 27db06ace8acbf3fdc609d3975c26b76da5f2f2e..432a53ea595281587d1f2d7d45bd1c1f1fc8a6a3 100644 --- a/services/src/fileoper/external_storage_utils.cpp +++ b/services/src/fileoper/external_storage_utils.cpp @@ -59,7 +59,7 @@ static bool GetRealPath(string &path) return true; } -static bool GetFileInfo(const std::string &path, const std::string &name, unique_ptr &fileInfo) +static bool GetFileInfo(const std::string &path, const std::string &name, shared_ptr &fileInfo) { std::string fullPath(path); size_t len = fullPath.size(); @@ -104,42 +104,31 @@ static bool ConvertUriToAbsolutePath(const std::string &uri, std::string &path) return true; } -int ExternalStorageUtils::DoListFile(const std::string &type, const std::string &uri, MessageParcel &reply) +int ExternalStorageUtils::DoListFile(const std::string &type, const std::string &uri, + std::vector> &fileList) { std::string path; - int fileCount = 0; if (!ConvertUriToAbsolutePath(uri, path)) { ERR_LOG("invalid uri[%{public}s].", uri.c_str()); - reply.WriteInt32(fileCount); return E_NOEXIST; } DIR *dir = opendir(path.c_str()); if (!dir) { ERR_LOG("opendir path[%{public}s] fail.", path.c_str()); - reply.WriteInt32(fileCount); return E_NOEXIST; } - std::vector> fileList; for (struct dirent *ent = readdir(dir); ent != nullptr; ent = readdir(dir)) { if (strcmp(ent->d_name, ".") == 0 || strcmp(ent->d_name, "..") == 0) { continue; } - unique_ptr fileInfo = make_unique(); + shared_ptr fileInfo = make_shared(); if (!GetFileInfo(path, ent->d_name, fileInfo)) { continue; } fileList.push_back(move(fileInfo)); } closedir(dir); - fileCount = static_cast(fileList.size()); - reply.WriteInt32(fileCount); - if (fileCount == 0) { - return E_EMPTYFOLDER; - } - for (int i = 0; i < fileCount; i++) { - reply.WriteParcelable(fileList[i].get()); - } return SUCCESS; } diff --git a/services/src/fileoper/external_storage_utils.h b/services/src/fileoper/external_storage_utils.h index 61a139da2f78a11cac8d74788324ef114380334a..c11bcbef30061cf4ab4cd4b2d65e9a7c6b2d6d85 100644 --- a/services/src/fileoper/external_storage_utils.h +++ b/services/src/fileoper/external_storage_utils.h @@ -15,6 +15,7 @@ #ifndef STORAGE_SERIVCES_EXTERNAL_STORAGE_UTILS_H #define STORAGE_SERIVCES_EXTERNAL_STORAGE_UTILS_H +#include #include #include @@ -28,7 +29,8 @@ class ExternalStorageUtils { public: ExternalStorageUtils(); ~ExternalStorageUtils(); - static int DoListFile(const std::string &type, const std::string &uri, MessageParcel &reply); + static int DoListFile(const std::string &type, const std::string &uri, + std::vector> &fileList); static int DoCreateFile(const std::string &uri, const std::string &name, MessageParcel &reply); static bool PopFileInfo(FileInfo &fileInfo, MessageParcel &reply); static int DoGetRoot(const std::string &name, const std::string &path, MessageParcel &reply); diff --git a/services/src/fileoper/file_info.cpp b/services/src/fileoper/file_info.cpp index 46168b766553ee2e69abf24383153a876ac23528..3eaf90067c1e03dc9f90954ed76c30fed9b15017 100644 --- a/services/src/fileoper/file_info.cpp +++ b/services/src/fileoper/file_info.cpp @@ -28,11 +28,13 @@ bool FileInfo::Marshalling(Parcel &parcel) const parcel.WriteInt64(modifiedTime_); return true; } + FileInfo* FileInfo::Unmarshalling(Parcel &parcel) { auto *obj = new (std::nothrow) FileInfo(); if (obj == nullptr) { ERR_LOG("Unmarshalling fail"); + return nullptr; } obj->path_ = parcel.ReadString(); obj->name_ = parcel.ReadString();