semwai / javafirst

первое задание

Hi there 👋

🐱 My GitHub Data

📦 107.0 kB Used in GitHub's Storage

🏆 120 Contributions in the Year 2023

🚫 Not Opted to Hire

📜 46 Public Repositories

🔑 21 Private Repositories

I'm a Night 🦉

🌞 Morning                43 commits          █░░░░░░░░░░░░░░░░░░░░░░░░   02.26 % 
🌆 Daytime                460 commits         ██████░░░░░░░░░░░░░░░░░░░   24.22 % 
🌃 Evening                759 commits         ██████████░░░░░░░░░░░░░░░   39.97 % 
🌙 Night                  637 commits         ████████░░░░░░░░░░░░░░░░░   33.54 % 

📅 I'm Most Productive on Wednesday

Monday                   132 commits         ██░░░░░░░░░░░░░░░░░░░░░░░   06.95 % 
Tuesday                  236 commits         ███░░░░░░░░░░░░░░░░░░░░░░   12.43 % 
Wednesday                449 commits         ██████░░░░░░░░░░░░░░░░░░░   23.64 % 
Thursday                 340 commits         ████░░░░░░░░░░░░░░░░░░░░░   17.90 % 
Friday                   266 commits         ████░░░░░░░░░░░░░░░░░░░░░   14.01 % 
Saturday                 160 commits         ██░░░░░░░░░░░░░░░░░░░░░░░   08.43 % 
Sunday                   316 commits         ████░░░░░░░░░░░░░░░░░░░░░   16.64 % 

📊 This Week I Spent My Time On

🕑︎ Time Zone: Europe/Moscow

💬 Programming Languages: 
No Activity Tracked This Week

🔥 Editors: 
No Activity Tracked This Week

💻 Operating System: 
No Activity Tracked This Week

I Mostly Code in Kotlin

Python                   11 repos            █████░░░░░░░░░░░░░░░░░░░░   18.97 % 
JavaScript               4 repos             ██░░░░░░░░░░░░░░░░░░░░░░░   06.90 % 
TypeScript               2 repos             █░░░░░░░░░░░░░░░░░░░░░░░░   03.45 % 
Shell                    1 repo              ░░░░░░░░░░░░░░░░░░░░░░░░░   01.72 % 
VBA                      1 repo              ░░░░░░░░░░░░░░░░░░░░░░░░░   01.72 % 

Last Updated on 09/12/2023 12:18:53 UTC

javafirst's Issues

Замечания

1. В equals следует сначала проверить переданный объект на null.

2. JavaFirst/src/ru/sbpstu/semwai/javafirst/BinaryThree.java

   Line 98 in e4cd606

   if (this == o && o instanceof BinaryThree)

   Проверка на instanceof не имеет смысла, если объекты равны через ==, то есть переменные ссылаются на один и тот же объект.

3. JavaFirst/src/ru/sbpstu/semwai/javafirst/BinaryThree.java

   Line 100 in e4cd606

   if (o instanceof BinaryThree) {

   Тут лучше инвертировать условие и вернуть false, чтобы не помещать почти весь код метода в if.
4. Считается, что для генерации хэш-кода лучше подходит константа 31 (по крайней мере в она используется в стандартной библиотеке). А вообще можете просто передать объекты, из которых нужно сделать хэш-код, в Objects.hash().

5. JavaFirst/src/ru/sbpstu/semwai/javafirst/BinaryThree.java

   Lines 138 to 146 in e4cd606

   	public BinaryThree[] friends(int value) throws Exception {
   	if (value == this.value)
   	return new BinaryThree[]{null, left, right};
   	if (value > this.value && this.right != null)
   	return right.friends(value, this);
   	if (value < this.value && this.left != null)
   	return left.friends(value, this);
   	throw new Exception("Node hasn't exist");
   	}

   Чтобы не дублировать здесь код приватной версии этой функции, можно просто её здесь вызвать.
6. DeleteTest у Вас фактически не является тестом, так как в нём нет assert функций.