Moving Average

MovingAverage.swift

class MovingAverage {
  var samples: Array<Double>
  var sampleCount = 0
  var period = 5
  
  init(period: Int = 5) {
    self.period = period
    samples = Array<Double>()
  }
  
  var average: Double {
    let sum: Double = samples.reduce(0, combine: +)
    
    if period > samples.count {
      return sum / Double(samples.count)
    } else {
      return sum / Double(period)
    }
  }
  
  func addSample(value: Double) -> Double {
    var pos = Int(fmodf(Float(sampleCount++), Float(period)))
    
    if pos >= samples.count {
      samples.append(value)
    } else {
      samples[pos] = value
    }
    
    return average
  }
}

This is perfect. Thanks! FYI pos is never mutated and therefore should be declared with let instead of var

class MovingAverage {
var samples: Array
var sampleCount = 0
var period = 5

init(period: Int = 5) {
    self.period = period
    samples = Array<Double>()
}

var average: Double {
    let sum: Double = samples.reduce(0, +)
    
    if period > samples.count {
        return sum / Double(samples.count)
    } else {
        return sum / Double(period)
    }
}

func addSample(value: Double) -> Double {
    sampleCount = sampleCount + 1
    let pos = Int(fmodf(Float(sampleCount + 1), Float(period)))
    
    if pos >= samples.count {
        samples.append(value)
    } else {
        samples[pos] = value
    }
    
    return average
}

}

dont you think it is incorrect
it should look like this this