Chciałbym, aby dany blok instrukcji nie był wykonany, aż do spełnienia 2 warunków jednocześnie.
Jaką pętle zastosować, tak żeby się powtarzała i sprawdzała warunki? Nie wiem kiedy zostaną spełnione warunki.
- Rejestracja:ponad 13 lat
- Ostatnio:prawie 3 lata
- Postów:4882
2
albo mozna to zrobic troszke inaczej, jezeli te warunki sa polaczone ze soba jakas logika (biznesowa, albo po prostu w kodzie jeden mocno zalezy od drugiego) to wtedy mozna polaczyc to w klase i sprawdzac
while(nameOfValidator.IsValid()){}
gdzies w petli wywolujesz Validate() przez co moga zostac zmienione te parametry na ktorych Ci zalezy
Dlaczego tak zrobic?
Bo mozesz latwo wyciagnac zaleznosc i latwiej takie cos testowac
- Rejestracja:ponad 7 lat
- Ostatnio:ponad 7 lat
- Lokalizacja:UK
- Postów:25
0
Mozesz uzyc kazdej petli (nawet for, bo tam tez jest warunek logiczny), ale najbardziej sensowny wydaje sie wybor sporod: while lub do-while. Jezeli nie wiesz kiedy warunki zostana spelnione musisz jednak wziac pod uwage jak uniknac dzialania petli w nieskonczonosc.
Zobacz pozostałe 2 komentarze
utworz sztuczny iterator i co kazdy przebieg resetuj jego wskaznik - teoretycznie masz petle nieskonczona tylko w sumie nawet z nozem przy gardle bym tego nie zrobil :)
foreach to nie pętla.
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/foreach-in wedlug tego nie ma petli jako takiego pojecia ;)
A co to jest, Kononowicz Driven Development?
- Rejestracja:około 8 lat
- Ostatnio:prawie 7 lat
- Postów:34
0
Problem polega na tym, że chce przyjąć byte z serial portu. Program za szybko się wykonuje. Wiec pomyślałem sobie, że warto by było zrobić pętle, która się wykonuje i jak przyjdą dane SerialPort.IsOpen > 0 i wartość tego byte co przyjdzie będzie równa 0. Wykona się kod. Problem polega na tym, że jak zrobię przeciwieństwo czyli (SerialPort.IsOpen !=0 i wartość byte != 1) program nie reaguje. Reasumując chce, żeby pętla przechodziła tyle razy aż przyjdą dane na serial port i te dane będą równe 1 wykona się blok instrukcji.
- Rejestracja:ponad 13 lat
- Ostatnio:prawie 3 lata
- Postów:4882
0
Nie potrzebujesz petli by czekac
Autor nie sprecyzowal technologii, ale masz racje. Chociaz wewnatrz tego wrappera i tak na pewno chodzi petla, ktora "czeka".
0
@bartk: Nie ma tam żadnej pętli, jak coś się łączy na porcie szeregowym to wyzwala przerwanie sprzętowe, które uaktywania eventy.
- Rejestracja:ponad 13 lat
- Ostatnio:prawie 3 lata
- Postów:4882
0
ale zeby byl wysluchany event to musisz miec petle ktora nasluchuje i sprawdza co jakis czas czy cos przyszlo. Wiec wewnetrznie jestem prawie pewien (bo nie widzialem kodu wiec nie jestem 100% pewny) jest petla ktora wyczekuje na eventy
Chodzi właśnie o to, żeby nie tworzyć dodatkowych event-ów. Bo wiem, że właśnie po danej instrukcji przyjdzie kolejny sygnał, która ma wykonać blok instrukcji jak przyjdzie. Chodzi o to, że wiem co się stanie. Ale nie wiem za ile sekund.
- Rejestracja:ponad 7 lat
- Ostatnio:ponad 7 lat
- Postów:77
0
Pytanie trochę z innej strony, komunikujesz się za pomocą ASCII, czy binarnie? SerialDataReceivedEventHandler nie koniecznie dobrze sobie radzi z komunikacją binarną, potrafi wyskoczyć po 1 bajcie, albo po kilku.
Przesyłane są Bytes w jedną i drugą stronę.
Masz jakieś timeouty? Transmisja master-slave, czy asynchroniczna?
Żadne, timeouty nie zostały predefiniowane. Jak to wpływa na działanie ?
timeout wyrzuci wyjątek jak będziesz czekał na dane, dzięki czemu program nie utknie w jednym miejscu. (Rozwiązanie mało eleganckie ale działa)
- Rejestracja:ponad 7 lat
- Ostatnio:ponad 7 lat
- Postów:77
0
Czy otwierasz w ogóle port:
sp = new SerialPort("COM3");
sp.Open();
sp.DataReceived += Sp_DataReceived;
otwarcie portu nie jest asynchroniczne, czeka aż się otworzy nie ma potrzeby za bardzo sprwdzać isOpen.
- Rejestracja:około 8 lat
- Ostatnio:prawie 7 lat
- Postów:34
0
Może inaczej. W Klasie KlasaObslugiUSB mam umieszczonego SerialDataReceivedEventHandler. Teraz chce stworzyć metodę w programie głównym, w której jeżeli program odbierze byte, to sprawdzi jaka jest jego wartość i wykona odpowiedni blok instrukcji. Największym problem jest to, że Program za szybko działa i po odebraniu wyskakuje błąd "System.NullReferenceException: 'Odwołanie do obiektu nie zostało ustawione na wystąpienie obiektu.'" Chodzi o to, że program dalej sie wykonuje, zamiast wykonywać się po sprawdzeniu warunków. Czyli w tym przypadku Odebrania bytes i sprawdzenia jego wartości.
public partial class MainWindow : Window
{
public MainWindow()
{
InitializeComponent();
KlasaObslugiUSB.Instancja.Open();
}
private void Start_Click(object sender, RoutedEventArgs e)
{
KlasaObslugiUSB.Instancja.StartGry(); //wysłanie sygnału StartGry
//Thread.Sleep(250);
ZapalLampki();
}
private void ZapalLampki()
{
while (KlasaObslugiUSB.Instancja.Wynik[0] != 1)
{
Ellipse01.Fill = Brushes.Blue;
Ellipse02.Fill = Brushes.Blue;
Ellipse03.Fill = Brushes.Blue;
Ellipse04.Fill = Brushes.Blue;
Ellipse05.Fill = Brushes.Blue;
Ellipse06.Fill = Brushes.Blue;
Ellipse07.Fill = Brushes.Blue;
Ellipse08.Fill = Brushes.Blue;
Ellipse09.Fill = Brushes.Blue;
Ellipse10.Fill = Brushes.Blue;
}
}
private void Zeruj_Click(object sender, RoutedEventArgs e)
{
Ellipse01.Fill = Brushes.Blue;
Ellipse02.Fill = Brushes.Blue;
Ellipse03.Fill = Brushes.Blue;
Ellipse04.Fill = Brushes.Blue;
Ellipse05.Fill = Brushes.Blue;
Ellipse06.Fill = Brushes.Blue;
Ellipse07.Fill = Brushes.Blue;
Ellipse08.Fill = Brushes.Blue;
Ellipse09.Fill = Brushes.Blue;
Ellipse10.Fill = Brushes.Blue;
}
}
- Rejestracja:ponad 7 lat
- Ostatnio:ponad 7 lat
- Postów:77
1
Idąc tym tokiem działania, (kod ogólnie nie powinien tak wyglądać, trzeba by dużo pozmieniać, no ale idąc tym tokiem)
zamiast
while (KlasaObslugiUSB.Instancja.Wynik[0] != 1)
daj
while (KlasaObslugiUSB.Instancja.IsValid())
a w klasie KlasaObslugiUSB dodaj metodę
IsValid() {
return = (wynik!=null&&wynik.length>0);
}
I dodaj
private void Sp_DataReceived(object sender, SerialDataReceivedEventArgs e)
{
//tu przypisywanie wartości do wynik
}
i
sp.DataReceived += Sp_DataReceived; //zaraz po sp.Open();
edytowany 1x, ostatnio: Andrzej Guziec
fasadinfasadin