PHP: betingelse

if (!preg_match("/^Re:/",$row['subject']) {
  $_POST['subject'] = "Re: ".$row['subject'];
}

Jeg takker! Smider du lige et svar?

Jeg samler slet ikke på point, tak.

Ellers kan du bruge nedenstående. Den løser også dit "problem" og er teoretisk set hurtigere end at skulle bruge regular expressions (som i det forslag ericjacobsen skriver).

if(substr($row['subject'], 0, 3) == "Re:")

Nu vi snakker hurtighed, så vil jeg mene dette er endnu hurtigere:

if(strpos($row['subject'],'Re:') == 0)

strpos skal gennemsøge hele strengen, det skal substr ikke

Morhan: Returns the numeric position of the first occurrence of needle in the haystack string.

i værste tilfælde, skal den søge hele strengen igennem

en lille benchmark siger, at strpos er en anelse hurtigere, hvis subject er en lille tekst (som et subject normalt er)

ved større tekst, er strpos en del langsommere, også selv om re: er tilstede

Jeg laver samme test med 100000 forsøg!
Resultatet kan ses her:
http://www.codebreaker.dk/exp/774488aexp.php
http://www.codebreaker.dk/exp/774488bexp.php

<?php
function microtime_float(){
  list($usec, $sec) = explode(" ", microtime());
  return ((float)$usec + (float)$sec);
}

$count = 100000;

$str = "this is a subject";

for($langstr="", $i=0; $i<130; $i++) $langstr .= $str;
//$str = $langstr; // kort eller lang streng


$str2 = "Re: $str";

$start1 = microtime_float();
for($i=0; $i<$count; $i++){
    if(substr($str, 0, 3) == 'Re :');
}
$slut1= microtime_float();

$start2 = microtime_float();
for($i=0; $i<$count; $i++){
    if(substr($str2, 0, 3) == 'Re :');
}
$slut2= microtime_float();


$start3 = microtime_float();
for($i=0; $i<$count; $i++){
    if(strpos($str, 'Re :') === 0);
}
$slut3= microtime_float();

$start4 = microtime_float();
for($i=0; $i<$count; $i++){
    if(strpos($str2, 'Re :') === 0);
}
$slut4= microtime_float();


echo "<pre>";
echo "substr uden re: ".($slut1-$start1)."\n";
echo "substr med re:  ".($slut2-$start2)."\n";
echo "strpos uden re: ".($slut3-$start3)."\n";
echo "strpos med re:  ".($slut4-$start4)."\n";
echo "</pre>";
?>


kort str:

substr uden re: 0.261208057404
substr med re:  0.223273038864
strpos uden re: 0.197156190872
strpos med re:  0.204102993011


lang str:

substr uden re: 0.251914978027
substr med re:  0.237639904022
strpos uden re: 0.192348957062
strpos med re:  0.296867847443

det var forkert copy/pasted

den lange streng:

substr uden re: 0.247492790222
substr med re:  0.247993946075
strpos uden re: 0.606462955475
strpos med re:  0.62971496582

Ja, så begynder er at være stor forskel kan jeg godt se.

Godt. Dvs. at istedet for at bruge den RegExp, som erikjacobsen gav i 1. kommentar, så bør jeg istedet bruge:
if(substr($row['subject'], 0, 3) == "Re:")

?

Når Re: er i starten, kan du bruge strpos().
Men er der tale om en længere streng, hvis Re: ikke er i starten, så skal du bruge substr().

jakobdo >>

Smider du lige et svar?

Svar!

Takker for point.

Tak for hjælpen ;)

Jeg kan se, at det vi har fået lavet ikke virker :(

Jeg har denne kode, som ikke virker:

<?
$subject="test";
echo "Start subject: ".$subject."<br>";
if(!strpos($subject,'Re:') == 0) {
    $subject = "Re: ".$subject;
}
echo "End subject: ".$subject;
?>

Håber der er en, som kan hjælpe? :)

substr returnerer også false (ikke -1, som man er vant til fra andre spog), hvis søgestrengen ikke findes. Så du skal bruge === når du sammenligner med 0

Jeg synes ikke det bliver bedre af at lave dette:

<?
$subject="test";
echo "Start subject: ".$subject."<br>";
if(!strpos($subject,'Re:') === 0) {
    $subject = "Re: ".$subject;
}
echo "End subject: ".$subject;
?>

! har højere prioritet end ===

if(!(strpos($subject,'Re:') === 0)) {

eller

if(strpos($subject,'Re:') !== 0) {

Jeg siger mange tak :)