ATNEL tech-forum

Re: Operacja If na liczbach.

2016-02-06T20:20:28+01:00

W linii 167 tworzysz string i umieszczasz go w wynik1.
Najlepiej zadeklaruj jakąś zmienną np. int xyz; i do niej daj xyz=(60.0 * obrotow_na_sekunde); i po kłopocie.

PS. Doczytaj o C String i o tablicach.

Statystyki: Napisane przez HomoChemicus — 6 lut 2016, o 20:20

Re: Operacja If na liczbach.

2016-02-06T18:38:24+01:00

Nadal nie rozumiesz, o co chodzi z tablicą wynik1[].
Tablica ta zawiera 16 znaków, które jeden po drugim są wysyłane na LCD. Co konkretnie na której pozycji jest nie wiem, ale może to być zestaw znaków ASCII np. taki

Code:

"OBROTY: 2560/min"

lub

Code:

"2500 obr/min. "

lub coś innego.
Zatem te twoje próby zamiany ascii na integer nie zadziałają. Wyciąganie z tej tablicy wartości obrotów jest możliwe, ale nieopłacalne i bezsensowne.
Załóżmy jednak, że ilość obrotów to cztery cyfry na pozycjach wynik1[8]do wynik1[11], wtedy możesz zrobić coś takiego (piszę na sucho):
[syntax=c]int obroty;
obroty = (wynik1[8] - '0') *1000 + (wynik1[9] - '0') *100 + (wynik1[10] - '0') *10 + (wynik1[11] - '0');[/syntax]
Myślę, że zadziała, ale czy jest sens, przecież wartość obrotów zanim została przeniesiona jako ASCII do tablicy wynik1[] musi być trzymana w innej zmiennej (np. typu integer) i musi być ona dostępna w programie. Ewentualnie jest wyliczana jakąś funkcją wcześniej; tak czy siak musi być dostępna gdzieś w programie.

Statystyki: Napisane przez PJS — 6 lut 2016, o 18:38

Re: Operacja If na liczbach.

2016-02-06T16:27:41+01:00

Próbowałem tak:
[syntax=c]int obroty;
obroty = atoi (wynik1);[/syntax]
i tak:
[syntax=c]int obroty = wynik1 - "0";[/syntax]
i tak:
[syntax=c]int obroty = (int)wynik1;[/syntax]
ale żaden z nich nie działa poprawnie z powyższym kodem..

Statystyki: Napisane przez damian5602 — 6 lut 2016, o 16:27

Re: Operacja If na liczbach.

2016-02-06T15:09:40+01:00

A więc tak:
1) wynik1[] to tablica znakowa - zawiera w sobie tekst w postaci znaków ASCII, a nie liczby całkowitej. Więc jak chcesz porównywać to z wartością 4000 czy inną?
2) zobacz co robi funkcja LCD_WriteText(wynik1);
3) zapoznaj się z BB, czyli poznaj podstawy C, a szczególnie typy.
Ad.1 w tablicy masz tekst: przykładowo 2400 czyli cztery znaki ASCII cyfr '2' '4' '0' '0', a nie liczbę 2400.
Ad2. osobiście nie znam zawartości LCD_WriteText(wynik1), ale myślę, że po prostu przekazujesz początek (adres) tablicy i funkcja wysyła kody znaków (zawarte w tej tablicy) na LCD.
Ad3. naprawdę zapoznaj się z BB lub inną książką o podstawach C, bo widać, że masz spore braki, a to w miarę czasu będzie miało wymierne negatywne skutki. Jak zrozumiesz typy zmiennych, a potem wskaźniki to całość będzie od razu dla ciebie jaśniejsza. A tak to z każdą modyfikacją, którą będziesz wprowadzał wystąpią problemy i pytanie o każdą najprostszą rzecz w pewnym momencie napierze tempa lawiny.

Statystyki: Napisane przez PJS — 6 lut 2016, o 15:09

Re: Operacja If na liczbach.

2016-02-06T13:15:08+01:00

PJS napisał(a):

Pierwsza sprawa - wynik1 to tablica char, więc przyjmuje wartości do 256, więc nic dziwnego, że masz problemy z porównaniami - poczytaj m.in. o niejawnej konwersji typów.
Druga sprawa - do tablicy odwołuj się przez wynik1[tu wstaw numer komórki], bo odwołanie typu wynik1 czy też *wynik1 jest błędne czy też niezbyt czytelne (w najlepszym razie) - przynajmniej dla początkującego.
Trzecia sprawa - czy opanowałeś podstawy języka C, czy też próbujesz rzucać się bez wiedzy: na zasadzie, a nuż się uda; jeśli to drugie to najpierw zapoznaj się z językiem C. Nie będę polecał innych książek, jako że te Mirka są dla początkującego wręcz idealne.

To dlaczego
[syntax=c]LCD_WriteText(wynik1);[/syntax]
wyświetla liczbę normalnie tzn. aktualne obroty.

A ok:
[syntax=c]char wynik1[16]; //bufor wyświetlacza[/syntax]
rozumiem.

Chcę dać taki kodzik:
[syntax=c]if(wynik1[16]<2000){
lcd_locate(0,16);
lcd_str("blue");
_delay_ms(150);
}else if( (wynik1[16]>2000) && (wynik1[16]<4000) ){
lcd_locate(0,16);
lcd_str("green");
_delay_ms(150);
}else if( (wynik1[16]>4000) && (wynik1[16]<6000) ){
lcd_locate(0,16);
lcd_str("yelow");
}else if( (wynik1[16]>6000) && (wynik1[16]<8000) ){
lcd_locate(0,16);
lcd_str("orange");
_delay_ms(150);
}else if( (wynik1[16]>8000) && (wynik1[16]<10000) ){
lcd_locate(0,16);
lcd_str("red");
_delay_ms(150);
}else if(wynik1[16]>10000){
lcd_locate(0,16);
lcd_str("pulsing red");
_delay_ms(150);
}[/syntax]

I tylko wykonuje "lcd str blue"

Statystyki: Napisane przez damian5602 — 6 lut 2016, o 13:15

Re: Operacja If na liczbach.

2016-02-06T13:00:32+01:00

Statystyki: Napisane przez PJS — 6 lut 2016, o 13:00

Re: Operacja If na liczbach.

2016-02-06T10:12:28+01:00

Tu jest sam kod do obrotów bez tych if:

[syntax=c]#include
#include
#include
#include
#include "HD44780.h"

//tutaj ustaw własne zasady pomiaru
#define POMIAR_ILOSC_IMPULSOW_NA_OBROT 4 //ilość inpulsów na jeden obrót
#define POMIAR_ILOSC_OBROTOW 10 //ile obrotów ma trwać pomiar

volatile unsigned long int pomiar_ilosc_cykli;
volatile unsigned int ilosc_przepelnien;
volatile unsigned char pomiar_zbocze_nr;
volatile unsigned char pomiar_gotowy;

volatile unsigned int ICR1_moment_A;
volatile unsigned int ICR1_moment_B;

char wynik1[16]; //bufor wyświetlacza

//------------------------------------------------------

ISR(TIMER1_CAPT_vect){

//Obsługa zdarzenia na ICP1

static unsigned int ICR1_aktualny; //wartość ICR1 złapana w aktualnie
//złapanym zboczu

//odczytaj złapany stan licznika
ICR1_aktualny = ICR1;

//czy to pierwsze złapane zbocze?
if(pomiar_zbocze_nr == 1){

//tak, pierwsze zbocze pomiarowe,

//uruchamiamy zliczanie przepełnień Timer1

//zgaś flagę przerwania od przepełnienia
TIFR |= (1<
//włącz przerwania od przepełnienia Timer1
TIMSK |= (1<
//zeruj licznik przepełnień Timera1
ilosc_przepelnien = 0;

//zapamiętujemy stan licznika początku obrotu (moment A)
ICR1_moment_A = ICR1_aktualny;

}else{

//każde następne zbocza pomiarowe

//czy to ostatnie zbocze (koniec pełnego obrotu silnika)?
if(pomiar_zbocze_nr == POMIAR_ILOSC_IMPULSOW_NA_OBROT
* POMIAR_ILOSC_OBROTOW){

//tak, wykryto ostatnie zbocze pomiarowe,
//czyli wykonał się pełny ostatni obrót tarczy

//wyłączamy przerwania z Input Capture Unit
//oraz przepełnienia Timer1
TIMSK &= ~( (1<
//zapamiętujemy zmienne dla main()
ICR1_moment_B = ICR1_aktualny;
pomiar_gotowy = 1;

}

//w pozostałych przypadkach nic nie robimy i czekamy na zliczanie
//dalszych zboczy

}

//zwiększ licznik wykrytych zboczy
pomiar_zbocze_nr++;

}

//------------------------------------------------------

ISR(TIMER1_OVF_vect){

//powiększ licznik przepełnień Timera1
ilosc_przepelnien++;

}

//------------------------------------------------------

void wykonaj_pomiar(void){

//Funkcja włącza pomiar

//zerujemy licznik przepełnień Timer0
ilosc_przepelnien = 0;

//ustawiamy nr pierwszego zbocza
pomiar_zbocze_nr = 1;

//zerujemy flagę gotowości pomiaru
pomiar_gotowy = 0;

//włącz przerwania z ICP1 Timer1
TIMSK |= (1<
}

//------------------------------------------------------

int main(void){

//zmienne do obliczeń
double czas_jednego_obrotu;
double obrotow_na_sekunde;

PORTB |= (1<
LCD_Initalize(); //inicjalizacja LCD i napis początkowy
LCD_Clear();
LCD_GoTo(0, 0);
LCD_WriteText("Obrotomierz");

//Ustawienie Timer1
TCCR1A = 0;
TCCR1B = (1< //canceler dla ICP1

//lub z modułem Noise Canceler:
//TCCR1B = (1< //noise canceler dla ICP1

sei(); //włącz przerwania globalne

wykonaj_pomiar(); //włącz pierwszy pomiar

//pętla główna
while(1){

if(pomiar_gotowy){

// gdy pomiar został wykonany

//obliczenia

pomiar_ilosc_cykli = 65536 * ilosc_przepelnien - ICR1_moment_A
+ ICR1_moment_B;

czas_jednego_obrotu = (double) pomiar_ilosc_cykli
/ POMIAR_ILOSC_OBROTOW / F_CPU ;

obrotow_na_sekunde = 1.0 / czas_jednego_obrotu;

//wyświetlenie wyników

LCD_Clear();

LCD_GoTo(13, 0);
LCD_WriteText("rpm");
//obroty na minutę
sprintf(wynik1,"%7u ", (int) (60.0 * obrotow_na_sekunde));
LCD_GoTo(9, 1);
LCD_WriteText(wynik1);

//pomiary co pół sekundy
_delay_ms(500);

//wykonaj następny pomiar
wykonaj_pomiar();

}
}
}[/syntax]

Statystyki: Napisane przez damian5602 — 6 lut 2016, o 10:12

Re: Operacja If na liczbach.

2016-02-05T20:15:26+01:00

pokaż więcej kodu bo cóś (pisownia zamierzona) kręcisz

Statystyki: Napisane przez MirkoT — 5 lut 2016, o 20:15

Re: Operacja If na liczbach.

2016-02-05T12:53:05+01:00

MirkoT napisał(a):

damian5602 napisał(a):

Mam pewien problem.
Jeżeli dam tak:
[syntax=c]if(*wynik1>3000)[/syntax]

To nic się nie dzieje.

a jeżeli dam:
[syntax=c]if(*wynik1<3000)[/syntax]
to jest to wykonywane poprawnie.

I bardzo dobrze!
Tak ma być. Przecież wszystko zależy od tego co masz w zmiennej wynik1
Nie może być równocześnie i większe i mniejsze

Chodzi o to że jak dam
[syntax=c]if(*wynik1>3000)[/syntax]
zamiast
[syntax=c]if(*wynik1<3000)[/syntax]
i wynik1 ma wartość np 4500 to nic się nie dzieje.

Statystyki: Napisane przez damian5602 — 5 lut 2016, o 12:53

Re: Operacja If na liczbach.

2016-02-05T12:13:08+01:00

damian5602 napisał(a):

Mam pewien problem.
Jeżeli dam tak:
[syntax=c]if(*wynik1>3000)[/syntax]

To nic się nie dzieje.

a jeżeli dam:
[syntax=c]if(*wynik1<3000)[/syntax]
to jest to wykonywane poprawnie.

I bardzo dobrze!
Tak ma być. Przecież wszystko zależy od tego co masz w zmiennej wynik1
Nie może być równocześnie i większe i mniejsze

Statystyki: Napisane przez MirkoT — 5 lut 2016, o 12:13

Re: Operacja If na liczbach.

2016-02-05T11:54:07+01:00

Mam pewien problem.
Jeżeli dam tak:
[syntax=c]if(*wynik1>3000)[/syntax]

To nic się nie dzieje.

a jeżeli dam:
[syntax=c]if(*wynik1<3000)[/syntax]
to jest to wykonywane poprawnie.

Statystyki: Napisane przez damian5602 — 5 lut 2016, o 11:54

Re: Operacja If na liczbach.

2016-02-05T10:46:27+01:00

damian5602 napisał(a):

BB czyli? (Blue Book?)

@EDIT
Zrobiłem tak:
[syntax=c]if(*wynik1>2000){
if(4000>*wynik1){

Coś tam
}
}[/syntax]
Ale kod nie działa

Tak chodzi o Niebieską Książkę
Zależy co chcesz osiągnąć. Może byś zrobił tak:
[syntax=c]if( (*wynik>2000) && (*wynik<4000) ){
coś_tam;
}[/syntax]

-------
Kolega marhef był szybszy

Statystyki: Napisane przez MirkoT — 5 lut 2016, o 10:46

Re: Operacja If na liczbach.

2016-02-05T10:45:36+01:00

damian5602 napisał(a):

Zrobiłem tak:
[syntax=c]if(*wynik1>2000){
if(4000>*wynik1){

Coś tam
}
}[/syntax]
Ale kod nie działa

Spróbuj zrobić tak:
[syntax=c]if((*wynik1>2000) && (*wynik<4000)){

Coś tam

}[/syntax]
Problemem może być wyrażenie >*

Statystyki: Napisane przez Marhef — 5 lut 2016, o 10:45

Re: Operacja If na liczbach.

2016-02-05T10:05:11+01:00

PJS napisał(a):

No przecież pisze co robisz źle: porównujesz wskaźnik z wartością integer. Może zrób tak:
[syntax=c]If(*wynik1<1000){
kawalek kodu
}[/syntax]

Dziękuje działa,
Jednak nie działa poprawnie, nawet gdy wynik1 jest 4000 to i tak wykonuje polecenie.

Teraz mam problem z tym kodem:

[syntax=c]}else if(4000>*wynik1>2000){[/syntax]

Code:

comparisons like 'X<=Y<=Z' do not have their mathematical meaning [-Wparentheses]

Czyli nie ma sensu matematycznego

rskup napisał(a):

Błąd oznacza, że próbujesz porównać zmienną będącą wskaźnikiem z liczbą.
Musisz użyć operatora wyciągania wartości z pod wskaźnika (*). W BB jest to pięknie opisane. I jak chcesz programować, to musisz to zrozumieć.

--
Pozdrawiam,
Robert

BB czyli? (Blue Book?)

@EDIT
Zrobiłem tak:
[syntax=c]if(*wynik1>2000){
if(4000>*wynik1){

Coś tam
}
}[/syntax]
Ale kod nie działa

Statystyki: Napisane przez damian5602 — 5 lut 2016, o 10:05

Re: Operacja If na liczbach.

2016-02-04T20:04:22+01:00

Statystyki: Napisane przez rskup — 4 lut 2016, o 20:04

Re: Operacja If na liczbach.

2016-02-04T20:03:39+01:00

No przecież pisze co robisz źle: porównujesz wskaźnik z wartością integer. Może zrób tak:
[syntax=c]If(*wynik1<1000){
kawalek kodu
}[/syntax]

Statystyki: Napisane przez PJS — 4 lut 2016, o 20:03

Re: Operacja If na liczbach.

2016-02-04T20:00:53+01:00

...a z jakiej litery piszesz if?

Statystyki: Napisane przez kicajek — 4 lut 2016, o 20:00

Operacja If na liczbach.

2016-02-04T19:17:44+01:00

Witam, bez bicia przyznaję od razu przepisałem kodzik z stronki:
....
i próbuje zrobić operację If np. taką:
[syntax=c]If(wynik1<1000){
kawalek kodu
}[/syntax]

to wywala mi taki błąd:

Code:

comparison between pointer and integer [enabled by default]

I ktoś mi wyjaśni jak sobie z tym poradzić ?

Statystyki: Napisane przez damian5602 — 4 lut 2016, o 19:17