KPOP - Możliwość odczytania aktualnej jednostki kanału

[krycha88]

Napisałem w temacie KPOP ale tam będzie mi się ciężko przebić z mało istotnym technicznym problemem Więc napisałem osobny wątek. Jeżeli to nie problem to proszę o dodanie w Supla::Channel możliwość odczytania jednostki kanału.

[Robert Błaszczak]

Popieram. Podaną w Cloud jednostkę będzie można wyświetlić na wyświetlaczu.

[klew]

W Supła:: Channel to nie przejdzie. W klasie od kpop są gettery i są też metody zwracające sformatowany tekst z jednostkami i przeliczone

[klew]

Napisz jeszcze co i dlaczego chciałbyś zrobić. Może jakoś to uda się dodać

[krycha88]

Nie chciałbym przechowywać dodatkowych obiektów po swojej stronie. Odczytywanie wartości np. dla OLED robię jak niżej jest to bardzo proste i uniwersalne

Code: Select all

  auto channel = getChanelByChannelNumber(oled[state->currentFrame].chanelSensor);

  if (channel && channel->getChannelType() == SUPLA_CHANNELTYPE_PRESSURESENSOR) {
    double lastPressure = channel->getValueDouble();

    displayUiGeneral(display, state, x, y, getPressureString(lastPressure), "hPa", pressure_bits);
  }

i do pełni szczęści brakuje mi

Code: Select all

const char* unit = channel->getChannelUnit(); 

[klew]

Mogę trochę te klasy KPOP/KLOP przeorganizować, aby te podstawowe parametry działały na klasie Supla::Channel.
Tylko mam tutaj kilka propozycji:
1. KPOP/KLOP mają nie tylko "jednostkę za wartością", ale też "przed", oraz ustawiane spacje między wartością i jednostką. Dlatego powstał getFormattedValue(), który zwraca gotowy char* z uwzględnieniem tych wszystkich ustawień, a także z obsługą wartości ---
2. Można tutaj ujednolicić interfejs i dla wszystkich typów kanałów zrobić metodę "getFormattedValue", która zwraca char* z wartością i jednostką (taką samą jak w apce). Wtedy nie trzeba będzie sprawdzać typów kanałów, tylko pobierać getFormattedValue()

[krycha88]

Mogę trochę te klasy KPOP/KLOP przeorganizować, aby te podstawowe parametry działały na klasie Supla::Channel.
Tylko mam tutaj kilka propozycji:
1. KPOP/KLOP mają nie tylko "jednostkę za wartością", ale też "przed", oraz ustawiane spacje między wartością i jednostką. Dlatego powstał getFormattedValue(), który zwraca gotowy char* z uwzględnieniem tych wszystkich ustawień, a także z obsługą wartości ---
2. Można tutaj ujednolicić interfejs i dla wszystkich typów kanałów zrobić metodę "getFormattedValue", która zwraca char* z wartością i jednostką (taką samą jak w apce). Wtedy nie trzeba będzie sprawdzać typów kanałów, tylko pobierać getFormattedValue()

Druga opcja brzmi ekstra. Ale ja nie będę narzekać i każda opcja będzie dobra

[SOYER]

Mogę trochę te klasy KPOP/KLOP przeorganizować, aby te podstawowe parametry działały na klasie Supla::Channel.
Tylko mam tutaj kilka propozycji:
1. KPOP/KLOP mają nie tylko "jednostkę za wartością", ale też "przed", oraz ustawiane spacje między wartością i jednostką. Dlatego powstał getFormattedValue(), który zwraca gotowy char* z uwzględnieniem tych wszystkich ustawień, a także z obsługą wartości ---
2. Można tutaj ujednolicić interfejs i dla wszystkich typów kanałów zrobić metodę "getFormattedValue", która zwraca char* z wartością i jednostką (taką samą jak w apce). Wtedy nie trzeba będzie sprawdzać typów kanałów, tylko pobierać getFormattedValue()

Druga opcja brzmi ekstra. Ale ja nie będę narzekać i każda opcja będzie dobra