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

Is the datastore bugging out or is it my script?

Asked by

4 years ago

In leaderstats are 3 intvalues (rocks wood and string) After collecting coins that increases value to the intvalues (in this case I would grab one coin that increases wood and one coin that would increase the rocks value increasing both the rocks and wood to 1000) After collecting these coins and leaving the game everything saves but one problem all values (the string included) somehow copy eachother when I never collected a coin that increases its value the string or the wood or rocks. If I were to collect two coins that increases rocks and make the rock's value 2000 but after I leave and join back it somehow copys its value and both wood and string have 2000. What is the problem?

01game.Players.PlayerAdded:connect(function(p)
02    local WF = Instance.new("Folder")
03    WF.Name = "leaderstats"
04    WF.Parent = p
05 
06    local Rocks = Instance.new("IntValue")
07    Rocks.Parent = WF
08    Rocks.Name = "Rocks"
09    local Wood = Instance.new("IntValue")
10    Wood.Parent = WF
11    Wood.Name = "Wood"
12    local Strings = Instance.new("IntValue")
13    Strings.Parent = WF
14    Strings.Name = "Strings"
15end)
16 
17 
18local players = game:GetService("Players")
19local datastore = game:GetService("DataStoreService")
20local ds1 = datastore:GetDataStore("Materials2")
21 
22players.PlayerAdded:connect(function(player)
23 
24    local folder = player:WaitForChild("leaderstats")
25    folder.Name = "leaderstats"
26    folder.Parent = player
27 
28    local currency1 = player.leaderstats.Rocks
29    currency1.Name = "Rocks"
30    currency1.Parent = player.leaderstats
31    currency1.Value = ds1:GetAsync(player.UserId) or 0
32    ds1:SetAsync(player.UserId, currency1.Value)
33 
34    local currency2 = player.leaderstats.Wood
35    currency2.Name = "Wood"
36    currency2.Parent = player.leaderstats
37    currency2.Value = ds1:GetAsync(player.UserId) or 0
38    ds1:SetAsync(player.UserId, currency2.Value)
39 
40    local currency3 = player.leaderstats.Strings
41    currency3.Name = "Strings"
42    currency3.Parent = player.leaderstats
43    currency3.Value = ds1:GetAsync(player.UserId) or 0
44    ds1:SetAsync(player.UserId, currency3.Value)
45 
46    currency1.Changed:connect(function()
47        ds1:SetAsync(player.UserId, currency1.Value)
48    end)
49    currency2.Changed:connect(function()
50        ds1:SetAsync(player.UserId, currency2.Value)
51    end)
52 
53    currency3.Changed:connect(function()
54        ds1:SetAsync(player.UserId, currency3.Value)
55    end)
56end)
 View all 56 lines...

1 answer

Answered by

JailBreaker_13 350

4 years ago

Edited 4 years ago

You are saving it to the same datastore, you are trying to put the user into one data store 3 times which is not possible. You need to create a new data store for each value, such as, ds1, ds2, ds3.

Here is the fixed code:

01game.Players.PlayerAdded:connect(function(p)
02    local WF = Instance.new("Folder")
03    WF.Name = "leaderstats"
04    WF.Parent = p
05 
06    local Rocks = Instance.new("IntValue")
07    Rocks.Parent = WF
08    Rocks.Name = "Rocks"
09    local Wood = Instance.new("IntValue")
10    Wood.Parent = WF
11    Wood.Name = "Wood"
12    local Strings = Instance.new("IntValue")
13    Strings.Parent = WF
14    Strings.Name = "Strings"
15end)
16 
17 
18local players = game:GetService("Players")
19local datastore = game:GetService("DataStoreService")
20local ds1 = datastore:GetDataStore("Materials1")
21local ds2 = datastore:GetDataStore("Materials2")
22local ds3 = datastore:GetDataStore("Materials3")
23 
24players.PlayerAdded:connect(function(player)
25 
26    local folder = player:WaitForChild("leaderstats")
27    folder.Name = "leaderstats"
28    folder.Parent = player
29 
30    local currency1 = player.leaderstats.Rocks
31    currency1.Name = "Rocks"
32    currency1.Parent = player.leaderstats
33    currency1.Value = ds1:GetAsync(player.UserId) or 0
34    ds1:SetAsync(player.UserId, currency1.Value)
35 
36    local currency2 = player.leaderstats.Wood
37    currency2.Name = "Wood"
38    currency2.Parent = player.leaderstats
39    currency2.Value = ds1:GetAsync(player.UserId) or 0
40    ds2:SetAsync(player.UserId, currency2.Value)
41 
42    local currency3 = player.leaderstats.Strings
43    currency3.Name = "Strings"
44    currency3.Parent = player.leaderstats
45    currency3.Value = ds1:GetAsync(player.UserId) or 0
46    ds3:SetAsync(player.UserId, currency3.Value)
47 
48    currency1.Changed:connect(function()
49        ds1:SetAsync(player.UserId, currency1.Value)
50    end)
51    currency2.Changed:connect(function()
52        ds2:SetAsync(player.UserId, currency2.Value)
53    end)
54 
55    currency3.Changed:connect(function()
56        ds3:SetAsync(player.UserId, currency3.Value)
57    end)
58end)
 View all 58 lines...

Be sure to mark as the answer if this helped.

well it would be better to save it as a table kkfilms_1 68 — 4y

Yes but in his case this is what he was asking. He needed help with his code erroring, not a whole new code. JailBreaker_13 350 — 4y

Is the datastore bugging out or is it my script?

1 answer

Answer this question