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

1. 

   mykolaim

   03.12.2018 16:39

   #19447566

   Код очень нечитабельный.
   Переменные то написаны в camelCase то в snake_case.
   Есть ещё такая штука — Exception
   Для запросов советую использовать guzzle
   
   А ну и запись данных доступа напрямую в код — это просто фейспалм.
   
   

   
   

   1. 

      vlreshet

      03.12.2018 20:19

      #19448516

      +4

      Ладно бы просто нечитабельный код. Это отличный пример того, как делать никогда не надо

      
      

      1. 

         edogs

         03.12.2018 22:10

         #19448910

         +2

         Ох отхватим мы сейчас, но честное слово.
         «If It Looks Stupid but Works It Ain't Stupid»©
         У ТС была цель — продемонстрировать работу с тиньков апи (хз в чем там была сложность, но раз автор заявляет что была — просто поверим). Цель выполнена? Да.
         Почему с него требуют еще и код по всем стандартам оформлять? Ему зачем еще и на это время тратить, если его цель не идеально оформленный код показать, а просто концепт рабочий показать?
         Реально анекдот получается
         

         Звонок в дверь, открывает мужчина.На пороге женщина спрашивает:
         — это вы вчера на озере мальчика спасли?
         — Да, я.
         — А где шапочка ????!!!

         
         

         1. 

            vlreshet

            03.12.2018 23:27

            #19449156

            -1

            А в том то и прикол, что нет там сложностей никаких, незачем какие-то «рабочие концепты» показывать. Обычный OAuth, никакой экзотики, любой адекватный разработчик прекрасно знает что это, и как с этим работать. Что имеем в итоге: человек написал о том как решить тривиальную задачу крайне хреновым методом. За что хвалить?

         
         

         1. 

            T_Sun

            04.12.2018 07:42

            #19449802

            У ТС была цель — продемонстрировать работу с тиньков апи

            
            Ну демонстрация должна быть читабильной. Сколько сотен или тысяч человек будут анализировать этот код? Если бы он использовал его сам внутри своего проекта, то критерия «работает и ладно» было бы достаточно.
            

            Цель выполнена? Да.

            
            Смотря какая цель.
            Написать работающую программу — выполнена.
            Написать статью и поделиться с сообществом — тут другие критерии.

1. 

   optimusqp Автор

   03.12.2018 16:44

   #19447596

   От банка, думаю атак не будет ) Поэтому информация записывается напрямую в базу. В статье дан минимум кода, который позволил бы реализовать функционал. За советы спасибо!

   
   

   1. 

      optimusqp Автор

      03.12.2018 19:05

      #19448258

      -9

      Мда… вот и 0 под статусом статьи… Совесть есть? Минусовать под контентом, которого нет в сети… Вот и пиши туторы. В закладки 7 человек добавили статью к себе в закладки(!)… Кто на ап статьи? Ау… В этом мире Добро приветствуется....?..
      

      
      

      1. 

         Igor_Shumilov

         03.12.2018 20:35

         #19448560

         В закладки 7 человек добавили статью к себе в закладки

         Это для «потом прочитаю».

1. 

   dmitry_dvm

   03.12.2018 21:16

   #19448708

   Ад какой-то. В пхп нет десериализации в класс? Непонятно в чем сложность была. Да, OAuth2, да, рефреш и акцессорные токены, и что?

   
   

   1. 

      MaximChistov

      04.12.2018 13:48

      #19451600

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

      Если остальной код не рефакторить, а только последний цикл

      foreach ($tinkoff_array as $tinkoff){
      		$temp_id = $pdo->query("SELECT count(*) FROM `justtin`.`tinkoff` WHERE id=".$tinkoff['id'].";")->fetchColumn(); 
      		if ($temp_id==0){
      		if (Get_highly_likely_is_number_bill($tinkoff['paymentPurpose'])!=""){
      			$stmt = $pdo->prepare("INSERT INTO `justtin`.`tinkoff` (id, date, amount, drawDate, payerName, payerInn, payerAccount, payerCorrAccount, payerBic, payerBank, chargeDate, recipient, recipientInn, recipientAccount, recipientCorrAccount, recipientBic, recipientBank, operationType, uin, paymentPurpose, creatorStatus, payerKpp, executionOrder, date_of_save) VALUES (:id, :date, :amount, :drawDate, :payerName, :payerInn, :payerAccount, :payerCorrAccount, :payerBic, :payerBank, :chargeDate, :recipient, :recipientInn, :recipientAccount, :recipientCorrAccount, :recipientBic, :recipientBank, :operationType, :uin, :paymentPurpose, :creatorStatus, :payerKpp, :executionOrder, NOW())");
          foreach($tinkoff as $key => $val) {
                $stmt->bindParam(":$key", $val);
              }
      			$stmt->execute();
      		}
      

      
      

1. 

   peresada

   04.12.2018 07:32

   #19449792

   Насколько я знаю, у Тинькофф много различных SDK и API, которые «работаю, но не поддерживаются, и вообще их трудно найти, и вообще, откуда вы это откопали», как-то прикручивал попап на получение кредита по товару, использовал SDK с нормальной документацией, обычная .js библиотечка, в объект просто передаем нужные данные и получаем готовое решение в виде кнопки и попапа.
   
   Спустя какое-то время это все перестало работать, при обращение в техподдержку ответ был как раз из серии «Не поддерживается, мы не знаем, вот другое решение», другое решение заключалось в добавлении на сайт кнопки со скрытыми инпутами. в которых внесена та же самая информация, форма просто отправляется на нужную страницу этого банка, а документация в .docx
   
   А вообще, что касается документаций и работы с авторизацией по Oauth — у Твиттера самые большие проблемы с этим, слишком мудрят они по сравнению с остальными соцсетями.

1. 

   D01

   05.12.2018 09:34

   #19455642

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

   const async = require('async');
   const request = require('request');
   const dateFormat = require('dateformat');
   const MongoClient = require('mongodb').MongoClient;
   
   const config = require('./config.js');
   const datasaver = require('./datasaver');
   
   const client_id = '';
   const client_secret = '';
   const refresh_token = ''
   const myINN = 'ИНН своей компании';
   
   const LOADER_TYPE='tinkof';
   
   const URL = 'https://openapi.tinkoff.ru/sso/secure/token';
   
   const ACCOUNT_URL = 'https://openapi.tinkoff.ru/sme/api/v1/partner/company/' + myINN + '/accounts';
   const EXCERPT_URL = 'https://openapi.tinkoff.ru/sme/api/v1/partner/company/' + myINN + '/excerpt';
   
   var startDate = new Date();
   startDate.setFullYear(2017);
   startDate.setMonth(0);
   startDate.setDate(1);
   startDate.setHours(0);
   startDate.setMinutes(0);
   startDate.setSeconds(0);
   startDate.setMilliseconds(0);
   
   var endDate = new Date();
   
   function work() {
     var access_token;
     var sessionId;
     async.waterfall([
       function auth(callback) {
         const req = {
           form: {
             grant_type: 'refresh_token',
             refresh_token: refresh_token,
             client_id: client_id,
             client_secret: client_secret,
             sessionId: 'ANTIRIUS_WEB',
           }
         };
         request.post({
           url: URL,
           formData: req.form,
           json: true,
         }, function optionalCallback(err, httpResponse, body) {
   
           if (err) {
             console.log(err);
             callback(err, {
               code: 400,
               message: 'Ошибка авторизации: ' + err.message
             });
           } else {
             access_token = body.access_token;
             sessionId = body.sessionId;
   
             callback();
           }
   
         });
       },
       function(callback) {
   
         const req = {
           form: {
             grant_type: 'refresh_token',
             refresh_token: refresh_token,
             client_id: client_id,
             client_secret: client_secret,
             sessionId: 'ANTIRIUS_WEB',
           }
         };
         request.get({
           url: ACCOUNT_URL,
           json: true,
           headers: {
             Authorization: 'Bearer ' + access_token
           }
         }, function optionalCallback(err, httpResponse, body) {
   
           if (err) {
             console.log(err);
             callback(err, {
               code: 400,
               message: 'Ошибка получения списка счетов: ' + err.message
             });
           } else {
             async.mapLimit(body, 1, function(item, callback) {
               loadDataForAccount(item, access_token, function() {
                 callback();
               });
             }, function done(err, result) {
               callback();
             });
           }
         });
       },
     ], function done(err, result) {
       if (err) console.dir(err);
       if (result) console.dir(result, {
         colors: true
       });
       console.log('DONE');
     });
   }
   
   work();
   
   function loadDataForAccount(account, access_token, callback) {
     // console.log(account.accountNumber);
     request.get({
       url: EXCERPT_URL + '?accountNumber=' + account.accountNumber + '&from=' + encodeURIComponent(dateFormat(startDate, "yyyy-mm-dd+HH:MM:ss")) + '&till=' + encodeURIComponent(dateFormat(endDate, "yyyy-mm-dd+HH:MM:ss")),
       json: true,
       headers: {
         Authorization: 'Bearer ' + access_token
       }
     }, function optionalCallback(err, httpResponse, body) {
   
       if (err) {
         console.log(err);
         callback(err);
       } else {
         save_all(account, body, callback);
       }
   
     });
   }
   
   function save_all(account, data, callback) {
     var db;
     var client;
     async.waterfall([
       function connect(callback) {
         MongoClient.connect('mongodb://' + config.db.server, function(err, _client) {
           if (err) {
             callback(err, {
               code: 500,
               message: 'Ошибка подключения к базе данных',
             });
           } else {
             client = _client;
             db = client.db(config.db.name);
             callback(null);
           }
         });
       },
       function save(callback) {
         async.mapLimit(data.operation, 1, function(item, callback) {
           prepare_save_data(db, item, function() {
             callback();
           });
         }, function done(err, result) {
           callback();
         });
       },
     ], function done(err, result) {
       callback(err, result);
       if (client) client.close();
     });
   }
   
   function prepare_save_data(db, obj, callback) {
     console.dir(obj, {
       colors: true
     });
     var data1;
     var data2;
     async.waterfall([
   
       function platel(callback) {
         save_plat_data(db, obj, function(err, result) {
           data1 = result;
           callback(null);
         });
       },
       function poluch(callback) {
         save_pol_data(db, obj, function(err, result) {
           data2 = result;
           callback(null);
         });
       },
       function save_plat(callback) {
         create_move(db, obj, data1, data2, callback);
       },
     ], function done(err, result) {
       console.dir(data1);
       console.dir(data2);
       callback(null);
     });
   }
   
   function getDate(date_string) {
     console.log(date_string);
     var arr = date_string.split('-');
     var day = parseInt(arr[2]);
     var month = parseInt(arr[1]);
     var year = parseInt(arr[0]);
   
     var sort = arr[0] + arr[1] + arr[2];
   
     return {
       y: year,
       m: month,
       d: day,
       sort: sort,
     }
   }
   
   function create_move(db, obj, payer, recipient, callback) {
     var move = {};
     move.number = obj['id'];
     move.sum = obj['amount'];
     move.date_string = obj['date'];
     move.date = getDate(move.date_string);
     move.payer = payer;
     move.recipient = recipient;
     move.purpose = obj['paymentPurpose'];
     move.raw = obj;
     move.id = move.date_string + '.' + move.number + '.' + move.sum;
     move.loader = LOADER_TYPE;
     datasaver.save_move(db, move, function(err, result) {
       console.dir(result, {
         colors: true
       });
       callback(null);
     })
   }
   
   function save_plat_data(db, obj, callback) {
   
     var obj1;
     var obj2;
     var obj3;
   
     async.waterfall([
       function save_1(callback) {
         var op = {};
         op.name = obj['payerName'];
         op.inn = obj['payerInn'];
         op.kpp = obj['payerKpp'];
         op.loader = LOADER_TYPE;
   
         if (op.name && op.name.toLowerCase().indexOf('синюков') !== -1) {
           delete op.kpp;
         }
         datasaver.save_company(db, op, function(err, result) {
           obj1 = result.object;
           callback(null);
         });
       },
       function save_bank_1(callback) {
         var op = {};
         op.name = obj['payerBank'];
         op.bik = obj['payerBic'];
         op.loader = LOADER_TYPE;
   
         datasaver.save_bank(db, op, function(err, result) {
           obj2 = result.object;
           callback(null);
         });
       },
       function _save_bank_account(callback) {
         var op = {};
         op.number = obj['payerAccount'];
         op.bank = obj2;
         op.owner = obj1.id;
         op.loader = LOADER_TYPE;
   
         datasaver.save_bank_account(db, op, function(err, result) {
           obj3 = result.object;
           callback(null);
         });
       },
     ], function done(err, result) {
       callback(null, {
         company: obj1,
         bank: obj2,
         account: obj3,
       });
     });
   }
   
   function save_pol_data(db, obj, callback) {
   
     var obj1;
     var obj2;
     var obj3;
   
     async.waterfall([
       function save_1(callback) {
         var op = {};
         op.name = obj['recipient'];
         op.inn = obj['recipientInn'];
         op.kpp = obj['recipientKpp'];
         op.loader = LOADER_TYPE;
   
         if (op.name && op.name.toLowerCase().indexOf('синюков') !== -1) {
           delete op.kpp;
         }
         datasaver.save_company(db, op, function(err, result) {
           obj1 = result.object;
           callback(null);
         });
       },
       function save_bank_1(callback) {
         var op = {};
         op.name = obj['recipientBank'];
         op.bik = obj['recipientBic'];
         op.loader = LOADER_TYPE;
   
         datasaver.save_bank(db, op, function(err, result) {
           obj2 = result.object;
           callback(null);
         });
       },
       function _save_bank_account(callback) {
         var op = {};
         op.number = obj['recipientAccount'];
         op.bank = obj2;
         op.owner = obj1.id;
         op.loader = LOADER_TYPE;
   
         datasaver.save_bank_account(db, op, function(err, result) {
           obj3 = result.object;
           callback(null);
         });
       },
     ], function done(err, result) {
       callback(null, {
         company: obj1,
         bank: obj2,
         account: obj3,
       });
     });
   }
   
   

   
   
   p/s Некоторые модули отсутствуют, но по названию понятно, что делают и все равно это придется переделывать под свой формат.