CAShapeLayer不会离开

2022-01-28 07:39:04 标签 iosswiftuibezierpathcashapelayer

我有一个类,添加一个进度视图到给定的UIView。我在集合视图的单元格中使用这个。在前几个单元格上看起来不错,但在滑动几次后,你可以看到应该在那里的模糊的标记。我将附上我的意思的截图。不知道为什么会这样?

这是cellForItemAt

guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: self.cellId, for: indexPath) as? TakeCollectionViewCell else {return UICollectionViewCell()}
let service = TakeProgressView()
let random = Double(indexPath.row) / 10
service.progress = random
service.addView(view: cell.takeProgressView, withParticipants: false)
return cell

takeProgressView是storyboard中实现的UIView

下面是添加视图的函数:

func addView(view: UIView, withParticipants: Bool){
        disagreePath.removeAllPoints()
        agreePath.removeAllPoints()
        path.removeAllPoints()
        circlePath.removeAllPoints()
        agreeLayer.removeFromSuperlayer()
        disagreeLayer.removeFromSuperlayer()
        backgroundFill.removeFromSuperlayer()
    
        let width:CGFloat = 270
        let height:CGFloat = 360
        
        let viewX = (view.frame.size.width - width) / 2
        let viewY = (view.frame.size.height - height) / 2
        
        subView = UIView(frame: CGRect(x: viewX - 20, y: viewY, width: width + 40, height: height))
        path = UIBezierPath(ovalIn: CGRect(x: 20, y: 0, width: width, height: height))
        
        let rect = CGRect(x: 60, y: 40, width: width - 80, height: height - 80)
        
        circlePath = UIBezierPath(ovalIn: rect)
        path.append(circlePath)
        path.usesEvenOddFillRule = true
        
                
        backgroundFill = CAShapeLayer()
        backgroundFill.path = path.cgPath
        backgroundFill.lineWidth = 40.0
        backgroundFill.fillRule = .evenOdd
        backgroundFill.fillColor = #colorLiteral(red: 0.9371728301, green: 0.9373074174, blue: 0.9371433854, alpha: 1)
        backgroundFill.opacity = 0.9
        
        disagreePath = UIBezierPath(ovalIn: CGRect(x: 30, y: 10, width: width - 20, height: height - 20))
        disagreeLayer = CAShapeLayer()
        disagreeLayer.path = disagreePath.cgPath
        disagreeLayer.fillColor = UIColor.clear.cgColor
        disagreeLayer.strokeColor = #colorLiteral(red: 1, green: 0.4117892087, blue: 0.4117360711, alpha: 1)
        disagreeLayer.strokeStart = 0.0
        disagreeLayer.strokeEnd = CGFloat(1 - progress)
        disagreeLayer.lineWidth = 20.0
        disagreeLayer.lineCap = .round
        
        backgroundFill.addSublayer(disagreeLayer)
        
        agreePath = UIBezierPath(ovalIn: CGRect(x: 50, y: 30, width: width - 60, height: height - 60))
        agreeLayer = CAShapeLayer()
        agreeLayer.path = agreePath.cgPath
        agreeLayer.fillColor = UIColor.clear.cgColor
        agreeLayer.strokeColor = UIColor.uStadium.primary.cgColor
        agreeLayer.strokeStart = CGFloat(1 - progress) + 0.02
        agreeLayer.strokeEnd = 0.98
        agreeLayer.lineWidth = 20.0
        agreeLayer.lineCap = .round
        
        
        backgroundFill.addSublayer(agreeLayer)
        subView.layer.addSublayer(backgroundFill)
        view.addSubview(subView)
    }

# # #(在细胞UITableView or UIColl或UICollectionView) ar)是重用。

你的代码耦合太多,你应该改变它的架构。

如果我理解其中的逻辑:

TakeCollectionViewCell应该有一个属性TakeProgressView:

class TakeCollectionViewCell: UICollectionViewCell {
    var takeProgress = TakeProgressView()
    ...
}

所以你的代码应该是:

guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: self.cellId, for: indexPath) as? TakeCollectionViewCell else {return UICollectionViewCell()}
let service = TakeProgressView()
let random = Double(indexPath.row) / 10
cell.takeProgress.progress = random
cell.takeProgress.addView(view: cell.takeProgressView, withParticipants: false)
return cell

在func addView(view: UIView withParticipants: Bool)中,你只使用view: frame calculation + addSuview:

...
let viewX = (view.frame.size.width - width) / 2
let viewY = (view.frame.size.height - height) / 2
...
view.addSubview(subView)
...

因此,让我们返回子视图,并给出框架(甚至是尺寸,因为这是你使用的):

func progressSubview(with size: CGSize, withParticipants: Bool) -> UIView {
    ...
    // let viewX = (view.frame.size.width - width) / 2
    let viewX = (size.width - width) / 2
    // let viewY = (view.frame.size.height - height) / 2
    let viewY = (size.height - height) / 2
    ...
    // view.addSubview(subView)
    return subview
    ...
}

让我们把这个子视图作为TakeCollectionViewCell的属性,让我们使用这个新方法。

class TakeCollectionViewCell: UICollectionViewCell {
    var subProgressView: UIView?
    ...
    func updateProgressView(with progress: Double, withParticipants: Bool) {
        takeProgress.progress = progress
        subProgressView = takeProgress.progressSubview(with: self.takeProgressView.size, withParticipants: withParticipants)
    }
}

现在让我们重写重用:

class TakeCollectionViewCell: UICollectionViewCell {
    override func prepareForReuse() {
        super.prepareForReuse()
        subProgressView?.removeFromSuperview()
    }
}

在cellForRow现在应该是:

guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: self.cellId, for: indexPath) as? TakeCollectionViewCell else {return UICollectionViewCell()}
let random = Double(indexPath.row) / 10
cell.updateProgressView(with: random, withParticipants: false)
return cell

额外的返工可能会完成,但这应该是一个开始。

阅读全文

▼ 版权说明

相关文章也很精彩
推荐内容
更多标签
相关热门
全站排行
随便看看

错说 cuoshuo.com —— 程序员的报错记录

部分内容根据CC版权协议转载;网站内容仅供参考,生产环境使用务必查阅官方文档

辽ICP备19011660号-5

×

扫码关注公众号:职场神器
发送: 1
获取永久解锁本站全部文章的验证码