search

Cha0z / springBootHomeWork

0 stars 1 forks source link

Code review #2 map #2

Open hjvfyfyfy opened 5 years ago

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/InnerPair.java#L16

це дуже погана ідея

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L8

першим типом має бути ключ а не значення (OwnMap<K, V>)

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L17-L19

навіщо мені знати кількість бакетів? мене теба зенати кількість елементів

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L24

імплементаціяisPresent() : public boolean isPresent() { return value != null; }

не лекше написати так elements.get(numberOfBucket) == null ?

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L33

навіть не уявляю навіщо ти затер ключ

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L37

ніколи так не пиши, ти зациклив структуру даних, а тепер попробуй обійти її (доступитися до всіх елементів), або викликати toString

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L33-L44

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

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L26

цей метод дозволяє тобі вставити обєкт на початок (перед 1 обєктом), що призводить до росту List elements це не є стандартна поведінка мапи приклад OwnMap<String, Long> cMap = new OwnMap<>(); System.out.println("getSize " + cMap.getSize()); cMap.put(1L, "ob 1"); cMap.put(2L, "ob 2"); cMap.put(3L, "ob 3"); cMap.put(4L, "ob 4"); cMap.put(5L, "ob 5"); cMap.put(6L, "ob 6"); cMap.put(7L, "ob 7"); cMap.put(8L, "ob 8");

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L14

початковий розмір масиву 0 а тут ти ділиш на 0 https://github.com/Cha0z/springBootHomeWork/blob/f7003a9a6d1f5b0fdfc7efb378663aea85669a4f/src/main/java/com/homework/home/map/OwnMap.java#L58