OOP Practice Help

[Yarik_superpro (IMoggedOOP)]

If you don’t like inheritance, then you should use another type of OOP because metatable OOP is for inheritance.
C style OOP could also work, or you may use closures if you are paranoid
Metatable OOP is good if you need to store a lot of methods but is very bad when it comes to index speed.

@alerght_55

I tried to go for a composition style rather than inheritance, thinking that a light has a proximity prompt. Don’t know if it was the best approach though.

What other things could I try to write using OOP to practice more?

In light version you dont need any of OOP becouse you are not storing anythink also i think its a recursive

@alerght_55

Light module:

local Proximity = require(script.Parent.Proximity)

local Light = {}
Light.__indes = Light

local changes = {
	Material = Enum.Material.Neon,
}

function Light.new(object)
	local self = setmetatable({}, Light)
	
	self.Prompt = Proximity.new(1)
	
	local int = Instance.new("ProximityPrompt")
	int.HoldDuration = self.Prompt.HoldDuration
	int.ActionText = self.Prompt.ActionText
	int.Parent = object
	
	int.Triggered:Connect(function(player)
		Proximity.Activate(object, changes, player)
	end)
	
end

return Light

Also its generally better to set metatable in the end of a constructor if you are going for metatable OOP:

local function Module.new()
local self = {}

return setmetatable(self,Module)
end