Sending and saving ID's for communication

So i'm creating a set of programs that use computers to control an ICBM bunker. Right now I need to store and send user created ID's to a control panel computer that can store them and call on them for when a user tells a missile to launch. If anyone knows how to link a user created name and send the corresponding computer ID so that when a player calls

 launch <ID or Label>

that the computer can discern the what computer to send the launch command to.
I pushed the code i'm working on here:

https://bitbucket.org/REXMC/ccmissilecontrol/src/25f16cb3835f2763448460ef34a6f19a60a6200a/InDev/?at=develop

Thanks in advance!

Store the ids in a table, indexed by their labels.

tIds = {
  ["derp"] = 20,
  ["derpy"] = 35
}
print(tIds["derpy"])

Prints 35

CometWolf, on 22 January 2014 - 02:49 PM said:

Store the ids in a table, indexed by their labels.

tIds = {
  ["derp"] = 20,
  ["derpy"] = 35
}
print(tIds["derpy"])

Prints 35

Forgive me if i'm being ignorant and not seeing how to integrate the user input aspect and transmitting with modem.transmit()… I need the ids to be sent from multiple different computers then stored with the replyChannel from the modem_message event.

So something like this then?

local tIds = {}
while true do
  local launched = false
  local _e, _side, recChan, resChan, msg = os.pullEvent"modem_message" -- get modem messages
  msg = msg:lower() -- convert to lower case
  for k,v in pairs(tIds) do -- check ids table for received label
    if msg == "launch"..k then --if the label is already stored with a id then
	  modem.transmit(v,v,"LAUNCH YO MISSLE!") -- launch message
	  launched = true
	  break --end loop
    end
  end
  if not launched then
    tIds[msg] = resChan --adds label with id to table
  end
end

CometWolf, on 22 January 2014 - 04:07 PM said:

So something like this then?

local tIds = {}
while true do
  local launched = false
  local _e, _side, recChan, resChan, msg = os.pullEvent"modem_message" -- get modem messages
  msg = msg:lower() -- convert to lower case
  for k,v in pairs(tIds) do -- check ids table for received label
	if msg == "launch"..k then --if the label is already stored with a id then
	  modem.transmit(v,v,"LAUNCH YO MISSLE!") -- launch message
	  launched = true
	  break --end loop
	end
  end
  if not launched then
	tIds[msg] = resChan --adds label with id to table
  end
end

That looks like what i'm looking for… i'm going to have to try that and see if it works
Thanx!

Just a quick question…
what does the k,v do/mean in the code?

k and v is the values returned by the pairs iterator in the for loop. In this case the k is the key of the table and v is the variable held in that key. The loop loops once for eveyr key in the table.

Thank you for all the help! I'm going to add you in the credits for my project.

So… I'm still having problems with my code… I'm not quite sure why… you can get the files here and try them out… when i try and call for the id from the table it returns like this

 table: f79dbie9s 

You realize that this

modem.transmit( sChan, rChannel, "/ tostring(f.readAll()) /" )

Dosen't actually transmit your file contents, right?
How did you try calling the id btw?

CometWolf, on 30 January 2014 - 10:32 AM said:

You realize that this

modem.transmit( sChan, rChannel, "/ tostring(f.readAll()) /" )

Dosen't actually transmit your file contents, right?
How did you try calling the id btw?

Oh… yah… I just put it there as a shot to see if it would…
The id was something like launch1 or something like that…. i wanted it to store the rChannel variable as equal to sending the computer id

That dosen't even make any sense… You tried to call a function within a string.
Just get rid of the quotation marks and the slashes

modem.transmit( sChan, rChannel, tostring(f.readAll()) )

Everytime i try to print out the table once it's recived it does this

table:c645670

So i don't think it is actually pulling anything out of the table…
is there a better way to do this…? without using a table?
I tried using a file… but i couldn't get it working either…

You can't print out a table this way.

try this:

local myTable = {
  key1 = "value1";
  key2="value2";
}
for k,v in pairs(myTable) do
  print(k..":  "..v)
end

or this

for k,v in next,myTable do
  print(k..":  "..v)
end

or this

for i,v in ipairs(myTable) do
  print(i..":  "..v)
end

surferpup, on 03 February 2014 - 12:52 PM said:

You can't print out a table this way.

try this:

local myTable = {
  key1 = "value1";
  key2="value2";
}
for k,v in pairs(myTable) do
  print(k..":  "..v)
end

or this

for k,v in next,myTable do
  print(k..":  "..v)
end

or this

for i,v in ipairs(myTable) do
  print(i..":  "..v)
end

I've been doing the top one… but it doesn't work for some reason…

LuaCrawler, on 05 February 2014 - 10:19 AM said:

I've been doing the top one… but it doesn't work for some reason…

Please post your code. I have tested all three. They work fine.

Here is the repository it is in… feel free to comment on it
https://bitbucket.or...Dev/?at=develop

The directory with the files i'm working on is indev

Right link:
https://bitbucket.org/REXMC/ccb/src/ca22092b11a1efcc5012f6037d45c257aa45af88/InDev/?at=master

That is unhelpful, as I have no where to begin troubleshooting. Please post relevant sections of code for us to comment on this specific issue. I will take a look at the rest if I have time.

As it is you have three lua files – which one is giving you trouble? Where?

Its really all the files… but it think it is from the server
here is the code that doesn't work:

  local tIds = {}
while true do
  local launched = false
  local _e, _side, recChan, resChan, msg = os.pullEvent("modem_message") -- get modem messages
  msg = msg:lower() -- convert to lower case
  for k,v in pairs(tIds) do -- check ids table for received label
        if msg == "launch"..k then --if the label is already stored with a id then
            print("Launch command recieved from:"..resChan)
          modem.transmit(v,v,"launch") -- launch message
          launched = true
          break --end loop
        end
  end
  if not launched then
        msg = tostring(msg)
        tIds[msg] = resChan --adds label with id to table
        print(tIds)
  end
end      

Okay, I think you have a logic issue here.

Pay attention to your table tIds. I don't think it ever gets values.

surferpup, on 05 February 2014 - 10:48 AM said:

Okay, I think you have a logic issue here.

Pay attention to your table tIds. I don't think it ever gets values.

let me run a debug using the textutils.serialize then output it to a file and see…

He's printing the table wrong. Basically he just flat out ignored you lol.
Here's the relevant section with surf's suggestion for printing the table contents.

  if not launched then
	    msg = tostring(msg) -- this is unessacary, as a msg received via rednet is always a string.
	    tIds[msg] = resChan --adds label with id to table
	    for k,v in pairs(tIds) do
		  print(k..": "..v)
	    end
  end

As for the table not ever getting any values, i can't really see why it wouldn't, unless he's not specifying a reply channel.

CometWolf, on 05 February 2014 - 11:37 AM said:

He's printing the table wrong. Basically he just flat out ignored you lol.
Here's the relevant section with surf's suggestion for printing the table contents.

  if not launched then
		msg = tostring(msg) -- this is unessacary, as a msg received via rednet is always a string.
		tIds[msg] = resChan --adds label with id to table
		for k,v in pairs(tIds) do
		  print(k..": "..v)
		end
  end

As for the table not ever getting any values, i can't really see why it wouldn't, unless he's not specifying a reply channel.

I'm specifying a reply channel…it is the channel it recieved the message from… and i used the textutils.serialize() to but the table in a file but it wouldn't output it to a file..

I didn't say you were, i was just saying that was the only case i could think of where no ids would be stored in the table. Again, we can't help you with stuff that dosen't work, unless you actually post it, but forget about that for now and try surf's suggestion.

Ok… i think i have it working now… thanks for the help guys