[ShookOnesPtII]

#programowanie #csharp #siszarp #wordpress #logowanie

Jak zabezpieczyć ten kod przed wywalaniem aplikacji? Coś oprócz Try {} Catch{} (brzydki kod).

http://pastebin.com/t5kdpJb8

Jest to kawałek metody służącej do logowania na jakikolwiek blog WordPress.

Wywaliło mi przy TcpClient client = new TcpClient(fulladdress, 80). Komunikat związany z timeoutem.

[patrolez]

@ShookOnesPtII: no po to ci wyjątki wymyślili, żebyś je używał :> Metoda niech wywala wyjątek jeżeli timeout :>

Piszę to ogólnie, bo nie znam C#, ale no po to wymyślili to cudeńko jak wyjątki (które są w każdym poważnym języku programowania), żeby życie ułatwić :>

[ShookOnesPtII]

@patrolez: Ale kod brzydki... :(

[Yahoo_]

@ShookOnesPtII: try...catch to piękny kod w porównaniu do tego kilometrowego stringa z 6. linijki :P

[kielonek]

@ShookOnesPtII: nie ma to jak narzekac na to ze try catch brzydki, a ta 6ta linijka to pozal się boze

[ShookOnesPtII]

@Yahoo_: @kielonek: Dobra, zrobiłem traj kaczem. Heh.

[G.....t]

@ShookOnesPtII: używanie sleep to zły pomysł

[ShookOnesPtII]

@Godlike_Atheist: To akurat w osobnym wątku się robi, więc nie przeszkadza. W sumie nie wiem czy ten Sleep tam musi być czy nie... Jeżeli networkStream.Read(buffer, 0, buffer.Length); czeka aż dane się "załadują", to Sleep jest zbędny.

[G.....t]

@ShookOnesPtII: nie musi go tam być, jest zbędny ;-)

[ShookOnesPtII]

@Godlike_Atheist: @kielonek: @Yahoo_: Jak byście podzielili ten kod z 6 linijki, żeby to ładnie wyglądało :>?

[Yahoo_]

@ShookOnesPtII:

Najprostsze sprawa to skorzystać z StringBuildera. Generalnie powinno się unikać "dodawania" stringów. Można też np. napisać jakieś extension method np. public static StringBuilder AppendHeader(this StringBuilder sb, string header, string value) albo w ogóle zamknąć requesta w jakiejś klasie jak będziesz z niego często korzystał. Zasada jest taka - pomyśl jakbyś chciał z czegoś korzystać, a potem napisz kod, który Ci to umożliwi :).

[japer]

@ShookOnesPtII: HttpWebRequest to ty używaj

[ShookOnesPtII]

@japer: Niestety, nie da się tego użyć przy logowaniu do Wordpress (jakiś bug z ciastkami) ;). Jeżeli już, to lepiej WebClient z obsługą ciastek.

[japer]

@ShookOnesPtII: webclientem możesz uploadować wartości przez metodę post

[ShookOnesPtII]

@japer: Wiem, no o to mi chodzi. Ani httpwebrequest ani webclientem NIE DA SIĘ zalogować na wordpressie. Trza na socketach. Wtedy czytam ciastka i zapisuję ręcznie do cookiecontainer.

[japer]

@ShookOnesPtII: btw używaj Encoding.ASCII a nie tworzysz klasę jakoś dziko :P

[ShookOnesPtII]

@japer: To tam akurat nieważne, bo bardzo rzadko używam tej metody (na początku tylko, żeby się zalogować na pewne blogi).

[japer]

a jak łapać błędy to tylko wyjątki. Inaczej się nie da.

[ShookOnesPtII]

@japer:

Generalnie zrobiłem to w taki sposób:

http://pastebin.com/maJT6inn

Response sprawdzam później (trzeba sprawdzić czy zalogowano poprawnie, wiadomo).

[japer]

@ShookOnesPtII: napieprzasz tak, aż zazębi? O.O

[ShookOnesPtII]

@japer: Nie no, MaximumFailCount==10 razy i koniec. W sumie 10 razy to też sporo.