配列の数値を並び替え、ある条件下の規則的な数値に置換したい

Question

お世話になっております。

実現したい内容

2つの配列array,array2の配列をそれぞれ、ランダムに並び替え、要素である数値を-1,0,1に置換するプログラムを作成しております。
ランダムに並び替えるところ(shuffleArrayData)まではうまくいったのですが、要素を置換する関数switchnumを加えるとキャッシュフローしたような状態となります。
###全体のソースコード

#include <stdio.h>
#include <stdlib.h>
//数字をランダムに並び替える関数
void shuffleArrayData(int* array, int size){
    int i, target, tmp;
    
    for(i = size -1; i > 0; i--){
        target = rand() % i;
        tmp = array[target];
        array[target] = array[i];
        array[i] = tmp;
    }
    //３連続を防ぐ.
    for(i = size -1; i > 0; i--){
        target = rand() % i;
        if(array[i] == array[i+1] && array[i] == array[i+2]){
            tmp = array[target];
            array[target] = array[i];
            array[i] = tmp;
        }
        
    }
    
    // for(i = 0; i < size; i++){
    //     printf("%d ", array[i]);
    // }
}
//数字を置換する関数
void switchnum(int* array,int* array2,int size,int* array3){
    int i,j,j_i;
    j=0;
    
    //(int)(sizeof(array2) / sizeof(array2[0])
    for(i = 0; i < 10; i++){
        if(array2[i]==5){
            j_i=j+25;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 25;
        }else if(array2[i]==10){
            j_i=j+50;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 50;
        }else{
            j_i=j+75;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 75;
        }
        if(array[i]==-1){
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=-1;
            }
            j += 10;
        }else{
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=1;
            }
            j += 10;
        }
        //printf("array2 is sizeof:%d",size);
        printf("i:%d\n",i);
        printf("j:%d\n",j);
    }
}

int main(){
    int array[] = {-1, -1, -1, -1, -1, -1, 1,1,1,1,1,1};
    int array2[]={5,5,5,5,5,5,5,5,5,10,10,10,15};
    //
    int data[560];
    //int array[] = {4, 2};
    //int array[] = {4};
    
    for(int i=0;i<15;i++){
        printf("=======================================================\n");
        printf("%d回目\n",i+1);
        shuffleArrayData(array, (int)(sizeof(array) / sizeof(array[0])));
        printf("\n");
        shuffleArrayData(array2, (int)(sizeof(array2) / sizeof(array2[0])));
        
        //arrayとarray2を交互に出力
        for(int j = 0; j < (int)(sizeof(array) / sizeof(array[0])); j++){
             int size=(int)(sizeof(array) / sizeof(array[0]));
             printf("%d,%d,", array2[j],array[j]);
             //最後に追加
             if(j == size-1){
                printf("%d",array2[j+1]);
             }
        }
        printf("\n");
        switchnum(array,array2,(int)(sizeof(data) / sizeof(data[0])),data);
        //array3を出力
        for(int i = 0; i < (int)(sizeof(data) / sizeof(data[0]))-1; i++){
            int size=(int)(sizeof(data) / sizeof(data[0]));
            printf("%d,", data[i]);
            //最後に追加
            if(i == size-2){
                printf("%d",data[i+1]);
            }
        }
        printf("\n");
        printf("=======================================================");
    }
    return 0;
}

###該当のソースコード

//数字を置換する関数
void switchnum(int* array,int* array2,int size,int* array3){
    int i,j,j_i;
    j=0;

    //(int)(sizeof(array2) / sizeof(array2[0])
    for(i = 0; i < 10; i++){
        if(array2[i]==5){
            j_i=j+25;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 25;
        }else if(array2[i]==10){
            j_i=j+50;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 50;
        }else{
            j_i=j+75;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
            j += 75;
        }
        if(array[i]==-1){
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=-1;
            }
            j += 10;
        }else{
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=1;
            }
            j += 10;
        }
        //printf("array2 is sizeof:%d",size);
        printf("i:%d\n",i);
        printf("j:%d\n",j);
    }
}

何かヒントだけでもご教授いただければと思っております。
よろしくお願いいたします。

Comments

[jimbe]

なぜ「該当のソースコード」でswitchnumを再度載せられているのでしょう。何か違うのでしょうか。

[jimbe]

「キャッシュフローしたような状態」とは具体的にどのような状態でしょうか。
「要素である数値を-1,0,1に置換する」という仕様も、どのような判断基準なのでしょう。

[nori0519]

ご連絡ありがとうございます。
>>「キャッシュフローしたような状態」
表現が正しいのか自信がありませんが、最終的に格納する配列data[512]を出力した際に、4195346や163189952といったような数値が入っている状態のことです。
>>「要素である数値を-1,0,1に置換する」という仕様の判断基準
array[]の要素が1であった場合、array3[]に1を10こ追加する。
array[]の要素が-1であった場合、array3[]に-1を10こ追加する。
array[2]の要素が5であった場合、array3[]に0を25こ追加する。
array[2]の要素が10であった場合、array3[]に0を50こ追加する。
array[2]の要素が15であった場合、array3[]に0を75こ追加する。
となります。

[jimbe]

data[512] が switchnum によって設定されていなければ、変な値が入っている可能性がある(いうより大抵入っている)のは c では普通です。そして、 switchnum の i の for ループが 10 回ですので、 data に入る有効データ数は(array2 の中身に依りますので)一定ではありません。
そのことは、 switchnum の j の表示を見ればお分かりかと思います。

頂いた判断基準を言い換えますと、
array3[]に"array[]の要素"を10こ追加する。
array3[]に0を"array2[]の要素"×5こ追加する。
の二つになります。回答にはそのようなコードを載せてみました。
またこのコードの関数は int を返すように変更していまして、 data に設定した"有効データ数"を返すようにしています。
main 側で data を表示する際はこの戻り値を使って 0～"有効データ数"までを表示するようにすれば、変な値は出なくなると思います。

[nori0519]

ご連絡ありがとうございます。
ご回答といただいた内容と返信の内容を参考にさせていただきます。
勉強不足により、あまり理解が及ぼない点があり、気の利いたご返信ができず申し訳ございません。
また、私自身の説明不足な点により非常に分かりにくい質問となってしまい、重ねてお詫びいたします。
最後まで懇切丁寧にお答えいただき、誠にありがとうございました。

Answer 1

お世話になっております。

今回の原因としまして、
・switchnum関数内にて、jに対して、forによりarray3[]に数値を入力後、余分な数値を加算していた。
・main関数内における、data560の要素数が不足していた。→data[580]に変更
でした。
お手数おかけいたしました。

修正したソースコード

#include <stdio.h>
#include <stdlib.h>
//数字をランダムに並び替える関数
void shuffleArrayData(int* array, int size){
    int i, target, tmp;
    
    for(i = size -1; i > 0; i--){
        target = rand() % i;
        tmp = array[target];
        array[target] = array[i];
        array[i] = tmp;
    }
    //３連続を防ぐ.
    for(i = size -1; i > 0; i--){
        target = rand() % i;
        if(array[i] == array[i+1] && array[i] == array[i+2]){
            tmp = array[target];
            array[target] = array[i];
            array[i] = tmp;
        }
        
    }
    
    // for(i = 0; i < size; i++){
    //     printf("%d ", array[i]);
    // }
}
//数字を置換する関数
void switchnum(int* array,int* array2,int size,int* array3){
    int i,j,j_i;
    j=0;
    
    for(i = 0; i < size; i++){
        if(array2[i]==5){
            j_i=j+25;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
        }else if(array2[i]==10){
            j_i=j+50;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
        }else{
            j_i=j+75;
            for(j;j<j_i;j++){
                array3[j]=0;
            }
        }
        if(array[i]==-1){
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=-1;
            }
        }else{
            j_i=j+10;
            for(j;j<j_i;j++){
                array3[j]=1;
            }
        }
        printf("array[i]:%d\n",array[i]);
        printf("array2[i]:%d\n",array2[i]);
        printf("i:%d\n",i);
        printf("j:%d\n",j);
    }
}

int main(){
    int array[] = {-1, -1, -1, -1, -1, -1, 1,1,1,1,1,1};
    int array2[]={5,5,5,5,5,5,5,5,5,10,10,10,15};
    //
    int data[580];
    //int array[] = {4, 2};
    //int array[] = {4};
    
    for(int i=0;i<15;i++){
        printf("=======================================================\n");
        printf("%d回目\n",i+1);
        shuffleArrayData(array, (int)(sizeof(array) / sizeof(array[0])));
        printf("\n");
        shuffleArrayData(array2, (int)(sizeof(array2) / sizeof(array2[0])));
        
        //arrayとarray2を交互に出力
        for(int j = 0; j < (int)(sizeof(array) / sizeof(array[0])); j++){
             int size=(int)(sizeof(array) / sizeof(array[0]));
             printf("%d,%d,", array2[j],array[j]);
             //最後に追加
             if(j == size-1){
                printf("%d",array2[j+1]);
             }
        }
        printf("\n");
        switchnum(array,array2,(int)(sizeof(array2) / sizeof(array2[0])),data);
        //array3を出力
        for(int i = 0; i < (int)(sizeof(data) / sizeof(data[0]))-1; i++){
            int size=(int)(sizeof(data) / sizeof(data[0]));
            printf("%d,", data[i]);
            //最後に追加
            if(i == size-2){
                printf("%d",data[i+1]);
            }
        }
        printf("\n");
        printf("=======================================================");
    }
    return 0;
}

Answer 2

j の値が data のサイズを超えている辺りが問題のようですね。
仕様があっているか分かりませんが。

//数字を置換する関数
int switchnum(int* array, int* array2, int size, int* array3) {
    int j = 0;
    for(int i=0; i<10; i++) {
        for(int k=0; k<array2[i]*5; k++) array3[j++] = 0;
        for(int k=0; k<10;          k++) array3[j++] = array[i];
        printf("i:%d\n",i);
        printf("j:%d\n",j);
    }
    return j;
}