1. 🐻 Русский / Russian
  2. Bugs
+1
Fixed

Паттерн компоновщик - в псевдокоде возможная ошибка

snooper69 5 years ago • updated by anonymous 5 years ago • 2 
Вызовы:
 group.add(components)
 all.remove(components)
нужно обернуть в foreach. т.к. методы add и remove по определению принимают 
единичные экземпляры компонент а не массив

Answer

Answer
Fixed
Alexander 5 years ago

Большое спасибо, поправлю этот момент в следующей версии.

progr dk 5 years ago

Более того, разве можно в groupSelected использовать локальную переменную all другого метода?

Специально же создали композит group для хранения выделения группы...

// Группировка выбранных компонентов в один сложный
    // компонент.
    method groupSelected(components: array of Graphic) is
        group = new CompoundGraphic()
        group.add(components)
        all.remove(components)
        all.add(group)
        // Все компоненты будут отрисованы.
        all.draw()

P.S. Блинство, пока писал коммент, исправляя в нём HTML, 3 раза ткнул на иконку профиля слева сверху. Сделайте её меньше!...или сохраняйте коммент в куки...

Answer
Fixed
Alexander 5 years ago

Большое спасибо, поправлю этот момент в следующей версии.

Customer support service by UserEcho