Let's start from the issue in your code. You generate the request string through a function, which is actually called only inside the update function
$scope.update = function(data) {
$scope.str = generateQueryParameters($scope.str,
"no", $scope.no);
console.log($scope.str);
};
$scope.check = function() {
$scope.no = 2;
console.log($scope.str);
};
You change the version number in the check function, but without calling again the function in order to change the $scope.str, so the value of $scope.str is still the same.
You can easily test this doing the following steps in your snippet:
Click on update (v0)
Click on check (v0)
But then click again on update and you will see that now it is updated (v2)
So in step 2 you actually change the version, you simply don't generate again the str to be used.
So easy fix for your code is simply to arrange your code in order to call the function for assigning the new str, every time you change your version:
$scope.update = function(data) {
$scope.no = 1;
$scope.str = generateQueryParameters($scope.str,
"no", $scope.no);
console.log($scope.str);
};
$scope.check = function() {
$scope.no = 2;
// here you need to call again your function for composing your URL and changing the value of str
$scope.str = generateQueryParameters($scope.str,
"no", $scope.no);
console.log($scope.str);
};
function generateStringRequest() {
}
Otherwise, you need to watch on the no version parameter, and automatically refresh the str value every time the version changes. But this solution implies that you will have a watcher in every single controller where you call the APIs, and also all the parameters always hardcoded inside the controllers:
$scope.$watch('no', function() {
$scope.str = generateQueryParameters($scope.str, 'no', $scope.no);
});
Even if this solution works, actually it sucks.
The logic for managing calls is inside the controllers, which is a really bad practice (you should think to use a centralized service for that).
So much better, in AngularJS you can use a custom interceptor and manage there all the actions to be performed regarding HTTP requests.
So you can specify as a parameter of the HTTP request, the version of the API you want to use.
This will keep your code clean. Moreover, if you want to change in the future some request, you can simply change that request parameter.
If you want to change all the requests, inside the interceptor you can simply set that all the requests to version 1 will be replaced by version 2.
Here a sample code on how to define the interceptor:
angular.module('myApp').factory('MyHTTPFactory', MyHTTPFactory)
.config(function($httpProvider) {
// register the new interceptor in AngularJS
$httpProvider.interceptors.push('MyHTTPFactory');
});
MyHTTPFactory.$inject = [];
function MyHTTPFactory() {
// this is the base URL of your rest requests, so this interceptor will be applied only to the requests directed to your service
const MY_REST_URL = 'blablabla';
// methods exposed by the service
let factory = {
request: request,
requestError: requestError,
response: response,
responseError: responseError
};
return factory;
// ============================================================
function request(config) {
if (config.url.includes(MY_REST_URL)) {
let versionToUse = config.version;
// here use a function for elaborating your query string, directly in the interecptor code
config.url = elaborateQueryString();
}
return config;
}
function requestError(config) {
return config;
}
function response(res) {
return res;
}
function responseError(res) {
return res;
}
// function for getting the query string you want to
function elaborateQueryString() {
// elaborate your requests
}
}
Then simply perform as always the HTTP requests through $http adding the version you want to use inside the request as a parameter:
// perform your request as usual simply specifying the new version parameter
let request = {
url: `myserviceurl`,
version: '2' // or whatever you qNR
};
$http(request);
So the interceptor will "sniff" all your requests before to be performed, it will correctly compose the version and the query string as you want to, and all your operations are managed within it and centralized.
Just as the last tip, when defining constants like version numbers, the endpoint of the rest service, whatever, use AngularJS constant and inject them where you need to use them. Hard-coded strings are not a good practice.
