Удаления цепочек категории?

Question

[Артур Стрежинов]

Когда удаляется категория 1, то по цепочки должны удалится все за ней категории, нужно сделать рекурсию, но не понимаю как???



БД

id | parent_id | title
1 |        0        | Категория 1
2 |        1        | Категория  1.1
3 |        2        | Категория  1.1.1
4 |        3        | Категория  1.1.1.1
------------------------------------------
5 |        0        | Категория  2
6 |        5        | Категория  2.2
7 |        6        | Категория  2.2.2

В контроллере, тут полный бред

public function actionDelete($id)
    {
            //$this->findModel($id)->delete();
            $parent = Category::find()->where(['parent_id' => $id])->one();
            if ($parent) {
                $child1 = Category::find()->where(['parent_id' => $parent->id])->one();
                $child2 = Category::find()->where(['parent_id' => $child1->id])->one();

                VarDumper::dump($child1->id,11,1);
                VarDumper::dump($child1->parent_id,11,1);
                VarDumper::dump($child2->id,11,1);
                VarDumper::dump($id,11,1);die();
            }

        return $this->redirect(['index']);
    }

Comments

[Дмитрий]

Добрый день?
А вопрос то в чём?

[Артур Стрежинов]

slo_nik, описал, случайно нажал на комбинацию и пост появился

[Дмитрий]

Артур Стрежинов, используйте внешний ключ.

[Артур Стрежинов]

slo_nik, поподробней объясните?

[Дмитрий]

Артур Стрежинов, не та ссылка.
Нужен не первичный, а внешний ключ.
https://ru.wikipedia.org/wiki/%D0%92%D0%BD%D0%B5%D...

[Артур Стрежинов]

slo_nik, у меня не так реализовано в бд.

[Артур Стрежинов]

slo_nik, и как мне реализовать с моей структурой?

[Дмитрий]

Артур Стрежинов, Вы можете установить внешний ключ на parent_id.

[Дмитрий]

Артур Стрежинов, вот как-то так.

$this->addForeignKey('fk-parent_id', '{{%categories}}', 'pareng_id', '{{%categories}}', 'id', 'CASCADE');

[Максим]

slo_nik, у него нули в базе..))))))))))

[Дмитрий]

Максим Ворожцов, исправить недолго.

[Максим]

slo_nik, да я понимаю)) Это больше с юмором комментарий)) Но решение то ваше подойти должно) Или удалением через запросы.

[Дмитрий]

Максим Ворожцов, через запросы муторно получится. Если внешний ключ не поможет, то тогда через запросы надо будет делать.

[Артур Стрежинов]

slo_nik, я установил внешний ключ на parent_id, и что дальше делать (как то глупо, но я не понимаю????)

[Артур Стрежинов]

Максим Ворожцов, что вы имеете виду??

[Дмитрий]

Артур Стрежинов, попробуйте удалить родительскую категорию. Удалит дочерние?

[Артур Стрежинов]

slo_nik, удалил родительскую, а вот дочернию не удалило, теперь я понял что вы имели виду

[Дмитрий]

Артур Стрежинов, а как Вы foreign key устанавливали?

[Дмитрий]

Артур Стрежинов, и какой тип таблицы у Вас? InnoDB?

[Артур Стрежинов]

slo_nik, Добрый вечер, в phpmyadmin выбрал поле parent_id нажал на индекс

[Артур Стрежинов]

slo_nik, тип InnoDB!

[Дмитрий]

Артур Стрежинов, InnoDB это хорошо. Столбец parent_id изначально должен быть NULL.
Вы поставили индекс, а не внешний ключ. Внешний ключ на вкладке "Структура", внизу "индексы".
Сделайте через миграцию, так будет лучше.

[Дмитрий]

Артур Стрежинов, так же перед установкой ключа надо отключить, а затем включить проверку внешнего ключа.

[Артур Стрежинов]

slo_nik, вот код миграции

public function up()
    {
        $this->createTable('{{%category}}', [
            'id' => $this->primaryKey()->notNull(),
            'parent_id' => $this->integer()->null(),
            'status' => $this->integer()->null(),
            'title' => $this->string(255)->null(),
            'alias' => $this->string(100)->null(),
            'sort' => $this->integer()->null(),
            'meta_title' => $this->string(200)->null(),
            'meta_keywords' => $this->string(200)->null(),
            'meta_description' => $this->string(200)->null(),
        ]);
        $this->addForeignKey('fk-parent_id', '{{%category}}', 'parent_id', '{{%category}}', 'id', 'CASCADE');
    }

когда хочу сохранить, то получаю на скриншоте

[Дмитрий]

Артур Стрежинов,

так же перед установкой ключа надо отключить, а затем включить проверку внешнего ключа.

$this->db->createCommand('SET foreign_key_checks=0')->execute();
$this->addForeignKey('fk-parent_id', '{{%category}}', 'parent_id', '{{%category}}', 'id', 'CASCADE');
$this->db->createCommand('SET foreign_key_checks=1')->execute();

[Дмитрий]

Артур Стрежинов, в таблице уже существуют записи?
Если да, то поэтому и выдаёт ошибку. Надо в этом случае отключать проверку ключа. Так же индекс надо установить на parent_id.

$this->createIndex('idx-parent_id', '{{%category}}', 'parent_id');
$this->db->createCommand('SET foreign_key_checks=0')->execute();
$this->addForeignKey('fk-parent_id', '{{%category}}', 'parent_id', '{{%category}}', 'id', 'CASCADE');
$this->db->createCommand('SET foreign_key_checks=1')->execute();

[Артур Стрежинов]

slo_nik, Добрый вечер, в таблицы нету записей.

Я сделал так

class m190102_165446_category extends Migration
{
//    /**
//     * {@inheritdoc}
//     */
//    public function safeUp()
//    {
//
//    }
//
//    /**
//     * {@inheritdoc}
//     */
//    public function safeDown()
//    {
//        echo "m190102_165446_category cannot be reverted.\n";
//
//        return false;
//    }

    // Use up()/down() to run migration code without a transaction.
    public function up()
    {
        $this->createTable('{{%category}}', [
            'id' => $this->primaryKey()->notNull(),
            'parent_id' => $this->integer()->null(),
            'status' => $this->integer()->null(),
            'title' => $this->string(255)->null(),
            'alias' => $this->string(100)->null(),
            'sort' => $this->integer()->null(),
            'meta_title' => $this->string(200)->null(),
            'meta_keywords' => $this->string(200)->null(),
            'meta_description' => $this->string(200)->null(),
        ]);
        $this->db->createCommand('SET foreign_key_checks=0')->execute();
        $this->addForeignKey('fk-parent_id', '{{%category}}', 'parent_id', '{{%category}}', 'id', 'CASCADE');
        $this->db->createCommand('SET foreign_key_checks=1')->execute();
    }

    public function down()
    {
        echo "m190102_165446_category cannot be reverted.\n";

        return false;
    }
}

И получил ошибку

[Дмитрий]

Артур Стрежинов, а откуда берётся INSERT?

[Дмитрий]

Артур Стрежинов, и если по умолчанию у Вас для parent_id идёт NULL, то зачем в запросе Вы подсовываете значение "0"?
Ведь у Вас нет записи с id "0"...

[Артур Стрежинов]

slo_nik, Да Вы правы, сам "0" подсовывается от селекта, сам "0" стоит просто по умолчанию, я не обратил на это внимания!

<option value="0">Нет родителя</option>

Убрал "0", проверил, все работает.
Как бы вопрос решен, СПАСИБО ВАМ!
Хотелось бы понять как еще можно сделать с помощью рекурсии!

Вы писали у Максим Тимофеев, что можно

по первому id собираете массив parent_id первого уровня, потом, идя по первому уровню собираете в массив parent_id второго уровня и так далее.

, можете показать если не трудно на примере. Потому что может быть большая вложенность!

[Дмитрий]

Артур Стрежинов, option с названием "Нет родителя" не нужен атрибут value.
Чтобы решить с помощью рекурсии необходимо написать функцию, которая будет сама себя вызывать при определённых условиях.
Почитайте про рекурсивный вызов функций. Например здесь.

[Дмитрий]

Артур Стрежинов, но лучше использовать внешний ключ, так сохранится целостность данных.
Так же, вместо того чтобы удалять дочернюю директорию, можно устанавливать parent_id в NULL. Дочерняя директория при этом не удаляется, данные, которые привязаны к ней не теряют связь с категориями.

Answer 1

[Максим Тимофеев]

Ответ зависит от того как Вы храните свое дерево категорий. Способов хранения много (adjustment list, Nested Sets и т.д.), архитектура бд при этом совершенно разная, так что и решения будут принципиально разниться. А У Вас может быть еще и свой кастомный вариант. Так что прежде чем задавать такой вопрос, надо расписать свою архитектуру БД.

Но я уверен, что у Вас уже есть рекурсия необходимая, при постройке дерева на фронте. Так что не стоит изобретать велосипед. Хотя обычно юзают готовые классы типа https://github.com/creocoder/yii2-nested-sets и тогда вообще проблем нет никаких, так как методы уже все реализованы.

Comments

[Артур Стрежинов]

Здрасте, храню в бд!
Вот так в базе у меня скриншот прилагается, я хочу понять как мне в коде правильно удалять цепочки категории, мне не понятно и хочу понять как это реализовать!

[Максим Тимофеев]

Артур Стрежинов, Допустим удаляется id 125 соответственно мы должны удалить все записи у который parent_id = 125, при этом послать еще 1 запрос (например exists или count), что бы проверить есть ли записи для которых в parent_id id этих и т.д. В зависимости от того какой максимальный уровень вложенности предусмотрен. Так что рекурсии нет. Если он не ограничен, то лучше хранить не так а Nested Sets

[Артур Стрежинов]

Максим Тимофеев, да у меня есть рекурсия которая выводит само меню...... но как я могу ее использовать в удаление?

/**
     * Строим дерево категории
     * @param $data
     * @param int $rootID
     * @return array
     */
    protected function buildTree($data, $rootID = 0)
    {
        $tree = [];
        foreach ($data as $id => $node) {
            if ($node['parent_id'] == $rootID) {
                unset($data[$id]);
                $node['childs'] = $this->buildTree($data, $node['id']);
                $tree[] = $node;
            }
        }
        return $tree;
    }

    /**
     * Получаю все категории
     * @return array
     */
    public function getAllCategories()
    {
        $data = self::find()->asArray()->all();
        return $this->buildTree($data);
    }

вот что на выводе, подскажите как я могу использовать эту рекурсию для удаления??

[Дмитрий]

Артур Стрежинов, по первому id собираете массив parent_id первого уровня, потом, идя по первому уровню собираете в массив parent_id второго уровня и так далее.
Потом попробуйте через deleteAll() удалить категории.

[Артур Стрежинов]

slo_nik, а не чего что, рекурсия в модели находится?

[Максим Тимофеев]

Артур Стрежинов,

но как я могу ее использовать в удаление?

Немного подпилить. Все так же, только формируете не многоуровневый массив и только с id-шниками, а дальше https://www.yiiframework.com/doc/api/2.0/yii-db-ac...