Feedback Manual #2
Conversation
… of the WS server + minor changes
…emoved unnecessary traits
…ent and TIDY THE CODE U BASTARD
…s. Maps are optimized for uuids now.
|
Całość powinna być jednym projektem z odpowiednio wydzielonymi targetami do budowania i odpalania. |
/leave
Message sent successfully. Server code: 200 OK
Enter room name
my_room
my_room
Joined room 'my_room'
[2022-09-28 08:43:22.891307024 UTC] SERVER: piotr has left the chat
[2022-09-28 08:43:28.457892037 UTC] SERVER: piotr has joined the chatTo jest dosyć nieoczekiwane zachowanie. Trzeba dwa razy wpisać nazwę pokoju po wyjściu, żeby ponownie dołączyć (a raczej, dopiero to co się wpisze za drugim razem jest brane pod uwagę). Dodatkowo, czemu dołączenie do pokoju oznacza opuszczenie pokoju (jakiego?) i natychmiastowe dołączenie? |
|
Dobrze byłoby poinformować również jak ktoś opuszcza pokój |
client/src/main.rs
Outdated
| const ADDR: &str = "http://0.0.0.0:8080"; | ||
| const WS_ADDR: &str = "ws://127.0.0.1:8000/ws"; |
There was a problem hiding this comment.
Najlepiej jakby to było argumentem podawanym z linii komend, z defaultowymi wartościami
server/src/main.rs
Outdated
| mod utils; | ||
|
|
||
| const WS_ADDR: &str = "127.0.0.1:8000/ws"; | ||
| const SERVER_SIGNATURE: &str = "SERVER"; //TODO: zarezerwowanie tego nicka |
There was a problem hiding this comment.
Gdybyśmy chcieli potencjalnie rozwinąć ten projekt i klienci mogliby się podłączać do wielu serwerów, wtedy opcja podawania nazw serwera byłaby przydatna (i pewnie do podanej nazwy chcemy dodać jakis odpowiedni prefix, żeby wiedzieć, że to serwer). Ale to tylko hipotetyczna sytuacja.
server/src/main.rs
Outdated
| } | ||
| }); | ||
|
|
||
| let addr = "127.0.0.1:8080" |
There was a problem hiding this comment.
Jeden adres zahardkodowany na górze, a ten tutaj. A spodziewam się, że powinno być to to samo ip
There was a problem hiding this comment.
Innymi słowy, należy to przenieść do wspólnego miejsca, a także umożliwić podanie tego jako argument przez CLI
…ption, minor changes
Made the project work under one cargo.toml, added a readme and descri…
No description provided.