Header already sent

Nillas · 5 Januari 2012

Beste Allemaal,

Ik heb een probleempje in mijn script. Ik krijg namelijk deze fout te zien:

Citaat

Warning: Cannot modify header information - headers already sent by (output started at /home/proaja1b/public_html/header.php:5) in /home/proaja1b/public_html/header.php on line 88

Hier bevind zich de fout:

PHP

<?
            if (isset($_POST['knop'])) {
                $sQuery = 'SELECT id FROM leden WHERE gebruikersnaam = "' . mysql_escape_string($_POST['gebruikersnaam']) . '" AND wachtwoord = "' . md5($_POST['wachtwoord']) . '"';
                $iResult = mysql_query($sQuery);
                if (mysql_num_rows($iResult) > 0) {
                    $aObject = mysql_fetch_object($iResult);
                    $_SESSION['id'] = $aObject->id;
                    setcookie('id', $aObject->id, time() + 3600 * 24 * 100, '/');
                }else{
                    echo '<p>Gebruikersnaam/wachtwoord verkeerd.</p>';
                }
            }
            ?>

Toon Meer

Regel 88:

PHP

setcookie('id', $aObject->id, time() + 3600 * 24 * 100, '/');

Nu heb ik al geprobeerd om dit aan de bovenkant van de pagina toe te voegen:

PHP

<?php
ini_set("output_buffering", "1"); 
?>

Maar met nog weinig succes.

Iemand enig idee?

Alvast bedankt!

Malik · 5 Januari 2012

Je hebt een spatie na je <? haal die is weg...

T.Nijborg · 5 Januari 2012

Bij de sluit tag maakt het niet uit, bij begin tag wel, header already sent is dat er html staat voordat de sessie gestart word. Dus kijk of er html voorstaat?

Darsstar · 5 Januari 2012

Spaties binnen php tags maken niet uit...

Citaat

output started at /home/proaja1b/public_html/header.php:5

Aan regel 5 hadden we blijkbaar meer gehad.

output_buffering kan niet via ini_set() veranderd worden, wel via de ini files of .htaccess. Binnen php kan je ob_start() gebruiken.

Nillas · 5 Januari 2012

PHP

<?php
include 'config.php';
?>

Dat is regel 5 van de header.php.

Darsstar;

Kun je misschien uitleggen hoe ik de ob_start() en ob_flush() functie kan toepassen?

djordyh · 5 Januari 2012

Citaat van Nillas
PHP
<?php
include 'config.php';
?>
Dat is regel 5 van de header.php.

Darsstar;

Kun je misschien uitleggen hoe ik de ob_start() en ob_flush() functie kan toepassen?
Toon Meer

Waar je je session start hebt staan, zet de ob start daar onder, zo dus:

PHP

session_start();


ob_start();

Malik · 5 Januari 2012

@Sukel dat noem je niet oplossen maar verstoppen!

Darsstar · 5 Januari 2012

Liever zet je ob_start() helemaal boven aan in het script.

djordyh · 5 Januari 2012

Citaat van Malik

@Sukel dat noem je niet oplossen maar verstoppen!

Bij mij staat hij ook onderaan en hij werkt gewoon?

Nillas · 5 Januari 2012

En dan geen ob_flush()

Malik · 5 Januari 2012

Nillas

Na je <? heb je een spatie staan, die voor problemen kan opleveren. Probeer het is zo:

PHP

<?
            if (isset($_POST['knop'])) { 
                $sQuery = 'SELECT id FROM leden WHERE gebruikersnaam = "' . mysql_escape_string($_POST['gebruikersnaam']) . '" AND wachtwoord = "' . md5($_POST['wachtwoord']) . '"'; 
                $iResult = mysql_query($sQuery); 
                if (mysql_num_rows($iResult) > 0) { 
                    $aObject = mysql_fetch_object($iResult); 
                    $_SESSION['id'] = $aObject->id; 
                    setcookie('id', $aObject->id, time() + 3600 * 24 * 100, '/'); 
                }else{ 
                    echo '<p>Gebruikersnaam/wachtwoord verkeerd.</p>'; 
                } 
            } 
            ?>

Toon Meer

ob_start() zorgt er wel voor dat het werkt, maar niet op de nette manier .... Verstop het of los het op.

error_reporting('0');
error_reporting(E_ALL);

Darsstar · 5 Januari 2012

@Malik ik zei toch al dat spaties binnen PHP tags niets uithalen?

ob_start() is een hele nette manier ob headers already sent errors te voorkomen door een stuk output op te vangen en op te slaan in een variabele. ob_start() boven aan een script moeten plaatsen om die errors te voorkomen is slecht design. Aangezien criminals games over het algemeen slecht opgebouwd zijn heeft Nillas de volgende keuzes: heel de code base overhoop halen en verbeteren of iets aan het topje van de ijsberg toevoegen.

Zijn code base beter organiseren is natuurlijk het beste wat hij kan doen, maar aangezien er naar mijn idee een 98% kans is dat hij dat niet doet blijft ob_start() ergens bovenaan het script plaatsen de meest realistische optie.

Tim · 6 Januari 2012

Verplaats setcookie boven de output...?

Eigenlijk, verplaats alle functionele dingen voor de output? :\ Tenzij je dingen echt als laatste wilt doen of dingen die nooit echt van belang zijn.

Nillas · 6 Januari 2012

Ik heb geprobeerd het voor de output te zetten en ik heb ob_start neergezet. Geen van beide blijkt te helpen.

Tim · 6 Januari 2012

Citaat van Nillas

Ik heb geprobeerd het voor de output te zetten en ik heb ob_start neergezet. Geen van beide blijkt te helpen.

Niet goed geprobeerd
Op regel 5 in header.php is er dus al ouput.

Malik · 6 Januari 2012

zet ob_start(); bovenin op de 2e regel onder <?php

(desnoods in config, zolang hij bovenin staat en het werkt)

[offtopic]Maar het is niet de nette methode![/offtopic]

Nillas · 6 Januari 2012

PHP

<?php
ob_start();
?>
<?php
include 'config.php';
$iOptellen = 7161;


function getURL($iID, $sTitle) {
    $sTitle = preg_replace("/[^a-zA-Z0-9 -]/", "", $sTitle);
    $sTitle = strtolower($sTitle);
    $sTitle = str_replace(" ", '-', $sTitle);
    $sURL = 'nieuws/' . $iID . '/' . $sTitle;


    return $sURL;
}


if (isset($_COOKIE['id']) && !isset($_SESSION['id'])) $_SESSION['id'] = $_COOKIE['id'];


if (isset($_SESSION['id'])) {
    $sQuery999 = 'UPDATE leden SET ingelogd = ' . (time() + 3539 + 3600) . ' WHERE id = ' . $_SESSION['id'];
    $iResult999 = mysql_query($sQuery999);
}
?>




        <?php
        if (isset($_SESSION['id'])) {
        $sQuery1 = 'SELECT * FROM leden WHERE id = ' . $_SESSION['id'];
        $iResult1 = mysql_query($sQuery1);
        $aObject1 = mysql_fetch_object($iResult1);
		$data = $aObject1;
        ?>


  <?php
            if (isset($_POST['knop'])) {
                $sQuery = 'SELECT id FROM leden WHERE gebruikersnaam = "' . mysql_escape_string($_POST['gebruikersnaam']) . '" AND wachtwoord = "' . md5($_POST['wachtwoord']) . '"';
                $iResult = mysql_query($sQuery);
                if (mysql_num_rows($iResult) > 0) {
                    $aObject = mysql_fetch_object($iResult);
                    $_SESSION['id'] = $aObject->id;
                    setcookie('id', $aObject->id, time() + 3600 * 24 * 100, '/');
                }else{
                    echo '<p>Gebruikersnaam/wachtwoord verkeerd.</p>';
                }
            }
            ?>




<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>


	<meta http-equiv="Content-Type" content="text/html; charset=iso-8559-1" />
    <base href="http://www.proajax.nl/" />
	<link href="css/style.css" rel="stylesheet" type="text/css" />
	<script type="text/javascript" src="javascript/dddropdownpanel.js"></script>
	<title>proajax.nl - <?=(!isset($sTitle2)) ? 'Altijd het laatste nieuws over Ajax' : $sTitle2?></title>
	<link rel="alternate" title="Soccercentral.nl RSS" href="http://www.proajax.nl/rss/rss.php" type="application/rss+xml" />
        <link rel="shortcut icon" type="image/x-icon" href="favicon.ico">
	<!--[if IE 6]>
		<script type="text/javascript" src="javascript/dd_belatedpng.js"></script><script type="text/javascript">DD_belatedPNG.fix('h1 img, h2 img, #inloggentab img');</script>
		<link href="css/styleie.css" type="text/css" rel="stylesheet" />
	<![endif]-->


    <script type="text/javascript">
    var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");
    document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));
    </script>
    <script type="text/javascript">
    try {
    var pageTracker = _gat._getTracker("UA-3933867-3");
    pageTracker._trackPageview();
    } catch(err) {}</script>


</head>


<body>


	<div id="header" style="background:url(images/image-header.jpg) no-repeat top right;">


		<div class="left"><h1 title="Proajax.nl Altijd het laatste nieuws over Ajax!"><a href="http://www.proajax.nl/"><img src="images/h1-image.png" alt="Proajax.nl Altijd het laatste nieuws over Ajax!" /></a></h1></div>


		<div id="inloggen" class="ddpanel">


			<div id="inloggencontent" class="ddpanelcontent">
				<p id="ingelogd">
				<a href="instellingen">Instellingen &laquo;</a>
                <? if ($aObject1->rang != 'Gebruiker') echo '<a href="admin/">Adminpaneel &laquo;</a>'; ?><br />
				<a href="uitloggen">Uitloggen</a>
				</p>
			</div>
        
			<div id="inloggentab" class="ddpaneltab"><a href="#"><p>Welkom!</strong></p></a></div>
        <?
        }else{
        ?>
        	<div id="inloggencontent" class="ddpanelcontent">
          
				<form action="" method="post">


					<label for="gebruikersnaam"><img src="images/image-gebruikersnaam.png" alt="" /></label><input name="gebruikersnaam" id="gebruikersnaam" type="text" value="" />
					<label for="wachtwoord"><img src="images/image-wachtwoord.png" alt="" /></label><input name="wachtwoord" id="wachtwoord" type="password" value="" />
					<input name="knop" type="submit" class="submit" value="" />


					<p><a href="registreren">Registreren</a> | <a href="wachtwoordvergeten">Wachtwoord vergeten</a></p>


				</form>
			</div>


			<div id="inloggentab" class="ddpaneltab"><a href="#"><img src="images/image-inloggen.png" alt="" /></a></div>
        <?
        }
        ?>
		</div>


		<div id="kruimelpad">
			<p><strong><a href="index">Home</a></strong><?=$sTitle?></p>
			<form action="zoeken" method="post">
				<fieldset>
					<input type="text" class="search" name="zoek" />
					<input type="submit" class="submit" name="zoeken" value="" />
				</fieldset>
			</form>
		</div>


	</div>

Toon Meer

Zo doe ik het goed toch?

Malik · 6 Januari 2012

Zet hem in je config

Darsstar

Remember that header() must be called before any actual output is sent, either by normal HTML tags, blank lines in a file, or from PHP. It is a very common error to read code with include(), or require(), functions, or another file access function, and have spaces or empty lines that are output before header() is called.

Wil je dit dan aan mij uitleggen? Citaat van PHP.net

Tim · 6 Januari 2012

In config.php is er output. Simple as that.

Nillas · 6 Januari 2012

Hmm... Als ik hem in mijn config toevoeg dan werkt het ook niet. Lijkt me sterk dat er in mijn config.php iets verandert moet worden als dat niet wordt gezegd.

ProAjax.nl
Gebruikersnaam - Demo
Wachtwoord - demodemo

Kijk zelf maar is.

DNS update Directadmin

Graag advies voor mijn website

Ik wil op afstand studeren, kennen jullie Tech?

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

App bouwer gezocht

Dating software

250 Nieuwe domeinnamen April 2024

Xenforo buckaroo plugin

Header already sent

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen