Liczba postów: 880
Liczba wątków: 19
Dołączył: Jun 2018
Reputacja:
42
05-05-2021, 16:55
(Ten post był ostatnio modyfikowany: 05-05-2021, 16:56 przez Jarewa0606.)
Biblioteki się gryzą żeby to rozwiązać musiał byś przejrzeć je i o ile da radę ustawić SPI na sztywno, bo w tym przypadku każde urządzenie ma inna konfigurację SPI...
ogólnie to by działało tak na skróty
void loop() {
int pos1, pos2, pos3;
LoRa.begin (); // myślę ze chyba, albo przepisać ze setupa...
int packetSize = LoRa.parsePacket();
if (packetSize)
{
String LoRaData = LoRa.readString();
while (LoRa.available())
{
((char)LoRa.read());
}
pos1 = LoRaData.indexOf('/');
pos2 = LoRaData.indexOf('&');
pos3 = LoRaData.indexOf('!');
temperatura = LoRaData.substring(0,pos1);
cisnienie = LoRaData.substring(pos1 + 1, pos2);
wilgotnosc = LoRaData.substring(pos2 +1, pos3);
bateria = LoRaData.substring(pos3 +1, LoRaData.length());
Serial.println(temperatura);
Serial.println(cisnienie);
Serial.println(wilgotnosc);
Serial.println(bateria);
Serial.println("KONTROLA3");
}
Serial.println("KONTROLA4");
u8g2.begin ();
u8g2.clearBuffer();
u8g2.setFont (u8g2_font_ncenB14_tr);
u8g2.setCursor(2,16);
u8g2.print("EKRAN W LOOP");
u8g2.sendBuffer ();
} // KONIEC LOOP
Czyli za każdym razem jak zmieniasz urządzenie to inicjalizacja urządzenia wykonać....
Arduino zostało wymyślone po to, by robić dobrze jedną prostą rzecz – migać diodą.
Liczba postów: 880
Liczba wątków: 19
Dołączył: Jun 2018
Reputacja:
42
05-05-2021, 19:35
(Ten post był ostatnio modyfikowany: 05-05-2021, 19:36 przez Jarewa0606.)
Żeby to było czytelne i miało sens rodziel tak
void LCD (){
// program wykonawczy dla lcd
u8g2.begin ();
u8g2.clearBuffer();
u8g2.setFont (u8g2_font_ncenB10_tr);
u8g2.setCursor(2,16);
u8g2.print("EKRAN W LOOP");
u8g2.sendBuffer ();
}
void Lora(){
//program wykonawczy dla komunikacji
int pos1, pos2, pos3;
LoRa.begin ();
int packetSize = LoRa.parsePacket();
if (packetSize)
{
String LoRaData = LoRa.readString();
while (LoRa.available())
{
((char)LoRa.read());
}
pos1 = LoRaData.indexOf('/');
pos2 = LoRaData.indexOf('&');
pos3 = LoRaData.indexOf('!');
temperatura = LoRaData.substring(0,pos1);
cisnienie = LoRaData.substring(pos1 + 1, pos2);
wilgotnosc = LoRaData.substring(pos2 +1, pos3);
bateria = LoRaData.substring(pos3 +1, LoRaData.length());
Serial.println(temperatura);
Serial.println(cisnienie);
Serial.println(wilgotnosc);
Serial.println(bateria);
}
void setup(){
}
void loop(){
LCD();
Lora();
}
A pożniej właśnie dajesz jakes flagi żeby coś się wykonało jak coś zostanie spełnione i robisz naprzykład
void loop(){
if(stan){
LCD();
}else{
Lora();
}
}
Staraj się unikać funkcji blokujących unikaj "while" jak ognia...
Arduino zostało wymyślone po to, by robić dobrze jedną prostą rzecz – migać diodą.
Liczba postów: 880
Liczba wątków: 19
Dołączył: Jun 2018
Reputacja:
42
07-05-2021, 05:41
(Ten post był ostatnio modyfikowany: 07-05-2021, 05:59 przez Jarewa0606.)
bo źle resetujesz flagę a najlepsze jeszcze ze to float
a float to cieżko trafić by był równy "1" i "0". Flagi to najlepiej BOOLa jest albo nie ma... Nawet jak u ciebie działało to się zatrzymało na LCD bo tam nie resetowałeś flagi.
Spróbuj bez zmian
Kod:
[code]#include <SPI.h>
#include <LoRa.h>
#include <U8g2lib.h>
String temperatura;
String cisnienie;
String wilgotnosc;
String bateria;
bool flaga = false;
#define ss_lcd 2 //LCD
#define sclk 14 // LCD
#define mosi 13 // LCD
#define ss 15 // LoRa
#define rst 12 //LoRa
U8G2_ST7920_128X64_F_SW_SPI u8g2(U8G2_R2, sclk, mosi ,ss_lcd, U8X8_PIN_NONE);
void LCD (){
// program wykonawczy dla lcd
flaga = false;
u8g2.begin ();
u8g2.clearBuffer();
u8g2.setFont (u8g2_font_ncenB10_tr);
u8g2.setCursor(2,16);
u8g2.print("EKRAN W LOOP");
u8g2.sendBuffer ();
}
void Lora(){
//program wykonawczy dla komunikacji
int pos1, pos2, pos3;
LoRa.begin(433E6);
int packetSize = LoRa.parsePacket();
if (packetSize)
{
flaga =true;
String LoRaData = LoRa.readString();
while (LoRa.available())
{
((char)LoRa.read());
}
pos1 = LoRaData.indexOf('/');
pos2 = LoRaData.indexOf('&');
pos3 = LoRaData.indexOf('!');
temperatura = LoRaData.substring(0,pos1);
cisnienie = LoRaData.substring(pos1 + 1, pos2);
wilgotnosc = LoRaData.substring(pos2 +1, pos3);
bateria = LoRaData.substring(pos3 +1, LoRaData.length());
Serial.println(temperatura);
Serial.println(cisnienie);
Serial.println(wilgotnosc);
Serial.println(bateria);
Serial.println(flaga);
}
}
void setup(){
Serial.begin(9600);
}
void loop(){
if (flaga)
{
LCD();
}
else
{
Lora();
}
}
[/code]
Arduino zostało wymyślone po to, by robić dobrze jedną prostą rzecz – migać diodą.
Liczba postów: 880
Liczba wątków: 19
Dołączył: Jun 2018
Reputacja:
42
07-05-2021, 20:18
(Ten post był ostatnio modyfikowany: 07-05-2021, 20:31 przez Jarewa0606.)
A nie teraz zauwazyłem
Gdzie się zapodziało to z wczesniejszych kodów??
LoRa.setPins(ss,rst);
Tego małeś nie wycinać.
Wiec dodaj to i nie zmieniaj kodu co podałem...
Bo widzę że nie wiesz co robi beging(); ?? to jest przekonfigurowanie SPI w locie wiec nie usuwaj go z viod lora i lcd bo program nie będzie działał przed odczytem z lory musisz przekonfigurować spi dla lory tak samo jak przed wyświetleniem musisz przekonfigurować spi dla lcd.. Kiedy usuniesz to jak zrobiłeś to program sie zawiesi bo drugie "kółko" procesora i odczyt z lory zostaje przekonfigurowany SPI dla LCD..
Arduino zostało wymyślone po to, by robić dobrze jedną prostą rzecz – migać diodą.
Liczba postów: 880
Liczba wątków: 19
Dołączył: Jun 2018
Reputacja:
42
08-05-2021, 08:44
(Ten post był ostatnio modyfikowany: 08-05-2021, 08:45 przez Jarewa0606.)
Tu jeszcze błąd
#define rst 12 //LoRa
Nie możesz wykorzystać to RST bo to pin SPI MISO znajdź inny pin wolny...
Podał byś schemat podłączeń..
Arduino zostało wymyślone po to, by robić dobrze jedną prostą rzecz – migać diodą.