В чем ошибка при реплейсе?

Question

[Dmitri1998]

Код выдает ошибку

return tmp[len].replace(/{name}/g, (item, i) : string =>

export const likes = (a: string[]) : string => {
  let len: number;
  let tmp: string[] = [
    "no one",
    "{name}",
    "{name}, {name}",
    "{name}, {name}, {name} ",
    "{name}, {name} and others"
  ]
  a.length > 3 ? len = 4 : len = a.length;
  console.log(tmp[len]);
  return tmp[len].replace(/{name})/g, (item, i) : string => {
     if(item == '{name}') return a[i];
  })
}

Comments

[WbICHA]

if(item == '{name}')

В чём смысл этой проверки?

[Dmitri1998]

WbICHA, код был чуть длиннее, но из-за поиска ошибки стал максимально уменьшать его, чтобы найти, в чем причина

Answer 1

[Alexandroppolus]

/{name})/g - лишняя круглая скобка

Comments

[Dmitri1998]

Убрал

Function lacks ending return statement and return type does not include 'undefined'.

12   return tmp[len].replace(/{name}/g, (item, i) : string => {

~~~~~~

[WbICHA]

Dmitri1998, и в чём проблема?

[Dmitri1998]

WbICHA, что не включена возможность вернуть undefined.
Но он его никогда не вернет, а как это исправить, не могу найти.

[WbICHA]

Dmitri1998, ...
if(item == '{name}')

[Dmitri1998]

WbICHA, спасибо, но почему
работает
return (item == '{name}') ? a[i] : '';
а это выдает ошибку
return (item == '{name}') ? a.shift() : '';

Type 'string | undefined' is not assignable to type 'string'.
  Type 'undefined' is not assignable to type 'string'.

13      return (item == '{name}') ?  a.shift() : '';

[Alexandroppolus]

Dmitri1998, у метода shift возвращаемое значение может быть undefined

[WbICHA]

Dmitri1998, потому что шифт может вернуть андефайнд, если массив пустой, очевидно же.
Брать значение по индексу, фактически, тоже может, но по дефолту это не так.

[Dmitri1998]

WbICHA,
И как добавить в перебор результата функции не только строку, но и undefined, хотя он там никогда в жизни не сможем появится.
Можно как-то исключить это и оставить, что вернет строго строку и никакого андефаинд?

[WbICHA]

Dmitri1998, серьёзно? Совет - забудь о тс, вернись на жс.

[Dmitri1998]

WbICHA, (item) : string | number | undefined => {
В книге было вроде так написано, если нам нужно 2 и более типов возвращать, но выдает ошибку

No overload matches this call.
  The last overload gave the following error.
    Argument of type '(item: string) => string | number | undefined' is not assignable to parameter of type '(substring: string, ...args: any[]) => string'.
      Type 'string | number | undefined' is not assignable to type 'string'.
        Type 'undefined' is not assignable to type 'string'.

[WbICHA]

Dmitri1998, так прочти эту ошибку и проанализируй что у тебя не так.

[Dmitri1998]

WbICHA,
Эти типы не могут быть присвоены типу строка.
Но я не совсем понимаю, что он хочет, если я указал все возможные типы, которые могут быть.
Но тут и строка уже не может быть строкой.

Type 'string | number | undefined' is not assignable to type 'string'.
Type 'undefined' is not assignable to type 'string'

[WbICHA]

Dmitri1998, ...
Калбек реплейса хочет функцию, которая будет возвращать СТРОКУ, а ты возвращаешь ему строку И не строку.

[Dmitri1998]

WbICHA, а как правильно вернуть?
Кроме такого, у меня больше никак не работает, но это не сосем правильно с учётом того, что мы пишем ее на ts, как мне сказать колбеку, чтобы он мог в ответ получать не только строку, но и ещё число или андефаинд?
(item) : any => {

[WbICHA]

Dmitri1998, никак, калбек ждёт строки и должен возвращать строку и ничего более. В этом и смысл тса.

[Dmitri1998]

WbICHA, т.е. здесь нужно просто писать any?

[WbICHA]

Dmitri1998, нет, здесь нужно писать string.

[Alexandroppolus]

Dmitri1998, WbICHA, заспамили, разбойники! Как отписаться? Давайте переносите вашу дискуссию в комменты ответа RokeAlvo, его очередь ))

[WbICHA]

Alexandroppolus,

[WbICHA]

не

[WbICHA]

переживай,

[WbICHA]

я

[WbICHA]

сейчас

[WbICHA]

плюну

[WbICHA]

на

[WbICHA]

это

[WbICHA]

дело

[WbICHA]

)

Answer 2

[RokeAlvo]

tmp[len] может быть undefined
Надо либо проверить на это , либо скастовать, что тут точно всегда есть значение, либо в описание возвращаемого значения вставить undefined

Comments

[Dmitri1998]

Как это делается?
Вроде читал про тс, но не могу понять, как это исправить.

[RokeAlvo]

колбек в реплейсе должен всегда возвращать строку (какую именно зависит от того что надо в итоге получить)

export const likes = (a: string[]) : string => {
  let len: number;
  const tmp: string[] = [
    "no one",
    "{name}",
    "{name}, {name}",
    "{name}, {name}, {name} ",
    "{name}, {name} and others"
  ]
  a.length > 3 ? len = 4 : len = a.length;
  console.log(tmp[len]);
  return tmp[len].replace(/{name}/g, (item, i)  => {
    if(item == '{name}') return a[i];
    return ''
  })
}