Interface forståelses problem

public class Employees : IEmployeeService
{
// Her skal alt implementeres
}

IEmployeeService employees = new Employees();
string test = employees.employeeName;

Nu bruger du din klasse via dit interface. Et interface er bare en kontrakt som en klasse skal overholde. Du kan ikke oprette en instance af et interface.

Hmmmmmm nu gør du mig rundphorvirret *griner*

Sådan som jeg tilgår (og har fået at vide af Arne) er på følgende måde:

IEmployeeService employees = (IEmployeeService)Activator.GetObject(typeof(IEmployeeService), "tcp://127.0.0.1:50000" + "/Employees");

employees.GetEmployee(txtEmployeeId.Text);

Hvor er det jeg i stedet for at tilgå IEmployeeService kan tilgå Employees?

Altså det jeg tænker på er at på klienten kender jeg jo kun til mit Interface og ikke Employee (og sådan jeg ønsker det, så jeg ikke skal opdatere to steder, hvis der er en fejl i koden).

Pas ... nu kom der en masse andet information med som jeg ikke har arebjdet med.

Hvordan ser din service ud? Den må vel returnere et object når du kalder

employees.GetEmployee(txtEmployeeId.Text);

Smider den ikke en DataTable tilbage til dit klient program?

Jeg tror ikke du kan gøre det sådan, da Activator.GetObject kun laver en proxy ... altså sender kald videre til din Url du har angivet.

Hvis du ikke kender den konkrete implementation på din klient og du f.eks har en metode ... hvad kode skal den så kalde?

Logisk set kan jeg ikke se hvordan det skal kunne lade sig gøre uden at kalde endnu en proxy  hvor din implementation findes.

Men hvad er problemet i at din klient kender implementeringen? Jeg kan godt se problemet hvis du har en metode som er implementeret forkert, så ville det være nemmere at fikse server side og alle klienter ville automatisk komme til at virke.

Men så kan du også have lavet Proxy'er til dit IEmployee interface på din server.

Håber det giver lidt stof til eftertanke i dit design.

Jo, jeg får table med over og vil tænke over implementation videre (synes ikke jeg bestiller andet *griner)...

Jeg bruger så tabellen istedet for - havde bare håbet jeg kunne undgå at skrive tabel navne direkte ind i klienten.

Smid et svar og du får pointsene :-)

Svar.

Well ... du kan undgå mange ting, men for at hjælpe der skal vi nok have en bedre oversigt over din projekt.

Jeg ville nok lave nogen DTO(Data Tranfers Object) klasser som kun indeholder data(altså properties) og så sende dem tilbage så du har det type stært i stedet for at sende en DataTable tilbage.

mvh

Den dag, hvor jeg føler, jeg selv har styr på oversigten, skal du nok få den *griner* Tror Arne, er den der har mest styr på den lige nu :-)

Jeg har faktisk fået det til at virke....

Interface ser sådan her ud:

public interface IEmployeeService
    {
        string employeeId { get; set; }
        string employeeName { get; set; }
        string employeeGroupId { get; set; }
        string password { get; set; }
        bool isAdmin { get; set; }
        bool isActive { get; set; }

        DataTable GetEmployee(string employeeId);

        bool GetEmployeeIsRegistered(string employeeId, string password, out bool isAdmin, out bool isActive);
    }

og mit implementation ser det så sådan her ud:

public class Employees : MarshalByRefObject, IEmployeeService
    {
        public string employeeId { get; set; }
        public string employeeName { get; set; }
        public string employeeGroupId { get; set; }
        public string password { get; set; }
        public bool isAdmin { get; set; }
        public bool isActive { get; set; }

        public DataTable GetEmployee(string EmployeeId)
        {
            try
            {
                DALEmployees employees = new DALEmployees();

                DataTable getData = employees.GetEmployee(EmployeeId);

                employeeId = EmployeeId;
                employeeName = employees.employeeName;
                employeeGroupId = employees.employeeGroupId;
                password = employees.password;
                isAdmin = employees.isAdmin;
                isActive = employees.isActive;

                return getData;
            }
            catch
            {
                return null;
            }
        }

        public bool GetEmployeeIsRegistered(string employeeId, string password, out bool isAdmin, out bool isActive)
        {
            isAdmin = false;
            isActive = false;

            try
            {
                DALEmployees employees = new DALEmployees();

                return employees.GetEmployeeIsRegistered(employeeId, password, out isAdmin, out isActive);
            }
            catch
            {
                return false;
            }

        }
    }

og så har jeg selvfølgelig min DAL som du har set:

#region " Enums, Members, Properties "

        DBUtility DAL = new DBUtility();

        public string employeeId { get; set; }
        public string employeeName { get; set; }
        public string employeeGroupId { get; set; }
        public string password { get; set; }
        public bool isAdmin { get; set; }
        public bool isActive { get; set; }

        #endregion

        #region " Data Access Get "

        public DataTable GetEmployee(string employeeId)
        {
            DAL.AddParameter("employeeId", employeeId);

            DataTable getData = DAL.GetDataTable("GetEmployee", CommandType.StoredProcedure);

            employeeName = string.Empty;
            employeeGroupId = string.Empty;
            password = string.Empty;
            isAdmin = false;
            isActive = false;

            if (getData.Rows.Count > 0)
            {
                employeeId = Convert.ToString(getData.Rows[0]["employeeId"]);
                employeeName = Convert.ToString(getData.Rows[0]["employeeName"]);
                employeeGroupId = Convert.ToString(getData.Rows[0]["employeeGroupId"]);
                password = Convert.ToString(getData.Rows[0]["password"]);
                isAdmin = Convert.ToBoolean(getData.Rows[0]["isAdmin"]);
                isActive = Convert.ToBoolean(getData.Rows[0]["isActive"]);
            }

            return getData;
        }
        public bool GetEmployeeIsRegistered(string employeeId, string password, out bool isAdmin, out bool isActive)
        {
            bool isRegistered = false;
           
            DAL.AddParameter("employeeId", employeeId);
            DAL.AddParameter("password", password);

            DataTable getData = DAL.GetDataTable("GetEmployeeIsRegistered", CommandType.StoredProcedure);

            isAdmin = false;
            isActive = false;

            if (getData.Rows.Count > 0)
            {
                employeeId = Convert.ToString(getData.Rows[0]["employeeId"]);
                password = Convert.ToString(getData.Rows[0]["password"]);
                isAdmin = Convert.ToBoolean(getData.Rows[0]["isAdmin"]);
                isActive = Convert.ToBoolean(getData.Rows[0]["isActive"]);

                isRegistered = true;
            }

            return isRegistered;
        }

        #endregion

Når jeg så tilgår gør jeg det på følgende måde:

IEmployeeService employees = (IEmployeeService)Activator.GetObject(typeof(IEmployeeService), ServerSettings + "/Employees");

employees.GetEmployee(txtEmployeeId.Text);
employeeName = employees.employeeName;

og så får jeg fat i det rigtige :-)

Det virker helt mærkeligt.

Du kalder "GetEmployee" som sætter en værdi på server objectet på serveren. den DataTable du får tilbage smider du væk. Så return "null" hvis du ikke skal bruge den.

Derefter kadler du en property på dit object på serveren som blev sat af overstående kald. Hvilket jeg ikke helt kan forstå. Hvorfor returere du ikke hvad du sprøger efter i det første kald ?

mvh

Til inspiration:

Common.cs



Server.cs



Client.cs



Server.exe.config



build:

C:\Work\remoting>client
(123,Fusk,Aner det ikke)
(1,Dilbert,Engineer)
(2,Wally,Socalled engineer)
(3,PHB,Boss)

Og dt er faktisk Buzzzz's DTO koncept.

Du skal husk at de klasser skal vaere serializable.

Jeg har saa valgt at putte DTO klassen i common.

Det er efter min mening helt fint. Hvis man vil aendre i DTO klassen skal der som oftest ogsaa aendres i client.

Hvis man gerne vil undgaa det burde man kunne lave et IEmployee og kun brug dette i common og client.

Jeg proevede lige:

Men som sagt tror jeg ikke at jeg gad bruge interfaces for DTO.