Hi, trying to fill a list, below is my code and result. Toit #help

Hi, trying to fill a list, below is my code and re...

Macca

04/04/2023, 6:27 AM

import gpio import gpio.adc show Adc SampleSize := 3 Sample := 0 SampleTotal := 0.0 adc := Adc (gpio.Pin 34) Readings := [] main: for i := 0; i < SampleSize; i += 1: Readings[i] = adc.get sleep --ms=1 //print Readings[i] Calculate_RMS sleep --ms=10000 Calculate_RMS: Readings.do: SampleTotal = SampleTotal + it.SampleTotal print SampleTotal print Sample Decoding by

jag

, device has version ****************************************************************************** EXCEPTION error. OUT_OF_BOUNDS 0: List_.[]= /core/collections.toit:1865:24 1: main adc_test.toit:14:17 ******************************************************************************

Macca

04/04/2023, 6:30 AM

line 14 is Readings[i] = adc.get

Macca

04/04/2023, 6:57 AM

After much trial & error I fixed it. My answer was:

Macca

04/04/2023, 6:57 AM

for i := 0;i < SampleSize - 1; i += 1: Reading = adc.get Readings.add Reading

kasperl

04/04/2023, 9:11 AM

i < SampleSize was correct.

kasperl

04/04/2023, 9:12 AM

... and an alternative to calling add (which grows the list) is to do

Readings := List SampleSize

before the loop.

kasperl

04/04/2023, 9:13 AM

You can even try

Readings := List SampleSize: adc.get

and drop the loop.

kasperl

04/04/2023, 9:13 AM

It calls the block passed to the list constructor SampleSize times.

kasperl

04/04/2023, 9:14 AM

You can sleep from within there too.

Macca

04/05/2023, 7:46 AM

Thanks @kasperl, yes I discovered that Readings kept growing, so cleared it after each "loop" I will try your suggestion.

Previous Next