PHP Warning: Division by zero header_php_review.php

[cyberfolli]

I am getting this error in the logs?

PHP Warning: Division by zero in /var/www/vhosts/website.com/httpdocs/includes/modules/pages/product_info/header_php_review.php on line 87

  /* Average rating Calculation */
  $sizeOfAverage = count($reviewsAverageArray);
  for($i=0; $i < $sizeOfAverage; $i++) { 
    $averageRating += $reviewsAverageArray[$i];
  }
  $averageRating = round($averageRating/$sizeOfAverage); 

[mc12345678]

Suggest visiting the forum related to the plugin. There ought to be a check for the non-existence of reviews and an appropriate assignment if none have been provided. Perhaps if the average is 0 it knows that none have been provided and if so, then would modify to:

if ($sizeOfAverage>0) {
$averageRating = round($averageRating/$sizeOfAverage);
} else {
$averageRating = 0;
}

[swguy]

I would not compute an average if no data exists. The value only makes sense if $sizeOfAverage > 0.

[mc12345678]

I would not compute an average if no data exists. The value only makes sense if $sizeOfAverage > 0.

Generally would agree, also wouldn't program to have the possibility of a divide by zero... Thing is that without knowing how it deals with the condition otherwise, I chose to assgn it a value as it doesn't make sense inmost rating systems for the value of zero to be chosen by a customer... Typically would expect 1 to be theminmum and that a check for display would be if average is equal to or greater than 1 then show rating otherwise request first rating... But maybe that's just me...

[cyberfolli]

That Solved the Problem. I really appreciate the help. The Error was creating a ton of log files and now there are none.

[mc12345678]

That Solved the Problem. I really appreciate the help. The Error was creating a ton of log files and now there are none.

Please verify though that you don't have bad ratings on items that actually don't have a review yet provided. (This was the next step/test in case the code handled a zero differently than a non-assignment) and is one aspect to which I believe swguy was suggesting could appear as a problem.