Показать сообщение отдельно
Старый 27.06.2006, 17:32
etc вне форума Посмотреть профиль Найти все сообщения от etc
  № 8  
Ответить с цитированием
etc
Et cetera
 
Аватар для etc

Регистрация: Sep 2002
Сообщений: 30,787
2) Не нужно расширять Object, твой класс и так наследник Object. Запись "extends Object" не имеет смысла. И метода __drawRec нет в Object.
4) Тебе -- может и не нужно, но человеку, которому потребуется разобраться в твоём классе (пример -- эта тема), будет крайне сложно понять, что ты подразумевал под этими переменными. Мне, например, всё-равно неясно с первого взгляда и ковыряний в коде, что такое id, ini, iam.
5) До тех пор, пока ты работаешь один. Лучше использовать хоть какой-нибудь стандарт, чем не использовать его вообще и писать в стиле AS1 по десятку конструкций на одной строке. Ты не только другим, но и себе роешь яму -- через пару месяцев разобрать код будет сложно, потому что написан сплошной строкой.
7) У тебя нигде не определено, что такое Index
8) Путаница в названиях методов -- некоторые начинаются с двойного подчёркивания (такое название может использоваться в приватных методах, но не публичных точно), а другие со строчной буквы.
9) Отсутствие типизации в некоторых местах.
10) Т.к. класс статический, то конструктор в нём не вызывается, соответственно всё, что внутри конструктора, тоже не работает.
11) В статическом классе не может быть ссылки на this, и конструкцию
Код:
iam.ini=$ME;
надо заменить на
Код:
iam.ini=TRACE;
в данном случае. Но опять же -- не имеет смысла вообще так писать, т.к. TRACE и так является глобальным объектом и ссылка на конструктор не нужна.

В общем лучше всё начисто переписать.


Последний раз редактировалось etc; 27.06.2006 в 17:35.