Gemiddelde berekenen met weging.

L.Groot

Beste CP'ers,

Voor m'n school moest ik een overhoor script schrijven. (hier heb ik al vaker wat over gevraagd)
Dit is me nu helemaal gelukt.
Je kan nu bij een toets aanmaken de weging instellen (bv. 1, dus dan staat er 1 cijfer en bij 5 staan er 5 cijfers etc.)
Maar, hoe kan ik nu het gemakkelijkst daarmee het gemiddelde berekenen?

Ik heb nu onderstaand script voor het gemiddelde, maar dit gaat gewoon via MySQL, en daar kan ik dus nog geen weging instellen, terwijl ik dit wel wil.

PHP

<h2>Gemiddelden:</h2>
<table border="1" color="#66665E" cellspacing="0.1" cellpadding="3" width="50%">
<tr class="d0" align="center">
<td><b>Vak</b></td>
<td><b>Gemiddelde</b></td>
</tr>
<?php
$people = Array('Grieks','Latijns','Frans','Duits','Engels','Nederlands','Aardrijkskunde','Geschiedenis','Mythologie','Biologie','Wiskunde','Natuurkunde','Informatica','Scheikunde','Godsdienst','Beeldende Vorming','Economie');
for($i = 0; $i < sizeof($people); ++$i){
$result45 = mysql_query("SELECT AVG(cijfer) as cijfer FROM cijfers WHERE naam='".$get_userdata['username']."' && `vak`='".$people[$i]."'ORDER BY `vak`") or die(mysql_error());
$count2 = mysql_fetch_array($result45);
$countie = number_format($count2['cijfer'], 1, ',', '');
if($countie != '0,0'){
echo '<tr>';
echo '<td>'.$people[$i].'</td>';
if($countie < 5.5){ echo '<td><b><font color="red">'.$countie.'</font></b></td>'; } else{ echo '<td><b><font color="green">'.$countie.'</font></b></td>'; }
echo '</tr>';
}
}
?>
</table>

Toon Meer

Hoe krijg ik dit het kortst goed voor elkaar?
Hoop dat jullie het weten!

Lars Groot

NielsB

[offtopic]Ok vraag, welke school zit je?
En welk jaar?

Want ik zit op een webmaster opleiding in rotterdam.
En krijg bijv ook van die opdrachten wat ik moet maken
[/offtopic]

Ik heb je script ietsje korter gemaakt en makkelijker.
Verder laat ik het nu aan iemand anders over om het nog korter te krijgen.

PHP

<?php 
$people = array('Grieks','Latijns','Frans','Duits','Engels','Nederlands','Aardrijkskunde','Geschiedenis','Mythologie','Biologie','Wiskunde','Natuurkunde','Informatica','Scheikunde','Godsdienst','Beeldende Vorming','Economie'); 


for($i = 0; $i < sizeof($people); ++$i)
{ 
	$result45 = mysql_query("SELECT AVG(cijfer) as cijfer FROM cijfers WHERE naam='".$get_userdata['username']."' AND `vak`='".$people[$i]."'ORDER BY `vak`");
	$count2 = mysql_fetch_assoc($result45); 
	$countie = number_format($count2['cijfer'], 1, ',', '');
	
	if($countie != '0,0')
	{
		$kleur = $countie < 5.5 ? 'red' : 'green';
		?>
		<tr>
			<td><?php echo $people[$i]; ?></td>
			<td><strong><font color="<?php echo $kleur; ?>"><?php echo $countie; ?></font></strong></td>
		</tr>
		<?
	}
}
?>

Toon Meer

Volgens mij is het trouwens ook iets netter als je met een foreach gaat werken om alles uit te lezen, of heb ik het mis?

Ik ben me aan het vervelen, dus zit nu je code goed door te nemen.
Dit is al een verbetering! En je hebt een grote fout gemaakt met je num_format, weet ik wel zeker.

Dus ik wil graag jou standaardwaarde zien die jij mee geeft in je database ? Geef je 0.0 mee of 0,0 ?

Dit is de aangepaste:

PHP

<?php 
$talen = array('Grieks','Latijns','Frans','Duits','Engels','Nederlands','Aardrijkskunde','Geschiedenis','Mythologie','Biologie','Wiskunde','Natuurkunde','Informatica','Scheikunde','Godsdienst','Beeldende Vorming','Economie'); 


foreach($talen as $taal)
{ 
	$result45 = mysql_query("SELECT AVG(cijfer) as cijfer FROM cijfers WHERE naam='".$get_userdata['username']."' AND vak='".$taal."'ORDER BY vak ASC");
	$gegevens = mysql_fetch_assoc($result45); 
	$countie = number_format($gegevens['cijfer'], 1, ',', '');
	
	if($countie != '0,0')
	{
		$kleur = $countie < 5.5 ? 'red' : 'green';
		?>
		<tr>
			<td><?php echo $taal; ?></td>
			<td><strong><font color="<?php echo $kleur; ?>"><?php echo $countie; ?></font></strong></td>
		</tr>
		<?
	}
}
?>

Toon Meer

Darsstar

*gaat in op 'uitdaging' van NielsB*

PHP

<?php


$result45 = mysql_query("
SELECT
	vak,
	SUM(cijfer * weging) / SUM(weging) AS gemiddelde
FROM
	cijfers
WHERE
	naam='".$get_userdata['username']."'
GROUP BY
	vak
HAVING
	gemiddelde > 0
ORDER BY
	CASE vak
		WHEN 'Grieks' THEN 1
		WHEN 'Latijn' THEN 2
		WHEN 'Frans' THEN 3
		WHEN 'Duits' THEN 4
		WHEN 'Engels' THEN 5
		WHEN 'Nederlands' THEN 6
		WHEN 'Aardrijkskunde' THEN 7
		WHEN 'Geschiedenis' THEN 8
		WHEN 'Mythologie' THEN 9
		WHEN 'Biologie' THEN 10
		WHEN 'Wiskunde' THEN 11
		WHEN 'Natuurkunde' THEN 12
		WHEN 'Informatica' THEN 13
		WHEN 'Scheikunde' THEN 14
		WHEN 'Godsdienst' THEN 15
		WHEN 'Beeldende Vorming' THEN 16
		WHEN 'Economie' THEN 17
	END
ASC");


while($gegevens = mysql_fetch_assoc($result45))
{
	$kleur = floatval($gegevens['gemiddelde']) < 5.5 ? 'red' : 'green';
?>
        <tr>
            <td><?php echo $gegevens['vak']; ?></td>
            <td style="background-color: <?php echo $kleur; ?>; font-weight: bold;"><?php echo number_format($gegevens['gemiddelde'], 1, ',', ''); ?></td>
        </tr>
<?
}

Toon Meer

Het is korter wanneer je die query weer op een regel neer zet, maar minder overzichtelijk.
Verder heb ik er wat dingen in verwerkt die je op phpfreakz ook al zijn verteld.

NielsB

Ik geef toe je hebt jezelf weer lopen uitsloven.
Respect DOS! (L)

Maar die van mij is toch wat korter aangezien jij een grote query hebt :D!

Lars ik hoop dat je hier nog wat aan hebt
Als je meer van die opdrachten hebt (A), dan hoor ik het graag

MrMees

Woow, dat is echt de beste en meest ingewikkelde query die ik ooit heb gezien!
Ga zeker even kijken naar wat je nou gedaan hebt bij dat case gebeuren.
Dankje Dos:cheer:!

Als ik dit zelf had moeten doen, had ik eerder op dat van NielsB gekomen :).

EDIT: Verrekte handig die CASE END structuur!

L.Groot

Dankjullie wel, het is me gelukt

Ik heb net een MySQL5 & PHP 5 boek van de bieb geleend, ga die dus ff leren

Bedankt!

Lars

Nieuwe reactie samengevoegd met originele reactie op 06.03.10 22:38:07:
Ik heb nog een vraagje:
Hoe kan ik het beste beveiligen tegen XSS / JavaScripts? Ik heb het nu over de inputs, dus welke PHP functie kan ik hier het beste voor gebruiken? Ik dacht zelf aan htmlentities();, of toch beter htmlspecialchars();?

Hope you know!

Lars Groot

NielsB

Die case heeft dos gedaan om het leuker te maken
Je hebt die case helemala niet nodig eigenlijk, als je gewoon doet order by vak ASC

verder , htmlspecialchars is daar goed voor, maar ik denk dat daar iemand anders beter op in kan springen.

MrMees

Naar mijn weten is htmlspecialchars() voldoende inderdaad, maar dat kan je beter pas gebruiken als je het uit de database haalt.

Als je iets in de database doet kan je naar mijn weten het beste mysql_real_escape_string() gebruiken. Dit is voor strings.

Met getallen kan je het beste eerst controleren of het een getal is door middel van ctype_digit(), daarna kan je er eventueel nog (int) voor zetten. Zelf zet ik dit er niet voor omdat ik het dubbel op vindt.

Dit zijn de dingen die ik gebruik, verbeter alsjeblieft als ik iets fout zeg!

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

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

Van een pixelige afbeelding naar een strakke, moderne website

Gemiddelde berekenen met weging.

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

Gebruikers die dit topic bekijken