基于共享内存的聊天应用程序中的问题

时间:2012-05-03 10:30:47

标签: c++ linux memory shared-memory shared

我正在尝试构建一个具有两个使用共享内存来交换消息的进程的应用程序... 正如你所看到的那样,我正在请求共享内存,然后在其中放入一个结构。 struct由一个字符串,Bool标志和一个枚举值组成。 字符串应该保存消息,标志应该告诉对方是否已经看到此消息(因为如果内存中有未读消息,则不允许任何人添加消息) 我遇到了几个问题 1-我无法到达字符串处的字符串.... 2-当我用int替换字符串时,我在尝试到达内存时遇到客户端问题 这是代码......

服务器方:

#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <iostream>
#include <string>
#include <sys/wait.h>
using namespace std;
enum e {Server,Client};

struct chat             // struct that will reside in the memory
{


    bool ifread;        //boolian variable to determin if the message has been raed or not
    e who;
    char message[50];
    int msgl;


};

int main(void)
{
string temp;
int shmid;
//key_t key=ftok(".",'a');
key_t key=433;

if ((shmid = shmget(key, sizeof(chat), IPC_CREAT | 0666)) < 0)
{
cout<<"shmget"<<endl;
return(1);
}

chat *str ;
str = (chat *)shmat(shmid, NULL, 0);

pid_t pid;
pid=fork();
str->ifread==true;

str->who=Server;
if(pid==0)
{

    while(temp!="bye")
    {
        if(str->ifread==false && str->who==Client)
        {
            //cout<<"Client said : ";

            for(int i=0;i<str->msgl;i++)
            cout<<str->message[i];
    str->ifread==true;

        }

    }

}

else if (pid>0)
{
    while(temp!="bye")
    {
    getline(cin,temp);
    str->msgl=temp.length();
    if(str->ifread)
     {
        str->who=Server;
        for(int i=0;i<str->msgl;i++)
         {
            str->message[i]=temp.at(i);
         }

    str->who=Server;
    str->ifread==false;
      }

    else if (!str->ifread && str->who==Client)
       {
    sleep(1);
    waitpid(pid,NULL,0);
        }
}

}

shmctl (shmid, IPC_RMID, NULL);

}

客户方:

#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <iostream>
#include <string>
#include <sys/wait.h>
using namespace std;
enum e {Server,Client};

struct chat             // struct that will reside in the memory
{


    bool ifread;        //boolian variable to determin if the message has been raed or not
    e who;
    char message[50];
    int msgl;


};

int main(void)
{
string temp;
int shmid;
//key_t key=ftok(".",'a');
key_t key=433;

if ((shmid = s`hmget(key, sizeof(chat),  0666)) < 0)
{
cout<<"shmget"<<endl;
return(1);
}

chat *str ;
str = (chat *)shmat(shmid, NULL, 0);

pid_t pid;
pid=fork();


if(pid==0)
{

    while(temp!="bye")
    {
        if(str->ifread==false && str->who==Server)
        {
            //cout<<"Server said : ";

            for(int i=0;i<str->msgl;i++)
            cout<<str->message[i];
    str->ifread==true;

        }

    }

}

else if (pid>0)
{
    while(temp!="bye")
    {
    getline(cin,temp);
    str->msgl=temp.length();
    if(str->ifread)
     {
        str->who=Client;
        for(int i=0;i<str->msgl;i++)
         {
            str->message[i]=temp.at(i);
         }
    str->ifread=false;

      }

    else if (!str->ifread && str->who==Server)
       {
    sleep(1);
    //waitpid(pid,NULL,0);
        }
}

}

shmctl (shmid, IPC_RMID, NULL);

}

提前致谢,对不好的英语抱歉.....

编辑: 感谢aix,但还有另一个问题,即客户端首先无法从共享内存中获取任何数据,即使我使用int x在它们之间交换数字 客户端首先保持给0,即使服务器已经放置了另一个值,过了一会儿它在调用shmget()时开始给我一个错误;

编辑(2):我做了一些改动,但它仍然没有用......

编辑(3):解决问题谢谢大家,原来对标志的排序不好 再次感谢......

2 个答案:

答案 0 :(得分:4)

主要问题是封底std::string使用堆分配。

这意味着

struct chat
{
   string  letter;

不会将整个字符串放入结构中。它放置字符串 object ,但不一定是与字符串关联的字符数据

解决此问题的一种方法是将std::string替换为char[N]

通常,在将共享内存与涉及指针或引用的任何内容一起使用时应格外小心。

此外,为避免构造/破坏的复杂化,最好确保chatPOD

答案 1 :(得分:1)

我建议使用现有的库,而不是从头开始重新创建所有内容。

例如:Boost.Interprocess提供Message Queue

请注意,队列中的数据应为POD或使用interprocess allocation mechanisms。如果可能的话,我会建议序列化你的结构,然后再将它们写在共享内存中。以这种方式处理向后和向前兼容性更简单。