Avoiding long "if" checks

Help and Feedback Scripting Support
#1

So I just wanted to ask if there is some way (other than using typed Luau) to avoid these long and hard-to-read if checks when having to check the type of a specific data passed by the client.

image

1 Like
#2

You can just split them apart using enter key

#3

The if line can be spread into multiple lines, like ItsMeFelixAccept said.

#4

Got messaged by a friend saying that there is no way to actually shorten these checks other than splitting them apart using enter keys as ItsMeFelixAccept said, guess I’ll have to keep writing those long checks.

#5

A lot of the length comes from you repeating yourself, store Flashligh.Model in a variable.

local model = (Flashlight and Flashlight.Model)

if (typeof(model) == "Instance") and model:IsA("BasePart") and (typeof(newCFrame) == "CFrame") then
end

Assuming you can just exit if the data is invalid, you can write it more like this:

local model = (Flashlight and Flashlight.Model)

if (typeof(model) ~= "Instance") or (not model:IsA("BasePart")) then
	return
end

if typeof(newCFrame) ~= "CFrame" then
	return
end

Splits up the check for each parameter, not the worst thing.

2 Likes