Skip to content

Comments

Context değişkeni döndürmeye çalışıyor#1

Open
ercanerdogan wants to merge 1 commit intoCoskunKurtuldu:masterfrom
ercanerdogan:patch-1
Open

Context değişkeni döndürmeye çalışıyor#1
ercanerdogan wants to merge 1 commit intoCoskunKurtuldu:masterfrom
ercanerdogan:patch-1

Conversation

@ercanerdogan
Copy link

Method içlerinde MusicMarketDbContext olarak kullanıldığı için MusicMarketDbContext döndürmesi gerekirken Context olarak yazım yanlışı kalmış gibi.
Context olarak tanımlamak daha güzel olurdu aslında.

bir diğer alternatif olarak DependencyInjection ile yapabilirmiydik burada ? Aşağıda bir örnek bırakıyorum..

    private MusicMarketDbContext _context;

    public ArtistRepository(MusicMarketDbContext context)
        : base(context)
    {
        _context = context;

    }

    public async Task<IEnumerable<Artist>> GetAllWithMusicAsync()
    {
        return await _context.Artists.Include(a => a.Musics).ToListAsync();

    }

    public async Task<Artist> GetWithMusicsByIdAsync(int id)
    {
        return await _context.Artists.Include(a => a.Musics).SingleOrDefaultAsync(a => a.Id == id);
    }

Method içlerinde MusicMarketDbContext olarak kullanıldığı için MusicMarketDbContext  döndürmesi gerekirken Context olarak yazım yanlışı kalmış gibi. 
Context olarak tanımlamak daha güzel olurdu aslında. 

bir diğer alternatif olarak DependencyInjection ile yapabilirmiydik burada ? Aşağıda bir örnek bırakıyorum..

        private MusicMarketDbContext _context;

        public ArtistRepository(MusicMarketDbContext context)
            : base(context)
        {
            _context = context;

        }

        public async Task<IEnumerable<Artist>> GetAllWithMusicAsync()
        {
            return await _context.Artists.Include(a => a.Musics).ToListAsync();

        }

        public async Task<Artist> GetWithMusicsByIdAsync(int id)
        {
            return await _context.Artists.Include(a => a.Musics).SingleOrDefaultAsync(a => a.Id == id);
        }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant