Selling script not working!

Help and Feedback Scripting Support
, ,
#1

Hey, I’m currently making a simulator game and I’m wondering why my selling script is not working.

The script:

Capture
Capture727×456 14.8 KB

If anyone could help that would be great!

#2

can you show us the explorer? aaaa

2 Likes
#3

Hm, I am not sure what you are doing on line 10. A more clear way to write that would be:
money.Value += selling.Value
Edit: No Idea why you are multiplying by one.

#4

Could you show us the output, if there is any?

#5

Sure, 1 second. I didn’t get the hole explorer last time.

#6

You got it wrong too, it should be money.Value += selling.Value NOT money.Value =+ selling.Value

#7

Can you show us the console? Any errors?

#8

Oops, yeah, it was a typo, I’ll fix that right now.

1 Like
#9

Can I see the script that injects the leaderstats value into the player?

#10

Capture

#11

Why are you using a local script? Change that to a normal one.

#12

It says their’s an error with the expression ‘+=’ in the output.

#13

Can you change the script to be normal instead of local? Local scripts only work under client services i.e. StarterPlayer, StarterGui

#14

Can you show me the script and the console?

#15

change it to a normal script.please

#16

Capture
Capture1020×567 102 KB

Capture
Capture801×417 14.8 KB

#17

That’s not how += works, remove the = sign and the second money argument

#18

Local scripts don’t work in parts. Use a normal script. Plus, in a local script, someone can just exploit and give themself loads of money when selling if it is local.

#19

I have done that. Gotta write more to send

#20

Use this code in a normal script:

-- When a player touches the parent object, their items will be sold for gold
local sellPart = script.Parent

-- Gives the player gold for each item they have
local function sellItems(playerItems, playerGold)
	-- Gives players 10 pieces of gold for each item
	local totalSell = (playerItems.Value * 10)
	playerGold.Value = playerGold.Value + totalSell
	playerItems.Value = 0
end

local function onTouch(partTouched)
	-- Looks for a Humanoid
	local character = partTouched.Parent
	local humanoid = character:FindFirstChildWhichIsA("Humanoid")
	
	-- If a humanoid is found, gets leaderstats and calls sellItems function
	if humanoid then
		-- Get the player, so changes can be made to  the player's leaderstats
		local player = game.Players:GetPlayerFromCharacter(humanoid.Parent)
		local playerStats = player:FindFirstChild("leaderstats")
		local playerItems = playerStats:FindFirstChild("Clicks")
		local playerGold = playerStats:FindFirstChild("Coins")
		-- Sells Items and then changes the player's spaces and money
		sellItems(playerItems, playerGold)
	end
end

sellPart.Touched:Connect(onTouch)
2 Likes