это ещё раз доказывает мои слова. вы второй день в программировании.
я пишу на хаксе, где геттеры без сеттеров ( и наоборот ) могут жить абсолютно спокойно.
причина моей записи про ваш геттер - была не в том, что он без сеттера, а в том, что он нафиг не нужен. абсолютно бессмысленный лишний код.
ваш код не полный. начиная от "начинки" самой кнопки, заканчивая кучей не логичных проверок.
ещё раз повторюсь - вам нужно реструктурировать ваш код.
с нормальной структурой таких глупых вопросов не возникло бы, потому что ошибок бы не было.
пс: приватные не статические переменные логично прописывать в конструкторе, т.к. они являются частью любого из экземпляров и не будут доступны без создания хотя бы одного, что подразумевает пробег по конструктору. после компиляции разкомпильте свою свфку и посмотрите где все ваши переменные находятся. посмотрели? верно, в конструкторе.
"а надо наверное - mouseClickHandler писать ? "
не ёрничайте.
достаточно было бы

Код AS3:
onMinusLineClick(e:...
- сразу бы стало понятно, что это функция - обработчик мышиного события. опять начнёте ёрничать? или всё же умерите свой пыл и посмотрите фактам в лицо?
as3 расслабляет для неправильных мыслей. а вы их принимаете за правильные. мыслите сразу верно - и будет счастье.