Skip to content

Netbeans branch homework#3#4

Open
SychDan wants to merge 8 commits intoKinetikm:netbeansBranchfrom
SychDan:netbeansBranch
Open

Netbeans branch homework#3#4
SychDan wants to merge 8 commits intoKinetikm:netbeansBranchfrom
SychDan:netbeansBranch

Conversation

@SychDan
Copy link

@SychDan SychDan commented Sep 28, 2017

No description provided.

@ilya-lysko
Copy link
Collaborator

  1. Опять пункт про имена коммитов: давай им нормальные имена.
  2. Не нужно пушить бинарники (файлы .class). Добавь папку build в gitignore. Пока можешь не править, но в последующих дз отправлю на доработку.
  3. Папку с xml конфигами тоже добавь в gitignore, не надо ее пушить. Точно так же, пока не надо, дальше буду учитывать.
  4. MathService, метод dotProduct, строчка 23: if(a.length==0 & b.length==0). Почему логическое или не использовал? Не существенно в таком случае, но вспомни про разницу.
  5. Соблюдай правила именования пакетов и классов. Пакеты с маленькой буквы. Классы целиком из заглавных не надо делать.
  6. В классе MathService, метод sigmoid(double x): не надо выбрасывать исключение при x=0. Надо поправить и запушить.
  7. В классе MathService, метод sigmoid(**массивы): не надо, пожалуй, тут вставлять try-catch, лучше пробросить выше. Т.к. проблему ты решить не сможешь тут -- этому методу уже (свыше) пришли "плохие" массивы. Надо поправить и запушить.
  8. psvm main(..) убери из MathService. Надо вас тесты научить писать, вообще.

Жду исправлений.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants