Игра начинает тормозить. Помогите разобраться.

[pivnoibaron]

Привет. Сделал игру. Точнее делаю. Но вот игра при запуске, начинает есть все ресурсы системы. Если у кого есть желание, посмотрите что там не так. Сам не могу разобраться.

Игра напоминает игру "Жизнь". Нижний квадрат с права "Play" с лева "stop"

[stweet]

Ужасно!

1. переменные и методы должны явно указывать тип данных, в противном случае возвращать войд(:void).
2. при каждой попытке, в коде, изменить делай таймера у вас создаёться новый не удаляя стары, а это очень жрёт оперативную память.
3. Дикий ужас происходит в архитектуре кода, отсутствие инициализации блоков памяти, типов, читабильность на нуле. Повтор однозадачных методов, массовый! Сложно уловить мысль разработчика.

Приведите исходники в порядок!

[pivnoibaron]

" 2. при каждой попытке, в коде, изменить делай таймера у вас создаёться новый не удаляя стары, а это очень жрёт оперативную память. "
Получается что игра тормозит из за таймера?

Можно более подробней написать. Что поменять, что переделать?

[Inet_PC]

[quote]public function timePlayGo ( )
{
tmr = new Timer ( 1000, timeGo );
tmr.addEventListener(TimerEvent.TIMER, masiw );
tmr.start();
}[/QUOTE
Вы думаете, что потеряли ссылку на таймер и все он перестал работать?
Ну поменяйте хотябы так:

public function timePlayGo ( )
{
	if (tmr == null)
	{
		tmr = new Timer ( 1000, timeGo );
		tmr.addEventListener(TimerEvent.TIMER, masiw );	
	}
	else
	{
		tmr.reset();
	}
	tmr.start();
}

А вообще Вам бы книгу какуюнить прочитать.

[pivnoibaron]

Тут наверное не таймер виноват. Пробовал таймер закидывать в конструктор, и без воздействия на него кнопок. Просто сразу запускаю игру с уже готовой комбинацией и сразу запускается таймер. Но все равно игра начинает поглощать ресурсы. Может это из за того что Я создаю при цикле новые квадраты и они суммируютсья и при каждом цикле for они добавляю новые не удаляя старые?

private function masiw (e:TimerEvent ):void   //пробегает по массиву arayViz заменяет 1 и 0 на визуальные обекты.
		{
			for ( var i:uint = 0; i < arayViz.length; i++ )
			{
				for ( var j:uint  = 0; j < arayViz [ i ].length; j++ )
				{
					if (  arayViz [ i ] [ j ] == 0  )
					{
						var newBoxTwo = new Box ( arayGrid, arayViz, masivReload ); // Может обекты Box сумируются и начинают забирать память?
						newBoxTwo.x = j * 20;
						newBoxTwo.y = i * 20;
 
						newBoxTwo.ii = i;
						newBoxTwo.jj = j;
 
						arayViz [ i ] [ j ] = newBoxTwo;
 
						addChild ( arayViz [ i ] [ j ] );
					}  
					if ( arayViz [ i ] [ j ] == 1 )
					{
						var newMoveBoxTwo = new MoveBox ( arayGrid, arayViz, masivReload );
						newMoveBoxTwo.x = j * 20;
						newMoveBoxTwo.y = i * 20;
 
						newMoveBoxTwo.ii = i;
						newMoveBoxTwo.jj = j;
 
						arayViz [ i ] [ j ] = newMoveBoxTwo;
 
						addChild ( arayViz [ i ] [ j ] );
 
					}
				}
			}                                             
			moveBox ( );
		}

[venziver]

Каждый раз в таймере Вы ставите ребёнка в очередь: addChild(arayViz[ i ][ j ]); и все они остаются в памяти и продолжают работать. Попробуйте вставить:

removeChild(arayViz[ i ][ j ]);
arayViz[ i ][ j ]=null;

а после уже снова ставте в очередь.

[pivnoibaron]

Сообщение от venziver

Каждый раз в таймере Вы ставите ребёнка в очередь: addChild(arayViz[ i ][ j ]); и все они остаются в памяти и продолжают работать. Попробуйте вставить:
removeChild(arayViz[ i ][ j ]);
arayViz[ i ][ j ]=null;
а после уже снова ставте в очередь.

Куда их в коде поставить?

[venziver]

У вас есть функция:
function masiw (e:TimerEvent ):void
её выполняет таймер с заданной частотой и на экране получается движение. Поэтому в начале этой функции надо сначала удалить в цикле всех детей из очереди:
removeChild(arayViz[ i ][ j ]);
arayViz[ i ][ j ]=null;
а потом в цикле их снова создать и поставить в очередь

[СлаваRa]

А еще лучше объекты не пересоздавать. Вам по сути нужно просто менять их состояние, как я понял.

[pivnoibaron]

Переделал игру. Но опять начинает тормозить :-( Я кажется что то в ней не понимаю. Кому несложно. Посмотрите пожалуйста и объясните что не так.