Scripting Helpers is winding down operations and is now read-only. More info→

Please help me why this script dosent works? There is a string value that gets somehow the plr name. [closed]

Asked by

5 years ago

    local green = game.Workspace.green:GetChildren()

    function onClicked(plr) -- On click get player
        if green.PLRNAME == plr.Name.Value then
        script.Parent.Parent:Destroy()
    end
    end


    script.Parent.ClickDetector.MouseClick:Connect(onClicked)

can you explain this more clearly? what is green? magentacrasyguy -7 — 5y

green is a door where the string value that gets the player is. hvcu 3 — 5y

Closed as Too Broad by User#24403, ee0w, TheluaBanana, and evaera

This question has been closed because it is too broad and is generally unanswerable. Please ask a more specific question.

Why was this question closed?

2 answers

Answered by

TheluaBanana 946

5 years ago

Edited 5 years ago

ok so what im guessing is either

local green = game.Workspace.green -- i altered this part a bit

function onClicked(plr) -- On click get player
    if green.PLRNAME == plr.Name.Value then
    script.Parent.Parent:Destroy()
end
end


script.Parent.ClickDetector.MouseClick:Connect(function(plr) -- i also altered this part
    onClicked(plr)
end)

or that

local green = game.Workspace.green -- i altered this part a bit

function onClicked(plr) -- On click get player
    if green.PLRNAME == plr.Name then -- i also altered this part a bit
    script.Parent.Parent:Destroy()
end
end


script.Parent.ClickDetector.MouseClick:Connect(function(plr) -- i also also altered this part
    onClicked(plr)
end)

i apologize if this dusnt work ur question is also too broad btw

there is something wrong. The local green = game.Workspace.green:GetChildren() is very correct if you change it the script will fail. hvcu 3 — 5y

but it also fails anyways if u dont change it right? Logistically that would not prove its wrong it would simply not confirm that its correct TheluaBanana 946 — 5y

no. the problem is: The string is placed in a part. green is a model, GREEN is the acctual part. But after player goes in the door, the GREEN changes to player name, also the PLRNAME hvcu 3 — 5y

im certain GetChildren() is very wrong a simple way to test this theory is to just define PLRNAME by itself. Now after u said that, i am unsure what u r doing TheluaBanana 946 — 5y

View all comments (57 more)

thats why i put get child hvcu 3 — 5y

which line TheluaBanana 946 — 5y

also i have tested the probability of GetChildren() by defining a part in the workspace via TheluaBanana 946 — 5y

local part = workspace:GetChildren().Part; print(part) TheluaBanana 946 — 5y

which showed no success TheluaBanana 946 — 5y

so it is technically proven TheluaBanana 946 — 5y

also u do realize im spending a lot of time trying to correct u on this simple point TheluaBanana 946 — 5y

oh look at the model hvcu 3 — 5y

no offense meant but ur point is practically like trying to say that the earth is flat; unless if u do believe it is which in that case i have no words TheluaBanana 946 — 5y

green(model) - GREEN (brick) - PLRNAME (Stringvalue) and The script that gets the name. hvcu 3 — 5y

@hvcu i have tested that too and it also yielded the same results TheluaBanana 946 — 5y

so if u dont get this problem fixed we will not get answer TheluaBanana 946 — 5y

also i still dont understand ur purpose of this contraption TheluaBanana 946 — 5y

oh looks like i have to test. hvcu 3 — 5y

also where is the script located at? TheluaBanana 946 — 5y

is located in a brick that is located in a model hvcu 3 — 5y

how i can get a model's part without the exact name because THE NAME OF MODEL CHILD CHANGES maybe this will be the answer hvcu 3 — 5y

u could put the script in the model's part TheluaBanana 946 — 5y

it also gives me no error hvcu 3 — 5y

also that dusnt change a thing cus ur script also dusnt address that issue. Also also, u only need the name of the model before it changes. TheluaBanana 946 — 5y

@hvcu yes it does not yield error it yields "nil" if u attempt to print the part's name, which is basically means the script is unable to locate what u want TheluaBanana 946 — 5y

if i got backup from the community would u belive me? TheluaBanana 946 — 5y

dude im trying to help u here TheluaBanana 946 — 5y

wait a minute, isnt PLRNAME a string value and it should be PLRNAME.Value? hvcu 3 — 5y

correct that was what i said TheluaBanana 946 — 5y

well, not exactly, but its kinda TheluaBanana 946 — 5y

i made it more clear, now GREEN will remain GREEN and another thing will change name but why it still DOSENT WORK??? hvcu 3 — 5y

also i cant think of an idea o what ur gonna use this for TheluaBanana 946 — 5y

because green:GetChildren y wont u believe me dude TheluaBanana 946 — 5y

umm i am making a tycoon. Im making this for the new things to buy, because other players can click on it. Thats why i am trying to make. hvcu 3 — 5y

uhhhh u know that is a completely wrong strategy to do thing right TheluaBanana 946 — 5y

ok. . . hvcu 3 — 5y

this can be resolved very easily. TheluaBanana 946 — 5y

ok then how hvcu 3 — 5y

also im certain it will work if u dont use :GetChildren() TheluaBanana 946 — 5y

ok ill edit my answer gimme a minute TheluaBanana 946 — 5y

i removed it but still dosent work... hvcu 3 — 5y

cus there is more then one thing wrong here TheluaBanana 946 — 5y

im gonna test hvcu 3 — 5y

also have u not heard of the scientific method? TheluaBanana 946 — 5y

scientific method ... no hvcu 3 — 5y

ok that explains everything TheluaBanana 946 — 5y

it works but i think it still can be clicked by more persons... hvcu 3 — 5y

well is it disabled? TheluaBanana 946 — 5y

ok look very easy: make a whitelisted table; anyone thats whitelisted is capable of buying stuff TheluaBanana 946 — 5y

and set a limit to said table TheluaBanana 946 — 5y

it was but yestarday i was testing with my friend and when he clicked it worked for him (but i was owning the tycoon) hvcu 3 — 5y

ok look team create me a baseplate ill show u de wae TheluaBanana 946 — 5y

cuz your the best im 80% sure that i fixed it thanks hvcu 3 — 5y

its also a possibility ur friend sabotaged the script before he left to prevent u from learning the code scene as its ur game; leaving u to beleive false assumptions ultimately beleiveing GetChildren() which is actually just a table returning all the things in there TheluaBanana 946 — 5y

and @hvcu at this said time im actually probably am TheluaBanana 946 — 5y

this cant be real because 1 he dosent even know how to change roblox settings 2 he is my best friend . One more thing: Can i do if ... then and again if ... then? hvcu 3 — 5y

no u do for i, v in pairs(table) do TheluaBanana 946 — 5y

thanks hvcu 3 — 5y

gd it ill just give u an example of a table ill edit the answer, u know what nvm im guessin what i just said worked? TheluaBanana 946 — 5y

worked!!!!! hvcu 3 — 5y

xd ok TheluaBanana 946 — 5y

Answered by

hvcu 3

5 years ago

Edited 5 years ago

This was succesfully fixed:


    local green = game.Workspace.green 

    function onClicked(plr) -- On click get player
        if green.GREEN.PLRNAME.Value == plr.Name and plr.leaderstats.Money.Value >=50 and plr.leaderstats.Machines.Value == 5 then 
        script.Parent.Parent:Destroy()
    end
    end


    script.Parent.ClickDetector.MouseClick:Connect(function(plr) 
        onClicked(plr)
    end)

Worked for me!!!

: ) hvcu 3 — 5y