refactor: Simplify return statements in SetMultiDxlWrite function for cleaner code

Woojin-Crive · Woojin-Crive · commit a2c2eeb34d17 · 2025-06-25T09:19:14.000+09:00
diff --git a/src/dynamixel/dynamixel.cpp b/src/dynamixel/dynamixel.cpp
@@ -407,6 +407,7 @@ DxlError Dynamixel::SetDxlWriteItems(
 
   return DxlError::OK;
 }
+
 DxlError Dynamixel::SetMultiDxlWrite()
 {
   write_type_ = checkWriteType();
@@ -439,12 +440,10 @@ DxlError Dynamixel::SetMultiDxlWrite()
   }
 
   if (write_type_ == SYNC) {
-    SetSyncWriteItemAndHandler();
+    return SetSyncWriteItemAndHandler();
   } else {
-    SetBulkWriteItemAndHandler();
+    return SetBulkWriteItemAndHandler();
   }
-
-  return DxlError::OK;
 }
 
 DxlError Dynamixel::DynamixelEnable(std::vector<uint8_t> id_arr)

Original file line number	Diff line number	Diff line change
`@@ -407,6 +407,7 @@ DxlError Dynamixel::SetDxlWriteItems(`
`407`	`407`
`408`	`408`	`return DxlError::OK;`
`409`	`409`	`}`
	`410`	`+`
`410`	`411`	`DxlError Dynamixel::SetMultiDxlWrite()`
`411`	`412`	`{`
`412`	`413`	`write_type_ = checkWriteType();`
`@@ -439,12 +440,10 @@ DxlError Dynamixel::SetMultiDxlWrite()`
`439`	`440`	`}`
`440`	`441`
`441`	`442`	`if (write_type_ == SYNC) {`
`442`		`- SetSyncWriteItemAndHandler();`
	`443`	`+ return SetSyncWriteItemAndHandler();`
`443`	`444`	`} else {`
`444`		`- SetBulkWriteItemAndHandler();`
	`445`	`+ return SetBulkWriteItemAndHandler();`
`445`	`446`	`}`
`446`		`-`
`447`		`- return DxlError::OK;`
`448`	`447`	`}`
`449`	`448`
`450`	`449`	`DxlError Dynamixel::DynamixelEnable(std::vector<uint8_t> id_arr)`