diff --git a/frameworks/innerkits/file_access/src/js_file_access_ext_ability.cpp b/frameworks/innerkits/file_access/src/js_file_access_ext_ability.cpp
index c1cc9ed045f4e7441cb751185117df4c34b6072a..4239ed7e18eba4739727691f4e44d321efdd1786 100644
--- a/frameworks/innerkits/file_access/src/js_file_access_ext_ability.cpp
+++ b/frameworks/innerkits/file_access/src/js_file_access_ext_ability.cpp
@@ -423,6 +423,7 @@ int JsFileAccessExtAbility::CreateFile(const Uri &parent, const std::string &dis
         argc = ARGC_TWO;
         return true;
     };
+
     auto retParser = [value](NativeEngine &engine, NativeValue *result) -> bool {
         NativeObject *obj = ConvertNativeValueTo<NativeObject>(result);
         if (obj == nullptr) {
diff --git a/frameworks/innerkits/file_access/test/file_access_helper_test.cpp b/frameworks/innerkits/file_access/test/file_access_helper_test.cpp
index 3b22fbc7948bd60125da77910975dfbca6c24983..d95893eb0a783b50aad28aa8099beefaf6654713 100644
--- a/frameworks/innerkits/file_access/test/file_access_helper_test.cpp
+++ b/frameworks/innerkits/file_access/test/file_access_helper_test.cpp
@@ -13,7 +13,7 @@
  * limitations under the License.
  */
 
-#include <cstdio>
+#include <stdio.h>
 #include <unistd.h>
 
 #include <gtest/gtest.h>
@@ -29,7 +29,7 @@ namespace {
 using namespace std;
 using namespace OHOS;
 using namespace FileAccessFwk;
-int ABILITY_ID = 5003;
+const int ABILITY_ID = 5003;
 shared_ptr<FileAccessHelper> fah = nullptr;
 Uri g_newDirUri("");
 
diff --git a/frameworks/innerkits/file_access/test/file_extension_helper_test.cpp b/frameworks/innerkits/file_access/test/file_extension_helper_test.cpp
index ae68864ead86f18a5b4e6963e7637723d4ab4290..e3f71cbf0c67312f23f6ff78a22da08c7d0b914d 100644
--- a/frameworks/innerkits/file_access/test/file_extension_helper_test.cpp
+++ b/frameworks/innerkits/file_access/test/file_extension_helper_test.cpp
@@ -28,7 +28,7 @@ namespace {
 using namespace std;
 using namespace OHOS;
 using namespace FileAccessFwk;
-int ABILITY_ID = 5003;
+const int ABILITY_ID = 5003;
 shared_ptr<FileAccessHelper> fah = nullptr;
 OHOS::Security::AccessToken::AccessTokenID tokenId;
 
diff --git a/interfaces/kits/napi/file_access_module/napi_utils.cpp b/interfaces/kits/napi/file_access_module/napi_utils.cpp
index ecbe70a55c22dccb462633f27c49c70e76b1fe86..b25c930207d945b56b3b83d831b63f0d5ccd35a6 100644
--- a/interfaces/kits/napi/file_access_module/napi_utils.cpp
+++ b/interfaces/kits/napi/file_access_module/napi_utils.cpp
@@ -83,7 +83,7 @@ int GetFileFilterParam(const NVal &argv, FileFilter &filter)
     }
 
     if (argv.HasProp("fileSizeOver")) {
-        int64_t fileSizeOver;
+        int64_t fileSizeOver = 0;
         std::tie(ret, fileSizeOver) = argv.GetProp("fileSizeOver").ToInt64();
         if (!ret) {
             HILOG_ERROR("FileFilter get fileSizeOver param fail.");
@@ -95,7 +95,7 @@ int GetFileFilterParam(const NVal &argv, FileFilter &filter)
     }
 
     if (argv.HasProp("lastModifiedAfter")) {
-        double lastModifiedAfter;
+        double lastModifiedAfter = 0.0;
         std::tie(ret, lastModifiedAfter) = argv.GetProp("lastModifiedAfter").ToDouble();
         if (!ret) {
             HILOG_ERROR("FileFilter get lastModifiedAfter param fail.");
@@ -107,7 +107,7 @@ int GetFileFilterParam(const NVal &argv, FileFilter &filter)
     }
 
     if (argv.HasProp("excludeMedia")) {
-        bool excludeMedia;
+        bool excludeMedia = false;
         std::tie(ret, excludeMedia) = argv.GetProp("excludeMedia").ToBool();
         if (!ret) {
             HILOG_ERROR("FileFilter get excludeMedia param fail.");
diff --git a/services/src/client/file_manager_proxy.h b/services/src/client/file_manager_proxy.h
index 494648ed26703f008083ec301e84e68a30329f74..147eb72a9bc97eb0021fc93c1e0ad17cc3cd2b08 100644
--- a/services/src/client/file_manager_proxy.h
+++ b/services/src/client/file_manager_proxy.h
@@ -26,7 +26,7 @@ namespace FileManagerService {
 class FileManagerProxy : public IRemoteProxy<IFileManagerService>, public IFmsClient {
 public:
     explicit FileManagerProxy(const sptr<IRemoteObject> &impl);
-    virtual ~FileManagerProxy() = default;
+    virtual ~FileManagerProxy() override = 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<std::shared_ptr<FileInfo>> &fileRes) override;
diff --git a/services/src/fileoper/cmd_response.h b/services/src/fileoper/cmd_response.h
index f4a3b445b6d7634f95ea52e490f6f1f1e667f14b..4e1e8905fcf3e0967cd3ae4d1f68756a0091461e 100644
--- a/services/src/fileoper/cmd_response.h
+++ b/services/src/fileoper/cmd_response.h
@@ -62,7 +62,7 @@ public:
         return vecFileInfo_;
     }
 
-    virtual bool Marshalling(Parcel &parcel) const override
+    bool Marshalling(Parcel &parcel) const override
     {
         parcel.WriteInt32(err_);
         parcel.WriteString(uri_);
diff --git a/services/src/fileoper/ext_storage/ext_storage_subscriber.h b/services/src/fileoper/ext_storage/ext_storage_subscriber.h
index 18a34b779970bb9b509c5aa06372bfea2e18ed16..835dbb9a41268b58446f1b9cab4a61e3273f532f 100644
--- a/services/src/fileoper/ext_storage/ext_storage_subscriber.h
+++ b/services/src/fileoper/ext_storage/ext_storage_subscriber.h
@@ -41,7 +41,7 @@ public:
      * @brief System common event receiver.
      * @param eventData Common event data.
      */
-    virtual void OnReceiveEvent(const EventFwk::CommonEventData &eventData) override;
+    void OnReceiveEvent(const EventFwk::CommonEventData &eventData) override;
 
     bool CheckMountPoint(const std::string &path);
 
diff --git a/services/src/fileoper/external_storage_oper.h b/services/src/fileoper/external_storage_oper.h
index af5ee57a4d0c23f826e175370fbc8fcd06918b99..a21677b81a17321aff5069ee838f8a29b6534b9c 100644
--- a/services/src/fileoper/external_storage_oper.h
+++ b/services/src/fileoper/external_storage_oper.h
@@ -23,7 +23,7 @@ namespace FileManagerService {
 class ExternalStorageOper : public FileOper {
 public:
     ExternalStorageOper() = default;
-    virtual ~ExternalStorageOper() = default;
+    virtual ~ExternalStorageOper() override = default;
     int OperProcess(uint32_t code, MessageParcel &data, MessageParcel &reply) const override;
 private:
     int CreateFile(const std::string &uri, const std::string &name, MessageParcel &reply) const;
diff --git a/services/src/fileoper/external_storage_utils.cpp b/services/src/fileoper/external_storage_utils.cpp
index 1cff272a39f2eb3384b77be18c2e25cbe8afb438..207425ae1ba3dfeaf508550a847c44e566776d24 100644
--- a/services/src/fileoper/external_storage_utils.cpp
+++ b/services/src/fileoper/external_storage_utils.cpp
@@ -113,7 +113,7 @@ int ExternalStorageUtils::DoListFile(const std::string &type, const std::string
         ERR_LOG("invalid file count or offset.");
         return E_INVALID_FILE_NUMBER;
     }
-    DEBUG_LOG("limit %{public}lld, offset %{public}lld", (long long)count, (long long)offset);
+    DEBUG_LOG("limit %{public}lld, offset %{public}lld", static_cast<long long>(count), static_cast<long long>(offset));
     std::string path;
     if (!ConvertUriToAbsolutePath(uri, path)) {
         ERR_LOG("invalid uri[%{private}s].", uri.c_str());
diff --git a/services/src/fileoper/media_file_oper.h b/services/src/fileoper/media_file_oper.h
index bc4374b212fd9a7909a7bd1d2b5ab2e2c660b98d..b4f1b66bb37bbd78a8bb8245b2f186d28926d726 100644
--- a/services/src/fileoper/media_file_oper.h
+++ b/services/src/fileoper/media_file_oper.h
@@ -22,7 +22,7 @@ namespace FileManagerService {
 class MediaFileOper : public FileOper {
 public:
     MediaFileOper() = default;
-    virtual ~MediaFileOper() = default;
+    virtual ~MediaFileOper() override = default;
     int OperProcess(uint32_t code, MessageParcel &data, MessageParcel &reply) const override;
 private:
     int CreateFile(const std::string &name, const std::string &path, MessageParcel &reply) const;
diff --git a/services/src/server/file_manager_service.h b/services/src/server/file_manager_service.h
index 3410c40973e31563e488848469326085990e6c2e..c007b185b77655de15c0fd1247151fe61e50aee5 100644
--- a/services/src/server/file_manager_service.h
+++ b/services/src/server/file_manager_service.h
@@ -26,7 +26,7 @@ class FileManagerService : public SystemAbility, public FileManagerServiceStub {
 public:
     DISALLOW_COPY_AND_MOVE(FileManagerService);
     explicit FileManagerService(int32_t systemAbilityId, bool runOnCreate = true);
-    virtual ~FileManagerService() = default;
+    virtual ~FileManagerService() override = default;
     void OnDump() override;
     void OnStart() override;
     void OnStop() override;
diff --git a/services/src/server/file_manager_service_stub.h b/services/src/server/file_manager_service_stub.h
index e00697a98cf0b7bdda1cfd05700fc87ed0ee3143..cd7c4a1139545104c430dad0fea580f75cda3f43 100644
--- a/services/src/server/file_manager_service_stub.h
+++ b/services/src/server/file_manager_service_stub.h
@@ -33,8 +33,7 @@ public:
     FileManagerServiceStub() = default;
     virtual ~FileManagerServiceStub() = default;
     int OperProcess(uint32_t code, MessageParcel &data, MessageParcel &reply);
-    virtual int OnRemoteRequest(uint32_t code, MessageParcel &data,
-        MessageParcel &reply, MessageOption &option) override;
+    int OnRemoteRequest(uint32_t code, MessageParcel &data, MessageParcel &reply, MessageOption &option) override;
 };
 } // namespace FileManagerService
 } // namespace OHOS
diff --git a/services/test/client/fms_manager_proxy_mock.h b/services/test/client/fms_manager_proxy_mock.h
index 811807ed6f0d52ebdc692ea00dbe1545fb69ab72..fb83916e8a6eb04a5e2a78d8cb336f762db0d193 100644
--- a/services/test/client/fms_manager_proxy_mock.h
+++ b/services/test/client/fms_manager_proxy_mock.h
@@ -36,21 +36,21 @@ public:
         reply.WriteParcelable(cmdResponse);
         return ERR_NONE;
     }
-    virtual int Mkdir(const std::string &name, const std::string &path) override
+    virtual int Mkdir(const std::string &name, const std::string &path)
     {
         return ERR_NONE;
     }
     virtual int ListFile(const std::string &type, const std::string &path, const CmdOptions &option,
-        std::vector<std::shared_ptr<FileInfo>> &fileRes) override
+        std::vector<std::shared_ptr<FileInfo>> &fileRes)
     {
         return ERR_NONE;
     }
-    virtual int GetRoot(const CmdOptions &option, std::vector<std::shared_ptr<FileInfo>> &fileRes) override
+    virtual int GetRoot(const CmdOptions &option, std::vector<std::shared_ptr<FileInfo>> &fileRes)
     {
         return ERR_NONE;
     }
     virtual int CreateFile(const std::string &path, const std::string &fileName,
-        const CmdOptions &option, std::string &uri) override
+        const CmdOptions &option, std::string &uri)
     {
         return ERR_NONE;
     }