Рефакторинг кода angularjs?

Question

[Алекс]

Всем привет!

Изучаю angulars да и js вообще, назрел вопрос.
Можно как то оптимизировать код. Скажем написать функцию, которая подставляла id или что то в этом роде.
А то таких полей может быть сотня, и писать все это быдлокодом не хочется.

app.controller "addCitysPage", ($rootScope, $scope, $timeout, getDataService, setPlace ) ->

  $scope.show3 = () ->
    $scope.city3 = $rootScope.citys
  $scope.show4 = () ->
    $scope.city4  = $rootScope.citys
  $scope.show5 = () ->
    $scope.city5  = $rootScope.citys
  $scope.show6 = () ->
    $scope.city6  = $rootScope.citys

  objectResult = (e) ->
    { "id" : e.target.dataset.id,  "name" : e.target.dataset.name,  "cdr" : e.target.dataset.cdr  }

  $scope.selectionCity3 = (e) ->
    $scope.search3City           = setPlace.setCity(e.target.dataset, 3)
    $rootScope.Items.Citys.city3 =  objectResult(e)
  $scope.selectionCity4 = (e) ->
    $scope.search4City           = setPlace.setCity(e.target.dataset, 4)
    $rootScope.Items.Citys.city4 =  objectResult(e)
  $scope.selectionCity5 = (e) ->
    $scope.search5City           = setPlace.setCity(e.target.dataset, 5)
    $rootScope.Items.Citys.city5 =  objectResult(e)
  $scope.selectionCity6 = (e) ->
    $scope.search6City           = setPlace.setCity(e.target.dataset, 6)
    $rootScope.Items.Citys.city6 =  objectResult(e)

Comments

[Артур Мудрик]

Неужели кто-то еще использует говно-кофескрипт... Уродство какое-то.

Answer 1

[Philipp]

stackoverflow.com/questions/15666048/service-vs-pr...

Comments

[Алекс]

Не совсем понял, причем тут фабрики и т.д.
Что надо добавить какой то сервис, это и так понятно.
Если провести аналогию с PHP, то хотелось бы как то так:

// js $scope.city6  = $rootScope.citys
$this->scope->$show_number  = $this->rootScope->citys

[Philipp]

Алекс: это пример говнокода. Так писать нельзя, это антипаттерн.
Вам нужно добавить дополнительное измерение. Это как перейти от одномерного массива к двумерному.

[Sergej]

Philipp T: Весь symfony так реализован в bootstrap

public function __construct(array $parameters = array())
{
    $this->parameters = $parameters;
}

Почему вдруг это стало говнокодом?

[Sergej]

public function replace($parameters)
{
  return $this->$parameters[$root]
}

[Philipp]

Иван: потому что в Simfony это не говнокод и фреймворк построен для людей с пониманием дизайна приложений. Прежде чем писать на таком уровне, нужно понимать, что ты делаешь, а не просто херачить код. А понимание приходит после осознания необходимости тестирования приложений, уместности использования подходов в определенной ситуации. Можно взять к примеру Битрикс, в нем можно найти как разумные, так и откровенно говнокодные решения.