[FIXED]Whats Wrong with this Function? (Not Error)

Well if I type login("username1", "password1") it returns false when it should return true.

Here is the function code

function login(arg1, arg2)
local arg1, arg2 = nil
local user = { "username1" }
local pass = { "password1" }
if app.debug == true then
  for i, v in ipairs(user) do
   if arg1 == v then
	local valid = true
	local password = pass[i]
	break
   else
	return false
   end
  end
  if arg2 == password then
   return true
  else
   return false
  end
end
end

well, you set arg1 and arg2 to nil, so it will never work
get rid of that line and it will
you dont have to declare variables if they are arguments in a function

well it could be the local arg1,arg2 = nil inside the function

XP beat me to it

Only arg1 is set to nil. @tesla1889

Also, @FreePaidRemake, replace that for loop's return false with some fancy check if a user has been selected, right now it will just run once.

Also, local variables are only visible inside the block they're declared in.

if 5 < 6 then
  local test = "hi"
  print(tostring(test)) -- prints: hi
end
print(tostring(test)) -- prints: nil

Doyle3694, on 06 March 2013 - 08:22 PM said:

Only arg1 is set to nil. @tesla1889
–snip–

if you have more variables than values in parallel assignment, the last variables are set to nil

The linked page tells nothing about such assignment logic. You could very well be right but that page doesn't tell me anything in terms of logic like that

I will not be locking this. We generally don't lock Ask a Pro topics.