monkeytooth Posted November 24, 2010 Share Posted November 24, 2010 Ok this is not my code for starters.. <?php elseif($_GET['pg'] == 'stats'): ?> <?php require("snippets/tab.stats.php"); ?> <?php elseif($_GET['pg'] == 'app'): ?> <?php require("snippets/tab.app.php"); ?> <?php elseif($_GET['pg'] == 'triggers'): ?> <?php if(@$_GET['opt'] == "frm") { ?> <?php require("snippets/tab.trigger.form.php"); ?> <?php } else { ?> <?php require("snippets/tab.trigger.list.php"); ?> <?php } ?> <?php else:?> <?php require("snippets/tab.info.php"); ?> <?php endif; ?> Can anyone tell me how that makes any logical sense to do it like that, please.. Quote Link to comment https://forums.phpfreaks.com/topic/219649-what-the/ Share on other sites More sharing options...
PFMaBiSmAd Posted November 24, 2010 Share Posted November 24, 2010 It has two problems that I can see - 1) All the unnecessary opening and closing php tags. 2) Using both the traditional and alternative control structure syntax (which by the way, takes more typing in every case than using the traditional syntax.) Fixing both of those would at least make it readable. Quote Link to comment https://forums.phpfreaks.com/topic/219649-what-the/#findComment-1138826 Share on other sites More sharing options...
monkeytooth Posted November 24, 2010 Author Share Posted November 24, 2010 Thank you lol, thats what I already knew but I just needed confirmation if you will call it that, just to make 100% sure. Quote Link to comment https://forums.phpfreaks.com/topic/219649-what-the/#findComment-1138828 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.