Ah stimmt, du hast Recht :D

Es fehlt noch eine Kleinigkeit im Regulären Ausdruck.
Folgender Smiley gibt mit deinem Regex true zurück:
"::-)" oder ":-))"

1. Fair, das stimmt :)

2. Jo, hier finde ich das nicht so schlimmt, aber in einem richtigen Projekt hast du definitiv recht!

3. Wie 1.

4. Stimme ich dir auch zu, aber nur wenn '...' dann auch in einer Konstanten/einem Parameter steckt, meiner Meinung nach.

1. desiredLength ist immer gleich size. Das heisst, du kannst desiredLength weglassen und stattdessen size nutzen.
2. "..." in eine Variable auslagern. Falls man irgendwann mal die trim Funktion anpassen möchte (Bspw. nur noch 2 Punkte anzeigen), muss man das dann nur noch an einer Stelle tun.
3. Dasselbe gilt für desiredLength - 3 (bzw. size - 3)
4. Die 3 ist hier eine Magic-Number. Stattdessen kannst du "...".length (bzw. Variable, in die du "..." speicherst) nutzen, da es hier eine Korrelation gibt