Re^2: Why does my get_max_index function return zero? (High Water Mark Algorithm)

Sorry, I copy-pasted the original script's code (which didn't use warnings or strict). Please see my edited code above because my script is still returning zero, even when I run the function as its own perl script (I just run the above code in a file called test_max.pl). Also, I don't use $imax anywhere else in the original script that uses the function "get_max_index".

Comment on Re^2: Why does my get_max_index function return zero? (High Water Mark Algorithm)

Replies are listed 'Best First'.
Re^3: Why does my get_max_index function return zero? (High Water Mark Algorithm) by Paladin (Vicar) on Jun 03, 2019 at 17:53 UTC
The `$imax` inside your `for` loop is different from the `$imax` outside it (as you've declared it with `my` inside the `for` loop). Remove that `my`.	[reply] [d/l] [select]
Re^3: Why does my get_max_index function return zero? (High Water Mark Algorithm) by haukex (Archbishop) on Jun 03, 2019 at 18:04 UTC
Please see my edited code above Please have a look at How do I change/delete my post?, in particular, "It is uncool to update a node in a way that renders replies confusing or meaningless". But luckily I still had your original code sitting around: Read more... original code (399 Bytes) Other than that, Paladin has already pointed out the issue with the second `my` declaration inside the loop in your updated code. I just wanted to add that in general, one should not combine `my` with an `if` statement modifier, as its behavior is undefined (see Statement Modifiers).	[reply] [d/l] [select]