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

Ictscripters Chat

Het Grote Vibe Code Topic

PWYL source gezocht

Help testers nodig voor android app Urgent

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

Afspraken systeem met planbeperking

Developer Gezocht

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

370 Nieuwe Domeinnamen April 2026

Snel een website nodig?

Sicarras.com - Moderne Mafia Text-Based RPG

Gemiddelde berekenen met weging.

Participate now!

Google Cloud is in de problemen (en AWS weet het)

Nebius kondigt bouw aan van een van Europa's grootste datacenters

Samsung zal miljoenen verdienen aan Apple iPhone Fold dankzij zijn 12GB RAM

Functioneel ontwerp

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

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