Flutter对接Laravel API更新用户时不上传图片保留原有图片问题问询

问题根因 #

你当前的代码存在3个核心错误，直接导致了遇到的异常：

• 判断图片参数是否存在的逻辑有误：!is_null($input['photo']) 首先会在请求根本没传photo字段时报「未定义数组索引」错误，其次无法过滤空字符串、空白字符这类空值，导致没传有效图片时也进入了图片处理逻辑，解析空值抛出「Invalid image data」异常
• 图片判断的if代码块缺少闭合右大括号，导致商品基础字段（名称、价格等）的赋值逻辑被包裹在图片判断分支内，不传图片时基础字段也无法更新；如果是粘贴时的格式缩进错误，也会出现基础字段只有传图片时才更新的问题
• 没有先验证图片参数的有效性就直接做base64解析，进一步增加了异常概率

修复后的代码 #

$product = Products::findOrFail($id);
// 先做参数验证，可选参数photo加存在时的验证规则
$validator = Validator::make($request->all(), [
    'name' => 'required',
    'price' => 'required',
    'detail' => 'required',
    'quantity' => 'required',
    'discount' => 'required',
    // 新增：photo参数只有传的时候才验证是有效的base64格式png图片
    'photo' => 'sometimes|string|starts_with:data:image/png;base64,'
]);

// 验证失败处理，可按自身业务逻辑调整响应格式
if ($validator->fails()) {
    return response()->json(['errors' => $validator->errors()], 422);
}

// 先给基础字段赋值，不管传不传图片都要更新基础信息
$product->name = $request->input('name');
$product->price = $request->input('price');
$product->detail = $request->input('detail');
$product->quantity = $request->input('quantity');
$product->discount = $request->input('discount');

// 只有当确实传了有效的photo参数时才处理图片
if ($request->has('photo') && !empty($request->input('photo'))) {
    $image = $request->input('photo');
    $image = str_replace('data:image/png;base64,', '', $image);
    $image = str_replace(' ', '+', $image);
    $imageName = Str::random(50) . '.png';
    
    $uploadPath = '/uploads/products';
    if (!is_dir($uploadPath)) {
        mkdir($uploadPath, 0755, true);
    }
    // 删除旧图
    Storage::disk('public_uploads')->delete($product->photo);
    // 存储新图
    Storage::disk('public_uploads')->put($imageName, base64_decode($image));
    // 赋值新图路径
    $product->photo = $imageName;
}

// 统一执行更新，不管传不传图片都会生效
$product->save();

补充说明 #

• 用Laravel自带的$request->has()方法判断参数是否存在，!empty()判断参数值非空，避免直接访问数组索引的错误
• 验证规则里加了sometimes规则，只有请求传了photo参数时才会执行后面的验证，确保传的是符合要求的png格式base64图片
• 把基础字段的赋值逻辑移到图片判断分支外，确保不管传不传图片，基础信息都能正常更新
• 最后统一调用save()方法执行更新，Eloquent的save()方法会自动判断是新增还是更新操作，不需要单独调用update()

内容的提问来源于stack exchange，提问作者devtest