Замечания по паттерну Команда

[alexey-malov]

1>c:\teaching\ood\2017-ood\stepanenko\ood\05_command\documenteditor\editor.cpp(53): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
1>c:\teaching\ood\2017-ood\stepanenko\ood\05_command\documenteditor\editor.cpp(82): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data

• Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

[alexey-malov]

• shared_ptr должен передаваться в функцию по const-ссылке, если она не меняет свой аргумент
  http://stackoverflow.com/questions/3310737/shared-ptr-by-reference-or-by-value

[alexey-malov]

• delete paragrpah и delete image одинаковые. сделать один класс

[alexey-malov]

• Вставка изображения и вставка параграфа также могут быть реализованы одной командый InsertDocumentItem

[alexey-malov]

• Сейчас все манипуляции выполняются документом, в том числе и те, который могут быть выполнены его частями. Например, изменение текста параграфа или ресайз изображения можно выполнить средствами самого параграфа или изображения. Надо, чтобы можно было сказать параграфу SetText и это привело бы к созданию команды в истории.

[alexey-malov]

C.128: Virtual functions should specify exactly one of virtual, override, or final
https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Rh-override