Is there a better way to write this? ```val launchUri = try { if( isAvailable() ) {...

# getting-started

[Sam]

Is there a better way to write this?

val launchUri = try {
                if( isAvailable() ) {
                    Uri.parse( "..." )
                } else {
                    Uri.parse("..." )
                }
            } catch ( e : android.content.ActivityNotFoundException ) {
                if( isAvailable() ) {
                    Uri.parse( "..." )
                } else {
                    Uri.parse("..." )
                }
            }

[tipsy]

pull out the uri to parse as a variable

[Vladyslav Sitalo]

is "..." different in each case? also what is throwing the exception?

[tipsy]

val uri = if(available) ... else ...
Uri.parse(uri)

[Sam]

@Vladyslav Sitalo Yes, "..." is different

[Sam]

Nevermind, the exception was from a left over

[Sam]

This is the refactored code. Logic is to find an activity that can handle the URL if not launch a browser. This works fine except too many blocks without ?: operator

try {
                
                val launchUri = if( isAvailable() ) {
                    Uri.parse( "..." ) 
                } else {
                    Uri.parse("..." )
                }

                startActivity( Intent( Intent.ACTION_VIEW, launchUri ) )

            } catch ( e : android.content.ActivityNotFoundException ) {
                
                val launchUri = if( isAvailable() ) {
                    Uri.parse( "..." )
                } else {
                    Uri.parse("..." )
                }

                startActivity( Intent( Intent.ACTION_VIEW, launchUri ) )
            }

[Ernest Zamelczyk]

Maybe try it like this?

val launchUri = if(isAvailable())
    Uri.parse("...")
else
    Uri.parse("...")
val intent = Intent(Intent.ACTION_VIEW, launchUri)

if(intent.resolveActivity(getPackageManager()) != null) {
    startActivity(intent)
else {
    //resolve new url and start activity
}

[Ernest Zamelczyk]

I don’t know exactly what is your logic because you don’t tell us if there’s 4 different uris or 2 or 3

[ianbrandt]

@Sam, This and the prior are good posts for #codereview if you haven’t already seen that channel. 😊

[Sam]

Sure, good to know