packa Wrote:Единой и старался делать) в 3 переменной просто необходимо такое название, иначе не ясно разница между чем и чем.

Не, я про другое. Я про символы подчёркивания в именах. Их надо либо использовать везде, либо вообще не использовать.

packa Wrote:У меня все конструкторы вынесены ниже, рядом с методами. Почему в середине? Просто у меня еще 2 класса, группировал контруктор+метод+метод+...

Нет, функции-члены класса должны распаолагаться либо внутри него, либо сразу вслед за ним. Я не хочу рыскать по всему cpp-шнику в поисках нужной мне функции - особенно, если самих классов много.

packa Wrote:Исключительно для конструктора. Если заглянуть в сам конструктор там все предельно ясно

Ничего не ясно. В строке Tower ( int x, int y, int h, int d, int t, int r) - что такое x? Что такое h? Что такое d, t, r? Если я захочу создать экземпляр твоего класса - то я не смогу этого сделать, поскольку я понятия не имею, что же значат его параметры. Правильно писать Tower (int x_pos, int y_pos, int health, int damage, int target, int reload). Поскольку иначе мне придётся копаться во внунренностях твоего класса, а мне некогда.

packa Wrote:Не понял что не так) КоординатаХ, КоординатаУ, картинка - вроде так все пишется.

Почему именно 610 и 235, а не 510 и 234? С какого перепугу функция меняет конкретные пиксели экрана? А что, если я захочу уменьшить размеры окна? Или у меня экран 480*320? А если это число 610 используется в нескольких функциях, а я захотел его поменять, что тогда? Мне придётся править каждую функцию, что ли?
Запомни: вообще нельзя использовать какие-то конкретные числа внутри кода. В худшем случае - такие константы надо дефайнить, и внутри функций использовать именно константы, а не явно указанные числа. Единственные числа, которые можно указывать явно - это 0, 1 и -1 (ну и двойка также иногда разрешается). Разве что, ещё можно использовать 8 как число бит в байте, 10 и 16 для функций перевода в системы счисления, ну и 100 для подсчёта процентов. Не более!

packa Wrote:Ammo::Ammo (int x, int y, int n, int t) : xx(x), yy(y), number(n), target(t); // ?

Да, именно так.

packa Wrote:Одна есть и хватит)) А если серьезно - специально оставлял переменные - чтобы при надобности просто их менять, не правя при этом код.

Константы сильно экономят время, отлавливая ошибки ещё на этапе компиляции. Когда-нибудь ты это оценишь.

packa Wrote:Со вторым полностью согласен, а вот с первым нет. Потому что код явно теряет свою очевидность, что тупо мешает читаемости. Я бы сказал что здесь овчинка выделки не стоит)

Какое из двух высказываний для тебя звучит более понятно?
   1) Если группа башен пуста, то...
   2) Если верно, что группа башен пуста, то...
На мой взгляд - однозначно первое... Вот и с кодом то же самое. Особенно если условий много, и они вложены друг в друга.

packa Wrote:Нажали на кнопку - Вошли в режим строительства - каждый клик строит башню.
Нажали повторно, вышли из режима, можем кликать до посинения, ничего не произойдет

Део верно заметил, что можно просто написать build = !build и не мучаться.

packa Wrote:Мозгу нуба намного сподручнее пользоваться интами, чем абстрактными итераторами)
Кстати уже заюзал в одном месте, но все равно тяжко обрабатывать мозгу, как это работает, хоть и понятно достаточно.

Итератор - это просто "обёртка" над указателем. Если ты не до конца понимаешь адресную арифметику указателей - то, конечно, итераторами пользоваться ещё не стоит.

packa Wrote:Почему? 3 класса = 3 конструктора.

Дык это не конструкторы по умолчанию - у них параметры есть! Я же говорил именно про конструкторы по умолчанию...

P.S. Да, ещё забыл сказать: public-переменные в классе - это почти всегда плохо. Нужно везде использовать get- и set-функции для каждой переменной, к которой планируется "сторонний" доступ. Даже если они будут простейшими.