Conversation
- init client proj
-add tests
| /// <summary> | ||
| /// Скачивает файл с сервера | ||
| /// </summary> | ||
| public async Task<(long, byte[])> Get(string path) |
There was a problem hiding this comment.
Все длительные операции должны быть прерываемы, так что и List, и Get должны принимать CancellationToken
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
Вообще, уже почти, но надо кое-что поправить ещё
|
|
||
| <PropertyGroup> | ||
| <OutputType>Exe</OutputType> | ||
| <TargetFramework>net5.0</TargetFramework> |
There was a problem hiding this comment.
Можно уже на .net 6 и C# 10 переходить, там всякие крутые штуки типа file-scoped namespaces и глобальных юзингов.
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
Уже совсем почти зачтена :)
| { | ||
| using var handle = server.StartServer(); | ||
| Assert.Throws<AggregateException>(() => client.List("pluuuuug").Wait()); | ||
| Assert.Throws<AggregateException>(() => client.List("pluuuuug", tokenSource.Token).Wait()); |
There was a problem hiding this comment.
Если Вы не собираетесь отменяться по этому tokenSource-у, то передавайте просто CancellationToken.None
| Console.WriteLine("Введите, к какому порту обращаться:"); | ||
| int.TryParse(Console.ReadLine(), out var port); | ||
| var client = new Client(ip, port); | ||
| var client = new Client(args[0], int.Parse(args[1])); |
There was a problem hiding this comment.
Надо проверить, что они действительно были переданы, и если нет, вывести справку об использовании
| Console.WriteLine("Get:\nФормат запроса:2 <path1: String> <path2: String>\npath1 - путь к файлу относительно того места, где запущен сервер\n" + | ||
| "npath2 - путь к файлу на локальной машине, где запущен клиент"); | ||
| Console.WriteLine("Введите exit, если хотите закрыть соединение с сервером"); | ||
| var request = Console.ReadLine().Split(' '); |
There was a problem hiding this comment.
Это полумеры :) Ещё раз, хочу команду вида dotnet run -- 127.0.0.1 8888 --list .
2 participants
No description provided.