Hello all,Please Help me!....How to remove Graphics in Corona SDK? Cannot Remove Graphics in Corona SDK

apple_id7 · May 29, 2013, 10:32am

Hi all,

I would like to know how to remove graphics when I press on a button.

When I press “My Settings” Button,I want to remove “Select Player” graphics at the right hand side panel.When I press “Select Player”,I want to remove “My Settings” graphics at the right hand side panel.
…But I cannot remove object like this…

My code (in order to remove graphics) are as follows,

local function spButtonEvent( event )

local btn = event.target

if event.phase == “press” then

transition.to(panel, { time=400, rotation=360})

if panelWelcome ~= nil then

panelWelcome:removeSelf()

panelWelcome = nil

if animation1~= nil then

animation1:removeSelf()

animation1 = nil

end

if animation2~= nil then

animation2:removeSelf()

animation2 = nil

end

if animation3~= nil then

animation3:removeSelf()

animation3 = nil

end

if audioSettingTitle~=nil then

audioSettingTitle:removeSelf()

audioSettingTitle = nil

end

if voiceSlider~=nil then

voiceSlider.isVisible = false

voiceSlider= nil

end

if soundEffectsSlider~=nil then

soundEffectsSlider:removeSelf()

soundEffectsSlider = nil

end

if backgroundmusicSlider~= nil then

backgroundmusicSlider:removeSelf()

backgroundmusicSlider = nil

end

if fbLogOnButton ~= nil then

fbLogOnButton:removeSelf()

fbLogOnButton = nil

end

if pwdOnButton ~= nil then

pwdOnButton:removeSelf()

pwdOnButton = nil

end

if editMyProfileButtonSheet~= nil then

editMyProfileButton.isVisible = false

end

if adjustYourSettingMascot~= nil then

adjustYourSettingMascot:removeSelf()

adjustYourSettingMascot = nil

end

local spButton = widget.newButton{

id = “selectPlayerButton”,

width = 132*Xa, height = 164*Ya,

onEvent = spButtonEvent

}

spButton.x = displayOffsetX(565,offsetx)

spButton.y = displayOffsetY(500,offsety1,offsety2)

spButton.alpha = 0.01

May I know how to remove graphics properly in Corona SDK?..Please see the attached images.Please guide me…Thanks a lot!!!..

Warm Wishes,

John

torbenratzlaff · May 29, 2013, 10:54am

I think it’s pretty unefficient to create and remove the objects every time.

Just create a display group for every menu. When you press the button,

simply use the group.isVisible = false or group.isVisible = true propertie.

apple_id7 · May 29, 2013, 11:08am

Hi

@torbenratzlaff,

I will try as you mention.Thanks…

rob · May 29, 2013, 10:20pm

Please post your code using the and tags (no space) to make your code more readable.

apple_id7 · May 31, 2013, 10:39am

Hi all,

I would like to know how to remove group graphics.I tried to remove display group using isVisible = false or removeSelf() but I am not able to remove and not visible.

When I press “My Settings” Button,I want to remove “Select Player” display group at the right hand side panel and then display the My Settings Display Group at the right hand side panel.When I press “Select Player”,I want to remove “My Settings” display group at the right hand side panel and then display “Select Player” display groups again.
But I cannot remove and not visible like this…

I have attached my code as below,

local function spButtonEvent( event ) local btn = event.target &nbsp; if event.phase == "press" then &nbsp; transition.to(panel, { time=400, rotation=360}) &nbsp; if panelWelcome ~= nil then&nbsp; panelWelcome:removeSelf() panelWelcome = nil &nbsp;end if animation1~= nil then animation1:removeSelf() animation1 = nil end &nbsp; &nbsp;if animation2~= nil then animation2:removeSelf() animation2 = nil end &nbsp; if animation3~= nil then animation3:removeSelf() animation3 = nil end &nbsp; if audioSettingTitle~=nil then audioSettingTitle:removeSelf() audioSettingTitle = nil end &nbsp; if voiceSlider~=nil then voiceSlider.isVisible = false voiceSlider= nil end &nbsp; if soundEffectsSlider~=nil then soundEffectsSlider:removeSelf() soundEffectsSlider = nil end &nbsp; &nbsp; if backgroundmusicSlider~= nil then backgroundmusicSlider:removeSelf() backgroundmusicSlider = nil end &nbsp; if fbLogOnButton ~= nil then fbLogOnButton:removeSelf() fbLogOnButton = nil end &nbsp; if pwdOnButton ~= nil then pwdOnButton:removeSelf() pwdOnButton = nil end &nbsp; if editMyProfileButtonSheet~= nil then editMyProfileButton.isVisible = false end &nbsp; if adjustYourSettingMascot~= nil then adjustYourSettingMascot:removeSelf() adjustYourSettingMascot = nil end local spButton = widget.newButton{ id = "selectPlayerButton", &nbsp; width = 132\*Xa, height = 164\*Ya, onEvent = spButtonEvent } &nbsp;spButton.x = displayOffsetX(565,offsetx) &nbsp; spButton.y = displayOffsetY(500,offsety1,offsety2) spButton.alpha = 0.01

Your advise will be appreciate.Thanks a lot!!!..

Warm Wishes,

John

rgame · May 31, 2013, 2:51pm

@apple

a few things: using multiple if statements like this will slow down your program, I am assuming all that code is for the current scene and only one of those statements ever gets executed at a single time.

&nbsp; if voiceSlider~=nil then voiceSlider.isVisible = false voiceSlider= nil elseif soundEffectsSlider~=nil then soundEffectsSlider:removeSelf() soundEffectsSlider = nil end &nbsp;

that’s just an example I made using a block of your code, hopefully I am not crazy when saying that it’s “better” than what you currently have.

Also there’s a lot of code duplication when it comes to removing an object (though you’re doing it correctly, minimizing memory leaks)

you should take all that code that removes an object and place it inside a function to make it look neater

&nbsp; local function removeDisplayObject ( objectToRemove ) &nbsp; &nbsp; &nbsp;display.remove( objectToRemove ) &nbsp; &nbsp;objectToRemove = nil &nbsp; end

when using removeSelf, it doesn’t do a check to see if the object is already nil, so display.remove is the better choice here.

So now your code would look like

if&nbsp;voiceSlider~=nil&nbsp;then &nbsp;removeDisplayObject(voiceSlider) elseif&nbsp;soundEffectsSlider~=nil&nbsp;then &nbsp;removeDisplayObject(soundEffectSlider) end

Now to answer your question, if you placed things in a display group

group:remove(PlayerSettings)&nbsp; PlayerSettings = nil

should solve your problem, if you placed things into the appropriate group

torbenratzlaff · May 29, 2013, 10:54am

I think it’s pretty unefficient to create and remove the objects every time.

Just create a display group for every menu. When you press the button,

simply use the group.isVisible = false or group.isVisible = true propertie.

apple_id7 · May 29, 2013, 11:08am

Hi

@torbenratzlaff,

I will try as you mention.Thanks…

rob · May 29, 2013, 10:20pm

Please post your code using the and tags (no space) to make your code more readable.

apple_id7 · May 31, 2013, 10:39am

Hi all,

I would like to know how to remove group graphics.I tried to remove display group using isVisible = false or removeSelf() but I am not able to remove and not visible.

When I press “My Settings” Button,I want to remove “Select Player” display group at the right hand side panel and then display the My Settings Display Group at the right hand side panel.When I press “Select Player”,I want to remove “My Settings” display group at the right hand side panel and then display “Select Player” display groups again.
But I cannot remove and not visible like this…

I have attached my code as below,

local function spButtonEvent( event ) local btn = event.target &nbsp; if event.phase == "press" then &nbsp; transition.to(panel, { time=400, rotation=360}) &nbsp; if panelWelcome ~= nil then&nbsp; panelWelcome:removeSelf() panelWelcome = nil &nbsp;end if animation1~= nil then animation1:removeSelf() animation1 = nil end &nbsp; &nbsp;if animation2~= nil then animation2:removeSelf() animation2 = nil end &nbsp; if animation3~= nil then animation3:removeSelf() animation3 = nil end &nbsp; if audioSettingTitle~=nil then audioSettingTitle:removeSelf() audioSettingTitle = nil end &nbsp; if voiceSlider~=nil then voiceSlider.isVisible = false voiceSlider= nil end &nbsp; if soundEffectsSlider~=nil then soundEffectsSlider:removeSelf() soundEffectsSlider = nil end &nbsp; &nbsp; if backgroundmusicSlider~= nil then backgroundmusicSlider:removeSelf() backgroundmusicSlider = nil end &nbsp; if fbLogOnButton ~= nil then fbLogOnButton:removeSelf() fbLogOnButton = nil end &nbsp; if pwdOnButton ~= nil then pwdOnButton:removeSelf() pwdOnButton = nil end &nbsp; if editMyProfileButtonSheet~= nil then editMyProfileButton.isVisible = false end &nbsp; if adjustYourSettingMascot~= nil then adjustYourSettingMascot:removeSelf() adjustYourSettingMascot = nil end local spButton = widget.newButton{ id = "selectPlayerButton", &nbsp; width = 132\*Xa, height = 164\*Ya, onEvent = spButtonEvent } &nbsp;spButton.x = displayOffsetX(565,offsetx) &nbsp; spButton.y = displayOffsetY(500,offsety1,offsety2) spButton.alpha = 0.01

Your advise will be appreciate.Thanks a lot!!!..

Warm Wishes,

John

rgame · May 31, 2013, 2:51pm

@apple

a few things: using multiple if statements like this will slow down your program, I am assuming all that code is for the current scene and only one of those statements ever gets executed at a single time.

&nbsp; if voiceSlider~=nil then voiceSlider.isVisible = false voiceSlider= nil elseif soundEffectsSlider~=nil then soundEffectsSlider:removeSelf() soundEffectsSlider = nil end &nbsp;

that’s just an example I made using a block of your code, hopefully I am not crazy when saying that it’s “better” than what you currently have.

Also there’s a lot of code duplication when it comes to removing an object (though you’re doing it correctly, minimizing memory leaks)

you should take all that code that removes an object and place it inside a function to make it look neater

&nbsp; local function removeDisplayObject ( objectToRemove ) &nbsp; &nbsp; &nbsp;display.remove( objectToRemove ) &nbsp; &nbsp;objectToRemove = nil &nbsp; end

when using removeSelf, it doesn’t do a check to see if the object is already nil, so display.remove is the better choice here.

So now your code would look like

if&nbsp;voiceSlider~=nil&nbsp;then &nbsp;removeDisplayObject(voiceSlider) elseif&nbsp;soundEffectsSlider~=nil&nbsp;then &nbsp;removeDisplayObject(soundEffectSlider) end

Now to answer your question, if you placed things in a display group

group:remove(PlayerSettings)&nbsp; PlayerSettings = nil

should solve your problem, if you placed things into the appropriate group