[php/MYSQL]Header problem Opcje

[zaba12]

Witam mam problem z header mam taki poczatek kodu glownego indexu administracji oto kod

[PHP] pobierz, plaintext 
<? require("../config.php"); ob_start(); ?>
 
	<title><? echo $tytul_a; ?></title>
[PHP] pobierz, plaintext 

I niestety niedziala mi to oto bład:(

Kod

Warning: Cannot modify header information - headers already sent by (output started at D:\WebServ\httpd-users\cmsik\Administracja\index.php:4) in D:\WebServ\httpd-users\cmsik\Administracja\function\newsy.php on line 111

[TomASS]

A czy przed headerem nie dołączasz pliku ./function/newsy.php ? Bo prawdopodobnie to powoduje konflikt.

[zaba12]

plik jest nizej w

[PHP] pobierz, plaintext 
<?php
if ($cmd == 'news'){ include ("function/newsy.php"); newsy(); }
?>
[PHP] pobierz, plaintext 

Ten post edytował zaba12 11.04.2006, 13:43:06

[TomASS]

A czy mógłbym grzecznie poprosić o większą ilość kodu?

[zaba12]

Tak oczywiscie zapodam cały kod odrazu

[PHP] pobierz, plaintext 
<?php
//Newsy
 elseif ($cmd == '1') {
 
	echo 'Aktualnosci';
 
	include ("function/newsy.php");
 
	echo '<table align="center" width="100%"><tr><td height="20%" bgcolor="#F4F4F4">ID</td><td height="20%" bgcolor="#F4F4F4">Tytuł</td><td height="40%" bgcolor="#F4F4F4">Data</td><td height="20%" bgcolor="#F4F4F4">Dodał</td><td bgcolor="#F4F4F4">Inne</td></tr>';
 
	newsy();
 
	echo '</table>';
 
	echo '<table align="center" width="100%"><tr><td height="20%">ID</td><td height="20%">Tytuł</td><td height="40%">Data</td><td height="20%">Status</td><td height="20%">Dodał</td></tr>';
 
	newsy_zg();
 
	echo '</table>';
 
	dodaj_newsa();
 
} elseif ($cmd == 'd_ok') {
 
	include ("function/newsy.php");
 
	d_ok();
 
} elseif ($cmd == 'news_del') {
 
	include ("function/newsy.php");
 
	news_del();
 
} elseif ($cmd == 'e_kom_news') {
 
	include ("function/newsy.php");
 
	e_kom_news();
 
} elseif ($cmd == 'edit_n') {
 
	include ("function/newsy.php");
 
	edit_news();
 
} elseif ($cmd == 'edit_n_ok') {
 
	include ("function/newsy.php");
 
	edit_n_ok();
 
}
?>
[PHP] pobierz, plaintext 

Cała treść sie nie zmiesciła........

Ten post edytował zaba12 11.04.2006, 14:13:32

[TomASS]

Ja tutaj nie widzę żadnego headera, gdzie on jest i w jakim kontekście

[zaba12]

[PHP] pobierz, plaintext 
<?php
function edit_n_ok()
 
{
 
global $tytul,$text,$data,$zrodlo,$dodal,$id;
$text = nl2br($text);
 
$query = "UPDATE news SET tytul='$tytul', tresc='$text', zrodlo='$zrodlo' WHERE id=$id";
 
 $result = mysql_query($query);
 
header("location: index.php?cmd=1");
 
}
?>
[PHP] pobierz, plaintext 

oto chodzilo ? i on niedziala header ;/

[TomASS]

I nic wczesniej? Mysql nie wywala żadnego błedu?

[zaba12]

no wlasnie mysql nie wywala zadnego bledu all sie zmienia w bazie ale niedziala tylko ten header ;/ tylko w administracji a na www działa ;(

[Seth]

Blad, ktory podales oznacza tyle, ze przed wyslaniem headera() do przegladarki, cos wczesniej juz do niej zostalo wyslane.

Headera mozna uzyc tylko wtedy gdy zadne inne dane nie zostaly przeslane do klienta - czyli nic nie wypisujesz przez echo, print itd. itp.

Ten blad mogl tez pwostac przez to, ze wystapil jakis blad, zanim wyslales naglowek. A jak wiadomo bledy (w zaleznosci od konfiguracji) wysylaja na wyjscie (do przegladarki) komunikat - a to wystarczy aby headery nie dzialaly.

Jezeli zamieszalem to pytaj, ogolnie chodzi o to, ze header _nie moze_ byc wyslany po jakichkolwiek danych wyslanych do klienta.

[phobos]

masz takie cos

[PHP] pobierz, plaintext 
<?php
elseif ($cmd == 'edit_n_ok') {
 
	include ("function/newsy.php");
 
	edit_n_ok();
?>
[PHP] pobierz, plaintext 

a w funkcji masz header()
tylko ze przed funkcja masz inculde i temu masz blad
spróbuj zamienic

[PHP] pobierz, plaintext 
<?php
header("location: index.php?cmd=1");
?>
[PHP] pobierz, plaintext 

na takie cos

[PHP] pobierz, plaintext 
<?php
 
echo "<script>self.location.href=\" index.php?cmd=1\";</script>";
 
?>
[PHP] pobierz, plaintext 

powinno dzialac

[060156]

Aby unikacj takich problemow,
mo stosowac grupe funkcji zaczynajacych
sie od prefixu ob_... poszukaj
w podreczniku php sa bardzo pomocne

[zaba12]

a w funkcji masz header()
tylko ze przed funkcja masz inculde i temu masz blad
spróbuj zamienic

[PHP] pobierz, plaintext 
<?php
header("location: index.php?cmd=1");
?>
[PHP] pobierz, plaintext 

na takie cos

[PHP] pobierz, plaintext 
<?php
 
echo "<script>self.location.href=\" index.php?cmd=1\";</script>";
 
?>
[PHP] pobierz, plaintext 

powinno dzialac

To mi pomogło wielkie dzięki.

[TomASS]

[PHP] pobierz, plaintext 
<?php
echo "<script>self.location.href=\" index.php?cmd=1\";</script>";
?>
[PHP] pobierz, plaintext 

Ale to nie rozwiązuje problemu do końca!
Co będzie jeśli ktoś ma wyłączony JS?
Jaki jest cel w wysyłaniu czegoś do przeglądarki i zaraz po tym wywołanie header.... ? Tak nie powinno się robić i używanie self.location to zły nawyk.

[phobos]

wiem napisalem mu ze ma inlude przed header i dlatego ma blad
ALE popatrz sobie jaki on plik includuje "functions/news.php" tak wiec w tym pliku sa funkcje do ktorych sie odwoluje
i teraz jezeli include da za wywolaniem funkcji to bedzie mial komunikat "call to ....."
a jak da za inculde to ma header error tak wiec js w tym przypadku jest dobre bo on tak to potrzebuje tylko do przekierowanie na inna stronke po dodaniu newsa

[TomASS]

Wysztko się zgadza, tylko po co wywołuje jakąś funkcję z pliku (lub cały plik) która coś wyświetla, jak zaraz nastąpi przekierowanie? Co będzie jak ktoś ma wyłączony JS? (ponawiam swoje pytania)

To tylko 'kamufluje', zasłania problem.
To tak jaby pozbywać się Notice za pomocą konfiguracji Apache.
Dalej uważam, że powinno poprawić się skrypt a nie używać JS.
Wybór należy do autora wątku...

[phobos]

odpowiedz : jak ktos ma wylaczone js to skrypt js nie zadziala :-)
autor widac woli sobie wkleic plik z funkcjami niz "zasmiecac" sobie nimi plik ktory wykonuje okrelsona akcje w zalerznosci od argumentu "cmd"
teraz pomoglem mu tak a za jakis czas autor jak spojrzy sobie na ten skrypt to bedzie sie sam smial z siebie ze pisal jak noob i poprawi tak wszystko ze bedzie on zajmowal polowe mniej wiersz , ale to za jakis czas a narazie dopiero zdobywa doswiadczenie i dla niego to co teraz zrobil jest na 5, pozniej pewnie usunie ten skrypt js i znowu zastapi go header() i bedzie mu wszystko gralo

Ten post edytował phobos 11.04.2006, 23:07:59

[TomASS]

narazie dopiero zdobywa doswiadczenie

Ale właśnie chciałbym kolegę nauczyć jak poprawnie i z dobrymi nawykami pisać

EOT z mojej strony

[phobos]

jakbysmy chcieli nauczyc go dobrze pisac kod to nie powinno sie mu wpierw wypomniec to ze robi :

[PHP] pobierz, plaintext 
<?php
elseif(....)
{
inculde(...)
funkcja();
}
elseif(....)
{
inculde(...)
funkcja();
}
?>
[PHP] pobierz, plaintext 

jak moze wczytac plik z funkcjami tylko raz na poczatku, a zamiast elseif ..... zastosowac switch()

EOT z mojej strony rowniez

Ten post edytował phobos 12.04.2006, 01:01:25

[bim2]

Mhh widze ze kolega ma na gorze "enter" ja mialem problemy z header takie ze przed php nie moze byc zadnych liter spacji itp., a echo przed header set_cookie sesion dziala a jak dasz przed <?php spacje to nie dziala sprobuj bez entera ;P