node remove mq if never registered; refactor.

lichao

2021-04-16 c6964d5af25d4ec7ed9dbe7674dc4e3896b36ead

 src/shm_queue.cpp

@@ -21,6 +21,11 @@
#include <boost/uuid/uuid_generators.hpp>
#include <boost/uuid/uuid_io.hpp>

// TODO !!! add garbage collection, and handshake with gc-center.
// smart pointer will NOT work if user crash.
// problem of process release mq: client query success; server exits; client send.
//  client query success; server exits; client send.

namespace bhome_shm
{
using namespace bhome_msg;
@@ -55,68 +60,52 @@
}

ShmMsgQueue::ShmMsgQueue(ShmType &segment, const int len) :
    ShmMsgQueue(NewId(), segment, len)
    ShmMsgQueue(NewId(), segment, len) {}

ShmMsgQueue::~ShmMsgQueue() {}

bool ShmMsgQueue::Remove(SharedMemory &shm, const MQId &id)
{
   return Super::Remove(shm, MsgQIdToName(id));
}

ShmMsgQueue::~ShmMsgQueue()
ShmMsgQueue::Queue *ShmMsgQueue::FindRemote(SharedMemory &shm, const MQId &remote_id)
{
   Remove();
   return Find(shm, MsgQIdToName(remote_id));
}

bool ShmMsgQueue::Send(SharedMemory &shm, const MQId &remote_id, const MsgI &msg, const int timeout_ms)
bool ShmMsgQueue::Send(SharedMemory &shm, const MQId &remote_id, const MsgI &msg, const int timeout_ms, OnSend const &onsend)
{
   Queue *remote = Find(shm, MsgQIdToName(remote_id));
   return remote && remote->Write(msg, timeout_ms, [](const MsgI &msg) { msg.AddRef(); });
}

// bool ShmMsgQueue::Send(const MQId &remote_id, const MsgI &msg, const int timeout_ms)
// {
//     Queue *remote = Find(MsgQIdToName(remote_id));
//     return remote && remote->Write(msg, timeout_ms, [](const MsgI&msg){msg.AddRef();});
// }

bool ShmMsgQueue::Send(const MQId &remote_id, const BHMsg &data, const int timeout_ms)
{
   MsgI msg;
   if (msg.Make(shm(), data)) {
      if (Send(remote_id, msg, timeout_ms)) {
         return true;
   Queue *remote = FindRemote(shm, remote_id);
   if (remote) {
      if (onsend) {
         return remote->Write(msg, timeout_ms, [&onsend](const MsgI &msg) { onsend(); msg.AddRef(); });
      } else {
         msg.Release(shm());
         return remote->Write(msg, timeout_ms, [](const MsgI &msg) { msg.AddRef(); });
      }
   }
   return false;
}

/*
bool ShmMsgQueue::Send(const MQId &remote_id, const void *data, const size_t size, const int timeout_ms)
{
    // Test shows that in the 2 cases:
    // 1) build msg first, then find remote queue;
    // 2) find remote queue first, then build msg;
    // 1 is about 50% faster than 2, maybe cache related.

    MsgI msg;
    if(msg.BuildRequest(shm(), Id(), data, size)) {
        if(Send(remote_id, msg, timeout_ms)) {
            return true;
        } else {
            msg.Release(shm());
        }
    }
    return false;
}
//*/
bool ShmMsgQueue::Recv(BHMsg &msg, const int timeout_ms)
{
   MsgI imsg;
   if (Read(imsg, timeout_ms)) {
      DEFER1(imsg.Release(shm()););
      return imsg.Unpack(msg);
   } else {
      // SetLestError(eNotFound);
      return false;
   }
}

bool ShmMsgQueue::TrySend(SharedMemory &shm, const MQId &remote_id, const MsgI &msg, OnSend const &onsend)
{
   Queue *remote = FindRemote(shm, remote_id);
   if (remote) {
      if (onsend) {
         return remote->TryWrite(msg, [&onsend](const MsgI &msg) { onsend(); msg.AddRef(); });
      } else {
         return remote->TryWrite(msg, [](const MsgI &msg) { msg.AddRef(); });
      }
   } else {
      // SetLestError(eNotFound);
      return false;
   }
}

// Test shows that in the 2 cases:
// 1) build msg first, then find remote queue;
// 2) find remote queue first, then build msg;
// 1 is about 50% faster than 2, maybe cache related.

} // namespace bhome_shm