Humanoid is not a valid member of model?

Help and Feedback Scripting Support
#16

Yes ranp1 is an actual random generated person name.

#17

Is this a local or server script?

#18

It is a server script, yeah…

#19

Could we take a look at how hammertable is being generated and what it contains? I believe the problem might be with that.

#20

just remove all your timeouts so you can see where it actually breaks. you are silently catching errors, its horrible :frowning:

you can add them back when you know your controlled conditions work properly

#22 
game.ReplicatedStorage.Hammers.Event:Connect(function(plrtable)
	hammerevent = true
	hammertable = plrtable
	for i,v in pairs(hammertable) do
		print(v)
	end
end)
local amount = 3
local players = {}
function hi(x)
	if amount > 0 then
	amount = amount - 1
	local y = x.Backpack
	local z = game.Lighting["Hammer"] 
	z:Clone().Parent = y  
	table.insert(players, x.Name)
	if amount == 0 then
		game.ReplicatedStorage.Hammers:Fire(players)
	end
	local char = x.Character
	end
end

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

2 seperate scripts.

#23

I have had bad experience with math.random. It generates the same number again and again sometimes for me.

#24

I think the problem lies when I getranplayer() because it gets the person there.

#25

if you use the same seed and restart it each time, that happens

its an equation that cycles through numbers not actually random
seeding with tick() is popular

#26

Can we continue this in PMs? This post is getting unnecessarily long.

#27

Yes, but I don’t think that it is a random error. It is giving me the players names back fine.

#28

@ShutokouBattle

…what? I don’t see how math.random() is easier to use than the Random object. That’s not even the source of OP’s error. This is a fairly pointless suggestion. In addition: if Random fails to get a indice from a table, I don’t see how switching methods would change results.

Posts from the object's author if you're interested in learning about the object. Something you should be searching before making unsubstantiated assertions.

A Random Feature
Random:NextInteger() help?
Weighted crate system not choosing an item of the same rarity as another

@Pharyx_Styx

Not required with the Random object. It pulls a seed from an internal entropy source. A seed is typically for if you need predictable results - you can pass one seed at one moment and the same seed at another moment.

Honest best guess is that there’s a model named the same as a player. How are entries inserted into hammertable? Have you gone over each line of code and debugged it to find a potential source of error?

#29

Interesting, I have never used the Random object before. Just default to math for mathy things. I’ll have to look into that.

Thanks again for all the nifty info :slight_smile: This seems to be happening regularly, so much to learn.

#30

This shouldn’t happen. Unless you are using something like math.random(1,2) then you shouldn’t get the same numbers repeatedly.

#31

I used math.random() for my random npc spawns and they spawned all in the same place. I repeated the math.random() for every npc.

#32

The fact that they can spawn in the same place may have been the problem.

You may have been using math.random wrong. I use math.random a lot and I have never gotten the same result over and over again.

#33 
for i = 1, 100 do local spawn = math.random(1, 100)  end

What is wrong with that?

#34

You would have to spawn then NPC in that loop, or else spawn would equal the last math.random

#35

I did, im just getting the math.random

#37

Yeah, I did not make it a variable.