C# Nhà điều hành có điều kiện Không phải là tuyên bố?

Question

Tôi có một mảnh đơn giản chút mã được bực bội tôi:

HashSet<long> groupUIDs = new HashSet<long>(); 
groupUIDs.Add(uid)? unique++ : dupes++; 

Tại thời gian biên dịch, nó tạo ra các lỗi:

Only assignment, call, increment, decrement, and new object expressions can be used as a statement

HashSet.Add được ghi chép lại để trả về một bool, vì vậy ternary (?) nhà điều hành nên hoạt động, và điều này trông giống như một cách hoàn toàn hợp pháp để theo dõi số mục duy nhất và trùng lặp tôi thêm vào bộ băm.

Khi tôi định dạng lại dưới dạng if-then-else, nó hoạt động tốt.

Bất cứ ai có thể giải thích được lỗi và nếu có cách để thực hiện điều này như một toán tử bậc ba đơn giản?

Answer 1

Theo thông báo lỗi, toán tử bậc ba không thể được sử dụng làm câu lệnh. Bạn sẽ cần phải làm một cái gì đó như thế này để biến nó thành một bài tập:

int dummy = groupUIDs.Add(uid)? unique++ : dupes++; 

đó đang được nói, tôi khuyên bạn chỉ cần sử dụng if-then-else. Nó ít gây nhầm lẫn vì nó không liên quan đến việc tạo ra các biến giả "ma thuật" ...

Answer 2

Trình biên dịch không phàn nàn về Add nó phàn nàn về thực tế là biểu thức điều kiện của bạn không phải là một câu lệnh hoàn chỉnh.

Một số ngôn ngữ (như JavaScript) cho phép bạn sử dụng biểu thức có điều kiện cho logic nhánh như bạn đã làm ở đây nhưng C# yêu cầu bạn gán kết quả của biểu thức điều kiện cho biến. Một khi bạn gán kết quả của biểu thức, bạn đã thực hiện một câu lệnh đầy đủ và trình biên dịch rất vui.

Answer 3

Bạn không đặt kết quả giá trị của ternary thành bất kỳ thứ gì là lý do tại sao.

HashSet<long> groupUIDs = new HashSet<long>(); 
int count = groupUIDs.Add(uid)? unique++ : dupes++; 

Answer 4

Toán tử bậc ba không phải là câu lệnh. Vì vậy, nó không thể được sử dụng một mình trong một hướng dẫn - nó tương đương với việc viết

"something that is not a statement"; 

Để làm rõ, bạn nên dùng toán tử bậc ba và sử dụng if.

Answer 5

Nếu điều này không được chấp nhận, tại sao dòng của bạn lại là? Chỉ cần sử dụng câu lệnh if :-)

 bool b = false; 
     b?callB():callA(); 

Answer 6

gmcalab và sr pt là đúng; toán tử bậc ba có nghĩa là cung cấp cho bạn kết quả, giống như 1 + 1 cung cấp cho bạn 2. Bạn không thể chỉ viết:

1 + 1;

Sự nhầm lẫn ở đây (tôi nghĩ) là bạn đang nghĩ đến toán tử bậc ba giống như một hàm.

Answer 7

Bạn cần sử dụng giá trị từ toán tử bậc ba cho thứ gì đó ...

HashSet<long> groupUIDs = new HashSet<long>(); 
int newCount = groupUIDs.Add(uid)? unique++ : dupes++; 

hoặc - sử dụng một nếu

HashSet<long> groupUIDs = new HashSet<long>(); 
if (groupUIDs.Add(uid)) 
    unique++; 
else 
    dupes++; 

Answer 8

Các description của các nhà điều hành ternary trong tài liệu tham khảo ngôn ngữ nói rằng

If condition is true, first expression is evaluated and becomes the result; if false, the second expression is evaluated and becomes the result.

Dường như ternary chỉ có thể được sử dụng trong bối cảnh của bài tập, mặc dù tham chiếu ngôn ngữ không tuyên bố nó là sự giải thích. Bạn không làm bài tập về kết quả.

Theo ý kiến ​​của tôi, việc viết lại dưới dạng if/else sẽ rõ ràng hơn.

Answer 9

Như những người khác đã chỉ ra, toán tử điều kiện không phải là biểu thức tuyên bố pháp lý. (Các biểu thức tuyên bố pháp lý là các bài tập, cuộc gọi, số gia, giảm và công trình.)

Tuy nhiên, cũng có vấn đề về phong cách ở đây. Theo tôi, các biểu thức sẽ hữu ích cho các giá trị của chúng và sẽ hữu ích cho các tác dụng phụ của chúng. Những gì bạn đang chạy vào là bạn có một biểu thức chỉ hữu ích cho tác dụng phụ của nó, và đó là một mùi mã xấu.

Bạn có tác dụng phụ, vì vậy hãy sử dụng câu lệnh có điều kiện thay vì biểu thức có điều kiện.