String splitting function returning an empty table

I've been trying to code a simple file format for color files but this function returns an empty table:

function splitAt(chrchtr,strng,strt)
local ch = strt or 1
local parts = {}
local str = ""
local lngth = string.len(strng) - ch + 1
for i = 1,lngth do
  if strng.sub(strng,ch,ch) == chrctr then
   table.insert(parts,str)
   str = ""
  else
   str = str..string.sub(strng,ch,ch)
  end
  ch = ch + 1
end
return parts
end

This is the line that calls the code:

local args = splitAt(",",fnc(This is a function!,lol/test,[]),5)

Your problem is that strng.sub is nil so it's not going to be able to == chrctr try doing either string.sub(strng,ch,ch) or strng:sub(ch,ch)

Dragon53535, on 28 September 2014 - 07:05 AM said:

Your problem is that strng.sub is nil so it's not going to be able to == chrctr try doing either string.sub(strng,ch,ch) or strng:sub(ch,ch)

I did that and I still have the same problem.

As written, both your splitAt function and the line you try to call it from will error out as opposed to generating an empty table. I guess you copied them to the forums manually and made some typos in the process.

Exactly what does your "fnc" function look like?

You could use string.find() to save yourself the trouble of inspecting every character individually:

local function splitAt(searchFor, searchIn, startAt)
	if startAt then searchIn = searchIn:sub(startAt) end
	
	local results = {}
	
	while true do
		local index = searchIn:find(searchFor)
		
		if index then
			if index > 1 then results[#results+1] = searchIn:sub(1,index-1) end
			searchIn = searchIn:sub(index+#searchFor)
		else
			if #searchIn > 0 then results[#results+1] = searchIn end
			return results
		end
	end
end

local myTable = splitAt(",", "r4,r,4t,45t,r34,43",5)

for i=1,#myTable do print(myTable[i]) end

I've a nagging suspicion string.gmatch() could do it better.

Bomb Bloke, on 28 September 2014 - 08:03 AM said:

Exactly what does your "fnc" function look like?

What do you mean?

minecraftwarlock, on 28 September 2014 - 08:08 AM said:

Bomb Bloke, on 28 September 2014 - 08:03 AM said:

Exactly what does your "fnc" function look like?

What do you mean?

He means this

minecraftwarlock, on 28 September 2014 - 07:00 AM said:

This is the line that calls the code:

local args = splitAt(",",fnc(This is a function!,lol/test,[]),5)

See? You're calling the function splitAt with another function as an argument, unless you meant it to be a string, then you need to put quotes around it

local args = splitAt( ",", "fnc(This is a function,lol/test,[])", 5 )

TheOddByte, on 28 September 2014 - 09:18 AM said:

minecraftwarlock, on 28 September 2014 - 08:08 AM said:

Bomb Bloke, on 28 September 2014 - 08:03 AM said:

Exactly what does your "fnc" function look like?

What do you mean?

He means this

minecraftwarlock, on 28 September 2014 - 07:00 AM said:

This is the line that calls the code:

local args = splitAt(",",fnc(This is a function!,lol/test,[]),5)

See? You're calling the function splitAt with another function as an argument, unless you meant it to be a string, then you need to put quotes around it

local args = splitAt( ",", "fnc(This is a function,lol/test,[])", 5 )

I meant for it to be a string also that is just a representation of what's being read from a file.