Programming Fundamentals - Classes and Objects - Exercises - Student Groups

alex1966 255 Точки

Programming Fundamentals - Classes and Objects - Exercises - Student Groups

Здравейте, може да се каже, че съм попривършил със задачката, имам проблем който не мога да разбера от къде идва:
1. Условие - 7ма задача
2. Клас Group - http://pastebin.com/SBxK4wV1
3. Клас Student - http://pastebin.com/Fy0d1M39
4. Клас Town - http://pastebin.com/7780g4Ff
5. Класа ми с мейн метода: http://pastebin.com/tEFGsJr1

Дебъгвах открих множество бъгове, но един е този който не мога да оправя, намира се в метода DistributeStudentsInGroups(в мейн класа),
като създам List<Student> students = towns[i].Students и след това като го ордърна не влиза в While цикъла защото няма нищо в листа от студенти, но защо при положение че аз му присвоявам property-то с List<Students>, предполагам че е нещо много елементарно, за което не мога да се сетя толкова късно. Ако ви хрумват някакви идеи, споделяйте ги.
Поздрави!

Тагове:

10/06/2016 02:06:00 Programming Fundamentals 10/06/2016 02:20:51

annsta 305 Точки

Best Answer

Има проблем със студентите още преди DistributeStudentsInGroups метода. При четенето на студентите създаваш нов town и добяваш студентите към него, но самият нов town не отива никъде. Вместо това, трябва да добявяш студентите към посления създаден град в towns. Бих предложила такава модификация на кода (редове 41-57):

else
{
   Student student = new Student();
   //Town town = new Town();
   //town.Students = new List<Student>();
           .....
   //town.Students.Add(student);
   towns[towns.Count - 1].Students.Add(student);
}

10/06/2016 02:38:02

alex1966 255 Точки

Стига бе.. толкова много редове излишни, не бях забелязал че с town не правя нищо, толкова много време се мъчих да намеря грешка в другия метод, че този напълно съм го игнорирал. Така написано работи коректно, благодаря ти !

10/06/2016 02:52:52 10/06/2016 02:54:24

annsta 305 Точки

За нищо! Хвърли и един поглед на ред 75, според мен на Distinct() не му е мястото тук и следва да остане само .ThenBy(x => x.Name)

10/06/2016 03:08:51

alex1966 255 Точки

Да, вече ми хвърли null exception при дъбъг, набързо го махнах, благодаря отново ! :))

10/06/2016 03:11:24