View Single Post
Old 08-01-2004, 04:34 AM   #4 (permalink)
RelaX
Follower of Ner'Zhul
 
RelaX's Avatar
 
Location: Netherlands
Zend Studio rocks.

It works like this:
PHP Code:
/**************************************************************************************

        Browser Detection is provided by Leon Atkinson <leon@clearink.com>

/*------------------------------------------------------------------------------------*/ 
Notice that I removed the slash after the line of stars, which was closing the comment and causing the string to be parsed by PHP.

I would suggest you change it into:
PHP Code:
/**********************************************************************
 *                                                                    *
 * Browser Detection is provided by Leon Atkinson <leon@clearink.com> *
 *                                                                    *
 **********************************************************************/ 
Or if you want to be tha bomb and make it phpdoc compatible (phpdoc rules!) and generally more compact and human readable you could rewrite your code as follows:

PHP Code:
<?php

getBrowserInfo
();

/**
 * The plot cam function
 */
function plot_cam()
{
    global 
$filename,$img,$pause;
    
$k = 0;

    
Header("Content-type: multipart/x-mixed-replace;boundary=ThisRandomString");

    while (
$img[$k]==$filename) 
    {
        print(
"\n--ThisRandomString\n\n");
        
Header("Content-type: text/plain");
        
        
$cam = fopen($filename,"r");
        
fpassthru($cam);
        
fclose($cam);

        if (
$pause) { sleep($pause);} // time between reload, usefull for server usage

        
$k++;

        if (
$k==count($img)) echo("\n--ThisRandomString--\n");
    }
}

/**
 * Browser Detection is provided by Leon Atkinson <leon@clearink.com>
 */
function getBrowserinfo()
{
    
/* Get the name the browser calls itself and what version */

    
$Browser_Name = strtok($HTTP_USER_AGENT, "/");
    
$Browser_Version = strtok(" ");

    
/* MSIE lies about its name */
    
if(ereg("MSIE", $HTTP_USER_AGENT))    
    {
        
$Browser_Name = "MSIE";
        
$Browser_Version = strtok("MSIE");
        
$Browser_Version = strtok(" ");
        
$Browser_Version = strtok(";");
    }

    
/* Opera isn't completely honest, either ... */
    /* Modificaton by Chris Mospaw <mospaw@polk-county.com> */
    
if(ereg("Opera", $HTTP_USER_AGENT))    
    {
        
$Browser_Name = "Opera";
        
$Browser_Version = strtok("Opera");
        
$Browser_Version = strtok("/");
        
$Browser_Version = strtok(";");
    }

    
/* try to figure out what platform, windows or mac */
    
$Browser_Platform = "unknown";

    if(
ereg("Windows",$HTTP_USER_AGENT)
    || 
ereg("WinNT",$HTTP_USER_AGENT)
    || 
ereg("Win95",$HTTP_USER_AGENT)) 
        
$Browser_Platform = "Windows";

    if(
ereg("Mac", $HTTP_USER_AGENT)) 
            
$Browser_Platform = "Macintosh";

    if(
ereg("X11", $HTTP_USER_AGENT))
        
$Browser_Platform = "Unix";

    if((
$Browser_Platform == "Windows"))
    {
        if(
$Browser_Name == "Mozilla")    
        {
            
// SERVER PUSH WORK'S ONLY HERE <img src="images/smilies/smile.gif" border="0" alt="">
            
if($Browser_Version >= 4.0) plot_cam();
        }
    }
    elseif(
$Browser_Platform == "Macintosh") 
    {
        if(
$Browser_Name == "Mozilla") 
        {
            
// SERVER PUSH WORK'S ONLY HERE <img src="images/smilies/smile.gif" border="0" alt="">
            
if($Browser_Version >= 4.0)  plot_cam();
        }
    }

    elseif(
$Browser_Platform == "Unix") 
    {
        if(
$Browser_Name == "Mozilla") 
        {
            
// SERVER PUSH WORK'S ONLY HERE <img src="images/smilies/smile.gif" border="0" alt="">
            
if($Browser_Version >= 4.0) { plot_cam();}
        }
    }
}
?>
But that is just my style, doesn't have to be yours.
__________________
The most likely way for the world to be destroyed, most experts agree, is by accident. That's where we come in; we're computer professionals. We cause accidents.
- Nathaniel Borenstein

Last edited by RelaX; 08-01-2004 at 05:00 AM..
RelaX is offline  
 

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73