✓ ANSWER ACCEPTED

[string.sub help]

Asked Mar 15, 2014365 views2 answers

Can anyone see a problem with what is indexed below? I have made sure the variables are correctly found, I have tested this script in game with a report-error thing, and it doesn't show an error. (Yes, there is more to the script, this is just a piece of it.)

			elseif string.sub(string.lower(msg),1,13) == "openFrontDoor" then
				mdoor.Transparency = 0.4
				mdoor.CanCollide = false
			elseif string.sub(string.lower(msg),1,14) == "closeFrontDoor" then
				mdoor.Transparency = 0
				mdoor.CanCollide = true

AnonymousMar 15, 2014

2 answers

✓ Accepted answer

elseif string.sub(string.lower(msg),1,13) == "openfrontdoor" then
    mdoor.Transparency = 0.4
    mdoor.CanCollide = false
elseif string.sub(string.lower(msg),1,14) == "closefrontdoor" then
    mdoor.Transparency = 0
    mdoor.CanCollide = true

Your script was not working because you were calling string.lower on the message, which makes all letters lowercase, then comparing it to a string that contains capital letters.

AnonymousMar 16, 2014Permalink

2 comments

Thanks!

AnonymousMar 16, 2014#

msg:lower():sub(1,13) is a much shorter way to write the same thing!

AnonymousSep 24, 2014#

There's supposed to be a space in between else and if. I know I didn't fully answer this because I'm new to scripting also, but this is what I know.

else if string.sub(string.lower(msg),1,13) == "openFrontDoor" then
    mdoor.Transparency = 0.4
    mdoor.CanCollide = false
else if string.sub(string.lower(msg),1,14) == "closeFrontDoor" then
    mdoor.Transparency = 0
    mdoor.CanCollide = true

AnonymousMar 15, 2014Permalink

2 comments

That is false. It is elseif, but can also be "else if." As far in this situation, the difference does not matter.

AnonymousMar 15, 2014#

Actually, using else if in this situation is a bad idea and requires more ends.

AnonymousMar 16, 2014#

This discussion is preserved as it was. New answers, comments, and votes are closed.