I can't figure out what's wrong with this script

Join Date: 2009-05-06
Post Count: 2447

#40293173Monday, January 03, 2011 11:22 PM GMT

Here's the script: while true do wait() Tree = game.Workspace.TreePost Beach = game.Workspace.Beach if Tree.OriginColor.Value == "Dark green" and Beach.OriginColor.Value == "Dark green" then script.Parent.Blue.Value = true script.Parent.Red.Value = false elseif Tree.OriginColor.Value == "Bright red" and Beach.OriginColor.Value == "Bright red" then script.Parent.Blue.Value = false script.Parent.Red.Value = true end end I already fixed all the problems that the output told me about so now I have no idea why it's still not working. Can anybody tell me?

DangCool

Join Date: 2010-04-27
Post Count: 5568

#40293289Monday, January 03, 2011 11:23 PM GMT

[ Content Deleted ]

zephurus

Join Date: 2009-05-06
Post Count: 2447

#40293459Monday, January 03, 2011 11:25 PM GMT

Thanks, but knowing that it doesn't like me doesn't really make it work any better. Having a problem output won't tell me about happens to me all the time too, but usually I can figure it out on my own more easily than this time.

3dGen

Join Date: 2009-10-25
Post Count: 1478

#40293474Monday, January 03, 2011 11:25 PM GMT

I'm assuming "tree" and "beach" are both parts, but what classType is "OriginColor"?

zephurus

Join Date: 2009-05-06
Post Count: 2447

#40293550Monday, January 03, 2011 11:26 PM GMT

Actually "Beach" and "Tree" are spawn locations and "OriginColor" is a BrickColor Value.

3dGen

Join Date: 2009-10-25
Post Count: 1478

#40294818Monday, January 03, 2011 11:39 PM GMT

Haha that explains your problem then. == "brickcolorwhstever" Should be == BrickColor.new("brickcolorwhstever")

zephurus

Join Date: 2009-05-06
Post Count: 2447

#40305512Tuesday, January 04, 2011 1:40 AM GMT

I actually tried that and it didn't work but I'll try again anyway. Thanks though.

BJCarpenter

Join Date: 2008-11-04
Post Count: 4416

#40308772Tuesday, January 04, 2011 2:18 AM GMT

put a print statement, every other line and Output will be back to helping u again. while true do wait() Tree = game.Workspace.TreePost print("Tree = ", Tree) Beach = game.Workspace.Beach print("Conditions = ", (Tree.OriginColor.Value == "Dark green"), (Beach.OriginColor.Value == "Dark green")) if Tree.OriginColor.Value == "Dark green" and Beach.OriginColor.Value == "Dark green" then script.Parent.Blue.Value = true . . . . Print ALLWAYS finds the problem...

HotThoth

Forum Moderator
Join Date: 2010-08-24
Post Count: 1176

#40309296Tuesday, January 04, 2011 2:24 AM GMT

I usually find it easier to compare strings rather than more complicated objects like BrickColors. (Like saying: if Tree.OriginColor.Value.Name == "Dark green" ). And yeah, print statements really are one of the best debugging tools you have.

zephurus

Join Date: 2009-05-06
Post Count: 2447

#40311090Tuesday, January 04, 2011 2:44 AM GMT

Thanks, I'll add the prints to figure it out tomorrow and if I still can't figure it out I'll change the type of value I used for the colors and the scripting that tells it the value to be like you said, HotThoth.