current community

your communities

more stack exchange communities

Stack Exchange
tour help
stack overflow careers
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free.
up vote 3 down vote favorite

I want to retrieve an image from database and then send it to view and display it as thumbnail. Here is my code but It has some error and I can't return the image properly. I'm using ASP.Net MVC and aspx pages.

Controller :

public ActionResult showImg(int id)
{
   var imageData = from m in db.Products
                   where m.ShopId == 3
                   select Image.FromStream(new MemoryStream(m.Product_img.ToArray()));

   return new FileStreamResult(new System.IO.MemoryStream(imageData), "image/jpeg");
}

View :

<img src='<%= Url.Action("showImg", "image", new { id = ViewData["imageID"] } ) %>' />
share|improve this question
    
Have you ruled out any database issue, just trying to return a hardcoded filename instead of a memory stream? –  Nick Nov 2 '12 at 10:10
    
No it is just a test. I won't use hardcode in real situation. –  Doctor Nov 2 '12 at 10:13
    
I know you will not want to hardcode it - but are you sure that it is not a database connection issue? And is your query returning exactly what you want? –  Nick Nov 2 '12 at 10:16
    
Yes my friend, I have already inserted an image to the table successfully, but now I want to retrieve it. The imageData filled with System.Drawing.Image but FileStreamResult wants to return (byte[],string). Thats it. –  Doctor Nov 2 '12 at 10:26

2 Answers 2

up vote 2 down vote accepted

It looks like you do not need to recreate the Image object in your linq statement, just return the stream and pass it your FileStreamResult:

var image = (from m in db.Products
             where m.ShopId == 3
             select m.Product_img).FirstOrDefault();

var stream = new MemoryStream(image.ToArray());

return new FileStreamResult(stream, "image/jpeg");
share|improve this answer
    
Cannot convert System.Linq.IQueryable<System.IO.MemoryStrem> to System.IO.Stream –  Doctor Nov 2 '12 at 11:11
    
Have updated answer to account for this. –  Nick Nov 2 '12 at 11:19
    
Only parameterless constructors and initializers are supported in LINQ to Entities. –  Doctor Nov 2 '12 at 11:51
    
Have updated the answer again, now performing MemoryStream construction outside of the linq statement. –  Nick Nov 2 '12 at 13:14
up vote 0 down vote

Thanks a lot Nick. Your code is now correct. I wrote it in this way :

    var stream = (from m in db.Products where m.ShopId == 3 select m.Product_img).FirstOrDefault();

            return File(stream, "image/jpeg");
share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.