Help using return

function touch(xmin,xmax,ymin,ymax)
local b,e,x,y=os.pullEvent("mouse_click")
if e=="mouse_click" then
if b=="1" then
    if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
	   return "c1"
	   elseif
	    e=="mouse_click" then
		  if b=="2" then
		    if x>=xmin and x<=xmax and y>=ymin and y<=ymax then
		    return "c2"
		   else
		   end
		 end
	  end	 
    end
  end
end

function touch(xmin,xmax,ymin,ymax)
  local b,e,x,y=os.pullEvent("mouse_click")
  if e=="mouse_click" then
	if b=="1" then --this will never happen
	  if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
		 return "c1"
	   elseif e=="mouse_click" then
		 if b=="2" then
		   if x>=xmin and x<=xmax and y>=ymin and y<=ymax then
			 return "c2"
		   end
		 end
	  end	  
	end
  end
end

function touch(xmin,xmax,ymin,ymax)
  local e,b,x,y = os.pullEvent("mouse_click") --# switched e and b around... this is why meaningful variable names are important
  if e == "mouse_click" then
    if x >= xmin and x <= xmax and y >= ymin and y <= ymax then --# check where they've clicked
      if b == 1 then --# if it was left mouse click
        return "c1"
      elseif b == 2 then --# if ti was right mouse click
        return "c2"
      end
    end
  end
end

function touch(xmin,xmax,ymin,ymax)
  local e,b,x,y = os.pullEvent("mouse_click")
  if e == "mouse_click" then
    if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
      return "c"..tostring(B)/> --# add the button number to the end of "c", this will work for all mouse buttons now without updating the logic
    end
  end
end

CometWolf, on 01 April 2014 - 07:48 AM said:

Your indentation is way [redacted] up lol.

CometWolf, on 01 April 2014 - 07:48 AM said:

function touch(xmin,xmax,ymin,ymax)
  local b,e,x,y=os.pullEvent("mouse_click")
  if e=="mouse_click" then
	if b=="1" then --this will never happen
	  if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
		 return "c1"
	   elseif e=="mouse_click" then
		 if b=="2" then
		   if x>=xmin and x<=xmax and y>=ymin and y<=ymax then
			 return "c2"
		   end
		 end
	  end	  
	end
  end
end

Your indentation is way [messed] up lol. Anyways the first return value from os.pullEvent is the event name. In this case, that would be "mouse_click", not 1 :P/>/>

if x == true then
    print("This part works!")
else
    print("This other part also works!")
end

Jiloacom, on 01 April 2014 - 04:01 PM said:

Also, for future reference, I suggest adding a print statement on all of your conditionals for debugging purposes, such as this.

if x == true then
	print("This part works!")
else
	print("This other part also works!")
end

For instance, in your scenario, it was never reaching that point in your code, and with this tip, you would have been able to figure out the problem faster. ;)/>/>

function touch(xmin,xmax,ymin,ymax)
local e,b,x,y=os.pullEvent("mouse_click")
if e=="mouse_click" then
if b=="1" then
	if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
	   return "c1"
	   elseif
		e=="mouse_click" then
		  if b=="2" then
			if x>=xmin and x<=xmax and y>=ymin and y<=ymax then
			return "c2"
		   else
		   end
		 end
	  end	
	end
  end
end

if b=="1" then

if b==1 then

Chain99, on 02 April 2014 - 06:28 AM said:

-snip-

theoriginalbit, on 01 April 2014 - 08:50 AM said:

here is a slightly cleaner approach to your problem, note the cleaner conditionals for the if statements

function touch(xmin,xmax,ymin,ymax)
  local e,b,x,y = os.pullEvent("mouse_click") --# switched e and b around... this is why meaningful variable names are important
  if e == "mouse_click" then
	if x >= xmin and x <= xmax and y >= ymin and y <= ymax then --# check where they've clicked
	  if b == 1 then --# if it was left mouse click
		return "c1"
	  elseif b == 2 then --# if ti was right mouse click
		return "c2"
	  end
	end
  end
end

however it could be further improved like so

function touch(xmin,xmax,ymin,ymax)
  local e,b,x,y = os.pullEvent("mouse_click")
  if e == "mouse_click" then
	if x >= xmin and x <= xmax and y >= ymin and y <= ymax then
	  return "c"..tostring(B)/>/> --# add the button number to the end of "c", this will work for all mouse buttons now without updating the logic
	end
  end
end

as the comment in the code says, all mouse buttons will work without having to update the logic in the function

Bomb Bloke, on 02 April 2014 - 08:16 AM said:

if b=="1" then

Mouse click events return the buttons used as "number"-types, not as strings. Try:

if b==1 then

Same fix goes for when you check for a right-click.

You can also remove the second check for a mouse click event (use a straight up "else" instead of an "elseif"), as by that point in the code it's already been established what sort of event you've got. Finally, you'll want to shuffle your checks around a bit - currently you only check to see if that right button is used, if the left button has already been identified as used… That's a circumstance that'll never occur.