Advertisement
washinson

Untitled

Jan 25th, 2020
114
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 2.18 KB | None | 0 0
  1. (!) Важное замечание
  2. (?) Менее важное замечание
  3.  
  4. 1. (?) Выбрасываешь исключение Exception
  5. Я не знаю, что значит "выбрасывать исключение подходящего типа", но, наверное, стоитло выбрасывать ArgumentException
  6. 2. (?) В конструкторе Sparrow лишняя переменная bool isPredator.
  7. Старайся не требовать переменные, которые не используешь. Правильно было бы
  8. public Sparrow(string name, int maxDistance) : base(name, false, maxDistance)
  9. 3. (!) Неправильно считаешь случайное число int distance = rnd.Next((int)0.1 * MaxDistance, MaxDistance + 1);
  10. Если посмотреть, что делает (int), то заметишь, что (int)0.1 * MaxDistance эквивалентно 0 * MaxDistance
  11. Скорее всего ты забыла скобки, т.е. хотела так (int)(0.1 * MaxDistance). Тут важна внимательность(и знание приоритетов операций). Если не знаешь какой приоритет у какой-то операции, ставь скобки.
  12. 4. (?) Конструктор Eagle аналогично п.2
  13. 5. (!) Eagle distance аналогично п.3
  14. 6. (?) Кусок кода:
  15. if (choice1 == 1) name += (char)rnd.Next('A', 'Z' + 1);
  16. else
  17. {
  18. name += (char)rnd.Next('A', 'Z' + 1);
  19. for (int j = 1; j < choice1; j++)
  20. {
  21. name += (char)rnd.Next('a', 'z' + 1);
  22. }
  23. }
  24.  
  25. Можно записать так:
  26.  
  27. name += (char)rnd.Next('A', 'Z' + 1);
  28. for (int j = 1; j < choice1; j++)
  29. {
  30. name += (char)rnd.Next('a', 'z' + 1);
  31. }
  32. 7. (!) Ты выводишь суммарное максимальное расстояние каждых птиц, а надо было считать сумму от значений Move(). Читай условия, это достаточно серъезная ошибка.
  33. Совет. Когда закончишь прогу, старайся перечитать условие и свериться так ли ты сделала.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement