Header already sent

Nillas

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

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

T.Nijborg

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

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

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

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?

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

PHP

session_start();


ob_start();

Malik

@Sukel dat noem je niet oplossen maar verstoppen!

Darsstar

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

djordyh

Citaat van Malik

@Sukel dat noem je niet oplossen maar verstoppen!

Bij mij staat hij ook onderaan en hij werkt gewoon?

Nillas

En dan geen ob_flush()

Malik

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

@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

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

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

Tim

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

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

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

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

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

Nillas

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.

ICTscripters

Dé plek voor IT

Dé plek voor IT

Beta-testers gezocht voor Crypto-oefenplatform

Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

Developer Gezocht

Op zoek naar de legends

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

350 Nieuwe Domeinnamen Januari 2026

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

Header already sent

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

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

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