is this the right way to do?

Discussion in 'PHP' started by baris22, Feb 7, 2010.

  1. #1
    it works the way i did but is it the right way? is there anyother way to do this?

    thanks

    
    
    		if ($newReference == "A001" or $newReference == "B001" or $newReference == "C001" $newReference == "D001" or $newReference == "E001" or $newReference == "F001" $newReference == "G001" or $newReference == "H001" or $newReference == "I001" $newReference == "J001" or $newReference == "K001" or $newReference == "L001" $newReference == "M001" or $newReference == "N001" or $newReference == "O001" $newReference == "P001" or $newReference == "Q001" or $newReference == "R001" $newReference == "S001" or $newReference == "T001" or $newReference == "U001" $newReference == "V001" or $newReference == "W001" or $newReference == "X001" $newReference == "Y001" or $newReference == "Z001")
            {
            $optitem = "OPTIMIZE TABLE item";
            $opt_item_table = mysql_query($optitem); 
    		echo "optimised item table";
    		
    		$optorderr = "OPTIMIZE TABLE orderr";
            $opt_orderr_table = mysql_query($optorderr); 
    
    		echo "optimised orderr table";
            }
    
    
    PHP:
     
    baris22, Feb 7, 2010 IP
  2. danx10

    danx10 Peon

    Messages:
    1,179
    Likes Received:
    44
    Best Answers:
    2
    Trophy Points:
    0
    #2
    Instead of 'or' use the || operator, here's your code shortened.

    <?php
      if ($newReference == "A001" || $newReference == "B001" || $newReference == "C001" $newReference == "D001" || $newReference == "E001" || $newReference == "F001" $newReference == "G001" || $newReference == "H001" || $newReference == "I001" $newReference == "J001" || $newReference == "K001" || $newReference == "L001" $newReference == "M001" || $newReference == "N001" || $newReference == "O001" || $newReference == "P001" || $newReference == "Q001" || $newReference == "R001" $newReference == "S001" || $newReference == "T001" || $newReference == "U001" $newReference == "V001" || $newReference == "W001" || $newReference == "X001" $newReference == "Y001" || $newReference == "Z001") {
          if(mysql_query("OPTIMIZE TABLE item") && mysql_query("OPTIMIZE TABLE orderr")){
          echo "Optimised 'item' & 'orderr' table";
          } else {
          echo "Error Optimizing!";
          }
      }
    ?>
    PHP:
     
    Last edited: Feb 7, 2010
    danx10, Feb 7, 2010 IP
  3. baris22

    baris22 Active Member

    Messages:
    543
    Likes Received:
    4
    Best Answers:
    0
    Trophy Points:
    60
    #3
    i used || before using or but it did not work. it did not optimize the tables
     
    baris22, Feb 7, 2010 IP
  4. danx10

    danx10 Peon

    Messages:
    1,179
    Likes Received:
    44
    Best Answers:
    2
    Trophy Points:
    0
    #4
    $newReference == "X001" $newReference == "Y001"
    PHP:
    I think you have a typo in your if statement, I'm suprised the optimization worked??

    No wonder the || didn't work previously :p
     
    danx10, Feb 7, 2010 IP
  5. baris22

    baris22 Active Member

    Messages:
    543
    Likes Received:
    4
    Best Answers:
    0
    Trophy Points:
    60
    #5
    thanks it works perfect.
     
    baris22, Feb 7, 2010 IP