Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Помогите пожалуйста, если можно с объяснением (https://javascript.ru/forum/misc/83414-pomogite-pozhalujjsta-esli-mozhno-s-obyasneniem.html)

Aetae 03.12.2021 07:50

voraa, не согласен на счёт (object, options, filter). В options нормальные умолчания.)

Правилбьно было бы конечно, чтоб и так (object, options, filter) и так (object, filter, options). Но это некрасивую условную раскоряку писать.)

Давно хочу либу сделать для работы с аргументами функции в любых порядках и направлениях, да руки не доходят. Те что видел в npm мне эстетически не нравятся.)

voraa 03.12.2021 08:22

Цитата:

Сообщение от Aetae
Давно хочу либу сделать для работы с аргументами функции в любых порядках и направлениях, да руки не доходят.

Я тоже пытался. Но отказался, так как часто получается слишком тяжеловесно.
Простой пример - функция создания элемента
С вариантами использования
create (tag, options [, children [, parent]])
create (tag [, children | null [, parent]])
где
tag - string
children - Node | Array | string
parent - Element
options - object (но не Array | Node)

Слишком много разных проверок, что бы просто создать элемент Проще зафиксировать параметры и проверять на null или undefined.
Универсализм плохо сочетается с эффективностью.

Aetae 03.12.2021 14:02

voraa, я это себе как-то так представляю(чисто черновой набросок):
function create ({tag, options, children, parent}) {
  console.log({tag, options, children, parent})
}


function isType(arg, types) {
  if(Array.isArray(types))
    return types.some(type => isType(arg, type));
  if(typeof types === 'function')
    return types(arg);
  return arg === types;
}
function findArgs(args, list) {
  let maxArgsMatchLength = -1;
  let maxArgsMatchNames;
  list.forEach(({
    sublist,
    sublist: { length },
    names
  }) => {
    if(length <=  maxArgsMatchLength) return; // ничего не делаем если уже нашли больше аргументов
    for(let i = length; i--;) {
      if(!isType(args[i], sublist[i])) 
        return false;
    }
    maxArgsMatchLength = length;
    maxArgsMatchNames = names;
  });

  if(maxArgsMatchLength === -1)
    throw new Error('invalid arguments');

  return Object.fromEntries(
    maxArgsMatchNames
    .slice(0, maxArgsMatchLength)
    .map((key, i) => [key, args[i]])
  )
}

function withAttrs(func, list) {
  const bigList = list.flatMap(obj => {
    const sublist = [];
    const entries = Object.entries(obj);
    const current = [];
    const names = [];

    for(let arg in obj) {
      const type = obj[arg];
      if(arg.endsWith('?')) {
        arg = arg.slice(0, -1);
        sublist.push(current.slice());
      }
      names.push(arg);
      current.push(type)
    }
    sublist.push(current);

    return sublist.map(arr => ({ sublist: arr, names }));
  })


  return function(...args) {
    return func.call(this, findArgs(args, bigList)); 
  }
}

isNode = () => true;

createWithArgs = withAttrs(create, [
  {tag: isString, options: isPlainObject, 'children?': [isNode, isArray, isString], 'parent?': isElement},
  {tag: isString, 'children?': [isNode, isArray, isString, null], 'parent?': isElement},
]);

createWithArgs('1', {a: 3}, [])


Все isX функции из lodash, но естественно возможно использование своих.
+Обязательно должно бы кэширование: чтоб одна функция проверки типа вызывалась для одного конкретного аргумента только один раз.
+Разделить проверку списка аргументов не всех сразу, а по порядку.
+Добавить поддержку spread.
+Типизировать для typescript(это будет ад, если вообще возможно).
...
=)

voraa 03.12.2021 17:08

Это конечно жесть и ад.

Я плюнул на такие вещи. Если и делаю, то для конкретной функции и в случаях, когда просто можно сдвигать параметры влево

Vlasenko Fedor 03.12.2021 20:52

Все это называется код ради кода.
Если вы это используете то у вас ошибки в архитектуре, голове ...

Aetae 04.12.2021 00:20

Vlasenko Fedor,
Цитата:

Все это называется код ради кода.
Так здесь я этим и занимаюсь. Код ради дела - он на работе и в пет-проектах.)
Цитата:

Если вы это используете
Нет конечно.)

Просто иногда пишешь функцию с чётким интерфейсом, а мыслишка в голове "а ведь было бы круто если бы можно было и так, и вот так, и ещё так и чтоб TS сам на это overload'ы вывел".)
С TS кстати это не настолько ужасно, как может показаться: он должен проверять как возможные комбинации аргументов относительно переданной функции, так и делать чёткий интерфейс для новой. Т.е. тут нельзя будет опечататься или указать что-то не то. Как вариант обёртки для внешнего интерфейса какой-нить либы - почему нет.)

Vlasenko Fedor 04.12.2021 02:22

Цитата:

Сообщение от Aetae
Просто иногда пишешь функцию с чётким интерфейсом

автоматное програмирование здесь бы запилить, но дорого будет :lol:
мне кажется так достаточно для парметров
const test = (tag, params = {}) => {
  const def = {
    a: 'text',
    b: 2
  }
  params = { ...def, ...params }
  console.log(params)
}

test('tag', { b: 5, c: 3 })

без использования TS, глянув дефолтные всегда можно понять, что передать
TS не учу, я им vbs, vb не могу простить
только подумал MS продуктами уже 12 лет как и не пользуюсь и вроде ок :)

voraa 04.12.2021 12:30

Aetae,
Для ускорения строку 21 наверно можно так

if (length != args.length) return;

Aetae 04.12.2021 15:14

voraa, не, мыж не знаем что там за проверки, может ему ок, если последние аргументы будут undefined:
{foo: isString, bar: [isArray, undefined], 'buz': [isElement, undefined]},

По идее конечно в таком случае надо юзать 'foo?' но мало ли какие там сложные и хитрые проверки будут.)

voraa 04.12.2021 15:45

По идее
('str')
и
('str', undefined, undefined)
- разные вещи.
В последнем случае количество аргументов будет 3 и проверка нормально сработает

ЗЫ Я так нарвался на scrollIntoView()
у меня там вызов получался scrollIntoView(x), а x было = undefined
И долго чесал репу, почему не работает, как надо


Часовой пояс GMT +3, время: 19:04.