ソートされた構造体をメイン関数に戻すにはどうすればよいですか？

Question

目標：3ダイスとリターンソートのソート構造体

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

構造体

struct RolledDice 
{ 
    int die1 ; 
    int die2 ; 
    int die3 ; 
} dice; 

プロトタイプ

void sort_dice(struct RolledDice dice); 

メイン

int main() { 
srand (time(NULL)); 

dice.die1 = rand() % 6 + 1 ; 
dice.die2 = rand() % 6 + 1 ; 
dice.die3 = rand() % 6 + 1 ; 

sort_dice(dice); 

return 0; 
} 

ソートダイス機能

void sort_dice(struct RolledDice dice) { 

    printf("Die 1: %d \n", dice.die1); 
    printf("Die 2: %d \n", dice.die2); 
    printf("Die 3: %d \n\n", dice.die3); 

    int tempDie = 0; 

は（これを行うには良い方法があるかもしれません......これは私が思い付くことが最高です）

while (dice.die1 < dice.die2 || dice.die1 < dice.die3 || dice.die2 < dice.die3) 
    { 
     if (dice.die1 < dice.die2) 
     { 
     tempDie = dice.die1 ; 
     dice.die1 = dice.die2 ; 
     dice.die2 = tempDie ; 
     } 
     if (dice.die1 < dice.die3) 
     { 
     tempDie = dice.die1 ; 
     dice.die1 = dice.die3 ; 
     dice.die3 = tempDie ; 
     } 
     if (dice.die2 < dice.die3) 
     { 
     tempDie = dice.die2 ; 
     dice.die2 = dice.die3 ; 
     dice.die3 = tempDie ; 
     } 
    } 

    printf("Die 1: %d \n", dice.die1); 
    printf("Die 2: %d \n", dice.die2); 
    printf("Die 3: %d \n\n", dice.die3); 
} 

は私が変えてみましたvoid〜intとstructがエラーを起こしているか、またはmainのstructを更新しませんでした。

Answer 1

dice構造体のコピーをソートしていますが、この構造体は、関数が返ってきたら範囲外になるとすぐに失われます。

にあなたの関数を変更

：

struct RolledDice sort_dice(struct RolledDice dice) { 

と最後だけreturn dice

使用量：

int main() { 
srand (time(NULL)); 

dice.die1 = rand() % 6 + 1 ; 
dice.die2 = rand() % 6 + 1 ; 
dice.die3 = rand() % 6 + 1 ; 

dice = sort_dice(dice); 

// print the sorted struct here 
printf("Die 1: %d \n", dice.die1); 
printf("Die 2: %d \n", dice.die2); 
printf("Die 3: %d \n\n", dice.die3); 
return 0; 
} 

またはポインタとしてdiceを渡し、あなたの関数でdice->代わりのdice.を使用（重いリファクタリングではあるがメモリコピーが少なく、パフォーマンスが向上する）

void sort_dice(struct RolledDice *dice) { 
... 
dice->die1 = dice->die3 ; 
... 

用法：

int main() { 
srand (time(NULL)); 

dice.die1 = rand() % 6 + 1 ; 
dice.die2 = rand() % 6 + 1 ; 
dice.die3 = rand() % 6 + 1 ; 

sort_dice(&dice); // pass as pointer so it can be modified in the function 

// print the sorted struct here 
printf("Die 1: %d \n", dice.die1); 
printf("Die 2: %d \n", dice.die2); 
printf("Die 3: %d \n\n", dice.die3); 
return 0; 
} 

注：元のコードが正しく内のあなたのソート・ルーチンバブルソート値を出力します。私は、問題はあなたが呼び出し元の関数（ソートされた値を印刷するだけの関数を作成する関数を作成することは役に立たない）を更新する方法を見つけられなかったと仮定します。