search

Cha0z / springBootHomeWork

0 stars 1 forks source link

Code review #3 map #3

Open hjvfyfyfy opened 5 years ago

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L10

краще писати якось так private List<InnerPair<K,V>> elements; тоді тобі не прийдеться кожний раз кастити тип при витягувані ключа з elements

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L25-L34

по перше, ти рахуєш кількість бакетів в яких є елементи а не кількість елементів по друге, краще зробити зміну в якій буде зберігатися кількість елементів, всерівно це значення тобі треба щоб знати колим збільшувати кількість бакетів

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L54-L59

цю логіку можна винести в конструктор InnerPair

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L47-L48

а що буде якщо NextElement вже існує, ти просто затреш той елемент

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L61-L63

там немає паревірки на null, памятаєш ключати HashMap можуть бути null

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L72

метод названий цікаво

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/597b97b2396c7572b0e93c416c261fdefbd8f9aa/src/main/java/com/homework/home/map/OwnMap.java#L77-L79

тут вилетить StackOverflowError, ти запускаєш функцію з тимиж параметрами analyzeKey(innerPair, key)