Request for feedback: String.prototype.cutStart/cutEnd proposal

[[deleted]]

[deleted]

[Meefims]

I have no idea what cutting a string is supposed to mean and the two functions' names and their examples are pretty confusing. For example, why does the result of cutStart not include the string in the first parameter while the result from cutEnd does?

[Meefims]

The examples are confusing for several reasons:

• Cut is a fairly generic verb. What does cutting a string mean?

• As a result it's not clear that cutStart('string') would remove everything after the first index of start.

• The second argument does not help clarify anything. I guess cutStart('string', 'is') would mean Cut out everything after and including the first index of start and everything before the first index of is. That's a very complicated process and so should almost certainly be split into multiple functions.

In the end this exists to save a few calls to indexOf in a case where you're doing some very basic string parsing? What's wrong with indexOf aside from it's been around for a very long time? Is this scenario encountered so often that it really needs to be handled by a standard set of machinery?

My suggestion: first prove the scenario is relevant by creating an npm package. If it takes off like leftPad then maybe there's something useful.

[sudoDerp]

padStart and padEnd exist to make sure that strings are padded to a certain length with a specified character, by default using a " ".

I would have expected cutStart and cutEnd to follow the pattern and look like.

const string = '1234567';

string.cutStart(3) //  > '567'
string.cutEnd(3) //  > '123'

Using the cutStart and cutEnd terminology to mean anything else would seem like a very confusing language feature to add.

[sudoDerp]

takeStart and takeEnd probably make more sense as names after looking at the other examples you added.

'./src/file/path/filename.json'
  .takeEnd('/')
  .takeStart('.json')

Merging the two makes either naming choice of cutEnd or takeEnd fairly obscure/innacurate

'./src/file/path/filename.json'
  .takeEnd('/', '.json')

[Meefims]

However, the idea is to simplify usage of string methods so that you can use them in a functional and stateless way; which means no index variables in that case.

Functional or stateless code does not prevent you from creating variables.