Conversation
Теперь должен работать mvn test
|
|
||
| p.isTerminal = true; | ||
| return true; | ||
| } |
There was a problem hiding this comment.
странное форматирование, табы/пробелы?
|
|
||
| public class TrieImplTest | ||
| extends TestCase | ||
| { |
| assertEquals(0, b.howManyStartsWithPrefix("HeadZ")); | ||
|
|
||
| assertTrue(b.remove("Head")); | ||
| assertEquals(0, b.size()); |
There was a problem hiding this comment.
тесты стоит представлять в виде наименее возможных тестовых случаев, чтобы их можно было читать, понять, что проверено, а что нет
hw_01/src/main/java/hw_01/Bor.java
Outdated
| Node p = bor; | ||
| p.nwords++; | ||
| for (int i = 0; i < element.length(); i++) { | ||
| if (p.lifes[arr_num(element.charAt(i))] == null) { |
There was a problem hiding this comment.
arr_num(element.charAt(i)) используется три раза, что намекает на переменную
hw_01/src/main/java/hw_01/Bor.java
Outdated
| return true; | ||
| } | ||
|
|
||
| return false; |
hw_01/src/main/java/hw_01/Bor.java
Outdated
| } | ||
|
|
||
| private Node bor; | ||
| private int size = 0; |
There was a problem hiding this comment.
Я думал, что size это количество символов в боре.
There was a problem hiding this comment.
нет, это количество строк в боре, что эквивалентно количеству терминальных узлов или же bor.nwords
hw_01/src/main/java/hw_01/Bor.java
Outdated
| } | ||
|
|
||
| private Node bor; | ||
| private int size = 0; |
hw_01/src/main/java/hw_01/Bor.java
Outdated
| return p.nwords; | ||
| } | ||
|
|
||
| private Node bor; |
hw_01/src/main/java/hw_01/Bor.java
Outdated
|
|
||
| private Node bor; | ||
| private int size = 0; | ||
| private int arr_num(char a) { |
Разделил тесты на блоки.
| return p.nwords; | ||
| } | ||
|
|
||
| private int arrNum(char a) { |
There was a problem hiding this comment.
на буквах экономить не стоит, особенно если они помогают понимать код, что верно почти всегда
| assertEquals(4, b.size()); | ||
| } | ||
| /* | ||
| public void testBorBigTest() { |
There was a problem hiding this comment.
ненужный код лучше удалять, чтобы он не мозолил глаза
в крайнем случае, он останется в истории системы контроля версий
| import junit.framework.TestCase; | ||
|
|
||
| public class TrieImplTest extends TestCase { | ||
| private Bor b; |
There was a problem hiding this comment.
можно бор создавать в каждом тестовом методе, будет чуть длиньше файл, зато "чище"
2 participants
No description provided.