Function for adding the arguments doesn't work
1
I'm having doubts about the following code.
When I use the function with arguments such as these sum_warehouses_stock("CALL", "2", "B", "10", "CALL", 1 ), it returns zero instead of adding the numbers. Have I overlooked something?
function sum_warehouses_stock(){
$total = 0;
$warehouses = func_get_args();
foreach($warehouses as $warehouse){
$warehouse = str_replace(["+","<",">","B"], "", $warehouse);
if (is_numeric($warehouse)){
$total += $warehouse;
}
}
return $total;
}
php regex wordpress
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
|
show 5 more comments
1
I'm having doubts about the following code.
When I use the function with arguments such as these sum_warehouses_stock("CALL", "2", "B", "10", "CALL", 1 ), it returns zero instead of adding the numbers. Have I overlooked something?
function sum_warehouses_stock(){
$total = 0;
$warehouses = func_get_args();
foreach($warehouses as $warehouse){
$warehouse = str_replace(["+","<",">","B"], "", $warehouse);
if (is_numeric($warehouse)){
$total += $warehouse;
}
}
return $total;
}
php regex wordpress
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
1
Var dump warehouses lets see if it is returning false or array of your args.var_dump($warehouses);
– Mohammad C
Nov 20 '18 at 2:35
2
Also worth mentioning thatfunc_get_args()is old fashioned, we have variadic functions since PHP 5.6:function sum_warehouses_stock(...$warehouses){automatically populates$warehouseswith an array of arguments.
– miken32
Nov 20 '18 at 2:41
1
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27
|
show 5 more comments
1
1
1
2
I'm having doubts about the following code.
When I use the function with arguments such as these sum_warehouses_stock("CALL", "2", "B", "10", "CALL", 1 ), it returns zero instead of adding the numbers. Have I overlooked something?
function sum_warehouses_stock(){
$total = 0;
$warehouses = func_get_args();
foreach($warehouses as $warehouse){
$warehouse = str_replace(["+","<",">","B"], "", $warehouse);
if (is_numeric($warehouse)){
$total += $warehouse;
}
}
return $total;
}
php regex wordpress
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
I'm having doubts about the following code.
When I use the function with arguments such as these sum_warehouses_stock("CALL", "2", "B", "10", "CALL", 1 ), it returns zero instead of adding the numbers. Have I overlooked something?
function sum_warehouses_stock(){
$total = 0;
$warehouses = func_get_args();
foreach($warehouses as $warehouse){
$warehouse = str_replace(["+","<",">","B"], "", $warehouse);
if (is_numeric($warehouse)){
$total += $warehouse;
}
}
return $total;
}
php regex wordpress
php regex wordpress
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
asked Nov 20 '18 at 2:19
ptrcaoptrcao
279112
279112
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
1
Var dump warehouses lets see if it is returning false or array of your args.var_dump($warehouses);
– Mohammad C
Nov 20 '18 at 2:35
2
Also worth mentioning thatfunc_get_args()is old fashioned, we have variadic functions since PHP 5.6:function sum_warehouses_stock(...$warehouses){automatically populates$warehouseswith an array of arguments.
– miken32
Nov 20 '18 at 2:41
1
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27
|
show 5 more comments
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
1
Var dump warehouses lets see if it is returning false or array of your args.var_dump($warehouses);
– Mohammad C
Nov 20 '18 at 2:35
2
Also worth mentioning thatfunc_get_args()is old fashioned, we have variadic functions since PHP 5.6:function sum_warehouses_stock(...$warehouses){automatically populates$warehouseswith an array of arguments.
– miken32
Nov 20 '18 at 2:41
1
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
1
1
Var dump warehouses lets see if it is returning false or array of your args.
var_dump($warehouses);– Mohammad C
Nov 20 '18 at 2:35
Var dump warehouses lets see if it is returning false or array of your args.
var_dump($warehouses);– Mohammad C
Nov 20 '18 at 2:35
2
2
Also worth mentioning that
func_get_args() is old fashioned, we have variadic functions since PHP 5.6: function sum_warehouses_stock(...$warehouses){ automatically populates $warehouses with an array of arguments.– miken32
Nov 20 '18 at 2:41
Also worth mentioning that
func_get_args() is old fashioned, we have variadic functions since PHP 5.6: function sum_warehouses_stock(...$warehouses){ automatically populates $warehouses with an array of arguments.– miken32
Nov 20 '18 at 2:41
1
1
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27
|
show 5 more comments
0
active
oldest
votes
Your Answer
StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
draft saved
draft discarded
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53385304%2ffunction-for-adding-the-arguments-doesnt-work%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
0
active
oldest
votes
0
active
oldest
votes
active
oldest
votes
active
oldest
votes
draft saved
draft discarded
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
draft saved
draft discarded
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53385304%2ffunction-for-adding-the-arguments-doesnt-work%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Works fine for me, I get 13 with your sample input.
– miken32
Nov 20 '18 at 2:22
Works for me too: 3v4l.org/omkfo
– Nick
Nov 20 '18 at 2:29
1
Var dump warehouses lets see if it is returning false or array of your args.
var_dump($warehouses);– Mohammad C
Nov 20 '18 at 2:35
2
Also worth mentioning that
func_get_args()is old fashioned, we have variadic functions since PHP 5.6:function sum_warehouses_stock(...$warehouses){automatically populates$warehouseswith an array of arguments.– miken32
Nov 20 '18 at 2:41
1
As long as you update your function signature, yes.
– miken32
Nov 20 '18 at 16:27