ATNEL tech-forum

Re: Pętle while

2017-10-18T20:41:35+01:00

Właśnie te dwie 1 mi nie pasowało ale zaufałem koledze poprawione
Zauważyłem że nie napisałem break`a w drugim case dlatego nie chodziło tak jakbym chciał.
Poprawione obie rzeczy i śmiga tak jak chciałem.
Dziękuję za pomoc a oto finalny kod jeśli ktoś chciałby zerknąć.

[syntax=c]while(1)
{
switch(stan)
{
case 0:
if(SENSOR_ACTIVE)
{
stan = 1;
}

if(s1_flag) {/* sprawdzanie flagi tyknięć timera programowego co 1 sekundę */

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 0 sprawdzaj ilość dostępnych czujników */
if( 0 == (sekundy%3) ) {

uint8_t *cl=(uint8_t*)gSensorIDs;// pobieramy wskaźnik do tablicy adresów czujników
for( uint8_t i=0; iczujniki_cnt = search_sensors();// ponownie wykrywamy ile jest czujników i zapełniamy tablicę
lcd_locate(0,8);
lcd_int( czujniki_cnt );// wyświetlamy ilość czujników na magistrali
}

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 1 wysyłaj rozkaz pomiaru do czujników */
if( 1 == (sekundy%3) ) DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 2 czyli jedną sekundę po rozkazie konwersji
* dokonuj odczytu i wyświetlania temperatur z 2 czujników jeśli są podłączone, jeśli nie
* to pokaż komunikat o błędzie
*/

if( 2 == (sekundy%3) )
{
if( klawisz1() )
{
if( licz1 == 0 )
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T1 T2")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 1)
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T3 T4")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[2], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[3], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 2)
{
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,0);
lcd_str("BRAK ZAGROZENIA!");
licz1=0;
}
}
}
/* zerujemy flagę aby tylko jeden raz w ciągu sekundy wykonać operacje */
s1_flag=0;
} /* koniec sprawdzania flagi */
break; /* koniec pętli pomiarowej*/
case 1:
if(!SENSOR_ACTIVE)
{
stan = 0;
}

if(SENSOR_ACTIVE)
{
FAN1_ON;
BUZZER_ON;
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,1);
lcd_str("WYKRYTO DYM!!!");
_delay_ms(500);
}
else
{
BUZZER_OFF;
FAN1_OFF;

}

break;
}
}
}[/syntax]

Statystyki: Napisane przez padre159 — 18 paź 2017, o 20:41

Re: Pętle while

2017-10-18T20:29:58+01:00

No to poprzednio wstawiłeś inny kod
W miejscu które wskazałeś miałeś
[syntax=c]stan = 1;[/syntax]
To jak masz w kompilowanym kodzie?

Statystyki: Napisane przez MirkoT — 18 paź 2017, o 20:29

Re: Pętle while

2017-10-18T13:43:00+01:00

Zobacz to:

[syntax=c]while(1)
{
switch(stan)
{
case 0:
if(SENSOR_ACTIVE)
{
stan = 1;
}

if(s1_flag) { /* sprawdzanie flagi tyknięć timera programowego co 1 sekundę */

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 0 sprawdzaj ilość dostępnych czujników */
if( 0 == (sekundy%3) ) {

uint8_t *cl=(uint8_t*)gSensorIDs; // pobieramy wskaźnik do tablicy adresów czujników
for( uint8_t i=0; i czujniki_cnt = search_sensors(); // ponownie wykrywamy ile jest czujników i zapełniamy tablicę
lcd_locate(0,8);
lcd_int( czujniki_cnt ); // wyświetlamy ilość czujników na magistrali
}

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 1 wysyłaj rozkaz pomiaru do czujników */
if( 1 == (sekundy%3) ) DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 2 czyli jedną sekundę po rozkazie konwersji
* dokonuj odczytu i wyświetlania temperatur z 2 czujników jeśli są podłączone, jeśli nie
* to pokaż komunikat o błędzie
*/

if( 2 == (sekundy%3) )
{
if( klawisz1() )
{
if( licz1 == 1 )
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T1 T2")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 2)
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T3 T4")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[2], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[3], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 3)
{
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,0);
lcd_str("BRAK ZAGROZENIA!");
licz1=0;
}
}
}
/* zerujemy flagę aby tylko jeden raz w ciągu sekundy wykonać operacje */
s1_flag=0;
} /* koniec sprawdzania flagi */
break; /* koniec pętli pomiarowej*/
case 1:
if(!SENSOR_ACTIVE)
{
stan = 0; /* TU JEST ZMIANA*/
}

if(SENSOR_ACTIVE)
{
FAN1_ON;
BUZZER_ON;
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,1);
lcd_str("WYKRYTO DYM!!!");
_delay_ms(500);
}
else
{
BUZZER_OFF;
FAN1_OFF;

}

}
}
}[/syntax]

Statystyki: Napisane przez Jarecki — 18 paź 2017, o 13:43

Re: Pętle while

2017-10-18T13:39:45+01:00

A gdzie zerujesz flagę stan?
Bez tego nie powróci Ci do wykonywanie pierwszego warunku.

Statystyki: Napisane przez MirkoT — 18 paź 2017, o 13:39

Re: Pętle while

2017-10-18T12:50:23+01:00

Spróbuję.
Dzięki za szybką odpowiedź

------------------------ [ Dodano po: 28 minutach ]

Po podpowiedzi kolegi kod wygląda tak. Natomiast nie wyświetla temperatury. Reaguje na gaz włącza wiatrak buzzer i napis po odstawieniu gazu wiatrak i buzzer się wyłączają, napis zostaje i nie ma temp

[syntax=c]while(1)
{
switch(stan)
{
case 0:
if(SENSOR_ACTIVE)
{
stan = 1;
}

if(s1_flag) {/* sprawdzanie flagi tyknięć timera programowego co 1 sekundę */

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 0 sprawdzaj ilość dostępnych czujników */
if( 0 == (sekundy%3) ) {

uint8_t *cl=(uint8_t*)gSensorIDs;// pobieramy wskaźnik do tablicy adresów czujników
for( uint8_t i=0; iczujniki_cnt = search_sensors();// ponownie wykrywamy ile jest czujników i zapełniamy tablicę
lcd_locate(0,8);
lcd_int( czujniki_cnt );// wyświetlamy ilość czujników na magistrali
}

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 1 wysyłaj rozkaz pomiaru do czujników */
if( 1 == (sekundy%3) ) DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 2 czyli jedną sekundę po rozkazie konwersji
* dokonuj odczytu i wyświetlania temperatur z 2 czujników jeśli są podłączone, jeśli nie
* to pokaż komunikat o błędzie
*/

if( 2 == (sekundy%3) )
{
if( klawisz1() )
{
if( licz1 == 1 )
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T1 T2")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 2)
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T3 T4")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[2], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[3], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 3)
{
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,0);
lcd_str("BRAK ZAGROZENIA!");
licz1=0;
}
}
}
/* zerujemy flagę aby tylko jeden raz w ciągu sekundy wykonać operacje */
s1_flag=0;
} /* koniec sprawdzania flagi */
break; /* koniec pętli pomiarowej*/
case 1:
if(!SENSOR_ACTIVE)
{
stan = 1;
}

if(SENSOR_ACTIVE)
{
FAN1_ON;
BUZZER_ON;
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,1);
lcd_str("WYKRYTO DYM!!!");
_delay_ms(500);
}
else
{
BUZZER_OFF;
FAN1_OFF;

}

}
}
}[/syntax]

Statystyki: Napisane przez padre159 — 18 paź 2017, o 12:50

Re: Pętle while

2017-10-18T12:45:20+01:00

Zrób tak:

[syntax=c]uint8_t stan = 0; /* 0 - czesc pierwsza, 1- czesc druga programu */

int main(void)
{

while(1)
{

switch(stan )
{
case 0:

if(wykryto_gaz)
{
stan = 1;
}

/*
Czesc pierwsza programu
*/

break;

case 1:

if(brak_gazu)
{
stan = 1;
}

/*
Czesc druga programu
*/

break;
}/* switch */
}/* while */
}[/syntax]

Statystyki: Napisane przez Jarecki — 18 paź 2017, o 12:45

Pętle while

2017-10-18T12:38:19+01:00

Witam
Mam zagwostkę z 2 pętlami while.
Mianowicie mam sobie jedną pętlę która mierzy temp i ją wyświetla. Projekt 1_Wire Pana Mirka (chyba większości znany)
I teraz dołożyłem do tego czujnik gazów MQ2. W tej pierwszej pętli jako głównej dołożyłem if`a sprawdzajacego stan czujnika i w razie wykrycia gazu przejście do drugiej pętli while która realizuje załączenie wentylatora buzzera i napis na wyświetlaczu.
Po zaniku gazu nie mogę wrócić do poprzedniej pętli... Czasem nawet nie przeskakuje do tej drugiej po wykryciu gazu
Wykombinowałem to sobie tak

[syntax=c]while(x==1)
{
if(SENSOR_ACTIVE)
{
x++;
}

if(s1_flag) {/* sprawdzanie flagi tyknięć timera programowego co 1 sekundę */

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 0 sprawdzaj ilość dostępnych czujników */
if( 0 == (sekundy%3) ) {

uint8_t *cl=(uint8_t*)gSensorIDs;// pobieramy wskaźnik do tablicy adresów czujników
for( uint8_t i=0; iczujniki_cnt = search_sensors();// ponownie wykrywamy ile jest czujników i zapełniamy tablicę
lcd_locate(0,8);
lcd_int( czujniki_cnt );// wyświetlamy ilość czujników na magistrali
}

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 1 wysyłaj rozkaz pomiaru do czujników */
if( 1 == (sekundy%3) ) DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* co trzy sekundy gdy reszta z dzielenia modulo 3 == 2 czyli jedną sekundę po rozkazie konwersji
* dokonuj odczytu i wyświetlania temperatur z 2 czujników jeśli są podłączone, jeśli nie
* to pokaż komunikat o błędzie
*/

if( 2 == (sekundy%3) )
{
if( klawisz1() )
{
if( licz1 == 1 )
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T1 T2")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 2)
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR(" T3 T4")); /* wyświetl napisy w zerowej linii LCD */
lcd_locate(0,6);
lcd_int(licz1);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[2], &subzero, &cel, &cel_fract_bits) ) display_temp(0);
else {
lcd_locate(1,0);
lcd_str(" error ");
}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[3], &subzero, &cel, &cel_fract_bits) ) display_temp(9);
else {
lcd_locate(1,8);
lcd_str(" error ");
}
}
else
if(licz1 == 3)
{
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,0);
lcd_str("BRAK ZAGROZENIA!");
licz1=0;
}
}
}
/* zerujemy flagę aby tylko jeden raz w ciągu sekundy wykonać operacje */
s1_flag=0;
} /* koniec sprawdzania flagi */
} /* koniec pętli pomiarowej*/

while( x == 2 )
{
if(SENSOR_ACTIVE)
{
FAN1_ON;
BUZZER_ON;
lcd_cls();
lcd_locate (0,2);
lcd_str("czujnik dymu");
lcd_locate (1,1);
lcd_str("WYKRYTO DYM!!!");
_delay_ms(500);
}
else
{
BUZZER_OFF;
FAN1_OFF;
x = 1;
}

}
}[/syntax]

Statystyki: Napisane przez padre159 — 18 paź 2017, o 12:38