diff --git a/frameworks/ans/src/notification.cpp b/frameworks/ans/src/notification.cpp
index f368a78c2bd30ce8abe634d18203b604ae0fc67c..af1ca979bb1893ae67e3fd8a42a0d366f37faa0b 100644
--- a/frameworks/ans/src/notification.cpp
+++ b/frameworks/ans/src/notification.cpp
@@ -486,7 +486,7 @@ void Notification::SetSourceType(NotificationConstant::SourceType sourceType)
 std::string Notification::Dump() const
 {
     std::string vibrationStyle = "";
-    for (auto &style : vibrationStyle_) {
+    for (const auto &style : vibrationStyle_) {
         vibrationStyle += std::to_string(style);
         vibrationStyle += ", ";
     }
diff --git a/frameworks/ans/src/notification_distributed_options.cpp b/frameworks/ans/src/notification_distributed_options.cpp
index dbfb26a275de63819df7b2262afed6180f288f69..4f6349f67d05349babd396c986141bc467475f0f 100644
--- a/frameworks/ans/src/notification_distributed_options.cpp
+++ b/frameworks/ans/src/notification_distributed_options.cpp
@@ -56,13 +56,13 @@ std::vector<std::string> NotificationDistributedOptions::GetDevicesSupportOperat
 std::string NotificationDistributedOptions::Dump()
 {
     std::string devicesSupportDisplay = "";
-    for (auto &device : devicesSupportDisplay_) {
+    for (const auto &device : devicesSupportDisplay_) {
         devicesSupportDisplay += device;
         devicesSupportDisplay += ", ";
     }
 
     std::string devicesSupportOperate = "";
-    for (auto &device : devicesSupportOperate_) {
+    for (const auto &device : devicesSupportOperate_) {
         devicesSupportOperate += device;
         devicesSupportOperate += ", ";
     }
diff --git a/frameworks/js/napi/src/reminder/publish.cpp b/frameworks/js/napi/src/reminder/publish.cpp
index 8a3634270d073d9f012d1d6fd18d7b2bc92067be..4d4e65b1be9a0fa1d43916a9cc0b2b2317e390f5 100644
--- a/frameworks/js/napi/src/reminder/publish.cpp
+++ b/frameworks/js/napi/src/reminder/publish.cpp
@@ -479,7 +479,7 @@ void ParseReminderCalendar(const napi_env &env, const ReminderRequest &reminder,
 }
 
 void ParseReminder(
-    const napi_env &env, const ReminderRequest::ReminderType &type, ReminderRequest &reminder, napi_value &result)
+    const napi_env &env, const ReminderRequest::ReminderType &type, const ReminderRequest &reminder, napi_value &result)
 {
     switch (type) {
         case ReminderRequest::ReminderType::TIMER: {
diff --git a/frameworks/js/napi/src/reminder/reminder_common.cpp b/frameworks/js/napi/src/reminder/reminder_common.cpp
index d1a0afac321d34ea7f59dcc42d2d502a0cd14f30..95d0eefdcfb953e66a369d836b600d60a99e2ec9 100644
--- a/frameworks/js/napi/src/reminder/reminder_common.cpp
+++ b/frameworks/js/napi/src/reminder/reminder_common.cpp
@@ -552,8 +552,7 @@ void ReminderCommon::HandleErrCode(const napi_env &env, int32_t errCode)
     if (errCode == ERR_OK) {
         return;
     }
-    std::string errCodeMsg = FindErrMsg(env, errCode);
-    errCodeMsg = reminderErrCodeMsgMap[errCode];
+    std::string errCodeMsg = reminderErrCodeMsgMap[errCode];
     napi_throw_error(env, std::to_string(errCode).c_str(), errCodeMsg.c_str());
 }
 
diff --git a/frameworks/js/napi/src/subscribe.cpp b/frameworks/js/napi/src/subscribe.cpp
index c128d88c4a3a8c864eb24a1062c52546e09f943f..86cc4582ec3d34b759e9e7cab8c5a26a49e3518d 100644
--- a/frameworks/js/napi/src/subscribe.cpp
+++ b/frameworks/js/napi/src/subscribe.cpp
@@ -1038,7 +1038,7 @@ bool AddSubscriberInstancesInfo(const napi_env &env, const SubscriberInstancesIn
     return true;
 }
 
-bool DelSubscriberInstancesInfo(const napi_env &env, SubscriberInstance *subscriber)
+bool DelSubscriberInstancesInfo(const napi_env &env, const SubscriberInstance *subscriber)
 {
     ANS_LOGI("enter");
     if (subscriber == nullptr) {
diff --git a/services/ans/src/advanced_notification_service.cpp b/services/ans/src/advanced_notification_service.cpp
index 05481b8e720fd04bb2aec3fa680071243ec01504..c6d9f465d54b8c469d4cd6f3813acfa71f661c66 100644
--- a/services/ans/src/advanced_notification_service.cpp
+++ b/services/ans/src/advanced_notification_service.cpp
@@ -2095,7 +2095,7 @@ ErrCode AdvancedNotificationService::FlowControl(const std::shared_ptr<Notificat
         } else {
             std::list<std::shared_ptr<NotificationRecord>> sorted = notificationList_;
             sorted.sort(SortNotificationsByLevelAndTime);
-            recordToRemove = bundleList.front();
+            recordToRemove = sorted.front();
             SendFlowControlOccurHiSysEvent(recordToRemove);
             notificationList_.remove(sorted.front());
         }