Правильно ли я пишу? Работа с интерфейсом и двумя классами с аналогичными методами

@Lynatik001 · Регистрация: 28.09.2012

Author24 — интернет-сервис помощи студентам

Народ подскажите ли правильно я поступаю. У меня в задании по курсовой есть клиенты и обьекты у них есть добавление,удаление,поиск,редактирование и тдп. Я решил сделать интерфейс так как эти методы почти аналогичные

C#

        void add(object obj);
        void delete(object obj);
        void edit(int index, int indexparam, object info);
        object view(object obj);
        object viewall();
        object sort(int indexparam,object obj);

Вот как все в интерфейсе смотрится

а вот как в классе обьект

C#

        public void add(object realty)
        {
            realtys.Add((Realty)realty);
        }
 
        public void delete(object index)
        {
            realtys.RemoveAt((int)index-1);
        }
        public void edit(int index, int indexparam, object info)
        {
          int indexrealty = index -1;
          switch (indexparam)
          {
              case 1:
                  realtys[indexrealty].Name = (string)info;
                  break;
              case 2:
                  realtys[indexrealty].Vartist = (int)info;
                  break;
              case 3:
                  realtys[indexrealty].Mrozt = (string)info;
                  break;
 
          }

а вот в классе клиент

C#

        public void add(object client)
        {
            clients.Add((Client)client);
        }
        public void delete(object npassport)
        {
           clients.Remove(clients.Find(x => x.Npassport == (int)npassport));
        }
        public void edit(int npassport, int indexparam , object info)
        {
          int indexclient =  clients.FindIndex(x => x.Npassport == (int)npassport);
          switch (indexparam)
          {
              case 1:
                  clients[indexclient].Name = (string)info;
                  break;
              case 2:
                  clients[indexclient].Surname = (string)info;
                  break;
              case 3:
                  clients[indexclient].Npassport = (int)info;
                  break;
              case 4:
                  clients[indexclient].Adres = (string)info;
                  break;
              case 5:
                  clients[indexclient].Telefon = (int)info;
                  break;
              case 6:
                  clients[indexclient].Nombank = (int)info;
                  break;
              case 7:
                  clients[indexclient].Tipneruh = (string)info;
                  break;
              case 8:
                  clients[indexclient].Vcena = (int)info;
                  break;
          }
        }

Вопрос ли не крыво я делаю. Может не стоит все через интерфейс завязывать) Думаю будет страдать понимаемость что в том методе должно хранится)
Елементарные операции хранить в интерфейсе было написано в задании) Я как начал так и все вышло обобщить и кинуть в интерфейс))

@insite2012 · 29.11.2014, 14:59

Lynatik001, а напишите полностью текст задания и полностью ваш код.

@Lynatik001 · 29.11.2014, 15:11 **[ТС]**

Сообщение от insite2012

Lynatik001, а напишите полностью текст задания и полностью ваш код.

а Этого разве нахватает?
Это ж курсовая она болшая и много левого, я показал только то что надо что бы спросить ли норм написание вот такого кода).

@insite2012 · 29.11.2014, 15:13

Сообщение от Lynatik001

ли норм написание вот такого кода

Первый вопрос, который у меня возник: почему не используете Generic-и....

@Lynatik001 · 29.11.2014, 15:15 **[ТС]**

Вот просто один с пунктов задания) о котором я говорил
4. Описать элементарные операции с данными, используя интерфейсы и абстрактные классы (стандартные, например, IComparer, и собственные) **.

Добавлено через 1 минуту

Сообщение от insite2012

Первый вопрос, который у меня возник: почему не используете Generic-и....

типа view<Client> ?

@insite2012 · 29.11.2014, 15:17

Сообщение от Lynatik001

типа view<Client> ?

типа class<T>, Method<T>

@Lynatik001 · 29.11.2014, 15:22 **[ТС]**

Ну как видите мы тут работаем с 2 списками клиент и обьект))

C#

        public object view(object npassport)
        {
            return clients.Find(x => x.Npassport == (int)npassport);
        }

C#

        public object view(object index)
        {
            return realtys[(int)index-1];
        }

Где же лучше описать шаблонный метод заместь object
Он весьма удобен в плане если у нас метод допустим поиск используется для поиска по разному критерию например ли то инт ли то строка

@insite2012 · 29.11.2014, 15:26

Сообщение от Lynatik001

мы тут работаем с 2 списками клиент и обьект))

Я, к примеру, вижу только два метода и никаких списков (лопни мои глаза, что я проглядел

).
Именно поэтому я просил весь код, что относится к вопросу, а не кусочки. Нет желания гадать, что у вас там в коде и как.

@Lynatik001 · 29.11.2014, 15:28 **[ТС]**

Сообщение от insite2012

типа class<T>, Method<T>

Но я же не буду обьеденять все в один класс, можно описать конечно абстракный<T> и от него наследоаваться. Сначало я так и решил. Правда остановился, забыл почему)) но помню что-то не выходило совмещать ирешил в интерфейсе писать)

Добавлено через 1 минуту

Сообщение от insite2012

Я, к примеру, вижу только два метода и никаких списков (лопни мои глаза, что я проглядел ).
Именно поэтому я просил весь код, что относится к вопросу, а не кусочки. Нет желания гадать, что у вас там в коде и как.

C#

class ClientManipulation : Manipulation
    {
        List<Client> clients = new List<Client>();
        public void add(object client)
        {
            clients.Add((Client)client);
        }
        public void delete(object npassport)
        {
           clients.Remove(clients.Find(x => x.Npassport == (int)npassport));
        }
        public void edit(int npassport, int indexparam , object info)
        {
          int indexclient =  clients.FindIndex(x => x.Npassport == (int)npassport);
          switch (indexparam)
          {
              case 1:
                  clients[indexclient].Name = (string)info;
                  break;
              case 2:
                  clients[indexclient].Surname = (string)info;
                  break;
              case 3:
                  clients[indexclient].Npassport = (int)info;
                  break;
              case 4:
                  clients[indexclient].Adres = (string)info;
                  break;
              case 5:
                  clients[indexclient].Telefon = (int)info;
                  break;
              case 6:
                  clients[indexclient].Nombank = (int)info;
                  break;
              case 7:
                  clients[indexclient].Tipneruh = (string)info;
                  break;
              case 8:
                  clients[indexclient].Vcena = (int)info;
                  break;
          }
        }
 
        public object view(object npassport)
        {
            return clients.Find(x => x.Npassport == (int)npassport);
        }
        public object viewall()
        {
            return clients;
        }
        public object sort(int indexparam,object obj)
        {
            List<Client> sortclients = new List<Client>();
            switch(indexparam)
            {
                case 1:
                    sortclients = clients.FindAll(x => x.Name == (string)obj);
                    break;
                case 2:
                    sortclients = clients.FindAll(x => x.Surname == (string)obj);
                    break;
                case 3:
                     Regex regex = new Regex(@"^" + (int)obj + "");
 
                     sortclients = clients.FindAll((x) =>                         x.Nombank.ToString() != ""
                    
                    );
                    break;
 
            }
            return sortclients;
        }
    }
}

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace cursova6
{
    class RealtyManipulation : Manipulation
    {
        List<Realty> realtys = new List<Realty>();
        public void add(object realty)
        {
            realtys.Add((Realty)realty);
        }
 
        public void delete(object index)
        {
            realtys.RemoveAt((int)index-1);
        }
        public void edit(int index, int indexparam, object info)
        {
          int indexrealty = index -1;
          switch (indexparam)
          {
              case 1:
                  realtys[indexrealty].Name = (string)info;
                  break;
              case 2:
                  realtys[indexrealty].Vartist = (int)info;
                  break;
              case 3:
                  realtys[indexrealty].Mrozt = (string)info;
                  break;
 
          }
        }
        public object view(object index)
        {
            return realtys[(int)index-1];
        }
 
        public object viewall()
        {
            return realtys;
        }
        
    }
}

Он еще в режиме писания) сорта нету во 2ром))

@Anderok · 29.11.2014, 15:30

Реально, дядя, сделай дженериком хотя бы интерфейс.. А то вообще по-конченому выглядит постоянное приведение.)

Добавлено через 1 минуту

Сообщение от Lynatik001

Но я же не буду обьеденять все в один класс

у меня сложилось впечатление, что ты не знаешь, что такое дженерики.

Сделай интерфейс дженериком и при его реализации ты избавишься от постоянного приведения.

@Lynatik001 · 29.11.2014, 15:42 **[ТС]**

Сообщение от Anderok

Реально, дядя, сделай дженериком хотя бы интерфейс.. А то вообще по-конченому выглядит постоянное приведение.)

У меня постоянны приведения только из-за разности типов параметров.

C#

        public void edit(int index, int indexparam, object info)
        {
          int indexrealty = index -1;
          switch (indexparam)
          {
              case 1:
                  realtys[indexrealty].Name = (string)info;
                  break;
              case 2:
                  realtys[indexrealty].Vartist = (int)info;
                  break;
              case 3:
                  realtys[indexrealty].Mrozt = (string)info;
                  break;
 
          }

Пример, у меня тут вмещает редактирование всех параметров метод.
аналогом бы был
public void edit<T>(int index, int indexparam, T info)

C#

1
2
3

              case 1:
                  realtys[indexrealty].Name = info;
                  break;

и потом прийдется писать что то типа

C#

if(i==1)
edit<int>(1,1,1)
if(i==2)
edit<string>(1,1,"lkj")

неоч выглядит как то)
Заместь того что бы раз его вызвать)

@Anderok · 29.11.2014, 15:47

Lynatik001, так я предлагаю сам интерфейс сделать дженериком, а не каждый метод в отдельности...
<> придётся написать только два раза - при реализациях его..

@Lynatik001 · 29.11.2014, 15:55 **[ТС]**

Сообщение от Anderok

Сделай интерфейс дженериком и при его реализации ты избавишься от постоянного приведения.

у меня не все методы возращают тот же список а некоторые возращают самых клиентов или обьектов- и они не будут совпадать с типом T

Добавлено через 5 минут

Сообщение от Anderok

Lynatik001, так я предлагаю сам интерфейс сделать дженериком, а не каждый метод в отдельности...
<> придётся написать только два раза - при реализациях его..

ну да вы правы по отношению к 2ум методам ето будет правильно. Но всеровно многого не поменяется) Спасибо за критику

@Anderok · 29.11.2014, 15:55

C#

void Main()
{
    var a = new A();
    int i = 14;
    a.Edit(ref i);
    
    Console.WriteLine(i);
}
 
 
public class A: Interface<int> 
{
    public void Edit(ref int item)
    {
        item = 0;
    }
    public string GetStringView(int[] itemArray)
    {
        var sb = new StringBuilder();
        foreach(var x in itemArray)
        {
            sb.Append(x.ToString());
            sb.Append(' ');
        }
        return sb.ToString();
    }
}
 
public interface Interface<T>
{
    void Edit(ref T item);
    string GetStringView(T[] itemArray);
    // и так далее
}
 
// Define other methods and classes here

@insite2012 · 29.11.2014, 15:56

Lynatik001, я (значение частоты) так и не пойму что надо, и было бы проще, если бы вы озвучили задание. Но примерный набросок как-то так....

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication2 {
    class Program {
        static void Main(string[] args) {
        }
    }
    interface Manipulation<T> {
        void Add(T arg);
        void Delete(T arg);
        T view(T arg);
    }
    class ObjectManipulation<T> : Manipulation<T> where T : class {
        List<T> list = new List<T>();
 
        public void Add(T arg) {
            list.Add(arg);
        }
 
        public void Delete(T arg) {
            
        }
 
        public T view(T arg) {
            
        }
    }
}

@Anderok · 29.11.2014, 15:57

Lynatik001, во-первых, дженерик может может быть от более чем одного типа.. X<T,K>

А во-вторых не обязательно все методы сносить в интерфейс... Вообще говоря, интерфейсы - инструмент инкапсуляции, а если её не будет, то может стоит задуматься о необходимости применения интерфейса в целом?)

@Lynatik001 · 29.11.2014, 15:59 **[ТС]**

Если вы про метод

C#

        public void edit(int npassport, int indexparam , object info)
        {
          int indexclient =  clients.FindIndex(x => x.Npassport == (int)npassport);
          switch (indexparam)
          {
              case 1:
                  clients[indexclient].Name = (string)info;
                  break;
              case 2:
                  clients[indexclient].Surname = (string)info;
                  break;
              case 3:
                  clients[indexclient].Npassport = (int)info;
                  break;
              case 4:
                  clients[indexclient].Adres = (string)info;
                  break;
              case 5:
                  clients[indexclient].Telefon = (int)info;
                  break;
              case 6:
                  clients[indexclient].Nombank = (int)info;
                  break;
              case 7:
                  clients[indexclient].Tipneruh = (string)info;
                  break;
              case 8:
                  clients[indexclient].Vcena = (int)info;
                  break;
          }

То он содержит не один какой то тип что бы косить его под T как в примере)
то же и в других методах)) Разве что для тех 2ух методов там где именно возращение списков идет)
Но самому етому методу он не поможет)

@Anderok · 29.11.2014, 16:04

Lynatik001, а ё-моё... у тебя метод разные типы суёт??? Жесть..

Так примени dynamic ))

Добавлено через 2 минуты
Lynatik001, а, нет ... погоди... ё-моё.. У тебя есть массив object? жесть[2].. Блин, ладно, тогда помолчу.

@Lynatik001 · 29.11.2014, 16:05 **[ТС]**

Сообщение от insite2012

набросок как-то так...

Ну в добавлении удалении да у меня обьект в параметрах, в удалении в параметрах индекс а не Т-обьект
ввобщем тут все же не пихнешь в Т тут же разные типы везде в параметрах типа инт стринг или же обьект может быть а не именно тот один что надо

@Anderok · 29.11.2014, 16:06

Lynatik001, стоп, массив объектов, в которые ты что-то суёшь? Ты зря сделал один метод на всё это... Мог бы воспользоваться инструментом перегрузки методов и делать то, что тебе надо.. А так это антипаттерн какой-то...

@Anderok 116 / 116 / 70 Регистрация: 10.11.2013 Сообщений: 445
	29.11.2014, 15:30	10
	Реально, дядя, сделай дженериком хотя бы интерфейс.. А то вообще по-конченому выглядит постоянное приведение.) Добавлено через 1 минуту Сообщение от Lynatik001 Но я же не буду обьеденять все в один класс у меня сложилось впечатление, что ты не знаешь, что такое дженерики. Сделай интерфейс дженериком и при его реализации ты избавишься от постоянного приведения. 1

@Anderok 116 / 116 / 70 Регистрация: 10.11.2013 Сообщений: 445
	29.11.2014, 16:04	18
	Lynatik001, а ё-моё... у тебя метод разные типы суёт??? Жесть.. Так примени dynamic )) Добавлено через 2 минуты Lynatik001, а, нет ... погоди... ё-моё.. У тебя есть массив object? жесть[2].. Блин, ладно, тогда помолчу. 0

@Lynatik001 47 / 39 / 15 Регистрация: 28.09.2012 Сообщений: 818
	29.11.2014, 16:05 [ТС]	19
	Сообщение от insite2012 набросок как-то так... Ну в добавлении удалении да у меня обьект в параметрах, в удалении в параметрах индекс а не Т-обьект ввобщем тут все же не пихнешь в Т тут же разные типы везде в параметрах типа инт стринг или же обьект может быть а не именно тот один что надо 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	29.11.2014, 14:59	2
	Lynatik001, а напишите полностью текст задания и полностью ваш код. 0

@Lynatik001 47 / 39 / 15 Регистрация: 28.09.2012 Сообщений: 818
	29.11.2014, 15:11 [ТС]	3
	Сообщение от insite2012 Lynatik001, а напишите полностью текст задания и полностью ваш код. а Этого разве нахватает? Это ж курсовая она болшая и много левого, я показал только то что надо что бы спросить ли норм написание вот такого кода). 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	29.11.2014, 15:13	4
	Сообщение от Lynatik001 ли норм написание вот такого кода Первый вопрос, который у меня возник: почему не используете Generic-и.... 0

@Lynatik001 47 / 39 / 15 Регистрация: 28.09.2012 Сообщений: 818
	29.11.2014, 15:15 [ТС]	5
	Вот просто один с пунктов задания) о котором я говорил 4. Описать элементарные операции с данными, используя интерфейсы и абстрактные классы (стандартные, например, IComparer, и собственные) *. Добавлено через 1 минуту* Сообщение от insite2012 Первый вопрос, который у меня возник: почему не используете Generic-и.... типа view<Client> ? 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	29.11.2014, 15:17	6
	Сообщение от Lynatik001 типа view<Client> ? типа class<T>, Method<T> 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	29.11.2014, 15:26	8
	Сообщение от Lynatik001 мы тут работаем с 2 списками клиент и обьект)) Я, к примеру, вижу только два метода и никаких списков (лопни мои глаза, что я проглядел ). Именно поэтому я просил весь код, что относится к вопросу, а не кусочки. Нет желания гадать, что у вас там в коде и как. 1

@Anderok 116 / 116 / 70 Регистрация: 10.11.2013 Сообщений: 445
	29.11.2014, 15:47	12
	Lynatik001, так я предлагаю сам интерфейс сделать дженериком, а не каждый метод в отдельности... <> придётся написать только два раза - при реализациях его.. 0

@Lynatik001 47 / 39 / 15 Регистрация: 28.09.2012 Сообщений: 818
	29.11.2014, 15:55 [ТС]	13
	Сообщение от Anderok Сделай интерфейс дженериком и при его реализации ты избавишься от постоянного приведения. у меня не все методы возращают тот же список а некоторые возращают самых клиентов или обьектов- и они не будут совпадать с типом T Добавлено через 5 минут Сообщение от Anderok Lynatik001, так я предлагаю сам интерфейс сделать дженериком, а не каждый метод в отдельности... <> придётся написать только два раза - при реализациях его.. ну да вы правы по отношению к 2ум методам ето будет правильно. Но всеровно многого не поменяется) Спасибо за критику 0

@Anderok 116 / 116 / 70 Регистрация: 10.11.2013 Сообщений: 445
	29.11.2014, 15:57	16
	Lynatik001, во-первых, дженерик может может быть от более чем одного типа.. X<T,K> А во-вторых не обязательно все методы сносить в интерфейс... Вообще говоря, интерфейсы - инструмент инкапсуляции, а если её не будет, то может стоит задуматься о необходимости применения интерфейса в целом?) 0

@Anderok 116 / 116 / 70 Регистрация: 10.11.2013 Сообщений: 445
	29.11.2014, 16:06	20
	Lynatik001, стоп, массив объектов, в которые ты что-то суёшь? Ты зря сделал один метод на всё это... Мог бы воспользоваться инструментом перегрузки методов и делать то, что тебе надо.. А так это антипаттерн какой-то... 0