GUI - wyjątek przy nie odczytaniu sygnału.

mateuszbaran / ECG-analyzer

Main ECG-analyzer repository

GNU General Public License v2.0

45 stars 38 forks source link

GUI - wyjątek przy nie odczytaniu sygnału. #15

Closed kn65op closed 11 years ago

kn65op commented 11 years ago

Należy zadbać by była odpowiednia reakcja na wartość zwróconą przy wywołaniu funkcji readFile:

true oznacza, że pliki udało się odczytać
false oznacza, że nie udało się odczytać plików. Przydatna by była informacja dla użytkownika, że nie udało się odczytać sygnału.

ghost commented 11 years ago

Zrobione, ale nie wydaje mi się nadal, że to nie był problem GUI (problem zgłoszony przez Fuko-Fabio). Jeżeli np. wczytamy pusty plik to też będziemy mieli błąd (w sensie plik zostanie otwarty, bo istnieje i da się go odczytać). Można sprawdzać rozmiar pliku jeśli jest podejrzanie mały nie wczytywać go ale co z tego każdy może sobie wczytać plik, który jest np. arkuszem excela, pdfem czy czymkolwiek co dla nas nie ma sensu.

ghost commented 11 years ago

Sorry mój błąd.. jeśli plik wczytywany jest plikiem pustym to już jest przed tym zabezpieczenie

kn65op commented 11 years ago

Chodzi mi o to, żeby jak nie uda się odczytać plików, to żeby nie leciał wyjątek, tylko informacja dla użytkownika.

ghost commented 11 years ago

A leci wyjątek?? Zrobiłem komunikat a nie rzucanie, jeśli coś rzuca wyjątek to podejrzewam:

ui.rawPlotWidget->setSignal(&(_ECGcontroller.raw_signal), &(_ECGcontroller.ecg_info), &(_ECGcontroller.r_peaks_data));

funkcja setSignal a nie GUI

kn65op commented 11 years ago

Ten: https://f.cloud.github.com/assets/2552823/54661/8e026f2e-5a9d-11e2-8de4-be71057a692f.png No tak, dlatego jeśli nie ma sygnału to lepiej nie wywoływać tej funkcji, widzę, że już jest to zamykam.