Is it inefficient to use includes and requires compared to functions ?

jeddiki

Hello,

I have been writing procedural php for a couple of years
and I have quite lot of little scrips that I use in different
large scripts.

I'll give and example
I display the category menu on onmany pages so I have this
little php file:

Expand|Select|Wrap|Line Numbers

 switch ($N_cat_cd)

     { 

     case "ex": $Dcat = " Expert-World";  break;

     case "se": $Dcat = " Self Improvement";  break;

     case "s.": $Dcat = " S.E.O.";  break;

     case "tr": $Dcat = " Traffic"; break;

     case "ni": $Dcat = " Niche-Affiliate"; break;

     case "co": $Dcat = " Copy writing"; break;

     case "sa": $Dcat = " Sales systems"; break;

     case "pr": $Dcat = " Product creation"; break;

     case "we": $Dcat = " Website Design"; break;

     case "gr": $Dcat = " Graphics"; break;

     }

     $SEOcat = strtolower(trim($Dcat));
 
$cat_menu ="

  <div class='menuCAT'>

    <ul>

    <li> <a href=\"/im/menu1.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu1.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu2.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu3.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu4.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu5.htm" > $SEOcat</a></li>

    <li> <a href=\"/im/menu6.htm" > $SEOcat</a></li>

   </ul>

   </div>";

So when ever I need this code, I require inside my main php script

For example

Expand|Select|Wrap|Line Numbers

 
$page="pr";

$title1 = " products and resources";  

$desc=" internet marketing products";
 
if (@$_SESSION['auth'] == "yes" ){

     require_once("mem_head.php");

     }  // end if

else {

     require_once("a_head.php");

     }  // end if
 
$dir = "products";

$pub = "prod";
 require_once("a_cat_menu.php");

?>     
 
<div class="page_name">

<h1>Resources - Expert IM Products</h1>

</div> <!-- End div: page_name -->

In fact there are three sets of require command, I have bolded them

Now I was quite happy with this until I saw this
thread in a forum about an email class called swift.

Quote:

The best way to work is usually by making a function in a file, and calling
that function instead of including / requiring in loop

Because the php works this way:

Read File,
Tokenize,
Compilation
Execution

If you include at each iteration, the whole process is repeated, if you use
functions, it will only execute the portion of code needed to send the mail

From a functionality point of view, it works the same

From a performance point of view, it will work way slower.

So - and here is my question:

Should I be re-writing my includes and requests as functions?

I assumed that the whole code would be read by the php engine before compilation and execution - so why the suggested slow down ?

Maybe this is just inside loops ?

The actual bit of code that was under discussion with this:

Here is the script.

Expand|Select|Wrap|Line Numbers

 START

$swift_path =  "/home/focus7/public_html/lib/swift_required.php";

$func_path =  "/home/focus7/public_html/my_functions.php";

require_once("$swift_path");

require_once("$func_path");
 
$transport = Swift_SmtpTransport::newInstance('mail.MY-SITE.com', 25)

  ->setUsername('MY-NAME')

  ->setPassword('MY-PASS')

  ;
 
$mailer = Swift_Mailer::newInstance($transport);
 
$sql_client = "SELECT * FROM clients ";                     // OPEN

DATABASE & GET CLIENTS

$result_client = mysql_query($sql_client)       or die("could not execute

FIND CLIENT query.". mysql_error());

while($row_client = mysql_fetch_assoc($result_client))

{                            // START THE LOOP

  extract($row_client);
 
  $message = $message.$ek.$ek1.$ek2;     // CREATE THE MESSAGE FROM

DATA
 
  $simdi = date('H:m:s, l, j F Y',$today);
 
  $to = $email;                                         // CREATE THE

HEADERS

  $contact = $contact;

  $fm_email="Account-Mana...@support-focus.com";

  $fm_name="Support Center";

  $subject="Your Weekly Report";

  $message_html = "Dear $contact,<br><br><b>Weekly Report: $simdi</

b><br><br>".$message”;
 
   require("send_auto.php");                      // SEE BELOW
 
} // end while         //    LOOPS THROUGH DATABASE  ( ONLY 25
 
RECORDS )
 
THIS IS THE send_auto.php
 
START

$message = Swift_Message::newInstance();

send_auto.php
 
// set the sender

$message->setFrom(array($fm_email => $fm_name));
 
// set the recipient

$message->setTo(array($to => $contact));
 
// set the subject

$message->setSubject($subject);
 
// set the html body

$message->setBody($message_html, 'text/html');
 
// set the text body

$message->addPart($message_text, 'text/plain');
 
//Send the message

if (!$mailer->send($message, $failures)) {

   // update failure database

     $sql = "INSERT INTO `email_fail` (type,email,send_dt) VALUES

( '$send_type', '$to','$today')";

                 mysql_query($sql) or die("could not execute email_fail update

query". mysql_error());

   } // end if

END

I would really appreciate you inputs as it may fundementally change the way I structure my code.

Thanks

Dormilich

Quote:

Originally Posted by jeddiki

I assumed that the whole code would be read by the php engine before compilation and execution - so why the suggested slow down ?

this is a case where you have to carefully read. the paragraph is about include()/require() in a loop structure:

Expand|Select|Wrap|Line Numbers

 while ($condition)

{

    include 'include_file.php';

}

then the meaning becomes obvious…

jeddiki

Thanks Dormilich for pointing that ,

I had only used the require because I have four scripts that
use the same sending code.

Apart from that, there is no need to put the code outside the
script.

So I guess the loop will perform better like this:

Expand|Select|Wrap|Line Numbers

 START

$swift_path =  "/home/focus7/public_html/lib/swift_required.php";

$func_path =  "/home/focus7/public_html/my_functions.php";

require_once("$swift_path");

require_once("$func_path");
 
$transport = Swift_SmtpTransport::newInstance('mail.MY-SITE.com', 25)

  ->setUsername('MY-NAME')

  ->setPassword('MY-PASS')

  ;
 
$mailer = Swift_Mailer::newInstance($transport);
 
$sql_client = "SELECT * FROM clients ";                     // OPEN

DATABASE & GET CLIENTS

$result_client = mysql_query($sql_client)       or die("could not

execute

FIND CLIENT query.". mysql_error());

while($row_client = mysql_fetch_assoc($result_client)) {        //

START THE LOOP

  extract($row_client);
 
  $message = $message.$ek.$ek1.$ek2;     // CREATE THE MESSAGE FROM

DATA
 
  $simdi = date('H:m:s, l, j F Y',$today);
 
  $to = $email;                                         // CREATE THE

HEADERS

  $contact = $contact;

  $fm_email="Account-Mana...@support-focus.com";

  $fm_name="Support Center";

  $subject="Your Weekly Report";

  $message_html = "Dear $contact,<br><br><b>Weekly Report: $simdi</

b><br><br>".$message”;
 
$message = Swift_Message::newInstance();
 
// set the sender

$message->setFrom(array($fm_email => $fm_name));
 
// set the recipient

$message->setTo(array($to => $contact));
 
// set the subject

$message->setSubject($subject);
 
// set the html body

$message->setBody($message_html, 'text/html');
 
// set the text body

$message->addPart($message_text, 'text/plain');
 
//Send the message

if (!$mailer->send($message, $failures)) {

   // update failure database

     $sql = "INSERT INTO `email_fail` (type,email,send_dt) VALUES

( '$send_type', '$to','$today')";

                 mysql_query($sql) or die("could not execute email_fail update

query". mysql_error());

                 } // end if
 
$content = "Sent Weekly Report to $contact at $email \r\n";

fwrite($handle, $content);

} // end while                                    //  END OF LOOP
 
fclose($handle);

?>

There seems no point in creating a function out of that code
... or is there ???

Dormilich

Quote:

Originally Posted by jeddiki

There seems no point in creating a function out of that code
... or is there ???

if you’re sending to multiple recepients (I think you are), you can use Swiftmailer’s batchSend() method

jeddiki

Yes,
I did read about that,
but I also saw in their forum that a lot of people have trouble with it.

It uses the same sending code in the end so I
think I will stick to the basic version and control
it with my while loop.

But thanks anyway :)

Dormilich

Quote:

Originally Posted by jeddiki

but I also saw in their forum that a lot of people have trouble with it.

I use it and I don’t have any trouble. (I even use the Decorator Plugin). Nevertheless, batch sending is more effective, than using an external loop.

Is it inefficient to use includes and requires compared to functions ?

re: Is it inefficient to use includes and requires compared to functions ?

re: Is it inefficient to use includes and requires compared to functions ?

re: Is it inefficient to use includes and requires compared to functions ?

re: Is it inefficient to use includes and requires compared to functions ?

re: Is it inefficient to use includes and requires compared to functions ?

/bytes/development

/bytes/it

/bytes/about