I refactored Routing builder class. Please review.

Example

Routing routing = new Routing.Builder(start, end)
        .travelMode(AbstractRouting.TravelMode.DRIVING)
        .avoid(AbstractRouting.AvoidKind.HIGHWAYS)
        .waypoints(point1, point2)
        .withListener(this)
        .build();
routing.execute();

1. I have added a callback wrapper for AsyncTask.OnCancelled(), useful if any UI modification is done in the other callbacks (onSuccess/onFailure). If the user moves away from the activity while directions are still being calculated then we can simply discard the results and avoid touching the UI at all.
2. Added rudimentary support for multiple waypoints as per Using Waypoints in Routes

Added the option to select between alternative routes (shortest, fastest) where RouteMode.FASTEST is the default.

Usage:

        Routing routing = new Routing.Builder()
                .travelMode(AbstractRouting.TravelMode.DRIVING)
                .routeMode(AbstractRouting.RouteMode.SHORTEST)
                .withListener(this)
                .waypoints(start, end)
                .build();

Hi Jd Alexander,

Thanks a lot for taking the time to build this library, it saved me a lot of time, I have a problem..it seems that the route retrieved for cycling method is the same as driving method, anyhow it does not coincide with the google maps app route.. According to the documentation the key work for transport method is "bicycling".. you have in your library declared biking, I wonder if that is the reason why.

Cheers

The commit adds the possibility for users to specify their own API key when accessing the Google Directions API, thus enabling more advanced features such as higher quotas and more waypoints.

To do so the API request must use SSL so the default API URL has been changed to use SSL instead of plain HTTP.

hi. Google-Directions-Android is very nice. I will use Google-Directions-Android for my app.

I support avoid parameters. Please review.

Example

Routing routing = new Routing.Builder(Routing.TravelMode.DRIVING)
        .avoid(Routing.AvoidKind.HIGHWAYS)
        .build();
routing.registerListener(this);
routing.execute(start, end);

thanks

Currently, the onRoutingFailure callback method does not provide much useful information about the nature of the error. It could be that the API KEY is incorrect, or that the daily quota has been reached, or a bunch of other reasons. The JSON response coming from the Google servers indeed inform the user about the nature of the problem, but the library doesn't relay that information. This commit attempt to fix that.

PS. I noticed the previous pull request had some useless debug logs, thus I'm submitting another updated pull request.

Hello there,

Please let me thank you for your great library, it saved me a lot of time. I've integrated it yesterday and I discovered a few issues with it.

I've made a quick patch for bug when the requets is made offline. The InputStream is then null and crashed the parser.

I have also separated the UI work from the ASync task in Routing.java

Please feel free to integrate or not these small changes.

Thanks a lot again for this lib! Cheers, Cyrille

Is it somehow possible to remove a route when another marker on a map is clicked? I have 9 markers to which the route from the current position can be asked by the user, but they just add up on top of each other (depending on the route/location). I would like to display just the route to the last clicked marker...tried to assign the Routing to a variable and then set this to null onMarkerClick but that didn't work. Any hints?

I am getting this error during gradle syn - Error:(2, 0) org/gradle/api/publication/maven/internal/defaultmavenfactory

error focus onto this `apply plugin: 'com.android.library' apply plugin: 'com.github.dcendents.android-maven' apply plugin: "com.jfrog.bintray"

dependencies { classpath 'com.github.dcendents:android-maven-gradle-plugin:1.3' } // This is the library version used when deploying the artifact version = "1.0.9"`

Hi

On line 91 on your GoogleParser class: segment.setDistance(distance / 1000);

distance is and int and 1000 is also an int and the segment distance is a double. And as a consequence, the distances are not calculated correctly. I have values like 0.0, 1.0, 9.0, etc

You need to cast both to doubles: segment.setDistance((double)distance / (double)1000);

Thanks for a great library

Parallel builds. This project contains multiple modules. Parallel builds can improve the build speed by executing tasks in parallel. We can enable this feature by setting org.gradle.parallel=true.

Configuration on demand. Configuration on demand tells Gradle to configure modules that only are relevant to the requested tasks instead of configuring all of them. We can enable this feature by setting org.gradle.configureondemand=true.

gradle caching. Shared caches can reduce the number of tasks you need to execute by reusing outputs already generated elsewhere. This can significantly decrease build times. We can enable this feature by setting org.gradle.caching=true.

Gradle daemon. The Daemon is a long-lived process that help to avoid the cost of JVM startup for every build. Since Gradle 3.0, Gradle daemon is enabled by default. For an older version, you should enable it by setting org.gradle.daemon=true.

===================== If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

hei,I clone your project,but i doesn't run. can you help me? there are some tips: Process: com.directions.sample, PID: 24051 java.lang.NoClassDefFoundError: Failed resolution of: Lcom/google/android/gms/common/api/Api$ClientKey; at com.google.android.gms.location.places.Places.(Unknown Source) at com.directions.sample.MainActivity.onCreate(MainActivity.java:82) at android.app.Activity.performCreate(Activity.java:6910) at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1123) at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2746) at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2864) at android.app.ActivityThread.-wrap12(ActivityThread.java) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1567) at android.os.Handler.dispatchMessage(Handler.java:105) at android.os.Looper.loop(Looper.java:156) at android.app.ActivityThread.main(ActivityThread.java:6523) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:941) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:831) Caused by: java.lang.ClassNotFoundException: Didn't find class "com.google.android.gms.common.api.Api$ClientKey" on path: DexPathList[[zip file "/data/app/com.directions.sample-2/base.apk"],nativeLibraryDirectories=[/data/app/com.directions.sample-2/lib/arm64, /system/lib64, /vendor/lib64, /system/vendor/lib64, /product/lib64]] at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:56) at java.lang.ClassLoader.loadClass(ClassLoader.java:380) at java.lang.ClassLoader.loadClass(ClassLoader.java:312) at com.google.android.gms.location.places.Places.(Unknown Source)  at com.directions.sample.MainActivity.onCreate(MainActivity.java:82)  at android.app.Activity.performCreate(Activity.java:6910)  at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1123)  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2746)  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2864)  at android.app.ActivityThread.-wrap12(ActivityThread.java)  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1567)  at android.os.Handler.dispatchMessage(Handler.java:105)  at android.os.Looper.loop(Looper.java:156)  at android.app.ActivityThread.main(ActivityThread.java:6523)  at java.lang.reflect.Method.invoke(Native Method)  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:941)  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:831)  05-15 17:52:06.538 620-877/? E/logserver: imonitor_add_dynamic_path, path:/data/system/dropbox/[email protected]

Inside the code there is request asking "&alternatives=true" but it is not showing multiple routes(I mean alternative routes). Is it bug or shall I implement something?

https://github.com/jd-alexander/Google-Directions-Android shows MIT license in the top right corner (maybe from https://github.com/jd-alexander/Google-Directions-Android/blob/master/LICENSE.md) and at the same time shows Apache 2.0 license at the bottom of that page (from https://github.com/jd-alexander/Google-Directions-Android/blob/master/README.md)