Register forum user name Search FAQ

Gammon Forum

Notice: Any messages purporting to come from this site telling you that your password has expired, or that you need to "verify" your details, making threats, or asking for money, are spam. We do not email users with any such messages. If you have lost your password you can obtain a new one by using the password reset link.
 Entire forum ➜ MUSHclient ➜ Lua ➜ If statement is true, then assign a boolean and other related questions
If statement is true, then assign a boolean and other related questions

It is now over 60 days since the last post. This thread is closed.     Refresh page

Posted by Hidelensman   (10 posts)  Bio
Date Mon 31 Oct 2022 12:28 AM (UTC)

Amended on Mon 31 Oct 2022 12:38 AM (UTC) by Hidelensman

Message
Hi Nick and all,

I only have limited experience with Lua and want to know if the following situation is possible:

If a condition is met (or not met), then assign a boolean value to a variable.

My code as following (purpose is to check if players and mobs are in the set list then assign "fighting" is true/false to determine next action):

 
function check_player()
    local match = nil
    -- pplroom = false

    for k, v in pairs (players) do
      if string.match (Char_Name, k:lower ()) then
        match = k  -- remember match
        Note ("Warning, player " .. match .. " here.")
        pplroom = true
      else
        pplroom = false
      end -- if
    end -- for

end -- function


function check_mob()
    local match = nil

    for k, v in pairs (mobs) do
      if string.match (Char_Name, k:lower ()) then
        match = k  -- remember match
        Execute ("say " .. "Mob " .. match .. " here.")
        mobroom = true
      else
        mobroom = false
      end -- if
    end -- for

end -- function


I tried several times but in vain (the code can be executed while pplroom & mobroom are always false) and didn't find any related topic in the forum, thus I'd like to seek for your advice.

Thanks in advance.

P.S. I have some other questions regarding matching values in Lua table which are related to the code I posted here - will ask questions later.
Top
Posted by Fiendish   USA  (2,533 posts)  Bio   Global Moderator
Date Reply #1 on Mon 31 Oct 2022 03:23 AM (UTC)

Amended on Mon 31 Oct 2022 03:43 AM (UTC) by Fiendish

Message
The problem is that you don't stop looking through your tables when you find a match. Stick a return after both of your true assignments.


Programming advice follows:

Setting a non-local pplroom variable instead of returning pplroom looks like a bad practice given what these functions do. If your "pplroom = true" and "mobroom = true" had instead been "return true", you wouldn't have had this problem.

If you're looking for exact matches, then those comparisons are a poor use for string.match. "Char_Name == k:lower()" would make more sense in that case.

Also if you want exact matches then those loops could be reduced to single statements if the players and mobs tables were lowercased, because then you could just check if e.g. players[Char_name] exists.

If you really don't want exact matches, then ignore those last two.
https://github.com/fiendish/aardwolfclientpackage
Top
Posted by Hidelensman   (10 posts)  Bio
Date Reply #2 on Mon 31 Oct 2022 04:59 AM (UTC)

Amended on Mon 31 Oct 2022 05:00 AM (UTC) by Hidelensman

Message
Thanks Fiendish - I'd like to further consult you on the 1st part.

Fiendish said:

The problem is that you don't stop looking through your tables when you find a match. Stick a return after both of your true assignments.


I probably know where you are coming from but could you give me an example on "stick a return after both of true assignments"?

Fiendish said:

Programming advice follows:

Setting a non-local pplroom variable instead of returning pplroom looks like a bad practice given what these functions do. If your "pplroom = true" and "mobroom = true" had instead been "return true", you wouldn't have had this problem.


Does that mean I need to "return pplroom" / "return mobroom" after checking if the Char_Name in my tables? Tried but pplroom and mobroom are shown as nil.

BTW, I like your suggestion on the exact match and single statement - already made adjustment accordingly. Thanks.
Top
Posted by Nick Gammon   Australia  (23,070 posts)  Bio   Forum Administrator
Date Reply #3 on Mon 31 Oct 2022 05:57 AM (UTC)
Message
Quote:

I probably know where you are coming from but could you give me an example on "stick a return after both of true assignments"?



if string.match (Char_Name, k:lower ()) then
        match = k  -- remember match
        Execute ("say " .. "Mob " .. match .. " here.")
        mobroom = true
        return   --<-- HERE
      else
        mobroom = false
        return   --<-- HERE
      end -- if


But that doesn't totally make sense, because you would return on the first iteration. Probably more like:


function check_mob()
    mobroom = false
    for k, v in pairs (mobs) do
      if string.match (Char_Name, k:lower ()) then
        Execute ("say " .. "Mob " .. match .. " here.")
        mobroom = true
        return
      end -- if
    end -- for

end -- function


And to follow Fiendish's suggestion:


function check_mob()
    for k, v in pairs (mobs) do
      if string.match (Char_Name, k:lower ()) then
        Execute ("say " .. "Mob " .. match .. " here.")
        return true  -- match
      end -- if
    end -- for
    return false  -- no match
end -- function



Then you can just call it like this:


  if check_mob () then
    -- mob found
  else
    -- mob not found
  end -- if
- Nick Gammon

www.gammon.com.au, www.mushclient.com
Top
Posted by Fiendish   USA  (2,533 posts)  Bio   Global Moderator
Date Reply #4 on Mon 31 Oct 2022 01:32 PM (UTC)

Amended on Mon 31 Oct 2022 02:24 PM (UTC) by Fiendish

Message
Nick Gammon said:
But that doesn't totally make sense, because you would return on the first iteration. Probably more like:

Yeah, only after the true assignments, not after the false assignments.


Hidelensman said:

I probably know where you are coming from but could you give me an example on "stick a return after both of true assignments"?
...
Does that mean I need to "return pplroom" / "return mobroom" after checking if the Char_Name in my tables? Tried but pplroom and mobroom are shown as nil.


function check_mob()
   for k, v in pairs(mobs) do
      if Char_Name == k:lower() then
         Execute ("say Mob " .. k .. " here.")
         mobroom = true
         return
      else
         mobroom = false
      end
   end
end


But I favor this version instead:

function check_mob()
   for k, v in pairs(mobs) do
      if Char_Name == k:lower() then
         Execute ("say Mob " .. k .. " here.")
         return true
      end
   end
end


mobroom = check_mob()


Or better yet:

function check_mob(which_mob)
   for k, v in pairs(mobs) do
      if Char_Name:lower() == k:lower() then
         return k
      end
   end
end


mobroom = check_mob()
if mobroom then
   Execute ("say Mob " .. k .. " here.")
end


Or better still because this lets you use only one function instead of two that do the same thing:

function find_in_list(find_what, the_list)
   for k, v in pairs(the_list) do
      if find_what:lower() == k:lower() then
         return k
      end
   end
end


mobroom = find_in_list(Char_Name, mobs)
if mobroom then
   Execute ("say Mob " .. mobroom .. " here.")
else
   pplroom = find_in_list(Char_Name, players)
   if pplroom then
      Note ("Warning, player " .. pplroom .. " here.")
   end
end

I assume there that Char_Name cannot be both a player and a mob.
https://github.com/fiendish/aardwolfclientpackage
Top
Posted by Hidelensman   (10 posts)  Bio
Date Reply #5 on Tue 01 Nov 2022 05:26 AM (UTC)

Amended on Tue 01 Nov 2022 05:31 AM (UTC) by Hidelensman

Message
Wow nice - thanks Nick and Fiendish for these great replies.

Last night I based on Nick's suggestion to re-write the code as below and will try Fiendish's suggestion tonight:


function check_player()

    if players[Char_Name] then
       Note ("Warning, player " .. Char_Name .. " here.")
       return true
    end -- if

    return false

end -- function


function check_mob()

    if mobs[Char_Name] then
       Execute ("say " .. "Mob " .. Char_Name .. " here.")
       return true
    end -- if
    
    return false

end -- function



Another 2 questions want to consult you at this stage -

  1. I use a trigger to activate these functions and check if players/mobs are in the room, then kill the mobs; while if there are many mobs in the same room, the trigger will be fired many times, leading my char easily gets kicked out from the MUD - any good suggestions to only fire trigger once, after a mob killed, then fire the trigger again? I only think of disabling trigger then re-activating it after the mob killed
  2. I understood from other posts that "Simulate" could be used for trigger - Do you have any hints on this? I tried to replace the "say ..." command in my check_mob() function with Simulate but in vain. I know using "Simulate" here might not be ideal but just want to explore more possibilities.


Thanks.
Top
Posted by Fiendish   USA  (2,533 posts)  Bio   Global Moderator
Date Reply #6 on Thu 03 Nov 2022 05:19 AM (UTC)
Message
Quote:
I only think of disabling trigger then re-activating it after the mob killed

It sounds like you have answered your own question then.

Quote:
I understood from other posts that "Simulate" could be used for trigger - Do you have any hints on this?

I don't understand what you're trying to accomplish here.
https://github.com/fiendish/aardwolfclientpackage
Top

The dates and times for posts above are shown in Universal Co-ordinated Time (UTC).

To show them in your local time you can join the forum, and then set the 'time correction' field in your profile to the number of hours difference between your location and UTC time.

8,101 views.

It is now over 60 days since the last post. This thread is closed.     Refresh page

Go to topic:           Search the forum

[Go to top] top

Information and images on this site are licensed under the Creative Commons Attribution 3.0 Australia License unless stated otherwise.

Home