[C# -OOP Advanced] Exercises: Interfaces Как е по-добре да подам сет от елементи?

simeon_petrov 45 Точки

[C# -OOP Advanced] Exercises: Interfaces Как е по-добре да подам сет от елементи?

Въпросът ми е за Exercises: Interfaces, Problem 8. Military Elite.

Там: LeutenantGeneral – holds a set of Privates under his command

Генерала ще има: public HashSet<int> PrivatesIdUnderCommand { get; set; }

Информацията, която се подава в инпута е във вида: LeutenantGeneral: “LeutenantGeneral <id> <firstName> <lastName> <salary> <private1Id> <private2Id> … <privateNId>” where privateXId will always be an Id of a private already received through the input.

Въпросът ми е как е най-добре да се подаде този сет от номера на редници към генерала?

Вариант първи в майн метода преобразувам редицата от стрингове в HashSet<int> и така го подавам в конструктора на генерала.

Вариант втори подавам стрингчетата на конструктора и вътре в него се преобразуват в HashSet<int> ? Това не знам как ще стане, като броят на стринговете не е известен предваретелно и може да е много голям.

Вариант трети подавам стринговете касаещи Idтата на редниците като един стринг на конструктора и вътре в него те се Splitват и се преобразуват в HashSet<int>? Демек в майн метода сплитвам инпута по интервал и после събирам само Idтата в един стринг, който се подава на конструктора на генерала.

Тагове:

14/07/2016 10:33:28 C# OOP Advanced

RoYaL Trainer 6849 Точки

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

private int age;

а неговията сетър да е стрингов

public string Age { get { return this.age.ToString() } set { this.age = int.Parse(value); } }

Публичните методи, пропъртита и конструктори на класа дефинират неговия програмен интерфейс (API). Ползвателите на АПИ-то трябва да се съобразяват с него. Ако АПИ-то очаква инт за сетър, ползвателите да се оправят да парсват. Ако очаква Set<> в конструктора, ползвателите да създадат този Сет (в случая ползвател е мейн метода ти) и да го подават.

Ако искаш да прикриеш това, че е Сет, може да приемаш params (variadic arguments), които в конструктора да wrap-неш в Set. (това ще има известен performance overhead, но често продаваме производителност за четимост и поддръжка).

Друг вариант е, ако е възможно да не получиш редниците в началото, а впоследствие, то тогава пък хората въобще няма да ги интересува, че отдолу стои Сет, тъй като можеш да го скипнеш от конструктора, и да имаш метод за добавяне на редник. (това обаче не е сценарий на тази задача)

14/07/2016 10:52:53

mbencheva 52 Точки

Аз пък пазя Privates в LeutenantGeneral в List<Private> - private проперти, като имам публичен метод за добавяне на Private в листа.

По условие пише „privateXId will always be an Id of a private already received through the input“ и съответно разчитайки на това в мейна създавам LeutenantGeneral и по подадените Ид-а намирам съответния Private, който трябва да съществува вече и с публичния метод от LeutenantGeneral го добавям към private листа.

Не знам дали е добра идея или не...

14/07/2016 12:34:57

RoYaL Trainer 6849 Точки

Да, това звучи много по-логично, релацията да е обектна, след като ги имаш като вече създадени обекти.

14/07/2016 12:38:06 14/07/2016 12:38:18

kaloyannikov 531 Точки

Моя подход -> добре ли е на 1 място пазя всички privates , и в генерала взимам тая колекция и я филтрирам по idto github

14/07/2016 12:58:41