Posts by Darsstar

Darsstar

Toestemming bevestigd

Darsstar

http://www.php.net/manual/en/language.oop5.basic.php
Lees daar eens iets over $this aub

Dan kunnen al die lelijke globals weg...

Darsstar

Nog een probleem: ik zie nergens "$this".

Oh, en waarom vang je die exception op?
Doe dat lekker ergens anders (niet binnen de class).
Bijvoorbeeld:

PHP

try
{
// Connect to MySQL
Connection::NewConnection( $sql_config['hostname'], $sql_config['database'], $sql_config['username'], $sql_config['password'] );
}
catch(Exception $e)
{
$message  = $e->getMessage();
echo <<<ENDHTML
<body bgcolor="#000000">




<center><table cellspacing='0' width='700' height='1%' style='border: 1px solid #444444; margin-bottom: 4px'  cellpadding='2' >
                        <tr>
                    <td height='35' width='700' background='' bgcolor='#333333' style='border-bottom: 1px solid #444444; text-align: left; padding-left:25px; vertical-align: middle;'>
                    <b><font color='#999999'>A MySQL has error occured!</b>
            </td>
            <TR>
                    <td bgcolor='#222222'  width='700' style='border-bottom:1px solid #444444;' height='25'>


                    <font color='#999999'>The website has failed to connect to the mysql server!<BR>
                    Error: $message<br><br>


                    Please inform a system administrator!</font>
            </td>
            
            
            </TR>




            </table></center>
ENDHTML;
exit();
}

Toon Meer

Oh, en globals zijn ook lelijk..

Doe mij een plezier (en hopelijk daarmee jezelf), en plaats de hele class zodat ik (en andere) je kunnen helpen met het verbeteren er van.

Darsstar

Ik zelf vind het erg fijn dat alles via een bestand binnen komt. (front controller)
OF je dat nou via GET, MultiViews of Mod_rewrite voor elkaar krijgt maakt me niet echt uit. (al is de laatste het mooist)
Je kan er zelfs voor zorgen dat je gewoon links blijven zoals ze nu zijn maar alles (behalve plaatjes, css en dergelijke wat in mappen staan die je mod_rewrite laat 'negeren') toch via index.php binnen komt.

Darsstar

Probeer dit eens:

PHP

<?php
include 'includes/config.php';
include 'includes/header.php';


if($core->leden->ingelogd == true)
{
    $BenodighedenQuery = $core->mysql->query("SELECT
                                            b.artikel_id,
                                            b.naam,
                                            COUNT(g.id) AS aantal
                                        FROM
                                            benodigheden b
                                        LEFT JOIN
                                            gekochte_spullen g
                                        ON
                                            (b.artikel_id = g.artikel_id)
                                        WHERE
                                            g.lid_id = ". (int) $core->leden->ledeninfo['eigen']['id']."
                                        GROUP BY
                                            g.artikel_id");
                                        


    ?>
    <h1>Benodighedenlijst</h1>
        <div id="contentgedeelte">
            <strong>U N I F O R M</strong>


            1. Drie effen werkgewaden (zwart)


            3. E&eacute;n paar beschermde handschoenen (drakenhuid of soortgelijk)
            4. E&eacute;n wintermantel (zwart, met zilveren speld)
        
        <small><strong>    N.B.: Alle kledingstukken moeten van naamlabels zijn voorzien</strong></small>


            <strong>V E R P L I C H T E &nbsp; L E C T U U R</strong>
            Alle leerlingen moeten in het bezit zijn van de volgende werken:


    <?php
    
    while($BenodighedenFetch = $core->mysql->fetch($BenodighedenQuery))
    {
        if ($BenodighedenFetch['aantal'] > 0)
        {
            $BenodighedenFetch['naam'] = '<strike>'.$BenodighedenFetch['naam'].'</strike>';
        }


        echo $BenodighedenFetch['naam'];
    }
         
                   ?>




            <strong>O V E R I G E &nbsp; BE N O D I G D H E D E N</strong>
            <?php
                echo $core->leden->ledeninfo['eigen']['toverstok'] == '' ? '1 toverstok' : '<s>1 toverstok</s>';
            ?>
            1 ketel (tin, standaardmaat 2)
            1 set glazen of kristallen flesjes
            1 telescoop
            1 set koperen weegschalen
            
            <strong><small>Leerlingen mogen tevens een uil of een kat of een pad meenemen.
            Ouders worden er aan herinnerd dat eerstejaars geen eigen bezemstelen mogen bezitten.</small></strong>
        </div>
        <div id="contentbot"></div>
        <?php        
        
    
}


include 'includes/footer.php';

Toon Meer

De paar dingen die je zelf zult moeten doen kan je vast wel zelf uitvinden.

Darsstar

http://nl2.php.net/microtime
Volgens die pagina is mictotime() een preciesere time()

Dus wat jessewillem zegt klopt niet...

Darsstar

Zouden we mogen weten waarom je dat wilt?

id userid amount

Dat lijkt mij beter namelijk...
Maar mogelijk kan het nog stukken beter als je ons verteld wat je er mee wilt doen.

Darsstar

HUH?

Ik wil dus duidelijk maken die die if's het getal 35E6 niet dekken...
Nu dat hopelijk duidelijk is.

waarom zou je nog controleren of $user['bank'] lager is dan 35E6?
het volgende werkt precies het zelfde als de code in de OP:

PHP

if ($user['bank'] >= 35000000)
{
    $percent    = 0.05;
    $per        = 5.0;
}
elseif ($user['bank'] >= 27000000)
{
    $percent    = 0.045;
    $per        = 4.5;
}
elseif ($user['bank'] >= 21000000)
{
    $percent    = 0.04;
    $per        = 4.0;
}
elseif ($user['bank'] >= 15000000)
{
    $percent    = 0.035;
    $per        = 3.5;
}
elseif ($user['bank'] >= 10000000)
{
    $percent    = 0.03;
    $per        = 3.0;
}
elseif ($user['bank'] >= 6000000)
{
    $percent    = 0.025;
    $per        = 2.5;
}
elseif ($user['bank'] >= 3000000)
{
    $percent    = 0.02;
    $per        = 2.0;
}
elseif ($user['bank'] >= 1000000)
{
    $percent    = 0.015;
    $per        = 1.5;
}
elseif ($user['bank'] >= 100)
{
    $percent    = 0.01;
    $per        = 1.0;
}
else
{
    $per        = 0;
}

Toon Meer

MINDER CODE!!!
Doel bereikt!

Maar het kan korter...
$percent en $per lopen nu af...
Dus als je een loop maakt die door een array loopt die de waardes uit de if bevat, dan haal je telkens iets van beide variabele af als het lager is dan de waarde.

Darsstar

PHP

if ($user['bank'] > 35000000)
{
    $percent    = 0.05;
    $per        = 5.0;
}
elseif ($user['bank'] >= 27000000 and $user['bank'] < 35000000)

Ik zie een is-groter-dan en een is-kleiner-dan controlle (voor het getal 35E6)...
Nogmaals: wat ls $user['bank'] precies 35E6 is?
De eerste keer dat je code poste stond er toch echt een is-groter-of-gelijk-aan controlle...

Darsstar

Kan het zijn dat je van subdomein verwisseld?
Dus dat je van http://www.bla.nl/blabla naar bla.nl/albalb gaat?
PHP zet de cookies voor een van beide, als je wilt dat het voor beide werkt zul je met [func]session_set_cookie_params[/func] het domein naar .bla.nl moeten veranderen.

Darsstar

Niet bepaald nee...

Wat als $user['bank'] nu precies 35E6 (35000000) is?

*gaat er van uit dat je van de eerste > een >= operator maakt*

Waarom staat de < 35E6 controle er nog?
Je weet dat je daar nooit aan zal komen als het groter of gelijk is aan 35E6...
Het zelfde geld voor alle andere controlles na de "and", die zijn tutaal nutteloos geworden...
Dus je code is verkort wanneer je die weg haalt...

*wacht nu op het aangepaste script in een edit of de volgende reactie*

Darsstar

Ja, dan hoef je alleen te controleren of het groter is dan x...
Dan is er dus geen kleiner dan y meer bij, waardoor je ifs kleiner worden...

Doe dat, en ik vertel het volgende idee...

Darsstar

Ik heb een paar ideeën...

Begin maar met de ifs in de andere volgorde te doen, dus van groot naar klein.

Darsstar

Citaat

en ga nu omdat ik eindelijk tijd heb beginne met OOP enz

Leer aub eerst functies maken (classes en methods (zo heten functies binnen classes) komen later pas)...
Met functes heb je namelijk al met de scope van variabele te maken.

Darsstar

Dit script:

PHP

<?php


	$teams = array('nederland', 'spanje');


	while (TRUE)
	{
		$nederland = max(0, mt_rand(mt_rand(-3, 6), mt_rand(-3, 6)));
		$spanje    = max(0, mt_rand(mt_rand(-3, 7), mt_rand(-3, 7)));


		$verlenging = (mt_rand(0, 100) + $nederland + $spanje) % 2;


		if ($nederland == $spanje)
		{
			shuffle($teams);


			$one = reset($teams) . '_penalties';
			$two = end($teams)   . '_penalties';


			$$one = $$two = 0;


			while ($$one == $$two)
			{
				for ($i = 5; $i > 0; $i--)
				{
					if ($$one + $i < $$two)
						break 2;
	
					$$one += mt_rand(0, 1);
	
					if ($$two + $i < $$one)
						break 2;
	
					$$two += mt_rand(0, 1);
				}
			}
		}


    	if (mt_rand(1, 3) == 2)
    		break;


		unset($nederland_penalties, $spanje_penalties);
	}
    
	echo $nederland . ' - ' . $spanje;


	if (isset($nederland_penalties, $spanje_penalties))
	{
		echo ' (' . ($nederland + $nederland_penalties) . 'p - ' . ($spanje + $spanje_penalties) . 'p)';
	}


	if ($verlenging)
	{
		echo ' (verlengingen)';
	}

Toon Meer

en met mijn ogen dicht op een willekeurig moment F5 los laten zegt mij dat het 4 - 1 wordt...

Darsstar

Ik gooi dit topic maar weer dicht, ik heb geen zin meer om alle reports die hier van daan kopen te behandelen...
Als een ander crew lid dit wel wilt, succes met het heropenen...

Darsstar

Citaat van V-Ronaldinho

octopus Kiest dat Nederland Wint maar als Nederland Super slecht speelt wie wint er dan Nederland of Spanje

Nederland, of het in dit universum of een van de oneindig veel andere parallelle universa is maakt me niet uit.
Doe nu eens gezellig, of het wordt een stuk ongezelliger voor jou...

Darsstar

http://www.php.net/manual/en/function.finfo-file.php
http://en.wikipedia.org/wiki/Internet_media_type

Darsstar

PHP

ini_set('session.use_trans_sid', 0);

resources:
http://www.php.net/manual/en/function.session-start.php
http://www.php.net/manual/en/ini.list.php
http://www.php.net/manual/en/function.ini-set.php

Darsstar

Citaat van gamecontrol
Oke bedankt. Ik heb nu in enkele script alles geplaatst. als ik nu dit in (bv. kladblok) plaats
PHP
<img src="&#106&#97&#118&#97&#115&#99&#114&#105&#112&#116&#58&#97&#108&#101&#114&#116&#40&#39&#88&#83&#83&#39&#41&#59">
krijg ik in het kladblok dit te zien
PHP
<img src=\\\"javascript:alert('XSS');\\\">
(geen popup maar alleen de tekst zelf)
Betekend dit dat mijn script nu goed is beveiligd?

Het zou handig zijn als die post begrijpelijk zou zijn.
"als ik nu dit in (bv. kladblok) plaats ... krijg ik in het kladblok dit te zien" is namelijk heel duidelijk.

PS. ik houd het trouwens niet voor onmogelijk dat de briljante coding van CP er voor zorgt dat je post zo *kuch* verbeterd *verbeterd* is...