Biblioteka sieciowa w C++

Odpowiedz Nowy wątek
2019-09-08 12:14

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

5

Hej. Napisałem łatwą w użyciu bibliotekę sieciową w C++. Na razie wsprarcie tylko pod linux (wsparcie pod windows jest planowane).
Bibliotekę będę używał głównie do własnych narzędzi adhoc, planuję też napisać "nad nią" serwer http. sprawdzić wydajność.
Dajcie znać co myślicie o kodzie / projekcie. Link do repo: https://github.com/lukascode/socket-nano.

Pozostało 580 znaków

2019-09-09 14:22

Rejestracja: 12 lat temu

Ostatnio: 1 minuta temu

3

W przykładzie jest jawnie użyty operator new: new HttpRequestHandler!
We współczesnym C++ jest to niezalecane.

Funkcjonalność HTTP, w której trzeba podać w całości nagłówki i body nie jest zbyt poręczna.

Tyle patrząc tylko na stronę główną. Do środka może jeszcze zajrzę.


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.
edytowany 4x, ostatnio: MarekR22, 2019-09-09 14:30
Przyzwyczaiłem się do surowych wskaźników ale mam w planach zmiane na smart. Jeżeli chodzi o HttpRequestHandler to jest tylko przykład jak używać biblioteki, nie jest częścią biblioteki. - lookacode1 2019-09-09 15:01

Pozostało 580 znaków

2019-09-09 15:24

Rejestracja: 6 lat temu

Ostatnio: 9 godzin temu

1

Przeglądnąłem na szybko.
IMHO słabo, że nie uzyłeś poll/epoll/select tylko bazujesz na callach blokujących. Fajnie by też było dołożyć obsługę wielu procesów, a nie tylko wątków.

Pozostało 580 znaków

2019-09-09 19:18

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

0
alagner napisał(a):

Przeglądnąłem na szybko.
IMHO słabo, że nie uzyłeś poll/epoll/select tylko bazujesz na callach blokujących. Fajnie by też było dołożyć obsługę wielu procesów, a nie tylko wątków.

Na razie bazuje na callach blokujących. Łatwiej przerzucić odpowiedzialność obsługi połączenia na użytkownika biblioteki, który zaimplementuje TcpConnectionHandler
i obsluga zostanie wystartowana w nowym wątku z puli. Do czego miałaby służyć obsługa wielu procesów?

Pozostało 580 znaków

2019-09-10 09:24

Rejestracja: 6 lat temu

Ostatnio: 9 godzin temu

0

Z tymi procesami jest trochę kwestia przyjętej konwencji i tego co Ci wyjdzie z pomiarów. Na linuxie imho wieloprocesowość jest łatwiejsza w utrzymywaniu, bo operujesz jednak na osobnych przestrzeniach adresowych i nie ryzykujesz, że weźmiesz nie swój zasób, bo współdzielone jest tylko to, co explicite takim uczynisz. Aczkolwiek może to mój osobisty przechył po latach programowania w C.
Do poczytania https://elinux.org/images/1/1c/Ben-Yossef-GoodBadUgly.pdf

Pozostało 580 znaków

2019-09-12 20:41

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

0

@alagner wydaje mi się, że wsparcie dla wątków jest wystarczające, czy jest jakaś biblioteka sieciowa, która daje wsparcie procesów?

Pozostało 580 znaków

2019-09-19 00:01

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

0

Jakieś rady co można by było ulepszyć?

Pozostało 580 znaków

2019-09-19 12:17

Rejestracja: 11 lat temu

Ostatnio: 10 godzin temu

1

Tak pobieżnie. Nie używaj surowych wskaźników, używaj deklaracji zapowiadających, dla typów integralnych masz do dyspozycji aliasy, np std::atomic_bool. Twój thread pool jest, hmm, taki sobie. Użyj może na początku jakiejś zgrabnej implementacji, np CTPL.


edytowany 1x, ostatnio: several, 2019-09-19 12:25

Pozostało 580 znaków

2019-09-19 17:53

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

0

Twój thread pool jest, hmm, taki sobie. Użyj może na początku jakiejś zgrabnej implementacji, np CTPL.

No nwm ja przynajmniej mam jakieś testy swojego, chyba zostane przy swoim.

Pozostało 580 znaków

2019-09-29 21:13

Rejestracja: 6 lat temu

Ostatnio: 3 miesiące temu

0

Zmieniłem surowe pointery na smart. Wygląda to dużo lepiej. Jeszcze jakieś rady / sugestie?
Czy powinienem wrzucić bibliotekę w przestrzeń nazw? Czy nazewnictwo jest ok?

Pozostało 580 znaków

2019-10-08 13:44

Rejestracja: 15 lat temu

Ostatnio: 22 godziny temu

1
void Socket::SendAll(const std::string &data)
{
    SendAll(std::vector<uint8_t>(data.begin(), data.end()));
}

Po co ten vector, skoro masz wersję funkcji przyjmujący wskaźnik i ilość bajtów?

Nie jestem też przekonany do modelu obsługi połączeń. Teraz jest tak, że połączenie na czas obsługi zajmuje wątek, co jest słabym rozwiązaniem, bo dobrze napisany serwer powinien być w stanie obsługiwać "jednocześnie" wiele połączeń na jednym wątku.

edytowany 1x, ostatnio: _0x666_, 2019-10-08 14:01

Pozostało 580 znaków

Odpowiedz

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0