optimization - Optimising a PHP If/Else statement -


i'm attempting optimise following php if/else statement. rewrite code make use case , switch, or should leave is, or what?

code:

if(empty($_get['id'])){     include('pages/home.php'); }elseif ($_get['id'] === '13') {     include('pages/servicestatus.php'); }elseif(!empty($_get['id'])){     $rawdata = fetch_article($db->real_escape_string($_get['id']));     if(!$rawdata){         $title = "";         $meta['keywords'] = "";         $meta['description'] = "";     }else{         $title = stripslashes($rawdata['title']);         $meta['keywords'] = stripslashes($rawdata['htmlkeywords']);         $meta['description'] = stripslashes($rawdata['htmldesc']);         $subs = stripslashes($rawdata['subs']);         $pagecontent = "<article>" . stripslashes($rawdata['content']) . "</article>";     }     include("includes/header.php");     echo $pagecontent;     if(!$rawdata){         error_404();     } }

thanks

i hate switch statements, personal preference honest. far further optimization i'd suggest taking @ form of assembly language. give general ideas on how make conditional statements more efficient. is, give different out on things.

if(!empty($_get['id']))      {      if($_get['id'] == '13')     {         include('pages/servicestatus.php');     }     else     {         $rawdata = fetch_article($db->real_escape_string($_get['id']));          if (!$rawdata) {              $title = "";             $meta['keywords'] = "";             $meta['description'] = "";         } else {              $title = stripslashes($rawdata['title']);             $meta['keywords'] = stripslashes($rawdata['htmlkeywords']);             $meta['description'] = stripslashes($rawdata['htmldesc']);             $subs = stripslashes($rawdata['subs']);             $pagecontent = "<article>" . stripslashes($rawdata['content']) . "</article>";         }          include("includes/header.php");         echo $pagecontent;         if (!$rawdata) {              error_404();         }     } }  else  {     include('pages/home.php'); }

Comments

Post a Comment

Popular posts from this blog

asp.net - repeatedly call AddImageUrl(url) to assemble pdf document -

i'm using abcpdf , i'm curious if can recursively call addimageurl() function assemble pdf document compile multiple urls? something like: int pagecount = 0; int theid = thedoc.addimageurl("http://stackoverflow.com/search?q=abcpdf+footer+page+x+out+of+", true, 0, true); //assemble document while (thedoc.chainable(theid)) { thedoc.page = thedoc.addpage(); theid = thedoc.addimagetochain(theid); } pagecount = thedoc.pagecount; console.writeline("1 document page count:" + pagecount); //flatten document (int = 1; <= pagecount; i++) { thedoc.pagenumber = i; thedoc.flatten(); } //now try again theid = thedoc.addimageurl("http://stackoverflow.com/questions/1980890/pdf-report-generation", true, 0, true); //assemble document while (thedoc.chainable(theid)) { thedoc.page = thedoc.addpage(); theid = thedoc.addimagetoc...
Read more

java - Android recognize cell phone with keyboard or not? -

in android app, want detect user's device if device has keyboard(like motorola milestone) , show corresponding user interface. guys know how that? ! regards justicepenny http://developer.android.com/reference/android/content/res/configuration.html public int keyboard the kind of keyboard attached device. 1 of: keyboard_nokeys, keyboard_qwerty, keyboard_12key. that's it!
Read more

iphone - How would you achieve a LED Scrolling effect? -

how achieve led scrolling effect example below? led sign - led ticker emulator iphone , ipad http://img.skitch.com/20101201-rsfh4p1bajb1wp94k466pdpiuj.preview.jpg sometimes remunerates comment in other posts :) have put code you, should implement font '8pin matrix' realistic feeling. uilabel *label; - (void)viewdidload { [super viewdidload]; label = [[uilabel alloc]initwithframe:cgrectmake(10, 50, 300, 20)]; label.font = [uifont fontwithname:@"courier-bold" size:16.0]; label.backgroundcolor = [uicolor blackcolor]; label.textcolor = [uicolor greencolor]; label.linebreakmode = uilinebreakmodecharacterwrap; // otherwise "…" label.text = @"this led text scrolled "; // blanks space between [self.view addsubview:label]; [nstimer scheduledtimerwithtimeinterval:0.12f target:self selector:@selector(scrolllabel) use...
Read more