Wyświetlenie 3 ostatnich "news`ow" Opcje

[fraksipon]

Cześć,
Czy ktoś potrafiłby dla mnie wyciągnąc z tego skryptu aktualności możliwość wyświetlenia np: tylko 3 ostatnich wpisów?
Prosze o napisania kodu.

[PHP] pobierz, plaintext 
<?php 
 
 
require 'config.php';
 
	function VerifyTxtFile(){
		if(!is_dir("content")) mkdir("content", "0777", true);
		if(!file_exists("content/news.txt")) {
			$file = fopen("content/coments.txt", "w");
			fwrite($file, "");
			fclose($file);
		}
		return true;
	}
 
 
 	function ReadNews(){
 		VerifyTxtFile();
		$fh = fopen("content/news.txt", 'r');
		$theData = fgets($fh);
		fclose($fh);
		return ExplodeNews($theData);	
 	}
 
 	function ExplodeNews($NewsArray){
 		$list = array(array());
 		if($NewsArray == "") return 0;
 		$Coments = explode("<X-_-X>", $NewsArray);
	 		for($i=0; $i!=sizeof($Coments)-1; $i++){
	 			$Coment = explode("<XXX_>", $Coments[$i]);
	 			$list[$i]['title'] = $Coment[0];
	 			$list[$i]['date'] = $Coment[1];
	 			$list[$i]['resumo'] = $Coment[2];
	 		}
	 	return $list;	
 	}
 
 	function FindNew($id_new){
 		VerifyTxtFile();
		$fh = fopen("content/news.txt", 'r');
		$theData = fgets($fh);
		fclose($fh);
		$new = array();
 		$Coments = explode("<X-_-X>", $theData);
 
		 	if($id_new < sizeof($Coments)-1 and $id_new >= 0){
 
		 		$Coment = explode("<XXX_>", $Coments[$id_new]);
		 		$new['title'] = $Coment[0];
		 		$new['date'] = $Coment[1];
		 		$new['completo'] = $Coment[3];
		 		if($Coment[4] != ""){
		 			$new['image'] = $Coment[4];
		 			$new['align'] = $Coment[5];	
		 		}		
 
		 	}else{
		 		return 0;
 
		 	}
 
		return $new;
 	}
 
?>
<link rel="stylesheet" type="text/css" href="images/style.css" />
<?php 
 
	if(isset($_GET['view'])){
		$new_c = FindNew($_GET['id']);
		if($new_c != 0){
			echo "<table width=\"100%\" border=\"0\">
  <tr>
    <td height=\"20\"><strong><p class=\"NewsTitle\">".$new_c['title']."</p></strong></td>
  </tr>
  <tr>
    <td height=\"20\" class=\"NewsBody\"><strong>".$new_c['date']."</strong></td>
  </tr>
  <tr>";
 
		if(isset($new_c['image'])){
			$size = RedimensionaImagem("images/".$new_c['image']);
			echo "<td><img style=\"margin-left: 5px; margin-right: 5px;\"  src=\"images/".$new_c['image']."\" align=\"".$new_c['align']."\" width=\"".$size['largura']."\" height=\"".$size['altura']."\" class=\"images\" ><span class=\"NewsBody\">".$new_c['completo']."</span></td>";
		}else 
 			   echo "<td><span class=\"NewsBody\">".$new_c['completo']."</span></td>";
 
  echo "</tr>
</table>
<p>&nbsp;</p>";
 
		}else{
			echo "ERROR, no new found with id ".$_GET['id'];	
		}
 
 
	}else{
		echo "<table width=\"100%\" border=\"0\"><tr> <td valign=\"top\">";
 
		$all_news = ReadNews();
 
		if($all_news != 0){
			 for($i = sizeof($all_news)-1; $i >= 0; $i--){
			echo "<table width=\"100%\" border=\"0\" align=\"center\">
      <tr>
        <td height=\"15\" class=\"NewsTitle\"><strong><a href=\"?view&id=".$i."\">".$all_news[$i]['title']."</a></strong></td>
      </tr>
      <tr>
        <td height=\"15\" class=\"NewsBody\"><strong>".$all_news[$i]['date']."</strong></td>
      </tr>
      <tr>
        <td class=\"NewsBody\">".$all_news[$i]['resumo']."</td>
      </tr>
    </table><p>&nbsp;</p>";
			 }
 
		}else{
 
			echo "No news yet";
 
		}	
		echo "</td></tr></table>";
	}
 
	function RedimensionaImagem($imagem){  
		if(is_file($imagem)){     
	    $size = getimagesize($imagem);
	     $height = $size[1];
	     $width = $size[0];
	     $tamanho = array();
 
		 if ($height > __MAXHEIGHT)
	          {
	               $height = __MAXHEIGHT;
	               $percent = ($size[1] / $height);
	               $width = ($size[0] / $percent);
	          }
			   else if ($width > __MAXWIDTH)
	          {
	               $width = __MAXWIDTH;
	               $percent = ($size[0] / $width);
	               $height = ($size[1] / $percent);
	          }
 
	     $tamanho['largura'] = (int) $width;
	     $tamanho['altura'] = (int) $height;
		}
		return $tamanho;
	}
 
?>
[PHP] pobierz, plaintext 

[Blame]

Linijka 102:

[PHP] pobierz, plaintext 
for($i = sizeof($all_news)-1; $i >= 0; $i--){
[PHP] pobierz, plaintext 

Musisz tak zmodyfikować tą linijkę, aby pętla wykonała się tylko 3 razy (IMG:style_emoticons/default/smile.gif)
Podpowiedzi:
1. for
2. $all_news ci w tej linijce nie potrzebne (IMG:style_emoticons/default/winksmiley.jpg)

[fraksipon]

Czy moglbys napisać mi jak ta linijka ma wygladac?
Ja użyłem

[PHP] pobierz, plaintext 
 for($i = 2; $i >= 0; $i--){
[PHP] pobierz, plaintext 

ale to mi pokazuje pierwsze 3 wiadomosci.
Prosze o napisanie gotowego dzialania, gdyz nie znam sie na programowaniu.
Dziekuje

[phpion]

Daj sobie w takim razie przed tą pętlą:

[PHP] pobierz, plaintext 
$all_news = array_reverse($all_news);
[PHP] pobierz, plaintext 

i powinno pomóc.

Aha - wtedy wystarczy taka pętla:

[PHP] pobierz, plaintext 
for($i=0; $i<3; $i++)
[PHP] pobierz, plaintext 

Ten post edytował phpion 13.01.2010, 10:05:47

[fraksipon]

Posortował dobrze, ale niepełnie.
tzn.
w kolejności 3 ostatnie wiadomości po rozwinięciu pokazują 3 pierwsze wiadomości.

Jak to naprawić?

[Blame]

Rozwinięciu? Nie sortuj nic, napisz tak jak poradził phpion za drugim razem czyli:

[PHP] pobierz, plaintext 
for($i=0; $i<3; $i++)
[PHP] pobierz, plaintext 

Jak dalej nie działa to opisz bardziej szczegółowo bo nie wiem o co ci chodzi.

[fraksipon]

Już wyjaśniam.
Po zastosowaniu kodu phpion`a

[PHP] pobierz, plaintext 
		echo "<table width=\"100%\" border=\"0\"><tr> <td valign=\"top\">";
 
		$all_news = ReadNews();
 
		$all_news = array_reverse($all_news);
		if($all_news != 0){
	            for($i=0; $i<3; $i++){
		echo "<table width=\"100%\" border=\"0\" align=\"center\">
[PHP] pobierz, plaintext 

Jest to skrypt news`ow
Skrypt pokazuje mi wiadomości od końca tak jak chciałem.
Tylko ten sam skrypt umożliwia rozwinięcie wiadomości gdy ta jest za długa.

Problem polega na tym ze skrypt wyświetla mi ostatnią wiadomość a po rozwinięciu jej zawartość jest z pierwszej. Analogicznie z kolejymi wiadomościami. Wyświetla przedostatnią a po rozwinięciu drugą.

Jeśli niezrozumiale napisałem proszę pisać.
Co w takim razie trzeba zrobić i jak?

[Blame]

Sory, nie doczytałem się. Możesz zamiast tego:

[PHP] pobierz, plaintext 
$all_news = array_reverse($all_news);
[PHP] pobierz, plaintext 

po linijce 34 dać:

[PHP] pobierz, plaintext 
$list = array_reverse($list);
[PHP] pobierz, plaintext 

Bo jeśli dobrze rozumiem to funkcja ExplodeNews zwraca tablicę z newsami dla funkcji wyświetlających więc powinno działać.

Ten post edytował Blame 13.01.2010, 18:18:24

[fraksipon]

Zastosowałem i działa nawet po rozwinięciu news`a.
Tylko że pokazuje 3 pierwsze a nie 3 ostatnie news`y (IMG:style_emoticons/default/sad.gif)

[Blame]

To już zagmatwałeś całkiem, najpierw piszesz, że wyświetla dobrze a potem że źle. Jak wyświetla trzy pierwsze to usuń w ogóle tą linijkę z array_reverse

[fraksipon]

No nie chodzi tak jak powinien.
Czy mógłbym prosić abyś zobacz sam jak się kod zachowuje?
Tutaj link do skryptu surowego

[Blame]

Proszę, oto gotowy plik news.php, wystarczy podmienić: http://wrzucacz.pl/file/4051263452756

Problem był w tej właśnie pętli for nad którą się głowiliśmy, powinno to wyglądać tak:

[PHP] pobierz, plaintext 
                $all_news = ReadNews();
 
                $all_news = array_reverse($all_news);
 
                $ile_news = count($all_news);
 
                $ile = $ile_news - 3;
 
		if($all_news != 0){
			 for($i = $ile_news; $i >= $ile ; $i--){
[PHP] pobierz, plaintext 

Bo chociaż dobrze się wyświetlały to id było przepisywane na odwrót.

Ten post edytował Blame 14.01.2010, 08:09:29

[phpion]

@Blame:
A nie prościej (w celu zachowania kluczy) dodać TRUE jako drugi parametr do array_reverse?

[Blame]

Masz rację. Nie jestem jeszcze po prostu na tyle dobry, żeby znać parametry każdej klasy. Miałem problem i go rozwiązałem, ale jak widać można prościej.

[phpion]

Masz rację. Nie jestem jeszcze po prostu na tyle dobry, żeby znać parametry każdej klasy.

Ja również nie znam parametrów wszystkich funkcji, ale od tego jest dokumentacja by z niej korzystać. Zawsze najpierw warto zapoznać się z możliwościami danej funkcji by nie wymyślać koła na nowo (IMG:style_emoticons/default/winksmiley.jpg)

[Blame]

Wiem, wiem, przyznaję się do błędu. Na swoje usprawiedliwienie mam tylko edycję skryptu w godzinach porannych i na szybko (IMG:style_emoticons/default/tongue.gif)

[fraksipon]

Proszę, oto gotowy plik news.php, wystarczy podmienić: http://wrzucacz.pl/file/4051263452756

Problem był w tej właśnie pętli for nad którą się głowiliśmy, powinno to wyglądać tak:

[PHP] pobierz, plaintext 
                $all_news = ReadNews();
 
                $all_news = array_reverse($all_news);
 
                $ile_news = count($all_news);
 
                $ile = $ile_news - 3;
 
		if($all_news != 0){
			 for($i = $ile_news; $i >= $ile ; $i--){
[PHP] pobierz, plaintext 

Bo chociaż dobrze się wyświetlały to id było przepisywane na odwrót.

Dzięki.
Czy da się zmodyfikować kod tak aby nie wyświetlała się pusta ramka nad wiadomością `test1`?
(IMG:http://img42.imageshack.us/img42/5636/kasuj2.jpg)
Tu chyba chodzi o kod

[PHP] pobierz, plaintext 
 for($i = $ile_news; $i >= $ile ; $i--){
[PHP] pobierz, plaintext