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

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

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

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

Q&A

3回答

1088閲覧

Xorshiftの実装、シフト演算によるエラー

milulutan

総合スコア0

C

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

0グッド

0クリップ

投稿2021/06/19 03:25

前提・実現したいこと

C言語を用いてXorshift関数をつくってさいころのぞろ目が生まれる回数を調べたくコードを書いたのですが
どうもなにかshift(<<)のところでエラーが出てしまいます。シフトをどのように変えたらよいのでしょうか。

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

エラーメッセージ Segmentation fault

該当のソースコード

#include <stdio.h>
#include <time.h>

unsigned long xorshift(unsigned long *a){
printf("%lu\n", a);
*a = *a^(*a<<13);
*a = *a^(*a>>17);
*a = *a^(*a<<43);
return *a;
}

unsigned long make_seed(){
time_t timer;
time(&timer);
return timer;
}

int roll_dice(unsigned long *seed){
int prev, curr;
prev = xorshift(seed)%61;
for(int i = 0; i < 5; i++){
curr = xorshift(seed)%61;
if(prev != curr) return 0;
}
return 1;
}

int main(int argc, char *argv[])
{
int num = 0;
int i;
for(i=0;i<=100000;i++)
{
if(roll_dice(make_seed()) == 1){
num += 1;
}
}

printf("%d",num);

}

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

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

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

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

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

guest

回答3

0

それはシフトしてるからエラーが出てるんではありません。
アドレスを与えるところに、数値を与えてしまっているので、メモリアクセス違反が起こっています

投稿2021/06/19 05:47

y_waiwai

総合スコア88026

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

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

0

unsigned long が 32bit以上の実行環境ですか?でないと
*a<<43 は未定義動作になります。

if(roll_dice(make_seed()) == 1){

make_seed()で返した値を一度変数に保存してそのポインタを
roll_dice()に渡すのが正しいのでは?

投稿2021/06/19 03:45

sigsegv

総合スコア895

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

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

0

unsigned longであるmake_seedの返り値を、unsigned long *seedを取るroll_diceに渡してしまっているので、型が一致していません。

投稿2021/06/19 03:40

maisumakun

総合スコア145952

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

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

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

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問