看看这段代码回出现什么问题，如何优化，谢谢大家，来者有分

[Alisa]

<?php
class cartstore{
var $ownerid;
function cartstore($ownerid){
$this->ownerid = $ownerid;
}
function add($extendobj){
$a = new avatar_cart_store();
$a->set('componentid', $extendobj->componentid);
$a->set('ownerid', $this->ownerid);
$a->set('flag', $extendobj->isinuse);
$n = $a->selectby();
if (count($n) > 0){
$n[0]->set('count', $n[0]->get('count') + 1);
return $n[0]->update();
}else{
$a->set('count', 1);
return $a->insert();
}
}
function del($id){
$a = new avatar_cart_store();
$a->set('id', $id);
return $a->delete();
}
function emptycartstore($ownerid, $typeid){
$a = new avatar_cart_store();
$a->set('typeid', $typeid);
$a->set('ownerid', $ownerid);
$a->deleteby();
}
function changetype($ownerid, $otypeid, $ntypeid){
$a = new avatar_cart_store();
$a->set('ownerid', $ownerid);
$a->set('typeid', $otypeid);
$n = $a->selectby();
if (count($n) == 0){
return false;
}else{
foreach ($n as $row){
$temp = new avatar_cart_store();
$temp->set('ownerid', $ownerid);
$temp->set('componentid', $row->get('componentid'));
$temp->set('typeid', $ntypeid);
$k = $temp->selectby();
if (count($k) > 0){
$k[0]->set('count', $k[0]->get('count') + $row->get('count'));
$k[0]->update();
$row->delete();
}else{
$row->set('typeid', $ntypeid);
$row->update();
}
}
}
}
function getall($ownerid, $typeid){
$a = new avatar_cart_store();
$a->set('ownerid', $ownerid);
$a->set('typeid', $typeid);
$array = $a->selectby();
$ra = array();
foreach ($array as $row){
$c = new cbme_component();
$c->set('id', $row->get('componentid'));
$c = $c->select();
$o = new extendobj();
$o->id = $row->get('id');
$o->componentid = $row->get('componenetid');
$o->count = $row->get('count');
$o->name = $c->get('name');
$o->sexid = $c->get('sexid');
$o->price1 = $c->get('price1');
$o->price2 = $c->get('price2');
$o->jpglocation = $c->get('jpglocation');
$ra[] = $o;
}
return $ra;
}
}
?>

[wen8u8]

这个购物类自己写的吗?

[Alisa]

不是的，想找高手帮忙看看

[xwsn]

有请接分

[SysTem128]

除了命名有待规范以外~~其他的没什么……
为什么要改这个？

[Alisa]

顶.....

[saga104]

sfsa

[caotian2000]

太长了,真不喜欢看

[idealabc]

顶

[caotian2000]

jf

[kenesyu]

看不懂帮顶

[aminconi]

1.$a = new avatar_cart_store();可能会报错,需要include_once包含avatar_cart_store的这个class
2.$a可以作为全局变量,每次都new,太浪费
3.需要给全局变量$ownerid设置互斥量,并且建立$a和$ownerid之间的关联,多个Client同时访问这个class的时候,会变更这些值

[anshenghao]

确实有点长~~~但是帮顶 JF

[Alisa]

我在顶.....

[PHP_CHRIS_BT1984]

变量名最好不要a,c,o...其他挺好的

[hzg7361]

晕,下次写代码加点注释.

[Alisa]

感谢SysTem128、aminconi和各位的关注。
再问aminconi：多个client同时访问的时候该如何处理呢，是在数据库方面处理还是在程序这边处理。谢谢