Не работает подготовленный запрос?

Question

[Ivan Ivanovich]

Здравствуйте, подскажите пожалуйста, почему не работает подготовленный запрос

const writeData = await getDataDB('INSERT INTO test (ownId, email, ip, category, productId) VALUES (?, ?, ?, ?, ?)', ["1212", "[e-mail]", "12.12.12", "test", "1"]);

Ошибка: UnhandledPromiseRejectionWarning: Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '?, ?, ?, ?)' at line 1

Но при этом если написать значения заранее

const writeData = await getDataDB('INSERT INTO test (ownId, email, ip, category, productId) VALUES ("12324", "[e-mail]", "12.12", "test", "11")');

Все нормально работает.

P.S. Функция

const getDataDB = (q, p = '')=>{
  return new Promise((resolve,reject)=>{
    pool.query(q, [p], (err,result)=>{
      if(err) reject(err);

      resolve(result);
    });
  });
};

Также если я например делаю так:

const writeData = await getDataDB(`INSERT INTO test (ownId, email, ip, category, productId) VALUES ("123245", ${email}, "12.12", "test", "11")`);

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '@ss.ss, "12.12", "test", "11")' at line 1
почта была в данном случае [e-mail]

Все равно ошибка

Подскажите пожалуйста, почему не работает вариант с плейсхолдерами? Я же вроде-бы правильно все делаю, в массив передаю значения, все дела

Answer 1

[Rsa97]

Судя по всему, тот, кто писал функцию getDataDB не предусмотрел варианта с плейсхолдерами.
Смотрите документацию на эту функцию или её код.

Comments

[Ivan Ivanovich]

Rsa97, доброе утро, вот функция

const getDataDB = (q, p = '')=>{
  return new Promise((resolve,reject)=>{
    pool.query(q, [p], (err,result)=>{
      if(err) reject(err);

      resolve(result);
    });
  });
};

[Rsa97]

Ivan Ivanovich, Вы всегда передаёте вторым параметром в query массив из одного элемента, находящегося в переменной p. Даже если на входе массив, вы зачем-то делаете его первым элементом нового массива.
В результате, первый плейсхолдер заменяется на значение этого элемента, а все последующие остаются знаками вопроса.

[Ivan Ivanovich]

Rsa97, понял, сейчас что-нибудь придумаю

[Ivan Ivanovich]

Rsa97, сделал пока костыль

const getDataDB = (q, p)=>{
  if(Array.isArray(p)){
    console.log('is')
  } else {
    console.log('no')
  }
  return new Promise((resolve,reject)=>{
    if(Array.isArray(p)){
      pool.query(q, [p], (err,result)=>{
        if(err) reject(err);
  
        resolve(result);
      });
    } else {
      pool.query(q, (err,result)=>{
        if(err) reject(err);
  
        resolve(result);
      });
    }
  });
};

но все равно ошибка осталась

[Rsa97]

Ivan Ivanovich,

const getDataDB = (query, params = []) =>
  new Promise((resolve, reject) => {
    pool.query(query, params, (err, result) => {
      if (err) {
        reject(err)
      }
      resolve(result)
    })
  })

[Ivan Ivanovich]

Rsa97, огромное спасибо! Все заработало

[Виталий]

Ivan Ivanovich, кстати нейминг функции некорректен, ты можешь через квери и инсерты и делеты и что угодно сделать

[Ivan Ivanovich]

Виталий, Согласен