1

Пытаюсь реализовать агрегацию между классами

Файл Apartmen.cs:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using ClientNS;

namespace ApartmenNS
{
    class Apartmen
    {
        ...

        Client Person;

        ...

        public void AddApartmen(string name, int passport)
        {
            Person.AddClient(name, passport); // Error
        }

        ...
    }
}

Файл Client.cs:

    using System;
    using System.Collections.Generic;
    using System.Linq;
    using System.Text;
    using System.Threading.Tasks;
    using ApartmenNS;

    namespace ClientNS
    {
        class Client
        {
            string name;
            int passport;

            public Client ()
            {
                name = "";
                passport = 0;
            }

            int Passport { get {...}
                           set {...} }

            public void AddClient(string N, int P)
            {
                name = N;
                Passport = P; // костыли, знаю
            }
        }
    }

Ошибка возникает во время работы программы в строке помеченной "//Error"

Jagailo
  • 2,081
  • 19
  • 32

2 Answers2

2

Член класса

Client Person;

инициализируется null-ссылкой.

Вы можете записать его как

Client Person = new Person();

Чтобы при создании объекта Apartmen данный член этого класса инициализировался ссылкой на конкретный объект класса Person

Либо такую инициализацию вы можете включить непосредственно в конструктор класса Apartmen.

Мне представляет также, что лучше назвать класс Apartment вместо Apartmen:)

Vlad from Moscow
  • 44,824
  • А не будет ли этот вариант композицией? – Jagailo Oct 11 '15 at 11:09
  • 1
    @Алексей Ягело Да, это композиция или по-иному агрегирование. – Vlad from Moscow Oct 11 '15 at 11:13
  • У меня в задании: Между парами классов спроектировать отношения композицию (1) и агрегацию (1). Тогда раз Client Person = new Person(); - композиция, то как тогда должна выглядеть агрегация? – Jagailo Oct 11 '15 at 11:37
  • @АлексейЯгело Во многих книгах композиция и агрегация используются взаимозаменяемо. Но вам надо смотреть те определения, которые вам давали. К сожалению я не знаю, какие определения этих терминов вам давали. – Vlad from Moscow Oct 11 '15 at 11:44
  • Те что на Википедии ссылка – Jagailo Oct 11 '15 at 11:53
  • 1
    @АлексейЯгело На самом деле это - игра слов: "Композиция (агрегирование по значению) — более строгий вариант агрегирования". Кто-то на пустом месте пытается создать новый смысл. Если следовать данному определению, то если вы включите создание объекта Person внутри класса Apartmen, то это будет композиция. А если объект Person будет создан вне объекта Apartmen, а обекту Apartmen будет передана ссылка на созданный объект, то это будет агрегация.:) – Vlad from Moscow Oct 11 '15 at 12:13
1

вы не инициализируете поле Person - и оно не указывает на объект в тот момент, когда вы пытаетесь вызывать Person.AddClient

class Apartment
{
    Client Person = new Client();

    //...
}