Foutmelding in script

[Daan]

Als ik in me error log kijkt zie ik deze regel die steeds terug komt

 

[24-Mar-2014 10:48:40 Europe/Amsterdam] PHP Warning:  number_format() expects parameter 1 to be double, string given in /home/xxxxxxx/public_html/xxxxxx/includes/cms.class.php on line 136

 

Zou om deze regel gaan

 

$oca['kmstand'] = number_format(str_replace(array(",", "."), "", $oca['kmstand']), 0);

 

Kan 123 er niets vinden wat de oorzaak kan wezen 

 

verder werkt het script gewoon gaat me even alleen om de foutmelding wat er mis gaat.

 

[Psychosammie]

Nou, met mijn Delphi programmeerkennis kom ik al een heel eind.

 

De functie str_replace geeft een string als resultaat.

De functie number_format verwacht als eerste parameter een float

 

Je krijgt dus de waarschuwing dat je een string in een float probeert te stoppen.

Lijkt me sterk dat het script werkt.

24 maart 2014 aangepast door Psychosammie

[Daan]

Ik zal er van de week er even meer tijd aan besteden,waarom zou het script niet werken? het script draait verder zonder fouten op de webpagina hij laat alles zien.

[Psychosammie]

Ik weet niet wat $oca['kmstand'] voorstelt maar waarschijnlijk zit daar al een waarde in en gaat het per ongeluk goed.

[Big fellow]

Maak er eens van:

$oca['kmstand'] = number_format((double)str_replace(array(",", "."), "", $oca['kmstand']), 0);

 

Niet getest, maar zou moeten werken gok ik zo.  

 

Zo niet, gebruik dan:

$oca['kmstand'] = number_format(floatval(str_replace(array(",", ".")), "", $oca['kmstand']), 0);

[Psychosammie]

Ik gok eerder dat hij geen functie voor numbers moet gebruiken als hij al een string heeft en ook een string wil hebben.

 

Waarschijnlijk is str_replace(array(",", "."), "", $oca['kmstand']), 0) al voldoende?

 

Maag goed, ik heb geen kennis van PHP.

[Daan]

@Psychosammie met jou regel wou het script niet meer werken gelijk een blanco pagina 

 

@Big fellow met jou regel werkt het  geen errors meer  in de log thanks   

 

l $oca['kmstand'] = number_format((double)str_replace(array(",", "."), "", $oca['kmstand']), 0);

 

Iedereen nog bedankt voor het meedenken.

27 maart 2014 aangepast door Daan

[Psychosammie]

Was ook een kleine kans dat mijn oplossing zou werken. De oplossing van onze grote vriend geloof ik direct. Maar het blijft vreemd dat je een string hebt waarin je wat vervangt om daar dan een nummer van te maken om vervolgens dat nummer weer in een string om te zetten.

 

@Big fellow

Dat moet toch eenvoudiger kunnen?

[Big fellow]

Inderdaad zou de regel "$oca['kmstand'] = str_replace(array(",", "."), "", $oca['kmstand']);" ook moeten werken. Het enige wat er gebeurt is de kommas en punten verwijderen uit de string... En daarvoor is helemaal geen heen-en-weer conversie nodig...  Ik had in eerste instantie (toen ik het antwoord dus schreef) niet goed naar de functie gekeken, en niet echt daarover nagedacht...

Maar nu ken je de "truuk" van typecasting :-) ...

[Daan]

@Big Fellow

 

Alles werkt zoals het moet op een schoonheidsfoutje na  bijvoorbeeld de kmstand met een coma hij geeft  bijvoorbeeld een 40,000 dat zou moeten met een .

 

 

 

hier een stukje van het script

 

 

$q = $this->_link->query("SELECT o.*, t.type_name as type, m.merk_name as merk FROM occasions o LEFT JOIN types t ON o.type = t.type_id LEFT JOIN merken m ON o.merk = m.merk_id{$sold}{$brand} ORDER BY o.id DESC{$limit}");

if($this->_link->num_rows($q) > 0) {

$return = array();

while($oca = $this->_link->fetch_assoc($q)) {

if($oca['verkocht'] == 0) {

$oca['image'] = $this->getRandomImage($oca['id']);

} else {

$oca['image'] = array('img_thumb' => 'verkocht.png', 'img_full' => 'verkocht.png');

}

$oca['seo_titel'] = seo($oca['merk'].'-'.$oca['type'].'-'.$oca['bouwjaar'].'-'.$oca['motortype']);

$oca['prijs'] = number_format($oca['prijs'], 2, ",", ".");

$oca['kmstand'] = number_format((double)str_replace(array(",", "."), "", $oca['kmstand']), 0);

$return[] = $oca;

}

return $return;

} else {

return array();

}

}

 

}

 

?>

[Big fellow]

Verander de regel maar hiernaar;

$oca['kmstand'] = str_replace(",", ".", $oca['kmstand']);

[Daan]

Regel toegepast en is nu ok