ऐसा करने के लिए एक और अधिक पठनीय तरीका है?

Question

मुझे पता है कि इस लेखन का एक और अधिक पठनीय तरीका है:

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') 

संपादित करें:

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

को समारोह का नाम बदलकर कुछ एक सा प्रयास करें अधिक पठनीय:

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

उपयोग के रूप में:

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

तुम भी पागल हो जाना कर सकते हैं और prototype में जोड़ना, और r.days.singleOrPlural('day', 'days') साथ खत्म हो, लेकिन मुझे नहीं लगता कि इसे यहाँ की जरूरत है या नहीं।

Answer 2

मैं कुछ इस तरह का प्रयोग करेंगे 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');