C言語でのfor文の効率化 処理の簡素化

Question

前提・実現したいこと

コードを書いてみたんですがもっとfor文のところを効率的にまとめる方法はありませんか？
最初にCとXOR
関数に入れてｄを取得
d値と次のa（a+16)とXOR
関数に入れて再びｄを取得
d値と次のa（a+16)とXOR
以降この繰り返し

    unsigned char a[64]= {0}; 
    unsigned char b[16]= {0};
    unsigned char c[16]=  {0x00,0x01,0x02,0x03,0x04,0x05,0x06,0x07,
                          0x08,0x09,0x0a,0x0b,0x0c,0x0d,0x0e,0x0f}; 
    unsigned char d[64]= {0}; 

-省略-
  fread( a, sizeof(char), sizeof(a), file1 );
  fread( b, sizeof(char), sizeof(b), file2);
 
  for(i=0;i<16;i++){
     a[i]=a[i]^c[i]);
  }

　for(i=0;i<16;i++){
     kansuu(&a[i],b,&d[i]);
  }
  for(i=0;i<16;i++){
     a[i+16]=a[i+16]^d[i]
  }
  for(i=0;i<16;i++){
     kansuu(&a[i+16],b,&d[i+16]);
  }
  for(i=0;i<16;i++){
     a[i+32]=a[i+32]^d[i+16]
  }
  for(i=0;i<16;i++){
     kansuu(&a[i+32],b,&d[i+32]);
  }
  for(i=0;i<16;i++){
     a[i+48]=a[i+48]^d[i+32]
  }
  for(i=0;i<16;i++){
     kansuu(&a[i+48],b,&d[i+48]);
  }
上記記入間違いでした。
訂正
  for(i=0;i<16;i++){
     a[i]^＝c[i];
  }
  kansuu(a,b,d);
  
  for(i=0;i<16;i++){
     a[i+16]=a[i+16]^d[i] ;
  }
  kansuu(a+16,b,d+16]);
  
  for(i=0;i<16;i++){
     a[i+32]=a[i+32]^d[i+16] ;
  }
 kansuu(a+32,b,&d+32);
  }
  for(i=0;i<16;i++){
     a[i+48]=a[i+48]^d[i+32]
  }
 kansuu(a+48,b,d+48);
  }

##試したこと

a[i]=a[i]^c[i]); 
for(i=0;i<64;i++){
  if(i==0){
     kansuu(&a[i],b,&d[i])
    }
   if(i>0){
    kansuu(&a[i],b,&d[i]);
    a[i+16]=a[i+16]^d[i];
    kansuu(&a[i+16],b,&d[i+16]);
   }
}

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

C言語　VisualStudioCode LINUX

Comments

[thkana]

kansuuの中身次第なところがあると思いますが。

[alles-durch]

そうですか、戻り値はそのまま同じように出る形にはしていますが
わかりました。

Answer 1

書く前に問題が変わってたので・・・
一応修正して、c,dまとめる部分だけを

cとdをまとめてる形で保持すると、それを配列eとするとforの中身は、かなりシンプルに

unsigned char e[16+64]={ cの中身,dの中身 }
unsigned char *d=&e[16];

for(j=0;j<4;j++){
	for(i=0;i<16;i++){
		a[i+j*16]^=e[i+j*16];
	}
	kansuu(&a[j*16],b,&e[j*16+16]);
}

Comments

[alles-durch]

回答ありがとうございます。
シンプルになりますね。
e［］のc.dの中身は直接入力ってことですか？

Answer 2

	for (i = 0; i < 16; i++) a[i] ^= c[i];
	for (i = 0; i < 16; i++) kansuu(a + i, b, d + i);
	for (i = 16; i < 64; i += 16) {
		k = i + 16;
		for (j = i; j < k; j++) a[j] ^= d[j - 16]
		for (j = i; j < k; j++) kansuu(a + j, b, d + j);
	}

追記
問題が変わったので書き直しました。

    int i, j, k;
    for (i = 0; i < 16; i++)
		a[i] ^= c[i];
    kansuu(a, b, d);
    for (i = 16; i < 64; i = k) {
        for (j = i, k = i + 16; j < k; j++)
			a[j] ^= d[j - 16]
        kansuu(a + i, b, d + i);
    }

Comments

[alles-durch]

回答ありがとうございました。
参考にさせていただきました。
載せていたコードに不備がありました。

[alles-durch]

ありがとうございました。とてもスマートにまとられて簡略化できました。jkの使い方がなぜそうなるのかがよくわかりませんが期待値がでました。

a∧cとkansuu後に
for文逆順で戻す場合は
int i, j, k;
for (i = 0; i < 16; i++)
a[i] ^= c[i];
kansuu(a, b, d);

for (i = 16; i < 64; i = k) {
for (j = i, k = i + 16; j < k; j++)
　　　kansuu(a +i,b,d +i)
　　　｝
　　｝
　 for (i = 0; i < 64; i = k) {
d［i +16 ] ^= a[i];
｝で戻すことができましたが、違うやり方もありますか？

[kazuma-s]

kansuu の仕様が不明であり、入力例も期待値も不明なので、戻すという処理がどういうものか分かりません。

[alles-durch]

わかりました。ありがとうございます

Answer 3

for(i=0;i<16;i++){ 
   a[i]^=c[i];
}
kansuu(a,b,d);
for (count=0;count<4;count++){
   if(count==0){
      kansuu(a,b,d); 
    }
   if(count==1){
     for(i=0;i<64;i++){
         a[i+(count*16)]=a[i+(count*16)]^d[i];
      }
      kansuu(a+(count*16),b,d+(count*16)); 
    }  
   if(count>1){
      for(i=0;i<64;i++){
          a[i+(count*16)]=a[i+(count*16)]^d[i+(count*16-16)];
       }
       kansuu(a+(count*16),b,d+(count*16));    
   }
}

で最低限の解決をしました。
アドバイスありがとうございました。
誤記等変換ミス等があり修正させていただきました。
参考にさせていただいたことで解決し、自己解決欄に当初間違えて記入してしまいました。
申し訳ありません。

Comments

[kazuma-s]

全角文字の「＝＝」や「(count×16)」がコンパイルエラーになります。
) や { の対応も取れていません。本当に解決ですか？

[alles-durch]

環境が違くて携帯で返答していため誤記や変換ができませんでした。

Answer 4

for(i=0;i<16;i++){
a[i]=a[i]^c[i]);
kansuu(&a[i],b,&d[i]);
a[i+16]=a[i+16]^d[i]
kansuu(&a[i+16],b,&d[i+16]);
a[i+32]=a[i+32]^d[i+16]
kansuu(&a[i+32],b,&d[i+32]);
a[i+48]=a[i+48]^d[i+32]
kansuu(&a[i+48],b,&d[i+48]);
}

でいけそうやけど

Comments

[alles-durch]

回答ありがとうございます。
その方法でも試しましたが最初の16バイトのみでそこからは値と違っていました。
言葉足りずですいません。
現在は６４バイトですが、６４バイト以上のサイズになった場合を考えると毎回増やしていくのが大変になってくるので効率化を考えていました。

[y_waiwai]

デバッグできる環境を整えて、どこのステップで数字が違うのか追いかけていけばどうでしょう。
コードだけ眺めてあれこれ言ってもしよーがないですよ

[alles-durch]

ありがとうございます。
デバッグしたり、コード変えたりしながら空き時間には色々ためしています。