My code for ADC button handling, it works, but can it be slimmed down?


Closed Thread
Results 1 to 28 of 28
  1. #1
    Join Date
    Feb 2013
    Posts
    1,078

    Default My code for ADC button handling, it works, but can it be slimmed down?

    Hello,
    This is my code for handling two buttons connected to ADC input.
    It works fine, and what was most important, it prevents repeated key action -
    user has to release the button and press it again, to call needed action again.
    However, there are quite a lot of variables, and their number will double with
    more buttons being added.
    So maybe there is a way to make it simpler, while still maintaining
    "do not repeat action" functionality?

    Code:
    menu: 'main loop for button handling
    adcin 1, adcval   'read keys
    if adcval=255 then n1=0: n2=0: left=0: right=0  'if no key, then reset both button variables
    if adcval<10 then left=left+1 'detect button presses
    if adcval>100 and adcval <130 then right=right+1 'detect button presses
    if left>100 and n1=0 then 'if button pressed enough and it was not pressed before
    left=0
    n1=1
    gosub action1 'button 1 tasks
    endif
    if right>100 and n2=0 then 'if button pressed enough and it was not pressed before
    right=0
    n2=1
    gosub action2 'button 2 tasks
    endif  
    pause 1 'input delay
    goto menu

  2. #2
    Join Date
    Apr 2014
    Location
    OK
    Posts
    557


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Code:
    adcin 1, adcval   'read keys
    SELECT CASE adcval
      CASE < 10 : left=left+1 'detect button presses
      CASE < 100
    @ NOP
      CASE < 130 : right=right+1 'detect button presses
    END SELECT
    You might try the SELECT CASE method of parsing through. The above snippet should give you an idea.[/FONT]

  3. #3
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Thanks!
    My main concern is number of variables needed...

  4. #4
    Join Date
    May 2013
    Location
    australia
    Posts
    2,383


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    code snippets do not provide any where near the required info to provide a meaningful answer, not even one var definition is even provided

    for a meaningful answer
    1 how often is routine called
    2 is it called on a regular timed basis
    3 how responsive does it need to be

    the minimum key data necessary would be a count to detect a debounced press a bit to indicate a new press detected and a bit to indicate
    the press has been acknowledged and acted on provided the right conditions exist

  5. #5
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    1. At least 10 times per second.
    2. Yes
    3. As responsive as any other device, where user presses key and expects immediate feedback.

  6. #6
    Join Date
    May 2013
    Location
    australia
    Posts
    2,383


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    1. At least 10 times per second
    if right>100
    so button has to held for ten seconds till you get a response ?

    As responsive as any other device
    not in my book, i doubt the thing is working after 300mS

    2. Yes
    like how regular how often really
    Warning I'm not a teacher

  7. #7
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    No.
    I mean keyboard is polled at least 10 times per second.
    This is general interactive device control, with direct feedback
    like your TV remote.

  8. #8
    Join Date
    May 2004
    Location
    NW France
    Posts
    3,611


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Why not use the genuine BUTTON command ?????????????????

    of course, needs some "brainwork" and practise - and of course at least " read that f...... manual " ... "

    but the result is worth the effort ...

    " Just my two cents " ...

    Alain
    ************************************************** ***********************
    Why insist on using 32 Bits when you're not even able to deal with the first 8 ones ??? ehhhhhh ...
    ************************************************** ***********************
    IF there is the word "Problem" in your question ...
    certainly the answer is " RTFM " or " RTFDataSheet " !!!
    *****************************************

  9. #9
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Read 1st post 1st

    This is ADC input.

  10. #10
    Join Date
    May 2004
    Location
    NW France
    Posts
    3,611


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Quote Originally Posted by CuriousOne View Post
    Read 1st post 1st

    This is ADC input.
    ADC for ONLY two buttons and " immediate response " ( sic ) ??? ...

    if it's your choice ... go on ! ( ) ...

    Alain
    ************************************************** ***********************
    Why insist on using 32 Bits when you're not even able to deal with the first 8 ones ??? ehhhhhh ...
    ************************************************** ***********************
    IF there is the word "Problem" in your question ...
    certainly the answer is " RTFM " or " RTFDataSheet " !!!
    *****************************************

  11. #11
    Join Date
    May 2013
    Location
    australia
    Posts
    2,383


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    adc key reading is a good tried and tested technique and can be very responsive, the question here is can it be done
    using less memory. the answer is yes of course, if done in a methodical way.
    c1 has opted not to discuss or elaborate on his implementation in any meaningful way. end of story
    Warning I'm not a teacher

  12. #12
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    This is just an example for 2 buttons
    for 20 buttons there should be 40 variables
    I wanted to avoid that
    this is why I asked

  13. #13
    Join Date
    Oct 2005
    Location
    Sweden
    Posts
    3,517


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    With 10 or 20 keys, how do you handle the event of more than one key being pressed at the same time?

  14. #14
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    By clever selection of ADC resistors.
    This had been solved long time ago, and not by me

  15. #15
    Join Date
    Apr 2014
    Location
    OK
    Posts
    557


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Recent posts got me thinking. Are you designing something with a bunch of buttons, each with a specific functionality? Or are you trying to do something more akin to a key pad? If you're using a standard key pad there are set-ups using resistors to create a narrow voltage range with the press of any individual key. That only requires one analog input to read any key press.

    https://aws1.discourse-cdn.com/ardui...2_542x500.jpeg

  16. #16
    Join Date
    May 2013
    Location
    australia
    Posts
    2,383


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    12 buttons is ok with 8 bit adc . with 16 the last row gets down to 2 or 3 counts separation , a bit close for comfort in my view
    Warning I'm not a teacher

  17. #17
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    At current stage, I'm looking for more efficient and small code
    for ADC key handling.

  18. #18
    Join Date
    Nov 2003
    Location
    Greece
    Posts
    3,796


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Doesn't post#2 with the use of Select Case do what you want with the least code/var usage?

    Ioannis

  19. #19
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    No it does not.
    It does not do "do not repeat action until user releases and pushes key again"

  20. #20
    Join Date
    Nov 2003
    Location
    Greece
    Posts
    3,796


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Code:
    if !flag then
        adcin 1, adcval   'read keys
        SELECT CASE adcval
          CASE < 10 : left=left+1:flag=1 'detect button presses
          CASE < 100
        @ NOP
          CASE < 130 : right=right+1 :flag=1'detect button presses
        END SELECT
    endif
    Something like that then?

    Ioannis
    Last edited by Ioannis; - 7th January 2022 at 21:40.

  21. #21
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Yes and we again will have extra "left" and "right" variables, one per each button - what I wanted to avoid

  22. #22
    Join Date
    Nov 2003
    Location
    Greece
    Posts
    3,796


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    maybe I am a bit slow.. But how you can have 40 buttons but not 40 variables to distinct between them?

    Ioannis

  23. #23
    Join Date
    Feb 2013
    Posts
    1,078


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    ADCIN X,ADVAL
    IF ADVAL=20 then Gosub Y1
    IF ADVAL=10 then GOSUB Y2
    IF ADVAL=30 then GOSUB Y3

    and so on.

  24. #24
    Join Date
    Nov 2003
    Location
    Greece
    Posts
    3,796


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    So at any given moment you will not know which button was hit and will have to scan again.

    But even so, a flag for each button pressed is needed to keep it from regarding it as new press.

    Ioannis

  25. #25
    Join Date
    Apr 2014
    Location
    OK
    Posts
    557


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Regarding flags, here is a trick I use:
    Code:
    ButtonA VAR WORD
     Button1 VAR ButtonA.0
     Button2 VAR ButtonA.1
     ......
     Button16 VAR ButtonA.15
    This creates bit flags. To use them:
    Code:
    Button4 = PORTC.3

  26. #26
    Join Date
    Oct 2005
    Location
    Sweden
    Posts
    3,517


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    If you have many inputs scattered around various ports and you want to "gather them" in one consecutive block so you can iterate over them using a loop then sure, why not. Otherwise I must say I don't see the point of doing that. You might as well simply alias the bit variable directly to the port bit/pin and be done (after all, the Port registers is just another byte in memory - right).
    Code:
    Button4 VAR PortC.3
    You'll save RAM, FLASH and processor cycles by not first copying the value of one bit variable (PortC.3) to another (Button4) only to then evaluate the state of Button4 variable.

    But, I might be missing the point alltogether, in which case I apologise.

    /Henrik,

  27. #27
    Join Date
    Nov 2003
    Location
    Greece
    Posts
    3,796


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    Here there are not ports... Only a ADVAL from ADC since the buttons are read as analog voltage.

    Ioannis

  28. #28
    Join Date
    May 2013
    Location
    australia
    Posts
    2,383


    Did you find this post helpful? Yes | No

    Default Re: My code for ADC button handling, it works, but can it be slimmed down?

    its a fairly simple process , 10 bits per button is easy , 8 bits with a bit of effort
    providing the button check is called in a loop at a relatively consistent rate.
    the program structure matters , spaghetti code won't get good results
    sudo method
    Code:
    b1cnt  var byte
    newb1  var bit 
    b1dun  var bit
    b2cnt  var byte
    newb2  var bit 
    b2dun  var bit
    osc=8
    t1con=$01;32mS
    main
    if pir1.0
    pir1.0=0
    gosub getkey
    if newb1 gosub dob1 
    if newb2 gosub dob2
    endif
    
    
    goto main
    
    
    
    
    getkey:
    read adc
     b1cnt = b1cnt<<1
     b2cnt = b2cnt<<1
    if adc in b1 range 
      b1cnt = b1cnt+1
    elseif if adc in b2 range
      b2cnt = b2cnt+1
    endif
    if !b1dun
      if b1cnt=255 then newb1=1  ;32*8mS
    else
      if b1cnt=0 then b1dun=0
    endif
    
    
    if !b2dun
       if b2cnt=255 then newb2=1
    else
       if b2cnt=0 then b2dun=0
    endif
    
    
    return
    
    
    dob1:
    b1dun=1
    newb1=0
    do b1 stuff
    return
    
    
    dob2:
    b2dun=1
    newb1=0
    do b2 stuff
    return
    Warning I'm not a teacher

Similar Threads

  1. Long / short press button...but reading ADC
    By fratello in forum mel PIC BASIC Pro
    Replies: 37
    Last Post: - 10th July 2016, 09:37
  2. Replies: 42
    Last Post: - 19th March 2012, 00:02
  3. How come my DS18S20 code still works?
    By Wilson in forum mel PIC BASIC Pro
    Replies: 9
    Last Post: - 30th July 2011, 03:59
  4. 12 bit or higher ADC on a PIC that works with PBP?
    By Brandon in forum mel PIC BASIC Pro
    Replies: 8
    Last Post: - 11th November 2007, 18:19
  5. ADC -how it works?, pic -
    By matias in forum mel PIC BASIC Pro
    Replies: 1
    Last Post: - 13th March 2007, 22:54

Members who have read this thread : 2

You do not have permission to view the list of names.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts