function addAds($n) {
for ($i=0;$i<=$n;$i++) {
while($row=mysql_fetch_array(mysql_query("SELECT * FROM users"))) {
$aut[]=$row['name'];
}
$author=$aut[rand(0,mysql_num_rows(mysql_query("SELECT * FROM users")))];
$name="pavadinimas".rand(0,3600);
$rnd=rand(0,1);
if($rnd==0) {
$type="siulo";
} else {
$type="iesko";
}
$text="tekstas".md5("tekstas".rand(0,8000));
$time=time()-rand(3600,86400);
$catid=rand(1,9);
switch ($catid) {
case 1:
$subid=rand(1,8);
break;
case 2:
$subid=rand(9,16);
break;
case 3:
$subid=rand(17,24);
break;
case 4:
$subid=rand(25,32);
break;
case 5:
$subid=rand(33,41);
break;
case 6:
$subid=rand(42,49);
break;
case 7:
$subid=rand(50,56);
break;
case 8:
$subid=rand(57,64);
break;
case 9:
$subid=rand(65,70);
break;
}
mysql_query("INSERT INTO advert(author,name,type,text,time,catid,subid) VALUES('$author','$name','$type','$text','$time','$catid','$subid')") or die(mysql_error());
}
echo "$n adverts successfully added.";
}
这个功能的问题在于它永远不会加载。正如我注意到的那样,我的while循环导致它。如果我评论它,一切都很好。它必须从我的数据库中获取随机用户并将其设置为变量$ author。
答案 0 :(得分:6)
问题是查询是在循环中,所以它每次都会运行(所以你每次都从头开始)。只需将mysql_query()部分移到while循环之前,然后将其存储在变量中:
$query = mysql_query("SELECT * FROM users");
while($row=mysql_fetch_array($query))
答案 1 :(得分:4)
您可以用一行代替这个巨型开关:
$subid = rand(($catid * 8) - 7, min($catid * 8, 70));
答案 2 :(得分:2)
每次迭代都会执行while
循环的条件并进行评估。因此,每次迭代都会调用mysql_query
,并且retunrs为true。
只需执行一次数据库查询并缓存结果:
function addAds($n) {
$result = mysql_query("SELECT * FROM users");
$aut = array();
while ($row = mysql_fetch_array($result)) {
$aut[]=$row['name'];
}
$rowCount = count($aut);
for ($i=0; $i<=$n; $i++) {
$author=$aut[rand(0,$rowCount)];
// …
mysql_query("INSERT INTO advert(author,name,type,text,time,catid,subid) VALUES('$author','$name','$type','$text','$time','$catid','$subid')") or die(mysql_error());
}
echo "$n adverts successfully added.";
}
答案 3 :(得分:2)
我也认为问题是你的功能太大而无法理解(很快)。您应该将它们缩小并使用像phpunit这样的单元测试框架进行测试。
答案 4 :(得分:1)
很多时候我不使用PHP,但我认为这个任务
$row=mysql_fetch_array(mysql_query("SELECT * FROM users"))
应始终返回true
,它会在每次迭代时反复执行查询..
答案 5 :(得分:0)
每次运行循环时都会启动一个新查询。