Как провести рефакторинг и ревью кода?

Владимир, на этот раз надеюсь, что сделал всё как нужно?

try {
               
                    serviceDataByOptions
                        .getPriority()
                        .then(function(data) {
                            //vm.choices = [{id:'0c4dc671-b202-44a2-bf15-d94773cc1ccd',name:'Низкий'}];
                            vm.choices = data;   
                        })
                        .then(function() {
                            serviceDataByOptions
                                .getTag()
                                .then(function(data) { 
                                    vm.choicesTag = data;
                                })
                        })
                        .then(function() {
                            serviceDataByOptions
                                .getStatus()
                                .then(function(data) { 
                                    vm.choicesStatus = data;
                                })
                        })
                        .catch(function(error) {
                            throw new Error('Произошла ошибка в serviceDataByOptions. Возникли проблемы с получением данных из БД! ' + error);
                        });

                } catch(error) {
                    vm.notify('error',error.message);
                    console.log(error);
                }

Как провести рефакторинг и ревью кода?

Владимир, вы это имели в виду (по поводу промиссов):

serviceDataByOptions
                    .getPriority()
                    .then(function(data) {
                        return vm.choices = data;   
                    })
                    .then(function() {
                        serviceDataByOptions
                            .getTag()
                            .then(function(data) { 
                                return vm.choicesTag = data;
                            })
                            .catch(function(error) {
                                console.log(error);
                            });
                    })
                    .then(function() {
                        serviceDataByOptions
                            .getStatus()
                            .then(function(data) { 
                                vm.choicesStatus = data;
                            })
                            .catch(function(error) {
                                console.log(error);
                            });
                    })
                    .catch(function(error) {
                        console.log(error);
                    });

Как провести рефакторинг и ревью кода?

>Кроме того в блоке catch вы гасите ошибки
Владимир, вы имеете ввиду выдавать ошибку при помощи throw (например так: throw new Error('description error');)

>по поводу ссылки
очень познавательная статья, понял трюк промисов, но трудновато применить к своему случаю, начал думать над этим.

>также по поводу "...небольшой кусок как бы делал я, правда без вложености" и "...$resource"
было бы здорово, если бы вы показали мне эти куски кода

Как провести рефакторинг и ревью кода?

Владимир, я благодарен вам за помощь, вы уже многое мне посоветовали. Но хотелось бы получить ещё один ваш совет - правильно ли я организовал "возврат promise" (см. чуть выше мой пример)? Предполагаю, что существуют и другие варианты?

Как провести рефакторинг и ревью кода?

По поовду "должны вовращать promise":

Таким вот образом организовать?

serviceDataByOptions
    .getPriority()
    .then(function(data) {                                                
        //vm.choices = [{id:'0c4dc671-b202-44a2-bf15-d94773cc1ccd',name:'Низкий'}];
        vm.choices = data;
        vm.priorityNew = '';

            serviceDataByOptions
                .getTag()
                .then(function(data) { 
                    vm.choicesTag = data;
                    vm.selectModelNew = {
                        tag: []
                    };

                        serviceDataByOptions
                            .getStatus()
                            .then(function(data) { 
                                vm.choicesStatus = data;
                                vm.statusNew = '';
                            })
                            .catch(function(error) {
                                console.log(error);
                            });

                })
                .catch(function(error) {
                    console.log(error);
                });

    })
    .catch(function(error) {
        console.log(error);
    });

Как провести рефакторинг и ревью кода?

Спасибо.

>По поводу "2) использовать es6":
вы имеете ввиду использовать anguler2 вместо angularJS?

>По 1) код разбить по модулям :
можете не много по подробней по этому пункту. в частности - какие части вынести в модули?

А вообще как хорошо по стилю, терпимо?

Как провести рефакторинг и ревью кода?

спасибо. сделал

Как передавать параметры из сервиса в контроллер?

понял вас, внёс коррективы

Как передавать параметры из сервиса в контроллер?

return response.data; - не работает, в контроллере получаю undefined

return начинает работать, только если выношу его из $http.get():

.....
get: function (callback) {
$http.get('getData.php')
.then(function(response) {
// callback(response.data);
},
function(error) {
console.log(error);
});

return 'success';
}
.....

Как с помощью php-curl создать пустой файл на ФТП?

>эмм... зачем это делать курлом, если у пыха есть и родное расширение для работы с фтп, и куча клиентов, написанных на самом пыхе?
Я в курсе. Потому что есть определенные причины и требования, но это другой вопрос. Мне нужна реализация на curl.
Как вариант можно загрузить пустой файл (имя файла задаёт пользователь) с помощью php-curl на фтп, но предполагаю, что так не правильно делать

Существуют ли решения (и какие) для реализации фонового исполнения задач?

я это понимаю, но было решено начать реализацию с этого варианта, т.к. ещё не приходилось работать с многопоточностью в javascript, а также было запланировано в дальнейшем разработанную реализацию запустить в последствии в фоновом режиме (в итоге так и получилось, что сейчас этот код работает в фоновом режиме)

Существуют ли решения (и какие) для реализации фонового исполнения задач?

Вчера задал вопрос и нагуглил про web workers. Начал реализовывать задуманное, всё получается, работает как задумывалось. В общем это именно то что мне нужно.