0

这是我第一次使用线程,我从一个简单的程序开始。该程序接受n参数并创建n-2线程。问题是我遇到了分段错误,我不知道为什么。

这是代码:

#include <stdio.h>  
#include <string.h> 
#include <pthread.h>
#include <stdlib.h>
#include <unistd.h>

void *
removeBytes (int i, char* argv[])
{
  printf ("%d, %s\n", i, argv[i]);
  return NULL;
}  


int main (int argc, char *argv[])
{

  pthread_t threads[argc - 3];
  int err;
  int i;  
  int *ptr[argc - 3];

  printf ("argc = %d\n", argc);

  for (i = 0; i < argc -3; i++)
    {
      err =
        pthread_create (&(threads[i]), NULL,
                        removeBytes(i+1,&argv[i+1]), NULL);
      if (err != 0)
        {
          printf ("\nCan't create thread: [%d]", i);
        }
      else
        { 
          printf ("\nThread created successfully\n");
        }
    }

  for (i = 0; i < argc - 3; i++)
    {
      pthread_join (threads[i], (void **) &(ptr[i]));
      printf("pthread_join - thread %d",i);
    }

  return 0;
}

示例:我的程序被调用mythread,所以当我运行它时./mythread f1 f2 f3 f4 f5 f6,输出是:

argc = 6

1,f2
Thread created successfully

2,f4
Thread created successfully
3, (null)

为什么需要f2asargv[1]f4as argv[2]

更新:

 typedef struct{
    int i;
    char* argv;
  }Data;

  void* removeBytes(void* arg){
    Data* data = (Data*)arg;   
    printf("%d, %s\n",data->i, data->argv);
    free(data);
    return NULL;
  }



  int main(int argc, char** argv){
    Data* data;

    pthread_t threads[argc-3];

    int i;
    int err;
    for(i=0; i < argc-3;i++){
      data = (Data*)malloc(sizeof(Data));
      data->i=i+1;
      data->argv=argv[i+1];
      err = pthread_create(&(threads[i]),NULL,removeBytes,data);
      if(err != 0){
        printf("\nCan't create thread %d",i);
      }
      else{
        printf("Thread created successfully\n");
      }
    }  

    return 0;
  }

对于 ./mythread f1 f2 f3 f4 f5 f6 f7 f8 输出为:

5 x“线程创建成功”。它不打印 i 或 argvi[i]。

4

4 回答 4

0

你没有pthread_create正确使用:

pthread_create (&(threads[i]), NULL,
                        removeBytes(i+1,&argv[i+1]), NULL);

在这里,您只是调用removeBytes()并将结果 ( NULL) 作为pthread_create().

int pthread_create(pthread_t *thread, const pthread_attr_t *attr,
                          void *(*start_routine) (void *), void *arg);

第三个参数应该是指向void* myThread(void*)函数的指针。如果要将参数传递给线程,则应使用void*参数并将其作为pthread_create.

您可以看一下this以了解如何使用 pthread 库。

此外,您可能想要这样的事情:

typedef struct {
    int i;
    char* argv;
} Data;

void * removeBytes (void* arg)
{
  Data* data = (Data*) arg;
  printf ("%d, %s\n", data->i, data->argv);
  free(data);
  return NULL;
}

然后像这样创建线程:

Data* data = (Data*)malloc(sizeof(Data));
data->i = i;
data->argv = argv[i+1];
err = pthread_create (&(threads[i]), NULL, removeBytes, data);
于 2013-10-31T16:38:50.420 回答
0

pthread_create (&(threads[i]), NULL,
                    removeBytes(i+1,&argv[i+1]), NULL);

您正在调用removeBytes()而不是将其作为参数传递。

此外,您只能将一个参数传递给线程函数。因此,您需要将多个参数放在一个结构中。就像是:

struct thread_args {
    int i;
    char *argv;
}

#your main code
struct thread_args *thargs;

for (i = 0; i < argc -3; i++)
{
  thargs = malloc(sizeof(*thargs));
  thargs->i = i+1;
  thargs->argv = argv[i+1];
  err =
    pthread_create (&(threads[i]), NULL,
                    removeBytes, thargs);
  if (err != 0)
    {
      printf ("\nCan't create thread: [%d]", i);
    }
  else
    { 
      printf ("\nThread created successfully\n");
    }
}
#make sure to free up thargs as well.

并将线程函数更新为

void *removeBytes (void *arg)
{
  int i;
  char *argv;
  struct thread_args *thargs =  (struct thread_args *) arg;
  i = thargs->i;
  argv = thargs->argv;
  printf ("%d, %s\n", i, argv);
  return NULL;
} 
于 2013-10-31T16:39:26.360 回答
0

里面有问题

pthread_create (&(threads[i]), NULL,
                    removeBytes(i+1,&argv[i+1]), NULL);

pthread_createis 的语法

int pthread_create(pthread_t *thread, const pthread_attr_t *attr,
                      void *(*start_routine) (void *), void *arg);

它将启动例程作为回调。在您的情况下,您在生成并返回 NULL 之前的主线程中调用 removeBytes。因此,回调为 NULL。

因此,根据您的需要相应地修改您的 removeBytes 并调用

pthread_create (&(threads[i]), NULL, removeBytes, argv[i+1]);

于 2013-10-31T16:41:01.197 回答
0

它将 f2 作为 argv[1] 和 f4 作为 argv[2] 仅仅是因为当你传递 &argv[i+1] 你实际上传递了指向数组的第 (i+1) 个元素的指针并且你也传递了 i+1作为索引。

因此,如果您首先有 argv 等于 [mythread, f1, f2, f3, f4, f5, f6] 您将在 removeBytes 中得到 [f1, f2, f3, f4, f5, f6]。当你访问 i+1 = 1 元素时,你会得到 f2。下次你会得到 [f2, f3, f4, f5, f6] 和 i+1 = 2 所以你会得到 f4

于 2013-10-31T16:50:31.370 回答