单片机流水灯，只循环一次，有老哥帮看看吗

ID:624696 · 发表于 2019-11-15 17:05

这是一个流水灯，先从两侧向中间亮，在从中间向两侧亮，为什莫只循环一次呢

有没有老哥给说下啊

先谢谢了

单片机源程序如下:

#include <reg52.h>
unsigned char code table[] = {
0xff,0x7e,0x3c,0x18,0x00, //从两侧向中间点亮
0xff,0xe7,0xc3,0x81,0x00 //从中间向两侧点亮
};
unsigned char i;
void delay()
{
unsigned int s;
for (s=0;s<30000;s++);
}
void main ()
{
while(1)
{
if (table[i]!= 0xee)
{
P1 = table[i];
i++;
delay();
}
else
{
i=0;
}
}
}

复制代码

ID:394424 · 发表于 2019-11-16 08:47

看门狗是不是复位了

ID:25310 · 发表于 2019-11-16 08:52

if (table[i]!= 0xee) 下一个条件这里如果相等了要让他等于0吧

ID:213173 · 发表于 2019-11-16 10:38

if(table[i]!= 0xee)的条件永远满足，i++;跑飞。

ID:275826 · 发表于 2019-11-16 11:01

table[]里没有OXEE，i一直加，上天了

ID:511501 · 发表于 2019-11-16 11:57

table[i]永远不会等于0xee,所以跑一遍就超出数组大小了，可以这样改:主函数while(1)里面全部替换成:

ID:511501 · 发表于 2019-11-16 12:00

for(i=0;i<10;i++){P1=table[i];delay（）;}

ID:643100 · 发表于 2019-11-16 12:51

跑完一次后，就会一直循环i++这个地方，跑飞了

ID:272485 · 发表于 2019-11-16 13:44

i赋初值为0，数组里面结尾加个0xee也行

ID:624696 · 发表于 2019-11-16 15:43

huh 发表于 2019-11-16 13:44
i赋初值为0，数组里面结尾加个0xee也行

谢谢了，；老哥，你这个方法是可以的

ID:624696 · 发表于 2019-11-16 15:45

wulin 发表于 2019-11-16 10:38
if(table!= 0xee)的条件永远满足，i++;跑飞。

是的，对的，东了谢谢老哥

ID:624696 · 发表于 2019-11-16 15:45

tyrl800 发表于 2019-11-16 11:01
table[]里没有OXEE，i一直加，上天了

谢谢了，没错，我加了0xee就可以了

ID:624696 · 发表于 2019-11-16 15:46

1842058863 发表于 2019-11-16 12:51
跑完一次后，就会一直循环i++这个地方，跑飞了

明白了，谢谢老哥

ID:624696 · 发表于 2019-11-16 15:46

huh 发表于 2019-11-16 13:44
i赋初值为0，数组里面结尾加个0xee也行

谢谢老哥，加了0xee确实可以了

ID:624696 · 发表于 2019-11-16 15:50

13144088433 发表于 2019-11-16 12:00
for(i=0;i

谢谢谢谢，这样也是可以的，厉害哦大家

ID:624696 · 发表于 2019-11-16 15:53

Wprogrammer 发表于 2019-11-16 08:47
看门狗是不是复位了

不太懂你的意思，还是谢谢了

ID:603411 · 发表于 2019-11-16 17:20

数组中没有oxee这个值，也就是i不停地加，一直执行if (table[i]!= 0xee)中的内容，所以就只循环一次

ID:263427 · 发表于 2019-11-16 19:00

看到楼上已经解决的差不多了，但是原理还不太清楚，在这儿我着重阐述一下原理。

if语句里面没有结束条件，所以永远不可能出if，更不可能去执行while语句，究其原因，是i++没有结束条件，解决的方法楼上已经给出，加一个限制条件就好了。

ID:245004 · 发表于 2019-11-16 20:21

#include <reg52.h>
#define TAB_LENGTH 10
unsigned char code table[TAB_LENGTH] = {
0xff, 0x7e, 0x3c, 0x18, 0x00, //从两侧向中间点亮
0xff, 0xe7, 0xc3, 0x81, 0x00}; //从中间向两侧点亮
void delay(void) {
unsigned int s;
for (s = 0; s < 30000; s++) {
}
}
void main(void) {
while (1) {
unsigned char i;
for (i = 0; i < TAB_LENGTH; i++) {
P1 = table[i];
delay();
}
}
}

复制代码

这样写试试。

ID:245004 · 发表于 2019-11-16 20:28

如果一定要用if语句，那就把主函数这样写：

void main(void) {
while (1) {
unsigned char i;
if (i < TAB_LENGTH) {
P1 = table[i];
delay();
i++;
} else {
i = 0;
}
}
}

复制代码

ID:643346 · 发表于 2019-11-16 20:37

这个是不是忘加while循环里了，还有要加个扫描

ID:93224 · 发表于 2019-11-17 15:48

数组里面没有0xee

ID:296956 · 发表于 2019-11-17 17:10

table[] 数组后面加个0xee 的值。

ID:624696 · 发表于 2019-11-26 22:05

13144088433 发表于 2019-11-16 11:57
table永远不会等于0xee,所以跑一遍就超出数组大小了，可以这样改:主函数while(1)里面全部替换成:

为什莫led灯点亮不是按照数组的顺序进行的呢？？？

ID:624696 · 发表于 2019-11-26 22:07

huh 发表于 2019-11-16 13:44
i赋初值为0，数组里面结尾加个0xee也行

为什么led灯的点亮顺序不是按照数组里的顺序来的呢？？为什么不是一个一个点亮呢，而是同时点亮

ID:213173 · 发表于 2019-11-27 09:56

TRX007 发表于 2019-11-26 22:07
为什么led灯的点亮顺序不是按照数组里的顺序来的呢？？为什么不是一个一个点亮呢，而是同时点亮

按你的题意，先从两侧向中间亮，在从中间向两侧亮，反复循环。

#include <reg52.h>
unsigned char code table[] = {0x7e,0xbd,0xdb,0xe7};
unsigned char i;
void delay()
{
unsigned int s;
for(s=0;s<30000;s++);
}
void main ()
{
while(1)
{
for(i=0;i<4;i++)
{
P1=table[i];
delay();
}
for(i=2;i>0;i--)
{
P1=table[i];
delay();
}
}
}

复制代码

ID:624696 · 发表于 2019-11-27 10:19

wulin 发表于 2019-11-16 10:38
if(table!= 0xee)的条件永远满足，i++;跑飞。

老哥，请问为什么led的点亮顺序是同时点亮呢，而不是按数组的顺序一个一个点亮呢？

ID:624696 · 发表于 2019-11-27 10:20

bh2030693 发表于 2019-11-16 20:28
如果一定要用if语句，那就把主函数这样写：

请问，为什么led点亮不是按照数组的顺序一个个点亮，而是同时亮呢？

ID:213173 · 发表于 2019-11-27 10:49

TRX007 发表于 2019-11-27 10:19
老哥，请问为什么led的点亮顺序是同时点亮呢，而不是按数组的顺序一个一个点亮呢？

按你的题意，先从两侧向中间亮，再从中间向两侧亮，反复循环。看26楼回复。

ID:415088 · 发表于 2019-11-30 10:23

if (table[i]!= 0xee) 这里的判断条件有问题

ID:653047 · 发表于 2019-11-30 20:16

很显然在数组table中，没有数能让table[i]==0xee，也就是说，while循环只是在table[i]!=0xee的条件下循环，i的值只会无限增大，不归零

帐号		自动登录	找回密码
密码			立即注册