Cách viết biểu thức toán tử bậc ba (còn gọi là if) mà không cần tự lặp lại

Ví dụ, một cái gì đó như thế này:

var value = someArray.indexOf(3) !== -1 ? someArray.indexOf(3) : 0

Có cách nào tốt hơn để viết điều đó không? Một lần nữa, tôi không tìm kiếm câu trả lời cho câu hỏi chính xác ở trên, chỉ là một ví dụ về thời điểm bạn có thể lặp lại các toán hạng trong biểu thức toán tử bậc ba ...

Cá nhân tôi thấy cách tốt nhất để làm điều này vẫn là ifcâu nói cũ :

var value = someArray.indexOf(3);
if (value === -1) {
  value = 0;
}

Mã phải dễ đọc, do đó, ngắn gọn không có nghĩa là ngắn gọn bất cứ giá nào - vì vậy bạn nên đăng lại lên https://codegolf.stackexchange.com/ - vì vậy thay vào đó, tôi khuyên bạn nên sử dụng biến cục bộ thứ hai có tên indexđể tối đa hóa khả năng đọc hiểu ( với chi phí thời gian chạy tối thiểu, tôi lưu ý):

var index = someArray.indexOf( 3 );
var value = index == -1 ? 0 : index;

Nhưng nếu bạn thực sự muốn cắt giảm biểu hiện này, vì bạn là một kẻ tàn bạo tàn nhẫn với đồng nghiệp hoặc cộng tác viên dự án của mình, thì đây là 4 cách tiếp cận bạn có thể sử dụng:

1: Biến tạm thời trong một varcâu lệnh

Bạn có thể sử dụng varkhả năng của câu lệnh để xác định (và gán) một biến tạm thời thứ hai indexkhi được phân tách bằng dấu phẩy:

var index = someArray.indexOf(3), value = index !== -1 ? index: 0;

2: Chức năng ẩn danh tự thực hiện

Một tùy chọn khác là một chức năng ẩn danh tự thực thi:

// Traditional syntax:
var value = function( x ) { return x !== -1 ? x : 0 }( someArray.indexOf(3) );

// ES6 syntax:
var value = ( x => x !== -1 ? x : 0 )( someArray.indexOf(3) );

3: Toán tử dấu phẩy

Ngoài ra còn có "toán tử dấu phẩy" khét tiếng mà JavaScript hỗ trợ, cũng có trong C và C ++.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Comma_Operator

Bạn có thể sử dụng toán tử dấu phẩy khi bạn muốn bao gồm nhiều biểu thức trong một vị trí yêu cầu một biểu thức duy nhất.

Bạn có thể sử dụng nó để giới thiệu các tác dụng phụ, trong trường hợp này bằng cách gán lại cho value:

var value = ( value = someArray.indexOf(3), value !== -1 ? value : 0 );

Điều này hoạt động vì var valueđược diễn giải trước tiên (vì đó là một câu lệnh), sau đó là valuephép gán ngoài cùng bên trái, bên trong nhất và sau đó là bên phải của toán tử dấu phẩy và sau đó là toán tử bậc ba - tất cả đều là JavaScript hợp pháp.

4: Chỉ định lại trong một biểu thức con

Bình luận viên @IllusingBrian đã chỉ ra rằng việc sử dụng toán tử dấu phẩy (trong ví dụ trước) là không cần thiết nếu phép gán cho valueđược sử dụng như một biểu thức con có dấu ngoặc đơn:

var value = ( ( value = someArray.indexOf(3) ) !== -1 ? value : 0 );

Lưu ý rằng việc sử dụng phủ định trong các biểu thức logic có thể khó làm theo đối với con người hơn - vì vậy tất cả các ví dụ trên có thể được đơn giản hóa để đọc bằng cách thay đổi idx !== -1 ? x : ythành idx == -1 ? y : x:

var value = ( ( value = someArray.indexOf(3) ) == -1 ? 0 : value );

Đối với những con số

Bạn có thể sử dụng Math.max()chức năng.

var value = Math.max( someArray.indexOf('y'), 0 );

Nó sẽ giữ các ranh giới của kết quả 0cho đến khi kết quả đầu tiên lớn hơn 0nếu đúng như vậy. Và nếu kết quả từ indexOflà -1nó sẽ trả về 0 lớn hơn -1.

Đối với giá trị boolean và boolean-y

Đối với JS không có quy luật chung AFAIK đặc biệt vì cách falsy giá trị được đánh giá.

Nhưng nếu điều gì đó có thể giúp bạn hầu hết thời gian là toán tử hoặc ( ||):

// Instead of
var variable = this_one === true ? this_one : or_this_one;
// you can use
var variable = this_one || or_this_one;

Bạn phải rất cẩn thận với điều này, bởi vì trong ví dụ đầu tiên của bạn, indexOfcó thể trở lại 0và nếu bạn đánh giá 0 || -1nó sẽ trở lại -1vì 0là một falsy giá trị.

Không thực sự, chỉ cần sử dụng một biến khác.

Ví dụ của bạn khái quát thành một cái gì đó như thế này.

var x = predicate(f()) ? f() : default;

Bạn đang kiểm tra một giá trị được tính toán, sau đó gán giá trị đó cho một biến nếu nó chuyển một số vị từ. Cách để tránh tính lại giá trị đã tính là hiển nhiên: sử dụng một biến để lưu trữ kết quả.

var computed = f();
var x = predicate(computed) ? computed : default;

Tôi hiểu ý bạn - có vẻ như cần phải có một số cách để làm điều này trông gọn gàng hơn một chút. Nhưng tôi nghĩ đó là cách tốt nhất (nói một cách thành ngữ) để làm điều này. Nếu bạn đang lặp lại nhiều lần mẫu này trong mã của mình vì lý do nào đó, bạn có thể viết một hàm trợ giúp nhỏ:

var setif = (value, predicate, default) => predicate(value) ? value : default;
var x = setif(someArray.indexOf(3), x => x !== -1, 0)

CHỈNH SỬA: Đây rồi, đề xuất cho liên kết Nullary ngay bây giờ trong JavaScript!

Sử dụng ||

const result = a ? a : 'fallback value';

tương đương với

const result = a || 'fallback value';

Nếu truyền ađể Booleantrả về false, resultsẽ được gán 'fallback value', nếu không giá trị của a.

Hãy lưu ý đến trường hợp cạnh a === 0, sẽ diễn ra falsevà resultsẽ (không chính xác) lấy 'fallback value'. Sử dụng các thủ thuật như thế này có nguy cơ của riêng bạn.

Tái bút. Các ngôn ngữ như Swift có toán tử liên kết nil ( ??), phục vụ mục đích tương tự. Ví dụ, trong Swift, bạn sẽ viết result = a ?? "fallback value"nó khá gần với JavaScriptconst result = a || 'fallback value';

Sử dụng cấu trúc lại biến trích xuất :

var index = someArray.indexOf(3);
var value = index !== -1 ? index : 0

Nó thậm chí còn tốt hơn với constthay vìvar . Bạn cũng có thể thực hiện một trích xuất bổ sung:

const index = someArray.indexOf(3);
const condition = index !== -1;
const value = condition ? index : 0;

Trên thực tế, sử dụng tên có ý nghĩa hơn index, conditionvà value.

const threesIndex = someArray.indexOf(3);
const threeFound = threesIndex !== -1;
const threesIndexOrZero = threeFound ? threesIndex : 0;

Có thể bạn đang tìm kiếm một nhà điều hành liên kết. May mắn thay, chúng tôi có thể tận dụng Arraynguyên mẫu để tạo ra:

Array.prototype.coalesce = function() {
    for (var i = 0; i < this.length; i++) {
        if (this[i] != false && this[i] != null) return this[i];
    }
}

[null, false, 0, 5, 'test'].coalesce(); // returns 5

Điều này có thể được tổng quát hóa thêm cho trường hợp của bạn, bằng cách thêm một tham số vào hàm:

Array.prototype.coalesce = function(valid) {
    if (typeof valid !== 'function') {
        valid = function(a) {
            return a != false && a != null;
        }
    }

    for (var i = 0; i < this.length; i++) {
        if (valid(this[i])) return this[i];
    }
}

[null, false, 0, 5, 'test'].coalesce(); // still returns 5
[null, false, 0, 5, 'test'].coalesce(function(a){return a !== -1}); // returns null
[null, false, 0, 5, 'test'].coalesce(function(a){return a != null}); //returns false

Cá nhân tôi thích hai biến thể:

1. Thuần túy nếu, như @slebetman đề xuất

2. Hàm tách biệt, thay thế giá trị không hợp lệ bằng giá trị mặc định, như trong ví dụ sau:

function maskNegative(v, def) {
  return v >= 0 ? v : def;
}

Array.prototype.indexOfOrDefault = function(v, def) {
  return maskNegative(this.indexOf(v), def);
}

var someArray = [1, 2];
console.log(someArray.indexOfOrDefault(2, 0)); // index is 1
console.log(someArray.indexOfOrDefault(3, 0)); // default 0 returned
console.log(someArray.indexOfOrDefault(3, 123)); // default 123 returned

Tôi thích câu trả lời của @ slbetman. Bình luận dưới đó bày tỏ lo lắng về việc biến đang ở "trạng thái trung gian". nếu đây là một mối quan tâm lớn đối với bạn thì tôi khuyên bạn nên gói gọn nó trong một hàm:

function get_value(arr) {
   var value = arr.indexOf(3);
   if (value === -1) {
     value = 0;
   }
   return value;
}

Sau đó chỉ cần gọi

var value = get_value( someArray );

Bạn có thể thực hiện nhiều chức năng chung hơn nếu bạn sử dụng chúng ở những nơi khác, nhưng đừng thiết kế quá mức nếu đó là một trường hợp rất cụ thể.

Nhưng thành thật mà nói, tôi sẽ chỉ làm như @slebetman trừ khi tôi cần sử dụng lại từ một số nơi.

Có hai cách tôi có thể thấy khi xem xét câu hỏi của bạn: bạn muốn giảm độ dài dòng hoặc bạn muốn tránh lặp lại một biến trong một số ba. Đầu tiên là tầm thường (và nhiều người dùng khác đã đăng ví dụ):

var value = someArray.indexOf(3) !== -1 ? someArray.indexOf(3) : 0;

có thể được (và nên được, với các lệnh gọi hàm) được rút gọn như vậy:

var value = someArray.indexOf(3);
value = value !== -1 ? value : 0;

Nếu bạn đang tìm kiếm một giải pháp chung chung hơn để ngăn chặn sự lặp lại của một biến trong một bậc ba, như sau:

var value = conditionalTest(foo) ? foo : bar;

nơi foochỉ xuất hiện một lần. Loại bỏ các giải pháp của biểu mẫu:

var cad = foo;
var value = conditionalTest(foo) ? cad : bar;

đúng kỹ thuật nhưng thiếu điểm, thì bạn đã hết may mắn. Có các toán tử, hàm và phương thức sở hữu cú pháp ngắn gọn mà bạn tìm kiếm, nhưng các cấu trúc như vậy, theo định nghĩa, không phải là toán tử bậc ba .

Ví dụ:

javascript, sử dụng ||để trả về RHS khi LHS là falsey:

var value = foo || bar; // equivalent to !foo ? bar : foo

Sử dụng một chức năng trợ giúp:

function translateValue(value, match, translated) {
   return value === match ? translated : value;
}

Bây giờ mã của bạn rất dễ đọc và không có sự lặp lại.

var value = translateValue(someArray.indexOf(3), -1, 0);

Thứ bậc của các mối quan tâm về mã hóa là:

1. Đúng (bao gồm hiệu suất thực sự hoặc các mối quan tâm về SLA)
2. Thông thoáng
3. Ngắn gọn
4. Nhanh

Tất cả các câu trả lời trên trang cho đến nay đều có vẻ đúng, nhưng tôi nghĩ phiên bản của tôi có độ rõ ràng cao nhất, điều này quan trọng hơn tính ngắn gọn. Nếu bạn không tính hàm trợ giúp — vì nó có thể được sử dụng lại — thì đó cũng là hàm ngắn gọn nhất. Đề xuất hơi tương tự để sử dụng một hàm trợ giúp không may lại sử dụng lambda mà đối với tôi, chỉ che khuất những gì nó đang làm. Đối với tôi, một hàm đơn giản hơn với một mục đích không sử dụng lambda, chỉ sử dụng các giá trị.

PS Nếu bạn thích cú pháp ES6:

const translateValue = (value, match, translated) => value === match ? translated : value;
let value = translateValue(someArray.indexOf(3), -1, 0); // or const

Tôi nghĩ rằng ||nhà điều hành có thể được điều chỉnh để indexOf:

var value = ((someArray.indexOf(3) + 1) || 1) - 1;

Giá trị trả về được dịch chuyển lên 1, tạo thành 0 từ -1, giá trị này là sai và do đó được thay thế bằng giá trị thứ hai 1. Sau đó, nó được chuyển ngược lại.

Tuy nhiên, hãy nhớ rằng khả năng đọc tốt hơn việc tránh lặp lại.

Đây là một giải pháp đơn giản với KHÔNG theo bit và một giá trị mặc định -1mà kết quả sau đó là 0.

index = ~(~array.indexOf(3) || -1);

Về cơ bản, nó hoạt động với hàm NOT bit theo chiều kép, trả về giá trị ban đầu hoặc giá trị mặc định, sau khi áp dụng bitwise NOT sẽ trả về 0.

Hãy xem bảng sự thật:

 indexOf    ~indexOf   boolean    default     value      result         comment
---------  ---------  ---------  ---------  ---------  ---------  ------------------
      -1          0     falsy          -1         -1          0   take default value
       0         -1    truthy                     -1          0
       1         -2    truthy                     -2          1
       2         -3    truthy                     -3          2

Bạn có thể sử dụng phân công lại:

• khởi tạo biến thành một giá trị
• sử dụng tuần tự hóa &&toán tử để gán lại, bởi vì nếu điều kiện đầu tiên là sai, biểu thức thứ hai sẽ không được đánh giá

Ví dụ.

var value = someArray.indexOf(3);
value == -1 && (value=0);

var someArray = [4,3,2,1];

var value = someArray.indexOf(1);
value == -1 && (value=0);
console.log('Found:',value);

var value = someArray.indexOf(5);
value == -1 && (value=0);
console.log('Not Found:',value);

Đưa ra mã ví dụ tại Câu hỏi, không rõ làm thế nào nó sẽ được xác định rằng 3được hoặc không được đặt ở chỉ mục 0của someArray. -1trả về từ .indexOf()sẽ có giá trị trong trường hợp này, với mục đích loại trừ trường hợp không khớp được cho là có thể là một kết quả trùng khớp.

Nếu 3không được bao gồm trong mảng, -1sẽ được trả về. Chúng ta có thể thêm 1vào kết quả của .indexOf()để đánh giá falsekết quả như thế nào -1, theo sau là || ORtoán tử và 0. Khi valueđược tham chiếu, hãy trừ đi 1để nhận chỉ số của phần tử của mảng hoặc -1.

Dẫn đến việc chỉ cần sử dụng .indexOf()và kiểm tra -1tại một ifđiều kiện. Hoặc định valuenhư undefinedđể tránh có thể nhầm lẫn về thực tế kết quả của tình trạng đánh giá liên quan đến tài liệu tham khảo ban đầu.

var someArray = [1,2,3];
var value = someArray.indexOf(3) + 1 || 1;
console.log(value -= 1);

var someArray = [1,2,3];
var value = someArray.indexOf(4) + 1 || 1;
// how do we know that `4` is not at index `0`?
console.log(value -= 1);

var someArray = [1,2,3];
var value = someArray.indexOf(4) + 1 || void 0;
// we know for certain that `4` is not found in `someArray`
console.log(value, value = value || 0);

Một con ba giống như một if-else, nếu bạn không cần phần khác, tại sao không chỉ một if thay vào đó ..

if ((value = someArray.indexOf(3)) < 0) value = 0;

Đối với trường hợp cụ thể này, bạn có thể sử dụng đoản mạch với ||toán tử logic . Vì 0được coi là sai, bạn có thể thêm 1vào chỉ mục của mình, do đó, nếu index+1có 0thì bạn sẽ nhận được phía bên phải của logic-hoặc kết quả của bạn, nếu không, bạn sẽ nhận được index+1. Vì kết quả mong muốn của bạn được bù đắp bởi 1, sau đó bạn có thể trừ đi 1để nhận chỉ mục của mình:

const someArray = [1, 2, 3, 4];
const v = ((someArray.indexOf(3)+1) || 1)-1;
console.log(v);