@nodejs_ru - страница 1471 - Telegram web archive

Михаил Макарычев

нужно чтоб return Promise заработал только после чего возвращается результат от create

Сделайте в самом начале функции return new Promise((res, rej) =>), а после create сделайте res(hook)

сomorsiс

А почему просто не вернуть тот промис?

Valeriu

вот весь код

Valeriu

А почему просто не вернуть тот промис?

вернуть его из then result?

Anonymous

Так, еще один вопрос. В ассинхронной функции возвращаю true, после вызываю эту функцию func().then((r)=>console.log(r)) Но r - undefined

Покеж func()

Willy

Покеж func()

https://gist.github.com/DuckerMan/997e405dfd5dd943f49488de733b94c5

сomorsiс

вернуть его из then result?

Написать внутри then return

Sergey

Так, еще один вопрос. В ассинхронной функции возвращаю true, после вызываю эту функцию func().then((r)=>console.log(r)) Но r - undefined

покажи

Willy

покажи

Функцию? Она выше

Sergey

https://gist.github.com/DuckerMan/997e405dfd5dd943f49488de733b94c5

че у тебя с отступами и длиной строки?

Михаил Макарычев

вернуть его из then result?

Я же ответил на вопрос

Sergey

установи линтер

Sergey

лол

Valeriu

Написать внутри then return

ok, спасибо за ответ, щас попробую

Willy

установи линтер

Это плагин?

Sergey

Это плагин?

eslint.org

Willy

eslint.org

Я быстро написал этот код, потом табы поставлю

Sergey

табы не нужны

Eldar

Табы не нужны

Sergey

Я быстро написал этот код, потом табы поставлю

когда я быстро пишу код у меня нет такого пиздеца как ты умудрился

Eldar

когда я быстро пишу код у меня нет такого пиздеца как ты умудрился

Шожтыхвастаешься то, а?

Eldar

Деревянная еловая сова

Anonymous

>SELECT * FROM messages WHERE fromU = '"+login+"' AND toU = '"+loginMy+"' UNION SELECT Храни тебя господь

Willy

когда я быстро пишу код у меня нет такого пиздеца как ты умудрился

Да, кстати, заметил что сначала выводит Promise pending, а через секунды 3 уже готов ответ

Valeriu

Я же ответил на вопрос

где?

Willy

>SELECT * FROM messages WHERE fromU = '"+login+"' AND toU = '"+loginMy+"' UNION SELECT Храни тебя господь

Что не так?

Sergey

Да, кстати, заметил что сначала выводит Promise pending, а через секунды 3 уже готов ответ

ты скинул не валидный код

Михаил Макарычев

Сделайте в самом начале функции return new Promise((res, rej) =>), а после create сделайте res(hook)

Михаил Макарычев

@vodnicear

Valeriu

аа

Valeriu

да

Valeriu

щас попробую обе варианты

Valeriu

спасибо за ответ

Sergey

async.forEachOf не нужен

Willy

async.forEachOf не нужен

обычный for юзать?

Sergey

есть for (;;){ await }

Anonymous

Неужели там нет sql с параметрами?

Sergey

а как вам это?)

Anonymous

Мои глаза

Sergey

Eldar

пжлст,прекрати

arts

Да там и цикл не нужен. Можно сразу сделать SELECT fromU AS from, toU AS to ...

Sergey

https://gist.github.com/sergeysova/cd6f988d62af491d0ce6949fa4b562f4

Sergey

чтобы было хоть немного проще читать этот ад

Willy

Да там и цикл не нужен. Можно сразу сделать SELECT fromU AS from, toU AS to ...

я пытался, но не хочет.

Anonymous

Заведи себе кота-ревьюера https://pbs.twimg.com/media/DQIAbIsX4AA7ySh.jpg

Sergey

Sergey возьми курс по ноджс и установи линтер

Sergey

нихера не понятно че ты хотел сделать

Willy

Sergey возьми курс по ноджс и установи линтер

Я брал уже курс. Лучше укажи на то, что исправить

Sergey

SELECT * FROM messages WHERE fromU = '${login}' AND toU = '${loginMy}' UNION SELECT * FROM messages WHERE fromU = '${loginMy}' AND toU = '${login}' ORDER BY date DESC О_о писать такие запросы руками… пиздец

Sergey

зачем?

Sergey

зачем тянуть эту либу?

Willy

зачем?

Она ассинхронно делает форыч

Sergey

зачем?

Sergey

какой смысл

Willy

зачем?

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

Willy

SELECT * FROM messages WHERE fromU = '${login}' AND toU = '${loginMy}' UNION SELECT * FROM messages WHERE fromU = '${loginMy}' AND toU = '${login}' ORDER BY date DESC О_о писать такие запросы руками… пиздец

Кстати, как штуки ${loginMy} называются?

Sergey

интерполяция

Sergey

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

чувак ты решаешь несуществующую проблему

arts

Кстати, как штуки ${loginMy} называются?

template strings А за вкидывание строк напрямую в sql ещё в 2005-м по рукам били.

Nikolay

Кстати, как штуки ${loginMy} называются?

Template string literals

сomorsiс

Template strings

Sergey

https://gist.github.com/sergeysova/cd6f988d62af491d0ce6949fa4b562f4

Sergey

всё

Sergey

~10 долбанных строк

Sergey

нахуя усложнять

arts

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

Для этого в SQL есть LIMIT

Nikolay

> SELECT * FROM messages WHERE fromU = '${login}' AND toU = '${loginMy}' UNION SELECT * FROM messages WHERE fromU = '${loginMy}' AND toU = '${login}' ORDER BY date DESC а че за сайт? дай адресок :D

Willy

template strings А за вкидывание строк напрямую в sql ещё в 2005-м по рукам били.

их обрабатывать надо, я знаю, я так делаю в функции в которой вызываю эту.

Sergey

ну и нахер возврат ошибки

Sergey

throw new TypeError

Sergey

https://gist.github.com/sergeysova/cd6f988d62af491d0ce6949fa4b562f4

Алексей

Действительно, дай человеку адрес сайта, где этот код будет крутиться. Он тебе поможет понять почему не надо вставлять параметры в запросы.

Sergey

пиздец

Sergey

fromU и toU

Sergey

это когда человеку лень писать User

Willy

fromU и toU

еще есть боль для глаз?

Sergey

но у сообщения нет from и to поэтому можно было просто писать from to чтобы потом не делать map

Sergey

еще есть боль для глаз?

https://gist.github.com/sergeysova/cd6f988d62af491d0ce6949fa4b562f4 Ну и исправить запрос

Sergey

выкинуть raw запросы и взять orm