Wie kann man diese Saite effizient aufbauen?

Question

Ich habe eine Zahl, die durch 4 teilbar ist, z. 20. Jetzt muss ich einen String erstellen, die wie folgt aussieht:

(1, 2, 3, 4), (5, 6, 7, 8), (9, 10, 11, 12), (13, 14, 15, 16), (17, 18, 19, 20) 

alle Zahlen von 1 bis 20, in Packungen mit 4, eingewickelt in Klammern, und alles durch Komma getrennt gruppiert also.

Mein aktueller Ansatz sieht wie folgt aus:

const placeholders = []; 

for (let i = 0; i < events.length; i++) { 
    const base = i * 4 + 1; 

    placeholders.push(`(${base}, ${base + 1}, ${base + 2}, ${base + 3})`); 
} 

const result = placeholders.join(','); 

Gibt es eine Möglichkeit, dies a) effizienter zu machen, und b) in einem besser lesbaren Art und Weise?

Answer 1

var results = []; 
for(var i = 1; i <= events.length * 4; i++){ 
    var m = i % 4; 
    results.push((m==1 ? "(" : "") + i + (m==0 ? ")" : "")); 
} 
results = results.join(", "); 

Besser lesbar? Wahrscheinlich nicht. Effizienter? Sie laufen ungefähr gleich.

Der beste Weg zum Mitnehmen ist wahrscheinlich, dass Sie Ihre Schleife so ändern können, dass sie bei 1 beginnt.

Answer 2

Einfach versucht. Ich weiß nicht effizient oder nicht.

res = '('; 
for (i = 1; i <= 20; i++) { 
    var mod = i % 4; 
    (mod == 1 && i != 1) ? res += ',(' : ''; 
    res += i; 
    res += (mod != 0) ? ',' : ')'; 
}