StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

POPHP Code efficiency in menu
primarykey
Id
5127947
data
AcceptedAnswerId
5128125
AnswerCount
3
ClosedDate
CommentCount
2
CommunityOwnedDate
CreationDate
2011-02-26T16:01:16.323
FavoriteCount
0
LastActivityDate
2011-02-26T16:50:23.173
LastEditDate
LastEditorUserId
0
OwnerUserId
635529
ParentId
0
PostTypeId
1
Score
0
ViewCount
208
LastEditorDisplayName
text
Body
<p>I currently have some VERY long winded code for the menu I use on my site. <a href="http://badnutbeats.co.uk/inotherwords" rel="nofollow">My website</a> It's almost 2000 lines long lol. I think I may be able to use a switch but I've tried and cannot implement it to work properly. </p> <p>On each menu when you click a button it stays highlighted, telling the user that they are on that page. The only way I could get this to work was like so...</p> <pre><code>if($subject == 'art') { echo '<div id="spacer2"><br></div>'; echo '<div class="idName2" id="menu2">'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=all&listtype='.$listtype.'">All</a>'; echo '<div id="spacer2"><br></div>'; echo '</div>'; echo '<div class="idName3" id="menu3">'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=art&listtype='.$listtype.'">Art</a>'; echo '</div>'; echo '<div class="idName2" id="menu2">'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=biology&listtype='.$listtype.'">Biology</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=english&listtype='.$listtype.'">English</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=chemistry&listtype='.$listtype.'">Chemistry</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=mathematics&listtype='.$listtype.'">Mathematics</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=history&listtype='.$listtype.'">History</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=religion&listtype='.$listtype.'">Religion</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=geography&listtype='.$listtype.'">Geography</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=music&listtype='.$listtype.'">Music</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=philosophy&listtype='.$listtype.'">Philosophy</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=psychology&listtype='.$listtype.'">Psychology</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=economics&listtype='.$listtype.'">Economics</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=sociology&listtype='.$listtype.'">Sociology</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=technology&listtype='.$listtype.'">Technology</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=electronics&listtype='.$listtype.'">Electronics</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=food&listtype='.$listtype.'">Food</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=law&listtype='.$listtype.'">Law</a>'; echo '<a href="browse.php?alphabet='.$alphabet.'&subject=politics&listtype='.$listtype.'">Politics</a>'; echo '</div>'; } elseif($subject == 'biology') { </code></pre> <p>and then there's however many 'elseifs' as there are menu items, which ends up totalling to 2000 lines of code which is obviously very inefficient and it also makes it unbelievably time consuming to change anything... can someone point me in the right direction in what I need to do please!</p>
Tags
<php><performance><switch-statement>
Title
PHP Code efficiency in menu
singulars
PostAcceptedAnswerId
1. PO
  singulars
  PostTypePostTypeId
  PTAnswer
PostParentId
1. This table or related slice is empty.
PostTypePostTypeId
1. PTQuestion
UserLastEditorUserId
1. This table or related slice is empty.
UserOwnerUserId
1. USnutman
plurals
PostLinksPostIdRelatedPostId
1. This table or related slice is empty.
PostLinksRelatedPostIdPostId
1. This table or related slice is empty.
PostsAcceptedAnswerId
1. This table or related slice is empty.
PostsParentIdCreationDate
1. PO
  singulars
  PostTypePostTypeId
  PTAnswer
2. PO
  singulars
  PostTypePostTypeId
  PTAnswer
3. PO
  singulars
  PostTypePostTypeId
  PTAnswer
VotesPostIdCreationDate
1. This table or related slice is empty.
CommentsPostId
1. COI would do something similar to duri's answer below. I would say, however, that the menu would be better if you used a `UL` instead of your 'left' `DIV` and `LI`s instead of the `DIV`s within that 'left' `DIV`. It'll make things easier to follow, and explain the function of each section to readers of your code.
  singulars
  PostPostId
  POPHP Code efficiency in menu
  UserUserId
  USGareth
2. COThis was done by an accountant. Programmers don't do this.
  singulars
  PostPostId
  POPHP Code efficiency in menu
  UserUserId
  USHyderA

Querying!

Guidance

A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.