如何在 Laravel 中删除具有相同列值的重复行？

Question

我正在尝试在 Laravel 中创建一个工匠命令，以删除所有具有相同地址的场地，并保留 ID 号最低的场地（因此首先创建）。

为此，我需要检查 3 个字段：'street'、'house_number'、'house_number_addition'

这是我有多远：

$venues = Venue::select('street', 'house_number', 'house_number_addition', DB::raw('COUNT(*) as count'))
    ->groupBy('street', 'house_number', 'house_number_addition')
    ->having('count', '>', 1)
    ->get();

foreach ($venues as $venue) {
    $this->comment("Removing venue: {$venue->street} {$venue->house_number} {$venue->house_number_addition}");
    $venue->delete();
}

只有删除不起作用，但也没有给出错误。

Answer 1

为了能够删除一个项目，Eloquent 需要知道它的 id。如果您确保查询了模型的 ID，则可以delete()毫无问题地调用。

但是，在您的查询中，这将不起作用，因为您有一条GROUP_BY语句，因此 SQL 不允许您选择id列（请参阅此处）。

这里最简单的解决方案是利用 Eloquent 的Collection类来映射模型，例如：

$uniqueAddresses = [];

Venue::all()
    ->filter(function(Venue $venue) use (&$uniqueAddresses) {
        $address = sprintf("%s.%s.%s",
            $venue->street,
            $venue->house_number,
            $venue->house_number_addition);

        if (in_array($address, $uniqueAddresses)) {
            // address is a duplicate
            return $venue;
        }

        $uniqueAddresses[] = $address;
    })->map(function(Venue $venue) {
        $venue->delete();
    });

或者，为了使您的删除查询更有效率（取决于您的数据集有多大）：

$uniqueAddresses = [];
$duplicates = [];

Venue::all()
    ->map(function(Venue $venue) use (&$uniqueAddresses, &$duplicates) {
        $address = sprintf("%s.%s.%s",
            $venue->street,
            $venue->house_number,
            $venue->house_number_addition);

        if (in_array($address, $uniqueAddresses)) {
            // address is a duplicate
            $duplicates[] = $venue->id;
        } else {
            $uniqueAddresses[] = $address;
        }
    });

DB::table('venues')->whereIn('id', $duplicates)->delete();

注意：最后一个将永久删除您的模型；它不适用于 Eloquent 的SoftDeletes功能。

当然，您也可以编写一个原始查询来完成所有这些。