b4a 2.70

[SoyEli]

Hello:

I'm getting this under:
Sub Activity_KeyPress (KeyCode As Int) As Boolean

not all code paths return a value (warning #2)


Why?

Thank you:

 

[PenguinHero]

Hi SoyEli,

I presume that you have some code in that Sub.

Is it possible that you have conditions (if statements) that would prevent a value being returned for the Sub, that you are only returning a value when a certain condition is met?

You can paste the code and let everyone have a look.

And its probably stating the obvious, but it is only a warning. If your code is doing what you want it to, its probably safe to ignore it.

 

[Erel]

My guess is that you are not returning any value from this sub. This event expects a boolean result.
True means that the keypress is consumed.

 

[SoyEli]

Thank you for any help;

Here is the code:

Is working ok, did not show any error on prev b4a., only on upgrade to 2.70

Sub Activity_KeyPress (KeyCode As Int) As Boolean
If KeyCode = KeyCodes.KEYCODE_BACK Then
If MapPanel.Visible=True Then
Main.BringToFront
MapPanel.SendToBack
MapPanel.Visible=False
MapPanel.RemoveAllViews
Return True
End If
If Main.Visible=True Then
HomeScreen
Return True
End If
End If
End Sub

 

[yttrium]

Please use code tags next time.

Your code should be the following, though I don't know if this is redundant:

Sub Activity_KeyPress (KeyCode As Int) As Boolean
    If KeyCode = KeyCodes.KEYCODE_BACK Then
      If MapPanel.Visible=True Then
         Main.BringToFront
        MapPanel.SendToBack
        MapPanel.Visible=False
        MapPanel.RemoveAllViews
       Return True
     End If    
     If Main.Visible=True Then
       HomeScreen
        Return True
      End If
   End If
   Return False
End Sub

 

[agraham]

You are missing a return value if the KeyCode is not KeyCodes.KEYCODE_BACK

Sub Activity_KeyPress (KeyCode As Int) As Boolean
    If KeyCode = KeyCodes.KEYCODE_BACK Then
      If MapPanel.Visible=True Then
         Main.BringToFront
         MapPanel.SendToBack
         MapPanel.Visible=False
         MapPanel.RemoveAllViews
         Return True
      End If     
      If Main.Visible=True Then
         HomeScreen
         Return True
      End If
    End If
    Return False ' You are missing this return value if the IF block above is not entered
End Sub

 

[SoyEli]

Thank you !!!

that worked.

Please explain:
"code tags"

 

[yttrium]

Thank you !!!

that worked.

Please explain:
"code tags"

This forum has a special set of BBCode tags when posting that formats your text nicely. For example,

[code]Dim Button1 As Button[/code]

displays as

Dim Button1 As Button