Code mogelijk korter maken

Superior

Goedenavond,

Ik zit al tijdje te brainstormen over een code hoe die korter gedaan kan worden.
Kom er alleen niet helemaal uit en weet niet wat ik hier het beste voor kan gebruiken.

PHP

<?php


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

Meer tonen

Weet iemand een functie en kan er ook een klein voorbeeld bij hoe je het ongeveer moet doen?

Alvast bedankt!

Mvg,
Superior

Darsstar

Ik heb een paar ideeën...

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

Superior

Darsstar
Hoe bedoel je van groot naar klein, maakt dit enig verschil uit?

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...

Superior

Hoop dat ik je correct begrepen heb, weet niet of dit is wat je bedoelde.

PHP

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

Meer tonen

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*

Superior

Snap echt niet wat je ermee bedoelt.......

Als het precies 35.000.000 is zal hij 5% geven, dit is getest.

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...

tcbhome

Die user['bank'] is gewoon kijken in de tabel(MYSQL) leden wie er zoveel geld geeft staan.

Alle denk ik dan toch maar volgens mij wel :p

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;
}

Meer tonen

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.

ICTscripters

Dé plek voor IT

Dé plek voor IT

[PROJECT] Gamekeuring.nl - Hét onafhankelijke keuringsplatform voor maffiagames & RPG's!

Ictscripters / Koen

🚀 WE ZOEKEN TESTERS! 🚀

Kweekrijk - van stekje tot Imperium

Crimora.nl

Programmeren geocht die samen met mij maffiaskrill.com

Gezocht mafiasnextkill

nieuwe game

[TE KOOP] CloutClubRPG.nl – Complete Maffia Rap & Street Clout RPG (Kant-en-klaar concept)

Straatbaas.eu – complete Nederlandstalige en Engelstalige online maffia-RPG

402 Nieuwe Domeinnamen Augustus 2026

Code mogelijk korter maken

Registratie

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