ATNEL tech-forum

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-21T13:48:46+01:00

xentis napisał(a):

andrews napisał(a):

W sumie autor wątku tak właśnie robi (przynajmniej tak to wygląda).

być może, ale samo nazewnictwo:

matekogon napisał(a):

wsk1 zawiera moja_nazwa

nasuwa podejrzenie, że autor albo z przyzwyczajenia źle nazywa to co przechowuje wskaźnik, albo do końca nie wie jak wskaźniki działają. Wskaźnik może pokazywać na daną moja_nazwa a nie ją zawierać. Wiem że to może czepialstwo ale z drugiej strony to forum przeglądają również początkujący którzy niekiedy posty innych traktują jako konkretne przykłady więc błędne nazewnictwo niepotrzebnie utrwala błędne pojęcia.

Niewątpliwie komentarze nie są adekwatne do tego, co kod faktycznie robi i bardzo dobrze, że na to zwróciłeś uwagę. Nie było moim zamiarem w jakiś sposób zdyskredytować Twojej odpowiedzi. Starałem się po prostu wskazać możliwe przyczyny nieprawidłowego działania kodu. Potraktowałem to jako uzupełnienie Twojej odpowiedzi, a nie jako jej krytykę

Mam wrażenie, że chyba trochę źle to odebrałeś

Statystyki: Napisane przez andrews — 21 sty 2018, o 13:48

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-21T12:13:55+01:00

xentis napisał(a):

że autor albo z przyzwyczajenia źle nazywa to co przechowuje wskaźnik, albo do końca nie wie jak wskaźniki działają.

Statystyki: Napisane przez matekogon — 21 sty 2018, o 12:13

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-21T12:00:41+01:00

andrews napisał(a):

W sumie autor wątku tak właśnie robi (przynajmniej tak to wygląda).

być może, ale samo nazewnictwo:

matekogon napisał(a):

wsk1 zawiera moja_nazwa

Statystyki: Napisane przez xentis — 21 sty 2018, o 12:00

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-21T11:51:17+01:00

andrews napisał(a):

[quote="xentis"
Moim zdaniem problemem może być czas życia zmiennej wskazywanej przez wskaźnik reszta. Istnieje prawdopodobieństwo, że w momencie wysyłania:[syntax=c]pakiet_str("+SAVETRANS:",konfiguracja.savetranslink);// funkcja która wysyła dane jednak nie wysyła mi tego co powinno
uart_puts(konfiguracja.savetranslink);// podejrzałem sobie co się znajduje w tej zmiennej i nie ma tam moja_nazwa[/syntax]zmienna wskazywana przez reszta już nie istnieje (np. była zadeklarowana jako zmienna lokalna i uległa degradacji po zakończeniu funkcji). Autor wątku przedstawił zbyt mało kodu, aby to można było ocenić.

Inną przyczyną może być to, że od momentu przypisania wskaźnika do zmiennej konfiguracja.savetranslink do momentu wysłania (pakiet_str("+SAVETRANS:",konfiguracja.savetranslink);) zawartość ciągu, na który wskazuje konfiguracja.savetranslink, została zmieniona. Ponownie dodam, że autor przedstawił zbyt mało informacji, aby jednoznacznie zdiagnozować problem.

Kolego masz rację teraz sobie to sprawdziłem i między wysłaniem komendy AT a wysłaniem AVRGET wysyłane sa również inne komendy a że "reszta" jest lokalna więc jest podmieniana

Czy jedynym rozwiązaniem jest napisanie funkcji która będzie mi kopiowała znak po znaku do tablicy aby nie stracić zawartości w postaci ciągu znaków "moja_nazwa"?

Moje założenie jest takie wysyłam komendę z ciągiem znaków "moja nazwa" i chce ja przechowywać w strukturze

Statystyki: Napisane przez matekogon — 21 sty 2018, o 11:51

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-21T10:25:15+01:00

xentis napisał(a):

Więc najpierw musisz utworzyć zmienną, która będzie zawierała coś co nazywasz "moja nazwa" a dopiero możesz we wskaźniku umieścić adres pamięci pod którym jest zapisany ww coś "moja_nazwa"

W sumie autor wątku tak właśnie robi (przynajmniej tak to wygląda). Rozumiem to tak, że zmienna reszta jest wskaźnikiem na ciąg znaków odebranej komendy AT. Funkcja strtok() zwraca adres początku podciągu "moja_nazwa", który to adres zostaje przypisany do elementu struktury konfiguracja.savetranslink (która, jak się domyślam, jest zdefiniowna jako globalna) i odbywa się to prawidłowo skoro:[syntax=c]uart_puts(wsk1);// wsk1 nadal zawiera moja_nazwa[/syntax]
Moim zdaniem problemem może być czas życia zmiennej wskazywanej przez wskaźnik reszta. Istnieje prawdopodobieństwo, że w momencie wysyłania:[syntax=c]pakiet_str("+SAVETRANS:",konfiguracja.savetranslink);// funkcja która wysyła dane jednak nie wysyła mi tego co powinno
uart_puts(konfiguracja.savetranslink);// podejrzałem sobie co się znajduje w tej zmiennej i nie ma tam moja_nazwa[/syntax]zmienna wskazywana przez reszta już nie istnieje (np. była zadeklarowana jako zmienna lokalna i uległa degradacji po zakończeniu funkcji). Autor wątku przedstawił zbyt mało kodu, aby to można było ocenić.

Inną przyczyną może być to, że od momentu przypisania wskaźnika do zmiennej konfiguracja.savetranslink do momentu wysłania (pakiet_str("+SAVETRANS:",konfiguracja.savetranslink);) zawartość ciągu, na który wskazuje konfiguracja.savetranslink, została zmieniona. Ponownie dodam, że autor przedstawił zbyt mało informacji, aby jednoznacznie zdiagnozować problem.

Statystyki: Napisane przez andrews — 21 sty 2018, o 10:25

Re: podpowiedz gdzie popełniłem błąd w kodzie

2018-01-20T21:17:22+01:00

Ale wskaźnik nie może zawierać "moja_nazwa" ponieważ jest wskaźnikiem i może zawierać jedynie adres pamięci w którym znajdować się może "moja_nazwa"
Więc najpierw musisz utworzyć zmienną, która będzie zawierała coś co nazywasz "moja nazwa" a dopiero możesz we wskaźniku umieścić adres pamięci pod którym jest zapisany ww coś "moja_nazwa"

Statystyki: Napisane przez xentis — 20 sty 2018, o 21:17

podpowiedz gdzie popełniłem błąd w kodzie

2018-01-20T19:20:36+01:00

Koledzy podpowie ktoś czy w dobrym miejscu namierzyłem błąd? Prawdopodobnie wyłożyłem się na wskaźnikach
To moja struktura
[syntax=c]typedef struct{

char *savetranslink;//ustawienie danych do programowania

}konfig;

konfig konfiguracja;[/syntax]

A tu wysyłam dane po udp

[syntax=c]//Wysyłam ciąg AT+SAVETRANS:moja_nazwa
if(strncmp("AT+SAVETRANS", reszta,12)==0){// ustawienie savetranslink
char *wsk1;
wsk1=strtok(reszta,":");
wsk1=strtok(NULL,":");// wsk1 zawiera moja_nazwa
konfiguracja.savetranslink=wsk1;// chyba tu jest błąd
uart_puts(wsk1);// wsk1 nadal zawiera moja_nazwa

}[/syntax]
Następnie wysyłam kolejne polecenie
[syntax=c]// wysyłam polecenie odczytu
if(strncmp("AVRGET", reszta,6)==0){// odczyt ustawien avr
pakiet_str("+SAVETRANS:",konfiguracja.savetranslink);// funkcja która wysyła dane jednak nie wysyła mi tego co powinno
uart_puts(konfiguracja.savetranslink);// podejrzałem sobie co się znajduje w tej zmiennej i nie ma tam moja_nazwa
}[/syntax]

Statystyki: Napisane przez matekogon — 20 sty 2018, o 19:20