230 lines
4.4 KiB
Markdown
230 lines
4.4 KiB
Markdown
# 批量排班接口修复说明
|
|
|
|
## 问题描述
|
|
|
|
接口报错：
|
|
```
|
|
app\common\controller\BaseLikeAdminController::success():
|
|
Argument #2 ($data) must be of type array, bool given
|
|
```
|
|
|
|
## 问题原因
|
|
|
|
`RosterLogic::batchSave()` 方法返回 `true`（布尔值），但控制器的 `success()` 方法要求第二个参数必须是数组类型。
|
|
|
|
## 解决方案
|
|
|
|
### 1. 修改 Logic 层返回值
|
|
|
|
**修改前**:
|
|
```php
|
|
public static function batchSave(array $params)
|
|
{
|
|
// ...
|
|
Db::commit();
|
|
return true; // 返回布尔值
|
|
}
|
|
```
|
|
|
|
**修改后**:
|
|
```php
|
|
public static function batchSave(array $params)
|
|
{
|
|
// ...
|
|
Db::commit();
|
|
return [
|
|
'success_count' => $successCount,
|
|
'create_count' => $createCount,
|
|
'update_count' => $updateCount,
|
|
]; // 返回数组，包含统计信息
|
|
}
|
|
```
|
|
|
|
### 2. 修改控制器
|
|
|
|
**修改前**:
|
|
```php
|
|
public function batchSave()
|
|
{
|
|
$params = (new RosterValidate())->post()->goCheck('batchSave');
|
|
$result = RosterLogic::batchSave($params);
|
|
return $this->success('批量保存成功', $result); // $result 是 bool
|
|
}
|
|
```
|
|
|
|
**修改后**:
|
|
```php
|
|
public function batchSave()
|
|
{
|
|
$params = (new RosterValidate())->post()->goCheck('batchSave');
|
|
$result = RosterLogic::batchSave($params);
|
|
if ($result === false) {
|
|
return $this->fail(RosterLogic::getError());
|
|
}
|
|
return $this->success('批量保存成功', $result); // $result 是 array
|
|
}
|
|
```
|
|
|
|
## 改进内容
|
|
|
|
### 1. 返回统计信息
|
|
|
|
现在批量保存接口会返回详细的统计信息：
|
|
|
|
```json
|
|
{
|
|
"code": 1,
|
|
"msg": "批量保存成功",
|
|
"data": {
|
|
"success_count": 20,
|
|
"create_count": 15,
|
|
"update_count": 5
|
|
}
|
|
}
|
|
```
|
|
|
|
- `success_count`: 成功处理的记录数
|
|
- `create_count`: 新创建的记录数
|
|
- `update_count`: 更新的记录数
|
|
|
|
### 2. 错误处理
|
|
|
|
如果批量保存失败，会返回错误信息：
|
|
|
|
```json
|
|
{
|
|
"code": 0,
|
|
"msg": "错误信息",
|
|
"data": []
|
|
}
|
|
```
|
|
|
|
### 3. 同样修复了 copy 方法
|
|
|
|
`copy` 方法也有同样的问题，已一并修复。
|
|
|
|
## 测试验证
|
|
|
|
### 测试1：批量创建
|
|
|
|
**请求**:
|
|
```json
|
|
POST /adminapi/doctor.roster/batchSave
|
|
{
|
|
"rosters": [
|
|
{
|
|
"doctor_id": 1,
|
|
"date": "2026-03-04",
|
|
"period": "morning",
|
|
"status": 1,
|
|
"quota": 20,
|
|
"max_patients": 30
|
|
},
|
|
{
|
|
"doctor_id": 1,
|
|
"date": "2026-03-04",
|
|
"period": "afternoon",
|
|
"status": 1,
|
|
"quota": 20,
|
|
"max_patients": 30
|
|
}
|
|
]
|
|
}
|
|
```
|
|
|
|
**响应**:
|
|
```json
|
|
{
|
|
"code": 1,
|
|
"msg": "批量保存成功",
|
|
"data": {
|
|
"success_count": 2,
|
|
"create_count": 2,
|
|
"update_count": 0
|
|
}
|
|
}
|
|
```
|
|
|
|
### 测试2：批量更新
|
|
|
|
再次执行相同的请求：
|
|
|
|
**响应**:
|
|
```json
|
|
{
|
|
"code": 1,
|
|
"msg": "批量保存成功",
|
|
"data": {
|
|
"success_count": 2,
|
|
"create_count": 0,
|
|
"update_count": 2
|
|
}
|
|
}
|
|
```
|
|
|
|
## 相关文件
|
|
|
|
修改的文件：
|
|
1. `server/app/adminapi/logic/doctor/RosterLogic.php` - 修改返回值类型
|
|
2. `server/app/adminapi/controller/doctor/RosterController.php` - 添加错误处理
|
|
|
|
## 注意事项
|
|
|
|
1. 所有返回布尔值的 Logic 方法都应该检查是否会传递给 `success()` 方法
|
|
2. 如果需要返回数据，应该返回数组而不是布尔值
|
|
3. 失败时应该使用 `fail()` 方法而不是 `success()`
|
|
|
|
## 最佳实践
|
|
|
|
### Logic 层返回值规范
|
|
|
|
```php
|
|
// ✅ 正确：返回数组或 false
|
|
public static function someMethod($params)
|
|
{
|
|
try {
|
|
// 处理逻辑
|
|
return ['id' => $id, 'count' => $count];
|
|
} catch (\Exception $e) {
|
|
self::setError($e->getMessage());
|
|
return false;
|
|
}
|
|
}
|
|
|
|
// ❌ 错误：返回 true
|
|
public static function someMethod($params)
|
|
{
|
|
try {
|
|
// 处理逻辑
|
|
return true; // 不要这样做
|
|
} catch (\Exception $e) {
|
|
return false;
|
|
}
|
|
}
|
|
```
|
|
|
|
### 控制器层处理规范
|
|
|
|
```php
|
|
// ✅ 正确：检查返回值
|
|
public function someAction()
|
|
{
|
|
$result = SomeLogic::someMethod($params);
|
|
if ($result === false) {
|
|
return $this->fail(SomeLogic::getError());
|
|
}
|
|
return $this->success('操作成功', $result);
|
|
}
|
|
|
|
// ❌ 错误：直接传递布尔值
|
|
public function someAction()
|
|
{
|
|
$result = SomeLogic::someMethod($params);
|
|
return $this->success('操作成功', $result); // 如果 $result 是 bool 会报错
|
|
}
|
|
```
|
|
|
|
## 总结
|
|
|
|
问题已修复，批量排班接口现在可以正常工作，并且返回详细的统计信息。
|