Комментарии (12)

1. 

   megahertz

   11.11.2019 03:47

   #20869960

   Для TypeScript полезно правило линтера promise-function-async. Инстинктивно стараюсь избегать вызова асинхронных функций из синхронных, в идеале then/catch остаются только в единственной точке входа.

1. 

   BerkutEagle

   11.11.2019 04:28

   #20869994

   +1

   Надо просто понимать, во что развернётся сахар async'а.
   Перепишите вашу функцию так:
   

   function fn(obj) {
     return new Promise((resolve, reject) => {
       try {
         const someProp = obj.someProp;
         resolve(someProp);
       } catch (e) {
         reject(e);
       }
     })
   }
   

   
   

   1. 

      mayorovp

      11.11.2019 11:02

      #20870752

      Это как раз пример того как делать НЕ надо. Во-первых, конструктор Promise сам поймает ошибку, try-catch тут полностью лишний. Ну или же лишним можно считать new Promise, поскольку try-catch тут и сам справится.

      
      

      Во-вторых, написать async гораздо проще.

   
   

   1. 

      vintage

      11.11.2019 13:50

      #20871702

      Он во что-то такое развернётся:

      
      

      function fn(obj) {
        return new Promise( resolve => {
          const someProp = obj.someProp;
          resolve(someProp);
        } )
      }

1. 

   BerkutEagle

   11.11.2019 05:23

   #20870020

   +1

   Перевод, это конечно круто. Но перевод ради перевода? Если хотели сделать полезное, то перевели бы и единственный комментарий к оригиналу, объясняющий, почему автор неправ.

1. 

   amakhrov

   11.11.2019 09:10

   #20870284

   +4

   Иногда встречаются рекомендации начинать цепочку промисов с Promise.resolve() — как раз чтобы избежать описанной проблемы.
   То есть вместо

   
   

   userModel.getUserById(req.params.id)
       .then(user => res.json(user))
       .catch()

   
   

   пишем

   
   

   Promise.resolve()
       .then(() => userModel.getUserById(req.params.id))
       .then(user => res.json(user))
       .catch()

   
   

   Минус — лишний код. Плюс — работает даже в случае, когда userModel.getUserById пришла из сторонней библиотеки, и мы не можем явно пометить ее как async

1. 

   justboris

   11.11.2019 12:28

   #20871218

   Выше уже заметили, что нужно помещать код внутрь new Promise((resolve, reject) => {...}), чтобы избежать такой проблемы (в случае если async-await по какой-то причине использовать нельзя, например, IE11).

   
   

   В общем-то именно ради возможности словить ошибку и появилось такое API c замыканием вместо классического Defferred.resolve/reject из jQuery и других ранних реализаций промисов. Вот тут можно почитать подробнее: https://stackoverflow.com/questions/28687566/difference-between-defer-promise-and-promise

   
   

   1. 

      RifleR

      12.11.2019 14:23

      #20876930

      в случае если async-await по какой-то причине использовать нельзя, например, IE11

      ну в IE11 и Promise нативно не поддерживается, так что реальных причин не использовать везде async-await не существует.

      
      

      1. 

         justboris

         12.11.2019 15:12

         #20877190

         Полифилл promise подключается один раз и весит 1Kb или около того, а транспиляция раздувает исходный код. Было

         
         

         async function loadJSON() {
           const response = await fetch('/url');
           const json = await response.json();
           return json;
         }

         
         

         стало

         
         

         function loadJSON() {
           var response, json;
           return regeneratorRuntime.async(function loadJSON$(_context) {
             while (1) {
               switch (_context.prev = _context.next) {
                 case 0:
                   _context.next = 2;
                   return regeneratorRuntime.awrap(fetch('/url'));
         
                 case 2:
                   response = _context.sent;
                   _context.next = 5;
                   return regeneratorRuntime.awrap(response.json());
         
                 case 5:
                   json = _context.sent;
                   return _context.abrupt("return", json);
         
                 case 7:
                 case "end":
                   return _context.stop();
               }
             }
           });
         }

         
         

         Поэтому я могу вполне себе представить вариант, где Promise доступен, а async/await – нет.

         
         

         1. 

            t_kanstantsin

            12.11.2019 15:21

            #20877226

            Не могу утверждть, работает ли это так как ожидается, но есть плагин для babel, который преобразует async-await в Promise.
            Т.е. решая проблему "в лоб" можно глобально подключить полифил + плагин для async->Promise

            
            

            1. 

               justboris

               12.11.2019 20:19

               #20878770

               Интересно, хороший плагин, очень жаль, что стандартный @babel/preset-env, производит намного больше кода.

            
            

            1. 

               mayorovp

               13.11.2019 06:48

               #20879920

               Судя по примеру по ссылке, этот плагин ломает код. При преобразовании URL.createObjectURL(blob) в .then(URL.createObjectURL) у метода createObjectURL теряется контест (this). И если конкретно для этого метода контекст не требуется, для других методов это не так.