Forum

> > CS2D > Scripts > string.len(Bad Argument?)
Forums overviewCS2D overview Scripts overviewLog in to reply

English string.len(Bad Argument?)

6 replies
To the start Previous 1 Next To the start

old string.len(Bad Argument?)

Ranu
User Off Offline

Quote
• I used string.len in my script, but cs2d couldn't find it and it said "ERROR" "LUA ERROR: sys/lua/******: bad argument #1 to 'len' (string expected, got nil)

• What is the problem?
edited 1×, last 02.01.19 04:08:23 pm

old Re: string.len(Bad Argument?)

Ranu
User Off Offline

Quote
1
2
3
4
5
6
7
8
9
10
11
12
13
function Color(r,g,b)
	if not r then r = 0 end
	if not g then g = 0 end
	if not b then b = 0 end
	local r,g,b = tostring(r),tostring(g),tostring(b)
	while string.len(r) < 3 do r = "0"..r end
	while string.len(g) < 3 do g = "0"..g end
	while string.len(b) < 3 do b = "0"..b end
	if string.len(r) == 3 and string.len(g) == 3 and string.len(b) == 3 then
		return "\169"..r..g..b
	end
	return "\169255255255"
end

old Re: string.len(Bad Argument?)

DC
Admin Off Offline

Quote
You left out important info: The line number (the part behind the ":"). Knowing the line would make it much easier to identify the problem.

The function runs fine in the Lua demo web console https://www.lua.org/cgi-bin/demo
(I added a
print(Color(1,2,3))
line for testing, even
print(Color())
works without error)

Are you sure that the error occurs in the snippet you showed here?

btw: The "local"-statement is misleading/useless. r, g & b are already local variables because they are passed in as function parameters. Therefore you can remove the "local"-statement (in more strict languages keeping it there would lead to an error).

old Re: string.len(Bad Argument?)

Vehk
User Off Offline

Quote
You're probably passing a bad value to the color function (although tostring should always return a string), or the problem is elsewhere in your code.

Show us the entire error message and script. I do not thing the error is raised here.

It looks like there is nothing that can be passed to tostring that would result in it returning nil, so the error must be raised somewhere else.

1
2
3
4
if string.len(r) == 3 and string.len(g) == 3 and string.len(b) == 3 then
          return "\169"..r..g..b
     end
     return "\169255255255"

The second return statement is never reached because you've made sure that r, g, and b are length 3. So theres no need for it.

1
local r,g,b = tostring(r),tostring(g),tostring(b)

As DC said this is redundant. And I'd like to point out that there is no need to use tostring here, the numbers will be automatically coerced into strings as needed.

1
2
3
4
5
6
while string.len(r) < 3 do r = "0"..r end
     while string.len(g) < 3 do g = "0"..g end
     while string.len(b) < 3 do b = "0"..b end
     if string.len(r) == 3 and string.len(g) == 3 and string.len(b) == 3 then
          return "\169"..r..g..b
     end

This is a lot of work. Check out string.format, it'll simplify this.

Here's a simplified version of this function using string.format
1
2
3
function Color(r, g, b)
	return string.format("\169%0.3u%0.3u%0.3u", r or 0, g or 0, b or 0)
end
edited 1×, last 06.01.19 01:07:08 am
To the start Previous 1 Next To the start
Log in to reply Scripts overviewCS2D overviewForums overview