עזרה |בקשר לייעול.

[Shon12]

שלום,

אז ככה יש לי תמונת GD שמראה את שם הדפדפן + הווינדוס.
עכשיו הבדיקה שלי זה משהו כזה:

PHP קוד:

$name = "";
   if (preg_match("/msie 6.0/i",$_SERVER['HTTP_USER_AGENT']))
   {
   $name .= "IE 6";
   }
   elseif (preg_match("/msie 7.0/i",$_SERVER['HTTP_USER_AGENT']))
   {
   $name .= "IE 7";
   }
   elseif (preg_match("/firefox/i",$_SERVER['HTTP_USER_AGENT']))
   {
   $name .= "fire fox";
   } 


והשיטה הזו נראה לי זוללת משאבים,
אשמח להצעות ייעול,

תודה

[psycho]

זוללת משאבים? כמה זמן לוקח לביטוי רגולרי ולהשמה של התוצאה בתוך משתנה? בחייאת גבר =)
תבדוק כמה זמן ריצה לוקח לכל שלב
ואז תגיד לי כמה זה זולל משאבים=]

[Meir]

אם אתה מחפש מחרוזות ללא צורך בתנאים מסויימים יש עדיפות לstrpos.

[Shon12]

נכתב במקור על ידי psycho

זוללת משאבים? כמה זמן לוקח לביטוי רגולרי ולהשמה של התוצאה בתוך משתנה? בחייאת גבר =)
תבדוק כמה זמן ריצה לוקח לכל שלב
ואז תגיד לי כמה זה זולל משאבים=]

כן,אבל תחשוב שאותו בדיקה אני עושה על שם הווינדוס.
ככה שיוצא איזה 7 ביטויים רגלורים.

[daMn]

http://il.php.net/get_browser
תקרא את ההערה, צריך לשנות בהגדרות של הphp משהו.
אם אין לך גישה קח פונקציה נחמדה שבניתי :

PHP קוד:

<?php
function userBrowser($UserAgent)
{
    $browsers = array("Firefox", "MSIE 6.0", "MSIE 7.0");
    for ($i=0;$i<=count($browsers);$i++)
    {
        $pos = strpos($UserAgent, $browsers[$i]);
        if ($pos == true)
            return $browsers[$i];
    }
}
echo userBrowser($_SERVER['HTTP_USER_AGENT']);
?>

[Meir]

אפשר גם ככה:

PHP קוד:

<?php

$browser = 'msie 6.0';
$browsers = array('msie 6.0' => 'IE 6', 'msie 7.0' => 'IE 7');

if (preg_match('/('.implode('|', array_keys($browsers)).')/i', $browser, $match))
{
    echo $browsers[$match[1]];
}

?>

[Shon12]

נכתב במקור על ידי daMn

http://il.php.net/get_browser
תקרא את ההערה, צריך לשנות בהגדרות של הphp משהו.
אם אין לך גישה קח פונקציה נחמדה שבניתי :

PHP קוד:

<?php
function userBrowser($UserAgent)
{
    $browsers = array("Firefox", "MSIE 6.0", "MSIE 7.0");
    for ($i=0;$i<=count($browsers);$i++)
    {
        $pos = strpos($UserAgent, $browsers[$i]);
        if ($pos == true)
            return $browsers[$i];
    }
}
echo userBrowser($_SERVER['HTTP_USER_AGENT']);
?>

תודה רבה !
רק למשל אני רוצה להציג את שם הווינדוס אני מעוניין שהוא יחזיר את שם הווינדוס הספציפי ולא: NT 5.1למשל..

[daMn]

נכתב במקור על ידי Shon12

תודה רבה !
רק למשל אני רוצה להציג את שם הווינדוס אני מעוניין שהוא יחזיר את שם הווינדוס הספציפי ולא: NT 5.1למשל..

אז תעשה ככה...

PHP קוד:

<?php
function userBrowser($UserAgent)
{
    $browsers = array("Firefox" => "ff", "MSIE 6.0" => "ie6", "MSIE 7.0" => "ie7");
    foreach ($browsers as $key => $val)
    {
        $pos = strpos($UserAgent, $key);
        if ($pos == true)
            return $val;
    }
}
echo userBrowser($_SERVER['HTTP_USER_AGENT']);
?>

[WebProject]

בתיכנות מודדים בכלליות, לא לפי כל פונקציה, הקוד שלך הוא סה"כ Big-O(n), לא משהו חריג.