refactor getResponse

Author: Remi-Gau

getResponse.m

@@ -1,4 +1,4 @@
-function responseEvents = getResponse(action, cfg, expParameters, getOnlyPress, verbose)
+function responseEvents = getResponse(action, cfg, expParameters, getOnlyPress)
 % wrapper function to use KbQueue
 % The queue will be listening to key presses on the response box as defined
 %  in the cfg structure : see setParameters for more details.
@@ -16,29 +16,29 @@
 %  - flush:
 %  - stop:
 %
-% - getOnlyPress: if set to 1 the function will only return the key presses and 
-%    will not return when the keys were released (default=1) 
+% - getOnlyPress: if set to 1 the function will only return the key presses and
+%    will not return when the keys were released (default=1)
 %    See the section on OUTPUT below for more info
 %
 %
-% 
+%
 % OUTPUT
-% 
+%
 % responseEvents: returns all the keypresses and return them as a structure
 % with field names that make it easier to save the output of in a BIDS
 % format
-% 
+%
 % responseEvents.onset : this is an absolute value and you should
 %   substract the "experiment start time" to get a value relative to when the
 %   experiment was started.
-% 
+%
 % responseEvents.trial_type = 'response';
-% 
+%
 % responseEvents.duration = 0;
-% 
+%
 % responseEvents.key_name : the name of the key pressed
-% 
-% responseEvents(iEvent,1).pressed : if 
+%
+% responseEvents(iEvent,1).pressed : if
 %   pressed == 1  --> the key was pressed
 %   pressed == 0  --> the key was released
 
@@ -47,10 +47,6 @@
     getOnlyPress = 1;
 end
 
-if nargin < 5
-    verbose = 0;
-end
-
 responseEvents = struct;
 responseEvents.onset = [];
 responseEvents.trial_type = [];
@@ -76,7 +72,7 @@
 
         % by default we listen to all keys
         % but if responseKey is set in the parameters we override this
-        keysOfInterest = ones(1,256); 
+        keysOfInterest = ones(1,256);
 
         fprintf('\n Will be listening for key presses on : ')
 
@@ -90,7 +86,7 @@
             end
 
             keysOfInterest(responseTargetKeys) = 1;
-           
+            
         else
 
             fprintf('ALL KEYS.')
@@ -105,19 +101,13 @@
 
     case 'start'
 
-        fprintf('\n starting to listen to keypresses\n')
-        
         KbQueueStart(responseBox);
 
 
     case 'check'
 
-        if verbose
-            fprintf('\n checking recent keypresses\n')
-        end
-        
         iEvent = 1;
-         
+        
         while KbEventAvail(responseBox)
 
             event = KbEventGet(responseBox);
@@ -131,32 +121,65 @@
                 responseEvents(iEvent,1).duration = 0;
                 responseEvents(iEvent,1).key_name = KbName(event.Keycode);
                 responseEvents(iEvent,1).pressed =  event.Pressed;
-
+                
             end
 
             iEvent = iEvent + 1;
 
         end
 
+        
     case 'flush'
 
-        if verbose
-            fprintf('\n reinitialising keyboard queue\n')
-        end
-       
         KbQueueFlush(responseBox);
 
 
     case 'stop'
 
-        fprintf('\n stopping to listen to keypresses\n\n')
-        
         KbQueueRelease(responseBox);
 
         % Give me my keyboard back... Pretty please.
         ListenChar(0);
 
+        
+end
+
+talkToMe(action, expParameters);
+
+
 end
 
 
-end
+function talkToMe(action, expParameters)
+
+if ~isfield(expParameters, 'verbose') || isempty(expParameters.verbose)
+    expParameters.verbose = false;
+end
+
+switch action
+    
+    case 'init'
+        
+    case 'start'
+        
+        fprintf('\n starting to listen to keypresses\n')
+        
+    case 'check'
+        
+        if expParameters.verbose
+            fprintf('\n checking recent keypresses\n')
+        end
+        
+    case 'flush'
+        
+        if expParameters.verbose
+            fprintf('\n reinitialising keyboard queue\n')
+        end
+        
+    case 'stop'
+        
+        fprintf('\n stopping to listen to keypresses\n\n')
+        
+end
+
+end