[PHP] Opsplitsen van query

[Verwijderd]

Beste lezers, ik heb een vraag...

De vraag
Ik haal een string uit de database, en heb deze met een function bewerkt, zodat hij na 200 karakters de eerste spatie opzoekt, en daar de string beëindigd. (Dat werkt)

Ik wil nu vanaf dat laatste woord weer beginnen, voor onder mijn "read more" button.

De Code

limit_string(); doet het wel
pick_string(); doet het niet

PHP:

//Limit - De functie die wel werkt - Stopt de string 
//na 200 karakters, na het vinden van een spatie.
//Het resultaat daarvan is dat ik alleen hele woorden heb.

                function limit_string($string, $charlimit)
                {
                   if(substr($string,$charlimit-1,1) != ' ')
                   {
                       $string = substr($string,'0',$charlimit);
                       $array = explode(' ',$string);
                       array_pop($array);
                       $new_string = implode(' ',$array);
                
                       return $new_string.'...';
                   }
                   else
                   {   
                       return substr($string,'0',$charlimit-1).'...';
                   }
                }

//Pickup - Deze functie werkt niet - Ik wil graag dat deze functie 
//verder gaat waar limit_string() ophield.

                  function pick_string($string, $charlimit)
                {
                   if(substr($string,$charlimit-1) != ' ')
                   {
                       $string = substr($string, $charlimit);
                       $array = explode(' ',$string);
                       array_pop($array);
                       $new_string = implode(' ',$array);
                
                       return $new_string;
                   }
                   else
                   {   
                       return substr($string,$charlimit-1);
                   }
                }

Zou ik nu dus:

PHP:

echo(limit_string("Dit is een test", 5));

Doen, dan output hij: "Dit is..."

Het is de bedoeling dat pick_string() weer verder gaat vanaf dat punt, dus "...een test"

Ik hoop dat jullie mij kunnen verder helpen.

[ Voor 11% gewijzigd door Verwijderd op 07-09-2006 02:42 ]

[dusty]

Een compleet andere oplossing dan je twee functies is gebruik maken van de mogelijkheden van je database.
Bijvoorbeeld de volgende code geeft je de twee delen tegelijk terug uit de database.

code:

select left(body,locate(" ",body,200)), substring(body,locate(" ",body,200)) from MyTable where id=9096

Een andere oplossing is om een wat makkelijkere algoritme te nemen door gebruik te maken van handige functies die PHP je bied, bijvoorbeeld strpos. Door gebruik te maken van de offset kan je dus midden in een string beginnen te zoeken, bovendien zal dat een stuk sneller zijn dan je huidige algoritme.

[ Voor 0% gewijzigd door dusty op 07-09-2006 03:23 . Reden: Ziek zijn zuigt. ]

[MBV]

je bedoelt hoop ik een algoritme
Dusty's oplossing vind ik erg netjes, als dat werkt. Wat je beter kan doen dan zoals je het nu doet: maak 1 functie die beide teruggeeft. Hoef je maar 1x op zoek, scheelt weer wat gedoe. Ik zou met strpos ofzo naar de eerste spatie vóór 200 zoeken, en dan daarop de string splitsen met substr ofzo. Vervolgens geef je de 2 strings in een array terug, die je vervolgens gebruikt in je output.

[dusty]

MBV schreef op donderdag 07 september 2006 @ 03:10:
je bedoelt hoop ik een algoritme

Uiteraard, ik gooi de verwarring op mijn griep en gebrek aan slaap de paar laatste dagen (vanwege de griep)

Dusty's oplossing vind ik erg netjes, als dat werkt.
[..]

Als de database MySQL is, werkt hij perfect en snel (zolang je uiteraard de juiste veldnamen en tabelnaam invult )

[sjhgvr]

Misschien heeft t nog wat debugging nodig, maar t werkt:

PHP:

<?php
function limit_string($string, $charlimit, $showmore) {
 if ($charlimit < strlen($string)) {
  if ($showmore) {
   $array = explode(' ', $string);
   $count = 0;
   foreach ($array as $piece) {
    $count += strlen($piece) + 1;
    if ($charlimit >= $count) array_shift($array);
    else break;
   }
   return '... ' . implode(' ', $array);
  } else {
   $array = explode(' ', substr($string, '0', $charlimit));
   array_pop($array);
   return implode(' ', $array) . ' ...';
  }
 } else if (!$showmore) return $string;
 return false;
}
$teststring = 'De PlayStation 3 komt in Europa pas volgend jaar maart op de markt. De console, die volgens eerdere berichten al op 17 november in de winkel zou liggen, kan niet in voldoende grote oplage geproduceerd worden vanwege problemen met de fabricage van de benodigde blauwelaserdiodes';
$cutoff = 200;
echo limit_string($teststring, $cutoff, false) . '<br>' . limit_string($teststring, $cutoff, true);
?>

[ Voor 14% gewijzigd door sjhgvr op 07-09-2006 05:56 . Reden: code stukkie herschreven ]

[DataGhost]

PHP:

function limit_string($string, $charlimit) {
    $limited = wordwrap($string, $charlimit, '<||>');
    $eerstedeel = substr($limited, 0, strpos($limited, '<||>'));
    return $eerstedeel;
}

function pick_string($string, $charlimit) {
    $limited = wordwrap($string, $charlimit, '<||>');
    $begintweededeel = strpos($limited, '<||>')+1;
    $tweededeel = substr($string, $begintweededeel);
    return $tweededeel;
}

$aap = "De snelle bruine vos springt over de luie hond.";
echo "Limit 15: ". limit_string($aap, 15) ."\n";
echo "Rest    : ". pick_string($aap, 15) ."\n";

code:

helios ~ # php limstr 
Limit 15: De snelle
Rest    : bruine vos springt over de luie hond.

Voordeel van wordwrap is dat je nooit over de 200 tekens heengaat, als dat een groot probleem is moet je de functie maar even aanpassen maar hier is in elk geval een begin
edit: voor de duidelijkheid dus, wordwrap zoekt niet na 200 tekens de eerste spatie maar voor 200 tekens de laatste.

[ Voor 6% gewijzigd door DataGhost op 07-09-2006 07:07 ]

[sjhgvr]

@ DataGhost
Kheb je script ff aangepast zodat t een beetje meer doet wat de TS wil (niet verkeerd bedoelt, alle respect gaat naar jou ):

PHP:

<?php
function stringCut($string, $charlimit, $showmore) {
 if ($charlimit < strlen($string)) {
  if ($showmore) return '... ' . substr($string, strpos(wordwrap($string, $charlimit, '<||>'), '<||>')+1);
  else {
   $limited = wordwrap($string, $charlimit, '<||>');
   return substr($limited, 0, strpos($limited, '<||>')) . ' ...';
  }
 } else if (!$showmore) return $string;
}
$string = 'The quick brown fox jumps over the lazy dog.';
$maxchr = 15;
echo stringCut($string, $maxchr, false) . '<br>' . stringCut($string, $maxchr, true);
?>

Nu kan de text korter zijn dan maxchr.
Oja .. en zorg aub dat <||> niet letterlijk voorkomt in de $string anders krijg je problemen, maar dat zal wel goed zitten aangezien je htmlchars escape-d .. hoop ik

[DataGhost]

mja <||> is niet echt een dagelijks voorkomende combinatie, zeker niet in html, het is geen geldige tag namelijk daarom dat ik die gekozen heb. Newlines kunnen er altijd inzitten, daar kan ik niet op vertrouwen dus.

[sjhgvr]

En dan kan t altijd nog korter

PHP:

<?php
function stringCut($s,$l,$e){
$r='<';$t=wordwrap($s,$l,$r);$u=strpos($t,$r);
return $l<strlen($s)?($e?'... '.substr($s,$u+1):
substr($t,0,$u).' ...'):(!$e?$s:0);}
$txt='The quick brown fox jumps over the lazy dog.';$m=15;
echo stringCut($txt,$m,0).'<br>'.stringCut($txt,$m,1);
?>

[ Voor 11% gewijzigd door sjhgvr op 07-09-2006 10:07 ]

[Zyppora]

DapinododiadeaL schreef op donderdag 07 september 2006 @ 07:56:
En dan kan t altijd nog korter

PHP:

<?php
function stringCut($s,$l,$e){
$r='<';$t=wordwrap($s,$l,$r);
return $l<strlen($s)?($e?'... '.substr($s,strpos($t,$r)+1):substr($t,0,strpos($t,$r)).' ...'):(!$e?$s:0);
}
$txt='The quick brown fox jumps over the lazy dog.';
$m=15;
echo stringCut($txt,$m,0).'<br>'.stringCut($txt,$m,1);
?>

Dat kan ... maar mocht je dit script na 6 maand aan moeten passen, kun je weer overnieuw beginnen (overdreven, weet ik, maar de moraal komt nog).

Je moet eigenlijk de variabelen zo willen benoemen dat je eruit kunt halen wat ze zijn/doen/etc. Dus niet $s, maar $start, niet $l, maar $length, niet $e, maar $end, etc.

Het ziet her zo heel strak uit, lekker kort en bondig, maar wanneer onderhoud (zeker wanneer je grotere scripts hebt) of uitbreiding nodig is, zul je spijt krijgen.

[YopY]

Daarnaast, PHP negeert spaties, returns, enzovoort. Het is wel kort, maar een paar spaties en returns maken niet uit voor de snelheid (voor zover ik weet)

Ik vind persoonlijk de SQL-optie iets mooier en hoogstwaarschijnlijk ook sneller, weet niet of die ook escaped op de spatie voor 200, maarja.

Ohja, en wiel opnieuw uitvinden hierzo: http://nl3.php.net/wordwrap

(weet niet of die precies hetzelfde doet, maar het lijkt er wel veel op)

[MBV]

ehm, dat staat in alledrie de 'code-posts' boven je

[Verwijderd]

Halleluja wat een replies, het is net een wedstrijd hier

Bedankt voor jullie antwoorden, met deze info kom ik er wel uit denk ik...zo niet, horen jullie het

[Verwijderd]

Helaas krijg ik het niet aan de gang...De SQL aanvraag (Mijn DB is inderdaad MySQL), levert de volgende error op:

code:

Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in c:\domains\messiah-clan.com\wwwroot\news_view.php on line 24

Als ik dit doe met de code:

PHP:

$id = $rij["id"];
$test = mysql_query("select left(item,locate(" ",item,200)), substring(item,locate(" ",$item,200)) from news where id=$id");

Ik ging ervan uit dat waar jij "body" had staan, dat daar de naam van het veld moest komen, dat werkte dus niet, ook niet als ik hier gewoon body liet staan. Nu moet ik eerlijk bekennen dat ik hier ook geen verstand van heb

Ik heb de functie van DapinododiadeaL ook geprobeerd. Deze scheen alles goed te doen, maar omdat de text die hij uit de database moet halen ook tags bevat voor de opmaak van de text, kreeg ik de output van alles, of niets. (Het eerste deel werd dus ..., het tweede deel begon met een stuk opmaaktags, gevold door de rest van de text, ik had $m toen wel op 200 staan.) Bij $m = 500; kreeg ik wel de hele text te zien, met de opmaak...alleen werd er dan onder "read more" niets meer weergegeven, simpelweg omdat hij alles al had gehad.

Screenshot: [Nadat ik op "Would you like to know more? heb geklikt]

---

---

Dat vond ik nogal vaag, omdat hij het eerste deel dan afkapt ruim voor de 200 karakters voorbij zijn.

[ Voor 8% gewijzigd door Verwijderd op 07-09-2006 13:44 ]

[Zyppora]

Da's geen SQL error, maar een PHP error.

Kun je ons een glimp van regel 24 (en omstreken) van je code laten zien?

Overigens kun je het beste verschillende quotes gebruiken voor je SQL query:

code:

$test = mysql_query("select left(item,locate(' ',item,200)), substring(item,locate(' ',$item,200)) from news where id=$id");

[Verwijderd]

jazeker kan dat...

PHP:

<?php
          mysql_connect("x", "x", "x");
          mysql_select_db("x");
          
          $sql = "SELECT * FROM news ORDER BY id DESC LIMIT 5";
          $result = mysql_query($sql);
    
          if (mysql_num_rows($result) >= 0)
          {
    
            while ($rij = mysql_fetch_array($result))
            {
                
                $id = $rij["id"];
                $test = mysql_query("select left(item,locate(' ',item,200)), substring(item,locate(' ',$item,200)) from news where id=$id");
                
                echo"
                <table width='600' border='0' cellpadding='0' cellspacing='0'>
                  <tr> 
                    <td width='600' height='20' align='center' valign='middle' background='../img/news/bar.jpg'><strong>".$rij["title"]."</strong></td>
                  </tr>
                  <tr> 
                    <td height='45' valign='top'><img src='img/news/".$rij["subject"].".jpg' width='600' height='70'></td>
                  </tr>
                  <tr> 
                    <td height='19' align='right' valign='middle'><div align='right'> 
                        <blockquote><strong>By ".$rij["author"]." on ".$rij["date"]."</strong></blockquote>
                      </div></td>
                  </tr>
                  <tr> 
                    <td height='19' valign='top'><blockquote><font size='2'>";
                    echo "$test";
                    echo"</font></blockquote></td>
                  </tr>";
                  ?>
                        <script type="text/javascript">
                          <!--
                            function toonVerbergInfo(divId) {
                              if(document.getElementById(divId).style.display == "none") {
                                //div weergeven
                                document.getElementById(divId).style.display = "block";
                              }
                              else {
                                //div verbergen
                                document.getElementById(divId).style.display = "none";
                              }
                            }
                          -->
                        </script>
<?php
                  echo"
                      <tr>
                        <td height='19' align='left' valign='top'> <img src='../img/plus.gif' width='11' height='11'>"; ?> <a href="#" onclick="toonVerbergInfo('<?php echo"".$rij["id"].""; ?>');return false"> Would you like to know more?</a> <?php echo"</td>
                      </tr>
                      <tr> 
                        <td height='19' valign='top'><blockquote><font size='2'>"; ?><div id="<?php echo "".$rij["id"].""; ?>" style="display:none;"><?php echo "$rest"; ?></div> <?php echo"</font></blockquote></td>
                      </tr>
                    </table>";
            }
          }
          
          if (mysql_num_rows($result) == 0)
          {
            echo "<hr>There are no items in the database to display</strong>";
          }
?>

Dat is de gehele code, ik heb de vorige functies eruit gehaald, dus regel 24 is nu regel 15

[Zyppora]

PHP:

$test = mysql_query("select left(item,locate(' ',item,200)), substring(item,locate(' ',$item,200)) from news where id=$id");

In het SUBSTRING() gedeelte heb je $item staan. Is dit een variabele of een typo?

PHP:

echo "$test";

Als je hier de quotes eens weghaalt (weet niet of het uitmaakt, maar het maakt de code wel overzichtelijker en IMHO netter). Hou je er wel rekening mee dat $test een array is van de LEFT() constructie (eerste 200 karakters) en de SUBSTRING() constructie uit je SQL query?

De rest heb ik niet met hele scherpe ogen aangekeken, maar op het eerste gezicht zag het er niet slecht uit.

Edit: Gequote zut opruimen ...
* Zyppora veegt nog wat [/quote] tags weg

[ Voor 14% gewijzigd door Zyppora op 07-09-2006 15:07 ]

[Verwijderd]

Ik wil iedereen hartelijk bedankten voor zijn bijdrage...ik heb het probleem nu opgelost

[dusty]

Verwijderd schreef op donderdag 07 september 2006 @ 13:41:
[..]
Als ik dit doe met de code:

PHP:

$id = $rij["id"];
$test = mysql_query("select left(item,locate(" ",item,200)), substring(item,locate(" ",$item,200)) from news where id=$id");

[..]

Dat klopt, je doet in principe nu:

code:

$hallo="hij zei:"hallo".";

Probleem is dus dat je de quotes BINNEN de string niet escaped.