Assistance with improving code

Question

Hi there,
I'm looking for a little bit of help on improving my code. I have a feeling the following could be made to look, and feel a lot better. Originally I had defined image 1-8 as separate variables (var image1:img1 new img1) but instead leave them on the scene now, set to invisible. Is this the correct way to do it??
My code is below, please tell me what you think of how im doing this. I realise I'm still on the timeline.
import flash.events.MouseEvent;
for(var i:uint=1; i<9; i++){
    this["btn"+String(i)].addEventListener(MouseEvent.CLICK, buttonappear, false, 0, true);
for(var p:uint=1; p<9; p++){
    this["image"+String(p)].visible=false;
closebutton.visible=false
closebutton.addEventListener(MouseEvent.CLICK, closeall, false, 0, true);
function closeall(e:MouseEvent):void
        for(var p:uint=1; p<9; p++){
        this["image"+String(p)].visible=false;
        closebutton.visible=false
function buttonappear(e:MouseEvent):void
    var buttonTarget:SimpleButton = SimpleButton(e.currentTarget); // The variable is allergyTarget
    switch(buttonTarget) // Each case should swap the lineTarget
    case btn1: // allergy 1 is the first button
    image1.visible=true
    image1.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn2: // allergy 1 is the first button
    image2.visible=true
    image2.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn3: // allergy 1 is the first button
    image3.visible=true
    image3.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn4: // allergy 1 is the first button
    image4.visible=true
    image4.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn5: // allergy 1 is the first button
    image5.visible=true
    image5.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn6: // allergy 1 is the first button
    image8.visible=true
    image8.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn7: // allergy 1 is the first button
    image6.visible=true
    image6.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next
        case btn8: // allergy 1 is the first button
    image7.visible=true
    image7.gotoAndPlay(1); // The textbox should display
    closebutton.visible=true
    break; // If this is not the case, break and check next

Answer 1

assuming btn1's name property is btn1 etc and these buttons are on-stage:
import flash.events.MouseEvent;
for(var i:uint=1; i<9; i++){
    this["btn"+i].addEventListener(MouseEvent.CLICK, buttonappear, false, 0, true);
this["image"+i].visible=false;
closebutton.visible=false
closebutton.addEventListener(MouseEvent.CLICK, closeall, false, 0, true);
function closeall(e:MouseEvent):void
        for(var p:uint=1; p<9; p++){
        this["image"+String(p)].visible=false;
        closebutton.visible=false
function buttonappear(e:MouseEvent):void
var i:int = int(e.currentTarget.name.substring(3));
this["image"+i].visible=true;
MovieClip(this["image"+i]).gotoAndPlay(1);
closebutton.visible=true;