Q&A
kansuuの中身次第なところがあると思いますが。
前提・実現したいこと
コードを書いてみたんですがもっと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
回答4件
0
書く前に問題が変わってたので・・・
一応修正して、c,dまとめる部分だけを
cとdをまとめてる形で保持すると、それを配列eとするとforの中身は、かなりシンプルに
C
1 2unsigned char e[16+64]={ cの中身,dの中身 } 3unsigned char *d=&e[16]; 4 5for(j=0;j<4;j++){ 6 for(i=0;i<16;i++){ 7 a[i+j*16]^=e[i+j*16]; 8 } 9 kansuu(&a[j*16],b,&e[j*16+16]); 10} 11
投稿2021/02/09 03:21
総合スコア304
0
ベストアンサー
C
1 for (i = 0; i < 16; i++) a[i] ^= c[i]; 2 for (i = 0; i < 16; i++) kansuu(a + i, b, d + i); 3 for (i = 16; i < 64; i += 16) { 4 k = i + 16; 5 for (j = i; j < k; j++) a[j] ^= d[j - 16] 6 for (j = i; j < k; j++) kansuu(a + j, b, d + j); 7 }
追記
問題が変わったので書き直しました。
C
1 int i, j, k; 2 for (i = 0; i < 16; i++) 3 a[i] ^= c[i]; 4 kansuu(a, b, d); 5 for (i = 16; i < 64; i = k) { 6 for (j = i, k = i + 16; j < k; j++) 7 a[j] ^= d[j - 16] 8 kansuu(a + i, b, d + i); 9 }
投稿2021/02/08 16:56
編集2021/02/09 04:07
総合スコア8224
回答ありがとうございました。
参考にさせていただきました。
載せていたコードに不備がありました。
ありがとうございました。とてもスマートにまとられて簡略化できました。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];
｝で戻すことができましたが、違うやり方もありますか？
kansuu の仕様が不明であり、入力例も期待値も不明なので、戻すという処理がどういうものか分かりません。
わかりました。ありがとうございます
0
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)); } }
で最低限の解決をしました。
アドバイスありがとうございました。
誤記等変換ミス等があり修正させていただきました。
参考にさせていただいたことで解決し、自己解決欄に当初間違えて記入してしまいました。
申し訳ありません。
投稿2021/02/09 03:32
編集2021/02/15 13:13総合スコア3
全角文字の「＝＝」や「(count×16)」がコンパイルエラーになります。
) や { の対応も取れていません。本当に解決ですか？
環境が違くて携帯で返答していため誤記や変換ができませんでした。
0
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]);
}
でいけそうやけど
投稿2021/02/08 15:16
総合スコア88042
回答ありがとうございます。
その方法でも試しましたが最初の16バイトのみでそこからは値と違っていました。
言葉足りずですいません。
現在は６４バイトですが、６４バイト以上のサイズになった場合を考えると毎回増やしていくのが大変になってくるので効率化を考えていました。
デバッグできる環境を整えて、どこのステップで数字が違うのか追いかけていけばどうでしょう。
コードだけ眺めてあれこれ言ってもしよーがないですよ
ありがとうございます。
デバッグしたり、コード変えたりしながら空き時間には色々ためしています。
あなたの回答
tips
プレビュー
