我一周前开始了解并使用信号量和共享内存,并且实际上创建了这个程序,问题是我无法找到它有什么问题我一直在看它几个小时,一切似乎都正确..代码编译和我可以创建构建但是当我执行它时没有任何事情发生......
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <stdio.h>
#include <sys/fcntl.h>
#include <semaphore.h>
#define MAXCHILDS 4
#define MAX_SIZE 10
#define MAX_WRITES 4
typedef struct{
int m[MAX_SIZE][MAX_SIZE];
} matrix;
/*fork variables*/
pid_t child[MAXCHILDS];
/*semphores variables */
sem_t *empty, *full, * mutex;
/*share memory id*/
int shmid;
/*shared memory array pointer */
matrix * sh_mem;
/*pointer to matrix*/
int **p;
void init(){
/*create pointer to matrix*/
p = &sh_mem->m;
/*semaphores unlink and creation */
sem_unlink("EMPTY");
empty=sem_open("EMPTY",O_CREAT|O_EXCL,0700,MAX_WRITES);
sem_unlink("FULL");
full=sem_open("FULL",O_CREAT|O_EXCL,0700,0);
sem_unlink("MUTEX");
mutex=sem_open("MUTEX",O_CREAT|O_EXCL,0700,1);
/*initialize shared memory */
shmid = shmget(IPC_PRIVATE,sizeof(matrix),IPC_CREAT|0777);
/*map shared memory*/
sh_mem = (matrix*)shmat(shmid,NULL,0);
if(sh_mem== (matrix*)(-1)){
perror("shmat");
}
}
void writer(int ** m){
int i,k;
for(i = 0;i<MAX_SIZE;i++){
for(k= 0;k<MAX_SIZE;k++){
m[i][k] = 0;
}
}
}
void reader(int **m){
int i = 0;
int k = 0;
for(i = 0;i<MAX_SIZE;i++){
for(k= 0;k<MAX_SIZE;k++){
printf(m[i][k]);
}
printf("\n");
}
}
void terminate() {
sem_close(empty);
sem_close(full);
sem_close(mutex);
sem_unlink("EMPTY");
sem_unlink("FULL");
sem_unlink("MUTEX");
shmctl(shmid, IPC_RMID, NULL);
}
int main(int argc, char **argv)
{
int i;
init();
for(i = 0;i<MAXCHILDS;i++){
if((child[i]= fork()) < 0) // error occured
{
perror("Fork Failed");
exit(1);
}
if((child[i] =fork())==0){
writer(p);
exit(0);
}
}
/*father*/
reader(p);
wait(NULL);
terminate();
return 0;
}
孩子应该在共享内存中写入矩阵,父亲应该读取共享内存数组并打印矩阵。 你能帮帮我吗?谢谢你的帮助...
答案 0 :(得分:2)
您的代码存在一些问题。
您声明p
变量如下:
int **p;
但是数组的实际类型是int[MAXSIZE][MAXSIZE]
。虽然这些都可以被认为是二维数组,但它们并不是完全相同的类型,并且在内存中不共享相同的结构:
int **
是指向整数的指针。当用作二维数组时,它描述了一个2级数组。第一级(取消引用指针一次)包含一个int *
指针数组,每个指针都指向一维整数数组。int[MAXSIZE][MAXSIZE]
是一个扁平的二维数组。它是指向MAXSIZE*MAXSIZE
整数数组的单个指针。它具有与1维数组int[MAXSIZE*MAXSIZE]
相同的内存结构,即使它提供了使用2级下标的便利。在C中传递指向type[][]
样式数组的指针是非直观的,很难做到正确。我不建议这样做。您可以考虑将指针传递给matrix
类型。仍然可以使用此声明:
typedef int array2d[MAX_SIZE][MAX_SIZE];
array2d *p;
然后您必须将m
和reader
中的writer
参数更改为array2d *m
或int **m
,并将其用作*m
代替仅m
。这变得丑陋。就像我说的那样,请考虑使用matrix
类型。
此问题应该已被编译器捕获。应该给你很多类型不匹配的警告。确保始终使用gcc -Wall
进行编译,目的是让您的程序编译时没有错误且没有警告。
在init()
中,您过早地设置了p
。您应该在分配sh_mem
之后将其设置在函数末尾,而不是在函数的开头。
您fork()
函数中的main()
进程太多,如上面的评论所述。你可能只想每次通过fork()
循环调用for
,而不是两次。
在你继续阅读其内容之前,你不要等到作者完成填写共享内存结构。
请注意,即使您在调用wait(NULL)
之前移动reader()
,这还不够,因为wait(NULL)
只会等待一个子进程完成,而不是全部。
一般情况下,您应该在启用警告的情况下编译程序,注意警告并修复它们,然后再想知道程序可能出现的问题。如果它仍然出现意外情况,您应该在调试器下运行它,使用断点,检查变量,并尝试查看正在发生的事情。使用这两种技术,您可能无需在此处发布就可以解决您的问题。