c# add parameter for each methods in class

Question

public class BaseDAL
{
    protected object GetOrder(int id)
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }

    public object GetByID(int id)
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }
}

Is there any option to add this param int id for any function in class without write it each function?

Somthing like:

public class BaseDAL : ADD SOME CODE THAT SHARE int id for all funcs
{
    protected object GetOrder()
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }

    public object GetByID()
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }
}

and GetOrder(12); will works. I want this done only to reduce the code.

You want a function that takes a parameter, but you don't want to include that parameter in the code? The compiler is at this time unable to read your mind. — Jonathon Reinhart
– Jonathon Reinhart, Commented Dec 28, 2014 at 19:06

Joel Coehoorn · Accepted Answer · 2014-12-28 19:12:31Z

You could set an ID parameter in the constructor, like so:

public class BaseDAL
{

    public BaseDAL(int ID)
    {
       this.id= ID;
    }

    int id;

    protected object GetOrder()
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }

    public object GetByID()
    {
        using (var DB = AccessManager.db)
        {
            return DB.Videos.Where(x => x.ID == id).SingleOrDefault();
        }
    }
}

That allows you to write code like this:

var result = new BaseDAL(5).GetOrder();

You could also combine this with the other answer, so that you have a property you can set and change. But I don't really think either is a good idea. I think what you already have is better. Also, I think you'd be better served to return Order and Video objects, rather than Objects.

Olivier Jacot-Descombes · Accepted Answer · 2014-12-28 19:04:20Z

0

Add a CurrentIndex property to your class

public class BaseDAL
{
    public CurrentIndex { get; set; }

    public Order GetOrder()
    {
        using (var DB = AccessManager.db) {
            return DB.Videos.Where(x => x.ID == CurrentIndex).SingleOrDefault();
        }
    }

    ...
}

and use it like this

var dal = new BaseDAL();
dal.CurrentIndex = 5;
var result = dal.GetOrder();

answered Dec 28, 2014 at 19:04

Olivier Jacot-Descombes

114k14 gold badges149 silver badges202 bronze badges

2 Comments

dor Over a year ago

Somthing like this but call the function like: var dal = new BaseDAL(); var result = dal.GetOrder(5);

Olivier Jacot-Descombes Over a year ago

No. If you want to call a method with parameters, you have to define them at the method declaration! What's wrong with declaring the parameter at each method declaration? int id has 6 characters, could it be that you are a bit lazy?

Collectives™ on Stack Overflow

c# add parameter for each methods in class

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related