Replace all occurrences of substr in str by newstr #359

tshemsedinov · 2020-12-06T00:03:29Z

code is properly formatted (npm run fmt)
tests are added/updated
documentation is updated (npm run doc to regenerate documentation based on comments)
description of changes is added under the Unreleased header in CHANGELOG.md

Usage: `replace(str, substr, newstr)`

belochub

@tshemsedinov Is this basically a simpler polyfill for String#replaceAll() without the support for regular expression and function arguments?

tshemsedinov · 2021-01-03T02:31:53Z

Something like that, just replace all occurrences (like replace global flag) of sub-string in the given string with another sub-string. @belochub

Semigradsky · 2021-09-03T13:42:11Z

lib/strings.js

+//
+// Example: replace('a20w10z2a22aa0', 'a2', 'z') = 'z0w10z2z2aa0'
+const replace = (str, substr, newstr) => {
+  if (substr === '') return str;


Mention: it works different than String.prototype.replaceAll:

replace('a2', '', 'z') // 'a2' 'a2'.replaceAll('', 'z') // 'zaz2z'

Replace all occurrences of substr in str by newstr

44e2c13

Usage: `replace(str, substr, newstr)`

tshemsedinov added the functionality label Dec 6, 2020

tshemsedinov requested a review from belochub December 6, 2020 00:03

tshemsedinov self-assigned this Dec 6, 2020

belochub reviewed Dec 30, 2020

View reviewed changes

Semigradsky reviewed Sep 3, 2021

View reviewed changes

Provide feedback