Hw2 Lzw #4
Hw2 Lzw #4
Conversation
hw2LZW/hw2LZW/LZW.cs
Outdated
| using System.Collections; | ||
| using System.IO; | ||
|
|
||
| namespace hw2LZW |
There was a problem hiding this comment.
Пространства имён в .NET именуются всегда с заглавной, и пространство имён всего проекта называется по умолчанию так же, как сам проект, поэтому и проекты стоит называть с заглавной
| } | ||
| } | ||
| return 1; | ||
| } |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
hw2LZW/hw2LZW/Trie.cs
Outdated
| } | ||
|
|
||
| private bool CheckAdd(byte value, Node node) | ||
| => node.IsFind(value) == null ? true : false; |
There was a problem hiding this comment.
? true : false можно не писать, == и так возвращает булевое значение
hw2LZW/hw2LZW/Trie.cs
Outdated
| if (runner == root) | ||
| { | ||
| var check = runner.Sons[value]; | ||
| if (check.IsUsed == false) |
There was a problem hiding this comment.
| if (check.IsUsed == false) | |
| if (!check.IsUsed) |
| if (codeOfBytes != -1) | ||
| { | ||
| codes.Enqueue(codeOfBytes); | ||
| trie.IsAdd(byter); |
There was a problem hiding this comment.
И снова добавляем байт в дерево? Хм, зачем?
There was a problem hiding this comment.
Надо придумать и написать этому рациональное объяснение. В реальном проекте нельзя просто взять и проигнорировать комментарий ревьюера.
| private bool CheckAdd(byte value, Node node) | ||
| => node.IsFind(value) == null ? true : false; | ||
|
|
||
| public int LastCode { get; set; } |
There was a problem hiding this comment.
По-хорошему, к свойствам (особенно с неочевидными названиями, как это) тоже надо комментарии
hw2LZW/hw2LZW/Trie.cs
Outdated
| /// вернем последний код | ||
| /// </summary> | ||
| /// <returns>возвращается последний код</returns> | ||
| public int GetLastCode() |
There was a problem hiding this comment.
А зачем, есть же public-свойство, которое делает то же самое
| string path1 = "..\\..\\..\\TestTxt.txt"; | ||
| string path2 = "..\\..\\..\\TestTxtCopy.txt"; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| var compressedFileSize = new FileInfo(args[0]); | ||
| var decompressedFileSize = new FileInfo(args[0] + ".zipped"); |
| hw2LZW.LZW.Compress(path1); | ||
| hw2LZW.LZW.Decompress(path1 + ".zipped"); | ||
| File.Delete(path1 + ".zipped"); | ||
| Assert.IsTrue(Compare(path1, path2)); |
There was a problem hiding this comment.
Я проделал это вручную и, если удалить исходный TestImg.jpg, получил:
Есть подозрение, что тест проходит только потому, что TestImg.jpg уже существует, а распаковщик не перезаписывает файл до конца (то есть в хвосте остаются байты оригинального изображения).
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
С одной стороны, стало лучше, с другой стороны, стало хуже :)
| if (codeOfBytes != -1) | ||
| { | ||
| codes.Enqueue(codeOfBytes); | ||
| trie.IsAdd(byter); |
There was a problem hiding this comment.
Надо придумать и написать этому рациональное объяснение. В реальном проекте нельзя просто взять и проигнорировать комментарий ревьюера.
|
|
||
| public bool IsUsed { get; set; } | ||
|
|
||
| public int IdByte; |
There was a problem hiding this comment.
Я пишу "у этих свойств видимость уж слишком широкая, сеттер можно было убрать", а Вы такой "о, почему бы не превратить свойство в public-поле?". Типа "сгорел сарай, гори и хата"? Нет, не прокатит.
| private bool CheckAdd(byte value, Node node) | ||
| => node.IsFind(value) == null ? true : false; | ||
|
|
||
| public int LastCode { get; set; } |
3 participants
No description provided.