Recursive function not returning anything

Help and Feedback Scripting Support
1

ok let me explain real quick, i have made a recursive function which doenst return anything. i have tried making a variable and then changing the variable with the function but it didnt work

local function getLowestBranch(branch)
		if branch:FindFirstChild("Rodzic") ~= nil then
			local lowerbranch = branch.Rodzic.Value
			getLowestBranch(lowerbranch)
		else
			return branch
		end
	end
1 Like
2

currently it has no ‘return’ function

you may need to add something like

return Table

at the end of it, once it has searched every descendant and added all of its ancestors into the table

3

aw man i have posted wrong function… i have edited and posted the one which doesnt work, my bad sorry lul

1 Like
4

aw man i think you’ve got the problem i had a while ago returning magnitude, it always comes back with ‘nil’ or ‘0’

I fixed it by just sticking it into a module script.

only change you need is to call them

local module = {}

function module.getAllBranchesAbove()
    -- the stuff
end

function module.getLowestBranch()
    -- the stuff again
end

return module

then require that in another script. if it doesnt work let me know, but its the only way i got around it when i started having this problem.

when you want to use a function within the module just do

module.getAllBranchesAbove()
5

hmm sounds promising let me try it out

6

image

7

can u call recursive functions in a module script?

8

image

1 Like
9

try using

module.getLowestBranch()
10

yes im dumb i just noticed lul

1 Like
11

haha! i do that all the time. its easy to forget when youre used to regular scripts.

let me know if it works :smiley:

12

image

13

nope i dont know if i did it right

1 Like
14

does ‘end’ need to be ‘end)’

sometimes functions need that

15

its working but now im having issues with player name

16

nevermind it is not working :frowning:

1 Like
17

im not sure how to fix this, perhaps try a remote function but im not sure it’ll work. sorry! :frowning:

18

Make it return getLowestBranch(lowerbranch) or else the initial call on the function will never return a value. Another way you could do this is by simply using a while loop:

local function getLowestBranch(branch)
    while true do
        if not branch:FindFirstChild("Rodzic") then
            return branch
        else
            branch = branch.Rodzic.Value
        end
    end
end
19

I see a problem here. You forgot to add the function keyword in front of the first line, so the script doesn’t perceive it as a function, which is why it’s confused about that last end

20

Here’s the original

local function getLowestBranch(branch)
	if branch:FindFirstChild("Rodzic") ~= nil then
		local lowerbranch = branch.Rodzic.Value
		getLowestBranch(lowerbranch)
	else
		return branch
	end
end

and here’s the solution. See if you can find what changed

local function getLowestBranch(branch)
	if branch:FindFirstChild("Rodzic") ~= nil then
		local lowerbranch = branch.Rodzic.Value
		return getLowestBranch(lowerbranch)
	else
		return branch
	end
end