am i a newbie? Voting script , operator?

HenryJohn

Im not that old in php programing but i still tryed the challange of making a simple voting script. My problem is that the values i have vritten to the text file which is storing the submitted votes doesnt seem to work in an operator.

code

olddata.txt

5 + 5 + 5 + 10 + 8

submitvote.php

//Add submitted value and add a "+" before.
$myfile = "olddata.txt";
$fh = fopen($myfile, 'a') or die ("CAn't open file");
$stringdata =" + ";
fwrite($fh, $stringdata);
$stringdata = "$optionboxvalue";
fwrite($fh, $stringdata);

$myfile = "olddata.txt";
$fh = fopen($myfile, 'r');
$thedata = fread($fh, 99999);

//Now the operation where we sum up the olddata.txt values and divede them and then echo ´them on the page.

$division = $thedata / 2;
echo "$division";

The result is that it echos only the first number in "olddata.txt" and divide it by two. I want it to sum up all the values in "olddata.txt" and then divide.
Did i miss somethin' in the operators class :queasy:

HenryJohn

OBS: FORGOT;

the first line in submitvote.php is

$optionboxvalue = $_POST['optionbox'];

maybe not very relevant cause this function works ;P but still

foid025

hmm... yes that's going to be a problem. the whole text file is being read as one big string, not a series of integers. That's why you can't add the different values to each other and then divide them. Try this:

$votesArray= explode(" + ", $thedata );
$sum = 0;
for($i=0;$i<length($votesArray);$i++)
{
   $sum  = $sum + (int)$votesArray[$i];
}
$division = $sum  / 2;
echo "$division";

foid025

oh and one more thing... you shouldn't do "$thedata = fread($fh, 99999);"

Bad programming... Might cause errors, that idk about. Do this instead:

"$thedata = fread($fh, filesize($myfile));"

HenryJohn

All i got was

Fatal error: Call to undefined function length() in D:\Program\wamp\www\voting\submitvote.php on line 19

Im not shure if i got it all right but i used it like this,

$optionboxvalue = $_POST['optionbox'];

//Append to file
$myfile = "olddata.txt";
$fh = fopen($myfile, 'a') or die("can't open file");
$stringdata = " + ";
fwrite($fh, $stringdata);
$stringdata = "$optionboxvalue";
fwrite($fh, $stringdata);

$myfile = "olddata.txt";
$fh = fopen($myfile, 'r');
$thedata = fread($fh, filesize($myfile));

//New Stuff
$votesArray= explode(" + ", $thedata );
$sum = 0;
for($i=0; $i<length($votesArray);$i++)
{
$sum = $sum + (int)$votesArray[$i];
}
$division = $sum / 2;
echo "$division";

Somethin' i missed ?

Thankz very much for the help btw, 🙂