Thay thế một phương pháp thông thường với một phương pháp vô danh trong C#/LINQ

Question

Tôi có một truy vấn LINQ trông như thế này:

public IEnumerable<Foo> SelectFooBars() 
{ 
    return 
     from 
      f in foos 
     join 
      b in bars 
      on f.BarId equals b.Id 
     select 
      AddMissingProp(f, b.MissingProp); 
} 

public void AddMissingProp(Foo foo, string missingProp) // substitute this with inline lambda 
{ 
    foo.MissingProp = missingProp; 
    return foo; 
} 

Tôi muốn thoát khỏi AddMissingProp và sử dụng một số hình thức của một lambda trong select của tôi thay vào đó.

Tôi đã cố gắng ...

... 
select 
    (f, b) => { f.MissingProp = b.MissingProp; return f } 

... nhưng tôi đã nhận được lỗi sau:

A local variable named 'f' cannot be declared in this scope because it would give a different meaning to 'f', which is already used in a 'parent or current' scope to denote something else.

Làm thế nào tôi có thể "lambda-ize" truy vấn của tôi?

---

Cập nhật

này cũng không hoạt động:

... 
select 
    () => { f.MissingProp = b.MissingProp; return f } 

tôi nhận được lỗi sau:

The type of one of the expressions in the join clause is incorrect. Type inference failed in the call to 'Join'.

tôi không thay đổi mệnh đề join tại tất cả, vì vậy tôi bối rối.

Answer 1

Bạn có thể cung cấp các loại cho tham số của mình trong biểu thức lambda nhưng bạn cần phải sử dụng các tên khác nhau vì bạn đã sử dụng f và b trong truy vấn.

(Foo f1, Bar b1) => ...

Sửa

return 
(
    from 
     f in foos 
    join 
     b in bars 
     on f.BarId equals b.Id 
    select 
     new {f, b} 
).select(foobar => {foobar.f.BarId = foobar.b.Id; return foobar.f}); 

Answer 2

chọn (f2, b2) => {f2.MissingProp = b2.MissingProp; return f2}

Answer 3

Viết lại điều này bằng cú pháp Lambda.

var vf2 = foos.Join(bars, f => f.id, b => b.id, (foo, bar) => { foo.MissingProp = bar.MissingProp; return foo; }); 

Nếu bạn cần giải thích về cú pháp này, hãy cho tôi biết.

Answer 4

Tôi nghĩ icambron là đúng, IMHO phiên bản có thể đọc được tốt hơn là thế này:

var fooBar = from 
       f in foos 
       join 
       b in bars 
       on f.BarId equals b.Id 
       select new {f,b}; 

    foreach(var x in fooBar) 
     x.f.MissingProp = x.b.MissingProp; 

    // EDIT due to comments: add this if you 
    // need IEnumerable<Foo> returned 
    return fooBar.Select(fb => fb.f); 

Các from join select báo cáo được cho các truy vấn, họ nên không được lạm dụng để biến đổi các nội dung của một chuỗi.

EDIT: Dưới đây là một số khác link cung cấp một số thông tin chi tiết về lý do sử dụng hình thức chức năng ForEach không phải là một ý tưởng hay.

Answer 5

Nếu trình biên dịch không thể suy ra đúng loại để chuyển đến một lambda, bạn tất nhiên có thể chỉ định loại chính mình.

này nên hoạt động tốt:

select 
    (Foo f2, b) => { f2.MissingProp = b.MissingProp; return f2; } 

Lưu ý rằng khi bạn đã nhận thấy, bạn không thể tái sử dụng f và hy vọng rằng nó sẽ giữ được ý nghĩa của nó.Đây là chữ ký phương thức mới, với các tham số mới, vì vậy bạn cần sử dụng tên riêng cho nó.

Khi thực hiện, bạn nhận thấy rằng trình biên dịch không thể tự tìm ra loại đối số đầu tiên, nhưng bạn có thể chỉ định đối số, như trên.