Question

角度相当新，所以我觉得我错过了一些简单的东西。我有一个功能，我需要从视图和控制器内调用。从视图中我给出了继续或重新开始的选项，所以我有两个函数声明如下。视图中的两个按钮都可以正常工作。

$scope.start_new = function(){
//logic in here
}

$scope.continue_work = function(){
//logic in here
}

我的问题是，如果这是第一次通过我不想提供选项，所以我有

//if the variable doesn't exist run the init
if(angular.isUndefined($localStorage.testVar)){
    //I've tried these ways
    //start_new();
    //$scope.start_new;
    //$scope.start_new();
}
else{
    $scope.active = false; //disply the layer which gives the choice
}

这些都没有解雇这个功能。如果我这样声明：

function start_new(){
//logic in here
}

它在控制器负载上运行正常，但视图中的按钮不再有效。按钮代码如下所示：

<button type="button" ng-click="start_new()" class="btn btn-info btn-lg btn200">Start New</button>

非常感谢任何帮助!!

这是实际代码：

stControllers.controller('WorkoutCtrl',['$scope','$routeParams','$localStorage','$filter','$location','Flash',function ($scope,$routeParams,$localStorage,$filter,$location,Flash) {

//set up counter background
$scope.strokeWidth = 20;
$scope.stroke = '#71d2f3';
$scope.background = '#eee';
$scope.size = 205;

$scope.show_list = false;
$scope.active = true;
if(angular.isUndefined($localStorage.currentWorkout)){
    //set up current workout

    $scope.start_workout();
}
else{
    $scope.active = false;

}


$scope.start_workout = function(){
    $scope.active = true;
    $scope.currentWorkout = $localStorage.userProgramTemplate[$routeParams['day_id']];
    $scope.setCount = $localStorage.setCounterTotal[$routeParams['day_id']];
    $localStorage.currentWorkout = $localStorage.userProgramTemplate[$routeParams['day_id']];

    //set up the workout detail
    $scope.workoutName = $localStorage.userProgramDay[$routeParams['day_id']]['program_day_name'];
    $scope.workoutDate = new Date();

    //workout progress circle
    $scope.currentSet = 1

    $scope.progress = $scope.currentSet/$scope.setCount;

    //start at first group
    $scope.workoutIndex = 0;
    $scope.groupCounter = 0;

    //start at first exercise
    $scope.exerciseIndex = 0;
    $scope.currentExerciseSet = 1;


    $scope.exerciseCounter = {};
    $scope.currentExercise = $scope.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex];
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id] = {};
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] = 1;
    $scope.currentExerciseSet = $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id];

    $scope.setProgress = ($scope.currentExerciseSet/$scope.currentExercise.sets) * 100;

    //reps
    $scope.reps = parseInt($scope.currentExercise.reps);
    //weight
    $scope.weight = parseInt($scope.currentExercise.weight);

    //create a storage variable to store counters in
    $localStorage.counters = {};
    $localStorage.counters['setCount'] = $scope.setCount;
    $localStorage.counters['workoutDate'] = $scope.workoutDate;
    $localStorage.counters['workoutName'] = $scope.workoutName;
    $localStorage.counters['exerciseIndex'] = $scope.exerciseIndex
$localStorage.counters['currentSet'] = $scope.currentSet;
$localStorage.counters['groupCounter'] = $scope.groupCounter;
$localStorage.counters['workoutIndex'] = $scope.workoutIndex;
$localStorage.counters['exerciseCounter'] = $scope.exerciseCounter;

list();

}

$scope.continue_workout = function(){
    $scope.active = true;

$scope.currentWorkout = $localStorage.currentWorkout;
    //set these values  
    $scope.setCount = $localStorage.counters['setCount'];
$scope.workoutDate = $localStorage.counters['workoutDate'];
    $scope.workoutName = $localStorage.counters['workoutName'];
$scope.exerciseIndex = $localStorage.counters['exerciseIndex'];//storage
$scope.currentSet = $localStorage.counters['currentSet'];//storage
$scope.groupCounter = $localStorage.counters['groupCounter'];//storage
$scope.workoutIndex = $localStorage.counters['workoutIndex'];//storage
$scope.exerciseCounter = $localStorage.counters['exerciseCounter'];//storage




$scope.currentExercise = $scope.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex];


if(angular.isUndefined($scope.exerciseCounter[$scope.currentExercise.user_workout_group_id])){
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id] = {};
}

if(angular.isUndefined($scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id])){
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] = 0;
}


$scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] = $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id];// + 1;

$scope.currentExerciseSet = $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id];


//increment the progress bars
$scope.setProgress = ($scope.currentExerciseSet/$scope.currentExercise.sets) * 100;
$scope.progress = $scope.currentSet/$scope.setCount;


    //set the projected reps and weight     
$scope.reps = parseInt($scope.currentExercise.reps);
    $scope.weight = parseInt($scope.currentExercise.weight);

list();

}

$scope.next = function(){


//record the prvious info
$localStorage.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex]['reps'] = $scope.reps;
$localStorage.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex]['weight'] = $scope.weight;
$localStorage.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex]['date'] = $filter('date')(new Date(), "yyyy-MM-dd HH:mm:ss");
$localStorage.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex]['set'] = $scope.currentExerciseSet;

    //increment the counters
$scope.exerciseIndex++;
$scope.currentSet++;
$scope.groupCounter++;

    //check for end of set
if($scope.currentWorkout[$scope.workoutIndex].length <= $scope.groupCounter){
    $scope.groupCounter = 0;
    $scope.exerciseIndex = 0;
        $scope.currentExerciseSet = 1;
    $scope.workoutIndex++;

    //check if it's the end of the workout
    if($scope.currentWorkout.length <= $scope.workoutIndex){
        var message = '<strong> Workour complete</strong>';
        Flash.create('success', message, 'custom-class');

        if(angular.isUndefined($localStorage.history)){
            $localStorage.history = [];
        }
        $localStorage.history.push($scope.currentWorkout);
        delete $scope.currentWorkout;
        delete $localStorage.currentWorkout;
        delete $localStorage.counters;
        //move workout into history and unset current workout variable

        $location.path('/home');    

    }
}

$scope.currentExercise = $scope.currentWorkout[$scope.workoutIndex][$scope.exerciseIndex];


if(angular.isUndefined($scope.exerciseCounter[$scope.currentExercise.user_workout_group_id])){
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id] = {};
}

if(angular.isUndefined($scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id])){
    $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] = 0;
}

$scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] = $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id] + 1;


//set up exercise progess   
$scope.currentExerciseSet = $scope.exerciseCounter[$scope.currentExercise.user_workout_group_id][$scope.currentExercise.exercise_id];

//increment the progress bars
$scope.setProgress = ($scope.currentExerciseSet/$scope.currentExercise.sets) * 100;
$scope.progress = $scope.currentSet/$scope.setCount;

    //set the projected reps and weight     
$scope.reps = parseInt($scope.currentExercise.reps);
    $scope.weight = parseInt($scope.currentExercise.weight);

    //set up some variable in local storage to use if we resume a workout
    $localStorage.counters['exerciseIndex'] = $scope.exerciseIndex;;
$localStorage.counters['currentSet'] = $scope.currentSet;
$localStorage.counters['groupCounter'] = $scope.groupCounter;
$localStorage.counters['workoutIndex'] = $scope.workoutIndex;
$localStorage.counters['exerciseCounter'] = $scope.exerciseCounter;

}

function list(){
$scope.workout_list = $localStorage.userProgramDay[$localStorage.counters['workoutIndex']];

//console.log($scope.workout_list)

}

}]）;

Answer 1

$scope.start_new = function(){
//logic in here
}

然后

//if the variable doesn't exist run the init
if(angular.isUndefined($localStorage.testVar)){
    $scope.start_new();
}
else {
    $scope.active = false; //display the layer which gives the choice
}

是对的。我无法确切地看到您的代码中发生了什么，但我怀疑您正在调用该函数 - 您只是遇到某种竞争条件导致它无法正常工作。

尝试在您的函数中放置一个console.log()，看看它是否会在您预期被击中时被击中。

修改

我很确定这种情况正在发生，因为您在定义之前使用start_workout()（第14行）（第22行）。尝试移动//set counter background上方的函数声明，原始建议的解决方案应该有效。

通常，JS以自上而下的方式评估语句;有一个称为变量提升的概念，可以立即评估直接function a(){}次调用 - 有效地将它们提升到代码块的顶部。这就是当你将它声明为独立功能而不是将其附加到$scope时，该功能正常工作的原因。

您可以在此SO答案中找到更多信息：https://stackoverflow.com/a/261682/2457037

从视图和控制器调用函数

1 个答案: