Generalisieren von zwei Funktionen, die auf zwei Eigenschaften invers wirken

Question

Ich habe diese beiden Funktionen, und ich denke, dass es eine elegante, funktionale Möglichkeit geben muss, sie zu verallgemeinern.

$scope.togglePick = function(){ 
    let targetPhoto = $scope.photoImport.data[indexService.current]; 
    targetPhoto.pick = !targetPhoto.pick; 
    if(targetPhoto.reject && targetPhoto.pick) { 
     targetPhoto.reject = false; 
    } 
} 

$scope.toggleReject = function() { 
    let targetPhoto = $scope.photoImport.data[indexService.current]; 
    targetPhoto.reject = !targetPhoto.reject; 
    if (targetPhoto.reject && targetPhoto.pick) { 
     targetPhoto.pick = false; 
    } 
} 

Ich glaube, ich in string params togglePick = toggle('pick', 'reject') passieren könnte und [] Notation in der toggle Funktion verwenden, aber das fühlt sich an wie Magie Saiten ... obwohl sie Eigenschaftsnamen sind, bin ich unsicher. Ich könnte auch haben:

function toggle(magicFlag){ 
    let primary = magicFlag ? 'pick' : 'reject'; 
    let secondary = magicFlag ? 'reject' : 'pick'; 
    ... 
    targetPhoto[primary] = !targetPhoto[primary]; 
    ... 
} 

Wieder, ick, magische Flagge. Gibt es einen eleganteren Weg, dies als Longhand in jeder Funktion zu tun?

Answer 1

Die Menge an Code in diesen Funktionen und die geringfügigen Unterschiede machen sie in Ordnung, so wie sie sind. Ohne die Eigenschaft String Verrücktheit zu arbeiten, sehe ich nicht, wie Sie wirklich diese generische machen, während lesbar bleiben. Manchmal ist es in Ordnung, nicht den allgemeinsten Code zu haben, den Sie möglicherweise haben können. Diese Linie im Sand muss irgendwo gezeichnet werden.