Index Being Recalculated

[marain]

I define $menuLabelIndex and then use it as an index item in the (highly excerpted) array that then follows:

$lemma = rand (0, count($quotations) -1);
$menuLabelIndex = $lemma;
unset ($lemma);

$menuItems = array(
  'Wire Fraud',
  'Case Review',
  'Super Links',

  $quotations [$menuLabelIndex],

  'Directions',
  'Parking',
  'PGP Public Key',
  'The Bottom Line', 
); 

Later in the script, I loop through the $menuItems table on two different occasions. On the second occasion, [$menuLabelIndex] is recalculated, so that $quotations [$menuLabelIndex] yields a different result from the $quotations array than from the first looping. You can see the disparate results in the left-hand and bottom menus on any page of www.MarainLaw.com. (It's the entry towards the bottom of the menue that immediately follows the "Super Links" item.)

Is there a straightforward way to keep [$menuLabelIndex] from being recalculated?

A.

[jscheuer1]

Is this the post you're talking about? If so, I would need to see the full code, or at least the fuller code. I would need enough to both run these two loops that you are talking about and in such a way that I could repeat the error/problem on a scratch page I could setup and run on wamp.

What exactly is the problem? Is it that you get a different random item each time? First time I got the same one. Second time I got two different ones.

I can tell you this, if you keep going:

$lemma = rand (0, count($quotations) -1); 
$menuLabelIndex = $lemma; 
unset ($lemma);

each time, the likelihood of getting a different result is high, because each time you do this, you potentially get a different random number.

[marain]

That IS the post in question. I can provide the full code if you like, although it exceeds 500 lines. (A fair number of those lines are comments.) I could likely pare it down, although that would take a bit of time.

You have accurately discerned the problem: What I was attempting to do was to obtain an initial random number, save it, and use the saved random number the next time I wanted the element from the table. I initially did not believe the unset command was needed. But in trying to solve the problem, even using unset did not work. Rather than "Index Being Recalculated," this thread could perhaps been more accurately entitled, "Unset Not Working".

Regardless, this bepuzzlement is now predominately of academic interest since, upon reflection, I'm not too unhappy with what the code is actually doing.

A.

Is this the post you're talking about? If so, I would need to see the full code, or at least the fuller code. I would need enough to both run these two loops that you are talking about and in such a way that I could repeat the error/problem on a scratch page I could setup and run on wamp.

What exactly is the problem? Is it that you get a different random item each time? First time I got the same one. Second time I got two different ones.

I can tell you this, if you keep going:

$lemma = rand (0, count($quotations) -1); 
$menuLabelIndex = $lemma; 
unset ($lemma);

each time, the likelihood of getting a different result is high, because each time you do this, you potentially get a different random number.

[DyDr]

The only way the index value would be recalculated during one execution/request of the script is if you are calling or including/requiring the posted section of code more than once, you have some other code that's modifying the value (perhaps a conditional test that's setting the value rather than comparing the value), you have a logic error or scope problem that's using a different value, you have code that always uses the first/last value, and a few more possibilities.

As already mentioned, it would take having enough of your code that reproduces the problem in order to specifically help you. There's no point in taking the time to paring down the code you post.

[marain]

The only way the index value would be recalculated during one execution/request of the script is if you are calling or including/requiring the posted section of code more than once, you have some other code that's modifying the value (perhaps a conditional test that's setting the value rather than comparing the value), you have a logic error or scope problem that's using a different value, you have code that always uses the first/last value, and a few more possibilities.

As already mentioned, it would take having enough of your code that reproduces the problem in order to specifically help you. There's no point in taking the time to paring down the code you post.

I do not believe any of the situations you describe are happening, but I could be wrong. (I've been maintaining this code for years, but I was not its original author.) Hesitant to post a file of that size here, I've uploaded the full code to http://www.marainlaw.com/pageContent/menuCode.txt .

A.

[DyDr]

Note, the second quotation is always the next entry in the array from the first quotation. The problem is due to this - $i++, on about line 358 in the posted code.

Almost all of this code will go away if you do two things -

1) Make a single array with the key being the current $menuLinks values and the value being the $menuItems values. You can then just use a foreach(){} loop to loop over the single array. This will make it easier to maintain the data since the key/values will be defined together.

2) You are currently building each line down the page, taking into account what may be on the left and right side of each line to control how the page is laid out. Use css to layout the page. The left-menu should be a div of content. The main-content should be a div of content. The bottom-menu should a div of content. Each section of content should be produced as an independent section.

[jscheuer1]

Note, the second quotation is always the next entry in the array from the first quotation.

No, it is not. Sometimes it is the same, sometimes it is another one not anywhere near being next in the array.

[jscheuer1]

OK, my diagnosis is that DyDr was right at first when he speculated: "you are calling or including/requiring the posted section of code more than once"

My reasoning is that I ran the code that you linked to and it only made one menu. In order for it to make two menus, it must be being used more than once, and that of course would make it be a different random number the next time.

[jscheuer1]

That's it. The code is being used twice. Once for the side menu when $menuType is 'button' and once for the bottom menu when $menuType is not 'button'. So to 'fix' it you would have to generate $menuLabelIndex on the calling (including) page or script and do it just one time per page.

Or perhaps - but this may or may not work, may or may not have unintended consequences (works here in testing though):

$lemma = rand (0, count($quotations) -1);
$menuLabelIndex = isset($menuLabelIndex)? $menuLabelIndex : $lemma;
unset ($lemma);

BTW unsetting $lemma was never going to fix this. So of course unset is working fine.

[DyDr]

Okay, so it's not always the next one now.

The supplied code cannot be producing the linked to web page. The supplied code is looping once. It is echoing a left-hand link syntax, followed by a bottom link, with | characters between items, in each pass through the loop. Even if the OP removed the end of the first loop/start of the next in the supplied code, the order of the left-hand links/bottom links in the output doesn't match the code.

In the view source of the output, the markup for the entire bottom menu, with the | separators, is first, followed by the markup for the left-hand link menu. The exact output and the order of the output cannot be different than when it was produced and sent to the browser.

I'm going to guess that the OP's real code is including this code twice, with some not-shown logic to control what it does, or there are actually two separate pieces of code, one for the left-menu and one for the bottom menu, and the random number code is producing two different numbers because it is running twice.

@OP, we cannot specifically help you with anything your code is doing when you don't supply accurate code.

Edit: I see there have been more posts while I was writing this. The $menuType value is 'button' for the links in both menus. See the class name in the view source in the links.

The only thing we can both tell is the code is being used twice and the OP is posting adulterated, time wasting, information.