代码出错

public function S(){
     if (!empty($_GET['id'])){
       $Urls=M('Urls');
       Load('extend');
      $time=strtotime(date('Y-m-d').' 00:00');
     if (!$Urls->where('urlid='.$_GET['id'].' && ip='.get_client_ip().' && time>'.$time)->count()){
       $dd['urlid']=$_GET['id'];
       $dd['ip']=get_client_ip();
       $dd['time']=time();
        $Urls->add($dd);
       $Url = M('Url');
       $Url->where('id='.$_GET['id'])->setField('time',time());
         redirect(__APP__);
    }else{
         redirect(__APP__);
     }
   }
}

请问班主，我的代码中 if (!$Urls->where('urlid='.$_GET['id'].' && ip='.get_client_ip().' && time>'.$time)->count()){ 哪里写错了，特别这样 ip='.get_client_ip().' 需要怎么写？

单看你这句
if (!$Urls->where('urlid='.$_GET['id'].' && ip='.get_client_ip().' && time>'.$time)->count()){ }
代码是没有错误的
但是我建议你以后这么写
$map['urlid']=$_GET['id'];
$map['id']=get_client_ip();
$map['time']=array('gt',$time);
if (!$Urls->where($map)->count()){ }
不知道你到底是哪里运行出了错误了哪？

你这样写 && 和 ||  要在哪里区别？
我测试过了 'ip=127.0.0.1' 就读不出数据   如果 ip="127.0.0.1" 就可以