My script that is supposed to save the kills stats is not working. What did I do wrong?

local DSService = game:GetService('DataStoreService'):GetDataStore('Pizza')
game.Players.PlayerAdded:connect(function(Player)
	
	local uniquekey = 'id-'..player.userId
	local leaderstats = Instance.new('IntValue',player)
	local savevalue = Instance.new('IntValue')
	leaderstats.Name = 'leaderstats'
	savevalue.Parent = leaderstats
	savevalue.Name = Kills
	
	local GetSaved = DSService:GetAsync(uniquekey)
	if GetSaved then
		savevalue.Value = GetSaved[1]
	else
	local NumbersForSaving = {savevalue.Value}
	DSService:SetAsync(uniquekey,NumbersForSaving)
end		
end)

game.Players.PlayerRemoving:connect(function(player)
	local uniquekey = 'id-'..player.userId
	local Savetable = {player.leaderstats.Kills.Value}
	DSService:SetAsync(uniquekey,Savetable)
	
end)

I made a script that is supposed to save your kills after you leave the game. This is what the script looks like:

What is wrong? If its something simple please change it and comment it. Thanks.

I think the problem is in line 9. The name of "savevalue" was supposed to be a string (word in quotation marks), and you didn't add them for the name. Apart from that, you should now use "UserId" instead of "userId" which was deprecated.

Here is the script:

local DSService = game:GetService('DataStoreService'):GetDataStore('Pizza')
game.Players.PlayerAdded:connect(function(Player)

    local uniquekey = 'id-'..player.UserId -- "userId" is deprecated, use "UserId" instead!
    local leaderstats = Instance.new('IntValue',player)
    local savevalue = Instance.new('IntValue')
    leaderstats.Name = 'leaderstats'
    savevalue.Parent = leaderstats
    savevalue.Name = "Kills" -- Here! You should use ""s for names.

    local GetSaved = DSService:GetAsync(uniquekey)
    if GetSaved then
        savevalue.Value = GetSaved[1]
    else
    local NumbersForSaving = {savevalue.Value}
    DSService:SetAsync(uniquekey,NumbersForSaving)
end     
end)

game.Players.PlayerRemoving:connect(function(player)
    local uniquekey = 'id-'..player.UserId --"userId" is deprecated, use "UserId" instead!
    local Savetable = {player.leaderstats.Kills.Value}
    DSService:SetAsync(uniquekey,Savetable)

end)

If you have any problems, please leave a comment below. Thank you, and I hope this will help.