質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

0回答

408閲覧

マルチスレッドで素数の数を見つけ出すプログラムを書きたい

oksinamon

総合スコア0

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2020/07/08 13:22

前提・実現したいこと

マルチスレッドC言語で決められた数字から素数を見つけ出して、素数なら1、そうでなければ0を配列に格納して、最終的に配列の中にある1のみを出力するプログラムを書いています。
自分では動くと思ったのですが以下の様なエラーメッセージが出てしまい動きません。
}の数が合わずにエラーが出ているのかと思い試してみたのですがどうやら違う様でした。

発生している問題・エラーメッセージ

Loop will run at most once (loop increment never executed)
Control may reach end of non-void function

該当のソースコード

c
ソースコード

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

#define MAX_NUM 100//素数を探したい数
#define THREAD_NUM 10//スレッド数

void *do_work(void *);
unsigned long data[MAX_NUM][3];
unsigned long param[THREAD_NUM];
pthread_mutex_t mutex;
unsigned char check[MAX_NUM + 1];//素数だったら1,そうでなければ0を格納する配列
unsigned long i;

int main(){
pthread_t tid[THREAD_NUM];//識別子
for(i=0;i<MAX_NUM;i++){//初期化
data[i][0] = 0;
data[i][1] = i;
data[i][2] = THREAD_NUM;
}

pthread_mutex_init(&mutex,NULL); for(i=0;i<THREAD_NUM;i++){ param[i] = i; pthread_create(&tid[i],NULL,do_work,&param[i]); } for (i=0; i<THREAD_NUM; i++) { pthread_join(tid[i], NULL); } for(i=0;i<=MAX_NUM+1;i++){//checkの配列の数だけfoe文を回す if(check[i]==1){ printf("%s\n",check); } } pthread_mutex_destroy(&mutex); return 0;

}

void *do_work(){

int j,divisor; for(i=0;i<MAX_NUM;i++){Loop will run at most once (loop increment never executed) pthread_mutex_lock(&mutex); divisor=0;//約数の初期化 for(j=1;j<=i;j++) { if(i%j==0) divisor++;//割り切れたらカウント } if(divisor==2){ check[i] = 1;//素数だったらcheckのv番目の配列を1にする }else{ check[i] = 0;//そうでなければ0にする } pthread_mutex_unlock(&mutex); return NULL;

}
}

試したこと

}の数はあっているか試しましたがどうやらそこではなかった様です。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yohhoy

2020/07/08 14:49

まずは適切なインデントをされては? 「return NULL; }}」→「} return NULL; }」
oksinamon

2020/07/08 15:36

ありがとうございます。エラ〜メッセージ消えました。完全に自分の見落としでした。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問