một cách dễ đọc hơn để làm điều này?

Question

Tôi biết có một cách dễ đọc hơn viết này:

var string = ( 
    (r.d != 0 ? r.d + (r.d == 1 ? ' day' : ' days') : '') + 
    (r.h != 0 ? r.h + (r.h == 1 ? ' hour' : ' hours') : '') + 
    (r.m != 0 ? r.m + (r.m == 1 ? ' minute' : ' minutes') : '') + 
    (r.s != 0 ? r.s + (r.s == 1 ? ' second' : ' seconds') : '')); 

Answer 1

Hãy thử một cái gì đó một chút dễ đọc hơn:

function singleOrPlural(val, single, plural){ 
    if(val == 1) 
    return val + ' ' + single; 
    if(val > 1) 
    return val + ' ' + plural; 
    return ''; 
} 

Sử dụng như:

singleOrPlural(r.day, 'day', 'days') 

Bạn thậm chí có thể đi các loại hạt và thêm video này vào các prototype, và kết thúc với r.days.singleOrPlural('day', 'days'), nhưng tôi không nghĩ rằng nó là cần thiết ở đây.

Answer 2

Tôi sẽ sử dụng một cái gì đó như thế này:

function pluralize(aVal, aSuffix) { 
    var r = ''; 

    if (aVal != 0) { 
     r = aVal + ' ' + aSuffix + (aVal > 1 ? 's' : ''); 
    } 

    return r; 
} 

var string = pluralize(r.d, 'day') + pluralize(r.h, 'hour') + 
    pluralize(r.m, 'minute') + pluralize(r.s, 'second') 

EDIT: đổi tên các chức năng để pluralize

Answer 3

function getvalue(val, unit) 
{ 
     if(val>0) 
      return val + unit + (val > 1 ? 's' : ''); 
     else 
      return ''; 
} 

var string = gets(r.d, 'day') + gets(r.h, 'hour') + gets(r.m, 'minute') + gets(r.s, 'second');