Hele simpele vraag

[bron]

Regel 16 en 17 veranderen, [0-5] wordt [1-5], het wordt dan

    // als adv format: cijfer [1-9], één of meer [0-9], cijfer [1-5]
    if (preg_match('/^[1-9][0-9]+[1-5]$/', $adv) == true) {

 

[pewie37]

Volgens mij is het nu helemaal toppie.
Dit was volgens mij de meest moeilijke "Hele simpele vraag" in de historie van helpmij.nl

hartelijk dank voor je perfecte hulp :thumb:

 

[pewie37]

Heb iets kleins moeten aanpassen.
Er zit nog 1 schoonheidsfoutje in, alleen weet ik niet of dat door mijn aanpassing komt?

Uitleg:
Ik heb na overleg met de groothandel gewoon de inkoopprijs met een factor 1.6 berekend.
Daarvoor heb ik onderstaande code vervangen met een regel zonder if-statement: "$adv = ($ink * 1.6);"
(Want er hoeft niet meer naar de adviesprijs gekeken)

if (empty($adv)) {
// bereken adv met ink formule
$adv = 2 - ($ink * 0.000107142857);
}​

Situatie:
Er zijn toch nog prijzen waarvan het "n"-getal (in 36n.00) wat uit de berekening komt op 0 komt te staan
225 Word uiteindelijk 360 terwijl ik deze eigenlijk op 5 had willen hebben staan.

 

[kenikavanbis]

...

 ...
  $ink = (float)$inkkoopprijs;
...

als je even terug kijkt dan zie je bovenstaande staan en vermoedelijk zal dit het probleem zijn.

 

[pewie37]

Float functie voor inkkoopprijs was niet de oorzaak (resultaat hetzelfde)

 

[bron]

Regel 1 t/m 11 (van code in #19) vervangen door deze code

// functie aanroepen met:  bereken_adviesprijs($inkkoopprijs)
function bereken_adviesprijs ($inkkoopprijs) {
  // maak van inkkoopprijs een getal (in geval het een tekst is)
  $ink = (float)$inkkoopprijs;
  // adviesprijs berekenen, afronden op 2 cijfers achter de komma
  $adv = round($ink * 1.6, 2);

In regel 16/17 is [0-5] veranderd in [1-5]
De functie heb ik een betere naam gegeven (zie code regel 1 hierboven).

Aanroepen met : bereken_adviesprijs($inkkoopprijs)

 

[pewie37]

Code aangepast maar 225 word nog steeds 360
zie afbeelding

 

[bron]

Klopt als een bus. inkoop 225 x 1,6 = 360

 

[pewie37]

Klopt als een bus. inkoop 225 x 1,6 = 360

 

[pewie37]

n Zou toch een 5 of een 9 worden?

 

[bron]

Zie onderste regel van #19 en jouw antwoord bij #20.
Als je toch alles op 5 of 9 wilt hebben dan hoor ik het wel.

Aanvulling
Gelijk maar oplossing erbij gezet

    // als adv format: cijfer [1-9], één of meer [0-9], laatste cijfer [0] of [6-9]
    } elseif (preg_match('/^[1-9][0-9]+(0|[6-9])$/', $adv) == true) {
      // dan afronden op 10-tal en 1 er vanaf halen (bijv. 137 > 140 > 139)
      $adv = round($adv, -1, PHP_ROUND_HALF_DOWN) - 1;
    }

adviesprijs 370.00 t/m 375.99 wordt 375.00
adviesprijs 376.00 t/m 379,99 wordt 379.00

 

[pewie37]

Dat antwoord was inderdaad niet goed bij nagedacht
excuseer

 

[pewie37]

Vraagstuk opgelost.
Dank jullie wel :thumb: