Bor - #4
Bor#4Palezehvat wants to merge 11 commits into
Conversation
…eworksCSharp into PostfixCalculator
…ь они идут отдельно с маленькой, при этом в каталоге отображаются с большой, а при добавлении с большой git status их не выделяет, поэтому этот комит их отдельное добавление
Другая ветка
| @@ -1,208 +0,0 @@ | |||
| namespace Sort; | |||
There was a problem hiding this comment.
Тут тоже надо сделать так, чтобы удаления BWT не было в диффе этого пуллреквеста
| using System.ComponentModel.DataAnnotations; | ||
| using System.Text; | ||
| using System.Xml.Linq; | ||
| using static System.Net.Mime.MediaTypeNames; |
There was a problem hiding this comment.
Правда ли, что все using-и нужны?
| namespace Bor; | ||
|
|
||
| // A container for storing strings, in the form of a suspended tree | ||
| public class BorClass |
There was a problem hiding this comment.
То, что он класс, и так понятно :)
| // A container for storing strings, in the form of a suspended tree | ||
| public class BorClass | ||
| { | ||
| const int sizeAlphabet = 65536; |
There was a problem hiding this comment.
| const int sizeAlphabet = 65536; | |
| private const int alphabetSize = 65536; |
| { | ||
| if (root == null) | ||
| { | ||
| throw new Exception(); |
There was a problem hiding this comment.
Нельзя кидать просто Exception, кидайте какие-нибудь более специфические исключения (тут, например, InvalidOperationException было бы уместно)
| Assert.AreEqual(2, bor.HowManyStartsWithPrefix("end"), "Problems with the prefix test!"); | ||
| } | ||
| } | ||
| } No newline at end of file |
| [Test] | ||
| public void AddTwoStringsWithTheSamePrefixBorTheNumberOfStringsWithThisPrefixIsTwo() | ||
| { | ||
| var bor = new BorClass(); |
There was a problem hiding this comment.
Почитайте про SetUp, чтобы не писать код инициализации трижды
| @@ -3,23 +3,23 @@ Microsoft Visual Studio Solution File, Format Version 12.00 | |||
| # Visual Studio Version 17 | |||
| VisualStudioVersion = 17.4.33403.182 | |||
There was a problem hiding this comment.
Внезапный стековый калькулятор в пуллреквесте про бор, его тоже надо убрать
| using Microsoft.VisualBasic; | ||
| using System.Xml.Linq; | ||
|
|
||
| namespace BorTest |
There was a problem hiding this comment.
Используйте file-scoped namespaces
| @@ -0,0 +1,35 @@ | |||
| using Bor; | |||
There was a problem hiding this comment.
Имя файла UnitTest1.cs не годится
…му-то не отображается
3 participants
Bor, 2 дз