Quick fix?

Currently I have two layers with a set of movieclips in each layer. I only have actions in one of the layers. This is an abbreviation of the code:


var counter:Number= 0;
var rightelbowtime:Timer = new Timer(3);
mbelly_mc.visible=false;

function rightelbowfall(event:TimerEvent): void {
	rightelbow_mc.y++;
	rightelbow_mc.rotation++;
	
	if (rightelbow_mc.y>=900) {
		rightelbowtime.stop();
		counter++;
		trace(counter);
	}
	
	}
rightelbowtime.addEventListener(TimerEvent.TIMER, rightelbowfall);

rightelbow_mc.addEventListener(MouseEvent.MOUSE_DOWN, rightelbowtimer);

function rightelbowtimer(event:MouseEvent): void {
	rightelbowtime.reset();
	rightelbowtime.start();
	
	}

function showmary(): void {
if (counter==1) {
mbelly_mc.visible=true;
}
}

Should i move the showmary function into the layer with the mbelly movieclip? Is there anything else I’m missing?