NodeJS i Crypto - zapisywanie do zmiennej

Question

W jaki sposób mogę zapisać wynik szyfrowania do zmiennej. Zmienna fileName zawiera undefined.

crypto.randomBytes(16, (err, buf) => {
          if (err) return reject(err);
          const fileName = buf.toString('hex') + path.extname(req.file.originalname);
        });

console.log(fileName);

Próbowałem też w ten sposób:

const fileName = crypto.randomBytes(16, (err, buf) => {
          if (err) return reject(err);
          return buf.toString('hex') + path.extname(req.file.originalname);
        });

console.log(fileName);

 

Answer 1

Użycie callbacka sprawia, że metoda crypto.randomBytes działa asynchronicznie. W związku z tym, nie możesz zwrócić z niej wartości (drugi przykład/kod). W pierwszym przypadku tworzysz zmienną, która ma zasięg w obrębie callbacka, stąd nie widać jej na zewnątrz.

Przy podejściu asynchronicznym możesz wspomóc się Promisem:

new Promise((resolve, reject) => {
    crypto.randomBytes(16, (err, buf) => {
        if (err) reject(err);
        resolve(buf.toString('hex') + path.extname(req.file.originalname));
    })
}).then(fileName => console.log(fileName));

Comments

Czy jest jakiś sposób, żeby użyć zmiennej fileName w dalszej części kodu, np. gdy tworzę nowy model produktu, który będę wysyłał do bazy danych? Teoretycznie mógłbym wstawić cały kod w segment .then(), ale nie wiem czy jest to dobra praktyka.

---

Albo, jak wspomniał @Ehlert, użyj async/await i przypisz wartość z rozwiązanego promisa do zmiennej:

// cały kod musi być umieszczony w funkcji z przedrostkiem async, inaczej pojawi się SyntaxError
const fileName = await new Promise((resolve, reject) => {
    crypto.randomBytes(16, (err, buf) => {
        if (err) reject(err);
        resolve(buf.toString('hex') + path.extname(req.file.originalname));
    })
});

, albo stwórz zmienną przed użyciem metody crypto.randomBytes i w jej callbacku przypisz do zmiennej to, co potrzebujesz. Pamiętaj jednak, że przypisanie do zmiennej wartości z callbacka odbędzie się asynchronicznie, czyli poza ciałem callbacka wartość zmiennej będzie undefined.

Jednak jeśli już zdecydujesz się na użycie Promisa, to możesz w then wykonać dalszy kod (np. wspomniane tworzenie modelu produktu):

function getRandomBytes() {
  return new Promise((resolve, reject) => {
      crypto.randomBytes(16, (err, buf) => {
          if (err) reject(err);
          resolve(buf.toString('hex') + path.extname(req.file.originalname));
      })
  });
}

function createNewProductModel(fileName) {
    // ...
}

getRandomBytes().then(fileName => {
    console.log(fileName);
    createNewProductModel(fileName);
});

Pamiętaj też, żeby obsłużyć ewentualne odrzucenie promisa z funkcji reject.

---

Wielkie dzięki

Answer 2

Nie możesz robić loga w taki sposób bo to wykonuje się asynchronicznie. Wrzuć go na koniec callbacka.

Zawsze możesz poszukać czy jest gdzieś w wersja tej funkcji z promisem i użyć await.