samples: matter: lock: Extract lambda to separate function

Address SonarQube issue regarding too long lambda body.

Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>

Author: adigie

samples/matter/lock/src/app_task.cpp

@@ -176,53 +176,9 @@ void AppTask::UpdateClusterState(const BoltLockManager::StateData &stateData)
 		return;
 	}
 
-	CHIP_ERROR err = SystemLayer().ScheduleLambda([stateData = stateDataCopy]() {
-		DlLockState newLockState;
-
-		switch (stateData->mState) {
-		case BoltLockManager::State::kLockingCompleted:
-			newLockState = DlLockState::kLocked;
-			break;
-		case BoltLockManager::State::kUnlockingCompleted:
-			newLockState = DlLockState::kUnlocked;
-			break;
-		default:
-			newLockState = DlLockState::kNotFullyLocked;
-			break;
-		}
-
-		chip::app::DataModel::Nullable<chip::app::Clusters::DoorLock::DlLockState> currentLockState;
-		chip::app::Clusters::DoorLock::Attributes::LockState::Get(kLockEndpointId, currentLockState);
-
-		if (currentLockState.IsNull()) {
-			/* Initialize lock state with start value, but not invoke lock/unlock. */
-			chip::app::Clusters::DoorLock::Attributes::LockState::Set(kLockEndpointId, newLockState);
-		} else {
-			LOG_INF("Updating LockState attribute");
-
-			Nullable<uint16_t> userId;
-			Nullable<List<const LockOpCredentials>> credentials;
-			List<const LockOpCredentials> credentialList;
-
-			if (!stateData->mValidatePINResult.IsNull()) {
-				userId = { stateData->mValidatePINResult.Value().mUserId };
-
-				/* `DoorLockServer::SetLockState` exptects list of `LockOpCredentials`,
-				   however in case of PIN validation it makes no sense to have more than one
-				   credential corresponding to validation result. For simplicity we wrap single
-				   credential in list here. */
-				credentialList = { &stateData->mValidatePINResult.Value().mCredential, 1 };
-				credentials = { credentialList };
-			}
-
-			if (!DoorLockServer::Instance().SetLockState(kLockEndpointId, newLockState, stateData->mSource,
-								     userId, credentials, stateData->mFabricIdx,
-								     stateData->mNodeId)) {
-				LOG_ERR("Failed to update LockState attribute");
-			}
-		}
-
-		Platform::Delete(stateData);
+	CHIP_ERROR err = SystemLayer().ScheduleLambda([stateDataCopy]() {
+		UpdateClusterStateHandler(*stateDataCopy);
+		Platform::Delete(stateDataCopy);
 	});
 
 	if (err != CHIP_NO_ERROR) {
@@ -231,6 +187,55 @@ void AppTask::UpdateClusterState(const BoltLockManager::StateData &stateData)
 	}
 }
 
+void AppTask::UpdateClusterStateHandler(const BoltLockManager::StateData &stateData)
+{
+	using namespace chip::app::Clusters::DoorLock::Attributes;
+
+	DlLockState newLockState;
+
+	switch (stateData.mState) {
+	case BoltLockManager::State::kLockingCompleted:
+		newLockState = DlLockState::kLocked;
+		break;
+	case BoltLockManager::State::kUnlockingCompleted:
+		newLockState = DlLockState::kUnlocked;
+		break;
+	default:
+		newLockState = DlLockState::kNotFullyLocked;
+		break;
+	}
+
+	Nullable<DlLockState> currentLockState;
+	LockState::Get(kLockEndpointId, currentLockState);
+
+	if (currentLockState.IsNull()) {
+		/* Initialize lock state with start value, but not invoke lock/unlock. */
+		LockState::Set(kLockEndpointId, newLockState);
+	} else {
+		LOG_INF("Updating LockState attribute");
+
+		Nullable<uint16_t> userId;
+		Nullable<List<const LockOpCredentials>> credentials;
+		List<const LockOpCredentials> credentialList;
+
+		if (!stateData.mValidatePINResult.IsNull()) {
+			userId = { stateData.mValidatePINResult.Value().mUserId };
+
+			/* `DoorLockServer::SetLockState` exptects list of `LockOpCredentials`,
+			   however in case of PIN validation it makes no sense to have more than one
+			   credential corresponding to validation result. For simplicity we wrap single
+			   credential in list here. */
+			credentialList = { &stateData.mValidatePINResult.Value().mCredential, 1 };
+			credentials = { credentialList };
+		}
+
+		if (!DoorLockServer::Instance().SetLockState(kLockEndpointId, newLockState, stateData.mSource, userId,
+							     credentials, stateData.mFabricIdx, stateData.mNodeId)) {
+			LOG_ERR("Failed to update LockState attribute");
+		}
+	}
+}
+
 #ifdef CONFIG_CHIP_NUS
 void AppTask::NUSLockCallback(void *context)
 {

samples/matter/lock/src/app_task.h

@@ -41,6 +41,7 @@ class AppTask {
 	static void LockActionEventHandler();
 	static void ButtonEventHandler(Nrf::ButtonState state, Nrf::ButtonMask hasChanged);
 	static void LockStateChanged(const BoltLockManager::StateData &stateData);
+	static void UpdateClusterStateHandler(const BoltLockManager::StateData &stateData);
 
 #ifdef CONFIG_THREAD_WIFI_SWITCHING
 	static void SwitchTransportEventHandler();