Java02. ДЗ 03, Гусев Андрей by ASGusev · Pull Request #4 · ASGusev/java_hw_2c

ASGusev · 2016-10-25T20:39:33Z

No description provided.

sproshev

надо поправить

sproshev · 2016-10-29T18:01:31Z

src/main/java/Collections.java

+            resList.add(function.apply(curElement));
+        }
+        return resList;
+    }


обычно длинные сигнатуры функций форматируются, как отформатирована сигнатура filter -- на первой строке первый параметр, последующие параметры под ним

остальных функций это тоже касается)
кроме того, видно, что форматирование кода не производится, Ctrl + Alt + L в помощь )

sproshev · 2016-10-29T18:02:26Z

src/main/java/Collections.java

+            }
+        }
+        return resList;
+    }


это можно было написать проще, вспомнив про break

sproshev · 2016-10-29T18:08:19Z

src/main/java/Predicate.java

+            return false;
+        }
+    };
+}


в этом классе маловато generic'ов

А ALWAYS_TRUE и ALWAYS_FALSE должны быть методами, final-объектами, или классами?

public static final полями

sproshev · 2016-10-29T18:09:37Z

src/test/java/CollectionsTest.java

+            sum += i;
+        }
+
+        Assert.assertEquals("Foldl doesn't work correctly.", sum, foldlSum);


этот тест не проверяет, действительно ли свертка левая

sproshev · 2016-10-29T18:12:29Z

src/test/java/PredicateTest.java

+            fail("Or should be lazy.");
+        }
+    }
+}


в F1Test, F2Test, PTest не хватает дженериков, и идея это все подсвечивает

и еще, в тестах не проверяются wildcard'ы (это которые ? extends ..., ? super ...)

sproshev · 2016-10-29T18:22:35Z

src/test/java/PredicateTest.java

+            fail("Or should be lazy.");
+        }
+    }
+}


и еще, в тестах не проверяются wildcard'ы (это которые ? extends ..., ? super ...)

ASGusev · 2016-10-31T22:13:35Z

Поправил: #4

29.10.2016 21:22, Semyon Proshev пишет:

@sproshev requested changes on this pull request.

In src/test/java/PredicateTest.java
#4 (review):
           Predicate.ALWAYS_FALSE.or(Predicate.ALWAYS_TRUE).apply(null));
   Assert.assertFalse("Or should perform disjunction.",
           Predicate.ALWAYS_FALSE.or(Predicate.ALWAYS_FALSE).apply(null));
   try {
       Predicate.ALWAYS_TRUE.or(new Predicate() {
           @Override
           public Boolean apply(Object x) throws Exception {
               throw new Exception();
           }
       }).apply(null);
   } catch (Exception e) {
       fail("Or should be lazy.");
   }
}
+}
и еще, в тестах не проверяются wildcard'ы (это которые ? extends ...,
? super ...)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (review),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFrdn5ZPZZW1SaI0nLbnzRUv3yTxNGs0ks5q447wgaJpZM4KgdRQ.

sproshev

10

sproshev · 2016-11-03T18:35:37Z

src/main/java/Collections.java

+            resList.add(function.apply(curElement));
+        }
+        return resList;
+    }


остальных функций это тоже касается)
кроме того, видно, что форматирование кода не производится, Ctrl + Alt + L в помощь )

sproshev · 2016-11-03T18:36:02Z

src/main/java/Collections.java

+            resList.add(curElem);
+        }
+        return resList;
+    }


еще короче, вообще без итератора

Functional Java, первый коммит.

af4012c

sproshev suggested changes Oct 29, 2016

View reviewed changes

Исправления в соответствии с замечаниями.

6d80eba

ASGusev changed the title ~~PR: Java02. ДЗ 03, Гусев Андрей~~ Java02. ДЗ 03, Гусев Андрей Oct 31, 2016

sproshev approved these changes Nov 3, 2016

View reviewed changes

Conversation

ASGusev commented Oct 25, 2016

Uh oh!

sproshev left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ASGusev commented Oct 31, 2016

Uh oh!

sproshev left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants