コードを短くする方法

Question

このコードを短くする方法はありますか？

include <3048f.h> //3048f.hの読み込み

/main関数/
int main(void)
{
int i,j; //待ち時間処理で使う変数の宣言

//LEDドットマトリクスの入出力設定
P4.DDR=0xff;
PB.DDR=0xff;

//点滅の永久ループ
while(1)
{
	//一行目のLEDがすべて点灯
	P4.DR.BYTE=0x80;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//二行目のLEDがすべて点灯
	P4.DR.BYTE=0x40;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
			;
	}

	//三行目のLEDがすべて点灯
	P4.DR.BYTE=0x20;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//四行目のLEDがすべて点灯
	P4.DR.BYTE=0x10;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//五行目のLEDがすべて点灯
	P4.DR.BYTE=0x08;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//六行目のLEDがすべて点灯
	P4.DR.BYTE=0x04;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//七行目のLEDがすべて点灯
	P4.DR.BYTE=0x02;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}

	//八行目のLEDがすべて点灯
	P4.DR.BYTE=0x01;
	PB.DR.BYTE=0x00;

	//待ち時間処理(500[m秒])
	for ( i=0; i<500; i++ ) //待ち時間処理(500*1[m秒])
	{
		for ( j=0; j<20000; j++ ) //待ち時間処理(1[m秒])
		;
	}
}
return 0;

}

Comments

[cateye]

なぜ、同じ処理（待ち時間処理など）を、関数にしないのでしょう？
待ち時間処理だけでも、48行が１４行程度に成ります。
基本、main()は、制御構造だけをかいたほうが良いですよｄ＾＾

[退会済みユーザー]

インデントが消えて見づらくなるので、ソースは「コードの挿入」で記入してください。

[thkana]

https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q11231031376
https://teratail.com/help#posted-otherservice
「やむを得ず複数のサイトに質問を投稿された場合は、質問内容にマルチポストをする理由を書き、他のサイトの投稿へのリンクを貼ってください。また、解決した際には必ずteratail及びすべての投稿に解決した旨と、どのように解決したかを記載してください。 」
まずはマルチポストの理由をどうぞ。

Answer 1

#include <3048f.h>

int main(void)
{
	for (P4.DDR = 0xff, PB.DDR = 0xff; ; )
		for (int i, m = 0x80; m; m >>= 1)
			for (P4.DR.BYTE = m, PB.DR.BYTE = i = 0; i < 500; i++)
				for (int j = 0; j < 20000; j++) ;
}

Comments

[cateye]

コンパイラがどういったコードを吐くか分かりませんが、
出力ポート（P4.DR.BYTE：８ビット）にintを出力するのは如何なものかと？

[kazuma-s]

質問のコード P4.DR.BYTE = 0x80; を否定するのですか？
0x80 は int です。

[cateye]

http://www008.upp.so-net.ne.jp/funfly/h8_2.html
には、即値（リテラル）で書いてあります。
まぁ、コンパイラがどう解釈するかなんでしょうが・・・通常int→charはワーニング対象だと思います。

Answer 2

31行

#include <stdint.h>

void func(uint8_t v)
{
    P4.DR.BYTE = v;
    PB.DR.BYTE = 0x00;
    //待ち時間処理(500[m秒])
    for(int i = 0; i < 500; i++) {      //待ち時間処理(500*1[m秒])
        for(int j = 0; j < 20000; j++)  //待ち時間処理(1[m秒])
            ;
    }
}
/*main関数*/
int main(void)
{
    //LEDドットマトリクスの入出力設定
    P4.DDR = 0xff;
    PB.DDR = 0xff;

    uint8_t v = 0x80;
    //点滅の永久ループ
    while(1) {
        //N行目のLEDがすべて点灯
        func(v);
        v >>= 1;
        if(v == 0x00) {
            v = 0x80;
        }
    }
    return 0;
}

「追記」これなら２９行

#include <stdint.h>

void func(uint8_t v)
{
    P4.DR.BYTE = v;
    PB.DR.BYTE = 0x00;
    //待ち時間処理(500[m秒])
    for(int i = 0; i < 500; i++) {      //待ち時間処理(500*1[m秒])
        for(int j = 0; j < 20000; j++)  //待ち時間処理(1[m秒])
            ;
    }
}
/*main関数*/
int main(void)
{
    //LEDドットマトリクスの入出力設定
    P4.DDR = 0xff;
    PB.DDR = 0xff;

    static uint8_t va[] = {0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01};
    size_t         pos  = 0;
    //点滅の永久ループ
    while(1) {
        //N行目のLEDがすべて点灯
        func(va[pos]);
        pos = (pos + 1) & 0x07;
    }
    return 0;
}

Comments

[cateye]

環境がないので、型名などは適当です。
stdint.hやuint8_tがなければ、unsigned charに読み替えてください。
・・・charは処理系依存なので、シフトした時の最上位ビットの扱いが違う場合があります。

Answer 3

• ｎ行目のLEDの点灯関数
• ｎm秒の待ち時間関数

の２つを作っておけば、main関数は、関数呼び出しを並べるだけでできますね

Answer 4

以下のような関数を作り、それを呼ぶように置き換えればコードの行数は短く出来るのではないでしょうか？
(関数名とか変数名とか適当です。wait2は引数とせず定数値のままでも良いかもしれません。)

void SetLED( int P4, int PB, int wait1, int wait2 ) {

    //一行目のLEDの操作
    P4.DR.BYTE=P4;
    PB.DR.BYTE=PB;

    //待ち時間処理
    for ( i=0; i<wait1; i++ ) //待ち時間処理(wait1*wait2[m秒])
    {
        for ( j=0; j<wait2; j++ ) //待ち時間処理(wait2[m秒])
            ;
    } 
}