C言語の文法について質問です。

質問

下記の文を長いのでfor文一つにまとめれないでしょうか？
下に自分で作ったのがあります。修正箇所を教えて頂けませんか？
n[6] = (n[1] - n[2]);
if(n[6] < 0){
n[6] = -n[6];
}
n[7] = (n[2] - n[3]);
if(n[7] < 0){
n[7] = -n[7];
}
n[8] = (n[3] - n[4]);
if(n[8] < 0){
n[8] = -n[8];
}
n[9] = (n[4] - n[5]);
if(n[9] < 0){
n[9] = -n[9];
}
n[10] = (n[6] - n[7]);
if(n[10] < 0){
n[10] = -n[10];
}
n[11] = (n[7] - n[8]);
if(n[11] < 0){
n[11] = -n[11];
}
n[12] = (n[8] - n[9]);
if(n[12] < 0){
n[12] = -n[12];
}
n[13] = (n[10] - n[11]);
if(n[13] < 0){
n[13] = -n[13];
}
n[14] = (n[11] - n[12]);
if(n[14] < 0){
n[14] = -n[14];
}
n[15] = (n[13] - n[14]);
if(n[15] < 0){
n[15] = -n[15];
}

////////////////////////////////
x=1;
for(i=6; i<16; i++){
n[i]=n[x]-n[x+1];
if(n[i]<0){
n[i]=-n[i];
}
x++;
}

回答No.4

gerugugu84 · ベストアンサー

途中の抜けてる部分があるから質問してんだよね？

とりあえず規則性を見出してつくってみたけど、
カウンタの範囲チェックとか抜けてる上に汚いので自分で修正して。

int main()
{
int n[ 16 ] = { 0 };
int limit = 4;
int kk = 1;
int jj = 0;
for( int ii = 6; ii < 16; ii++ )
{
if( jj && !( jj % limit ) )
{
if( 4 <= kk )
break;
limit += 4 - kk;
++kk;
}
n[ ii ] = n[ jj + kk ] - n[ jj + kk + 1 ];
++jj;

if( n[ ii ] < 0 )
{
n[ ii ] = -n[ ii ];
}
}
return 0;
}

回答No.7

Tacosan

この質問者のところでは
sizeof は使用禁止
らしいので (cf. http://okwave.jp/qa/q6916438.html )＞#5,
loopIndex の最後に {0, 0, 0} でも追加し, struct loopIndexDef * で for をまわす方法を提案しておこう.

abs がなければ ?: を使えばいいのに.

まあ, 「本当にわかっていてベストアンサーをつけた」かどうかは上のリンクを見れば一目瞭然でしょう＞#6.

回答No.6

麻野 なぎ（@AsanoNagi）

ちなみに、本当にわかっていて、

http://okwave.jp/qa/q6917741.html

にベストアンサーをつけたのなら、ここにあるソースがこの回答になっている。

for(i = maxStep - 1; i >= 1; i--)
{
int leftOfThisLine = doneCount;
int leftOfLastLine = doneCount - i - 1;

for (j = 0; j < i; j++)
cell[leftOfThisLine + j] = abs(cell[leftOfLastLine + j] - cell[leftOfLastLine + j + 1]);

doneCount += i;
}
}

回答No.5

麻野 なぎ（@AsanoNagi）

万能なやり方。

struct loopIndexDef
{
int result;
int from;
int to;
};

struct loopIndexDef loopIndex[] =
{
{6, 1, 2},
{7, 2, 3},
{8, 3, 4},
{9, 4, 5},
{10, 6, 7},
{11, 7, 8},
{12, 8, 9},
{13, 10, 11},
{14, 11, 12},
{15, 13, 14}
};

for(i = 0; i < sizeof loopIndex / sizeof loopIndex[0]; i++)
{
struct loopIndexeDef *ptr = &loopIndex[i];
n[ptr->result] = abs(n[ptr->from] - n[ptr->to]);
}

回答No.3

soraneko34

No2の方が言っているように
大元のソースに
(n[5] - n[6]);
(n[9] - n[10]);
(n[12] - n[13]);
が抜けています
回答したときに適当に気づかなかったので補足します
（ちゃんと質問文読めよ。。。って話ですが）

int i;
int x;
for(i=6,x=1; i<16; i++,x++){
if(x==5 || x==9 || x==12)x++;
n[i]=abs(n[x]-n[x+1]);
}

とすれば一応要求どおりにはなると思います
あまり美しくはないですが、、、

回答No.2

noname#147388

大本のソースに

n[ 5]-n[6]
n[ 9]-n[10]
n[12]-n[13]

がぬけてるからループはむりですな。

回答No.1

soraneko34

C触るのが久々なので間違ってるかも（？）
xが1からで良いのか疑問ですが取り合えず（配列は0番目から始まるため）

int i;
int x;
for(i=6,x=1; i<16; i++,x++){
n[i]=abs(n[x]-n[x+1]);
}

abs()関数は数値の絶対値を返します。
以上

補足

abs関数を習ってないので他の方法はありませんか？
宜しくお願いします。