Replace "magic numbers" with IntDef

[Sternbach-Software]

A magic number is "a unique value with unexplained meaning or multiple occurrences which could (preferably) be replaced with a named constant" - Wikipedia.

There are many instances where a number is used as a case in a switch or if statement to specify which action should be done (e.g. if(action == 5) { //create note), and it is a practice that has been found to often introduce bugs and reduce code readability.

The preferred solution in Android is to replace it with an IntDef.

Here are some examples that need changing (some are links to function definitions, so all calls to that function need to change; likewise for variables):

• GeniForAndroid/app/src/main/java/app/familygem/ConfirmationActivity.java

  Line 77 in f394c44

  case 1: // Note

• GeniForAndroid/app/src/main/java/app/familygem/ConfirmationActivity.java

  Line 54 in f394c44

  case 1: add++;

• https://github.com/Sternbach-Software/GeniForAndroid/blob/master/app/src/main/java/app/familygem/DetailActivity.java#L988

• GeniForAndroid/app/src/main/java/app/familygem/list/NotesFragment.java

  Line 101 in 8be0097

  if( item.getItemId() == 0 ) { // Delete

• GeniForAndroid/app/src/main/java/app/familygem/list/RepositoriesFragment.java

  Line 163 in 8be0097

  if( Global.settings.expert )

  • GeniForAndroid/app/src/main/java/app/familygem/list/RepositoriesFragment.java

    Line 62 in 8be0097

    case 1: // Sort by id

  • GeniForAndroid/app/src/main/java/app/familygem/list/RepositoriesFragment.java

    Line 171 in 8be0097

    case 1:

  • GeniForAndroid/app/src/main/java/app/familygem/list/RepositoriesFragment.java

    Line 195 in 8be0097

    if( item.getItemId() == 0 ) {

• GeniForAndroid/app/src/main/java/app/familygem/visitor/MediaList.java

  Line 38 in 8be0097

  private int mediaType;

  • GeniForAndroid/app/src/main/java/app/familygem/visitor/MediaList.java

    Line 48 in 8be0097

    if( mediaType == 0 )

  • GeniForAndroid/app/src/main/java/app/familygem/visitor/MediaList.java

    Line 91 in 8be0097

    list.addAll(gc.getMedia()); // (rakes?) all Gedcom shared media items //rastrella tutti gli oggetti media condivisi del Gedcom

• GeniForAndroid/app/src/main/java/app/familygem/CompareActivity.java

  Line 64 in 8be0097

  // Compare all the records of the two Gedcoms

• GeniForAndroid/app/src/main/java/app/familygem/CompareActivity.java

  Line 174 in 8be0097

  int modification = 0;

• GeniForAndroid/app/src/main/java/app/familygem/CompareActivity.java

  Line 277 in 8be0097

  String writeDifferences(int type) {

• GeniForAndroid/app/src/main/java/app/familygem/Comparison.java

  Line 55 in 8be0097

  int type; // number from 1 to 7 that defines the type: 1 Note -> 7 Family

• GeniForAndroid/app/src/main/java/app/familygem/Comparison.java

  Line 70 in 8be0097

  int destiny;

• GeniForAndroid/app/src/main/java/app/familygem/Diagram.java

  Line 103 in 8be0097

  startActivity(new Intent(getContext(), DiagramSettings.class));

[whittede]

Great explanation and intro to the work! @Sternbach-Software do you mind assigning me to this task?

Thanks!

[whittede]

@Sternbach-Software I finally have some time to begin dipping my toes into this project. I've initiated a pull request here to see if development is still able to happen on this project, or if you/anyone else are no longer planning on moving it forward.

[Sternbach-Software]

@whittede development has kind of stagnated for 3 reasons:

• I started developing the SDK for the API, but I couldn't figure out how to return specific values from the API.
• I am thinking of moving the app to Kotlin Multiplatform so that it will be an iOS, Android, web, and desktop app with a single codebase. In the process, I would want to reimplement all of the guts in a cleaner way.
• the only thing stopping me from doing the above is that I need a Compose family tree view. I have started developing one, but I need to figure out how to draw the lines between family members. Once I do that, it will be ready for use, and I will likely build the app from scratch.

[whittede]

Oh, wow! Well that's some very impressive dedication and work, I like the plan! I was also very impressed with your transition of the entire codebase from Java to Kotlin originally, to be honest. Let me know if there is anything I can do in terms of Googling or skills I could learn to help draw visual lines between the family trees in the app if that's something you'd like help with! I still think this project is a great idea that a ton of people will benefit from. Good to hear from you, just let me know if/when you have a slot that can require some additional help! El sáb., 12 de agosto de 2023 22:52, Sternbach-Software < ***@***.***> escribió:

…

@whittede <https://github.com/whittede> development has kind of stagnated for 3 reasons: - I started developing the SDK for the API <https://github.com/Sternbach-Software/GeniAndroidSDK>, but I couldn't figure out how to return specific values from the API. - I am thinking of moving the app to Kotlin Multiplatform so that it will be an iOS, Android, web, and desktop app with a single codebase. In the process, I would want to reimplement all of the guts in a cleaner way. - the only thing stopping me from doing the above is that I need a Compose family tree view. I have started developing one, but I need to figure out how to draw the lines between family members. Once I do that, it will be ready for use, and I will likely build the app from scratch. — Reply to this email directly, view it on GitHub <#9 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIM3JMHVFBKZZPKOITDADN3XVBTTTANCNFSM6AAAAAASHD2NSQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[Sternbach-Software]

The API SDK is not too hard. If you can get a working API request with specific field returned, that would be great. You can use something like postman, or write a script in python, Java, or Kotlin (preferably the latter).

I think the lines thing is pretty complicated. You can take a stab at it by googling Compose Layout (the Layout composable).

Conceptually, the algorithm I am working on to draw a line connecting all children is as follows:
Theory: a line comes out from each child’s center-top and connects to a horizontal line which extends from the left-most child to the right-most child.

1. Draw a straight line up from each child’s center-top (x = child center, y = child top) until parent’s bottom y ( (x = child center, y = parent bottom)
2. Let startX = left-most child’s center-top
3. Let endX = right-most child’s center-top
4. Draw a line from startX to endX

Then, to connect the children to the parents, and the parents to each other:

1. Draw a line from parent's middle (x = if(left parent) end x else start x, y = parent.height / 2) to halfway between parents (x = (end of left parent x + start of right parent x) / 2, = parent.height / 2). This connects parents.
2. Draw a line from halfway between parents to horizontal line connecting children (x = (end of left parent x + start of right parent x) / 2), y = parent bottom)
   There should probably be some padding between the horizontal child line and the parent bottom, but that is (hopefully) trivial.

The question is how to do this in Compose. This code lays out the children fine:

/**
 * Every person/[FamilyTree] is responsible for drawing themselves and their children.
 * */
@Composable
fun FamilyTree(
    modifier: Modifier = Modifier,
    root: Person,
) {
    val backgroundColor = colors.random(random) //for debug: draw bounding box to see bounds of this person
​
    Column(
        modifier
            .fillMaxWidth()
            .drawWithContent { //draw debug background
                drawRect(backgroundColor)
                drawContent()
            },
        horizontalAlignment = Alignment.CenterHorizontally,
    ) {
        Row(
            horizontalArrangement = Arrangement.Center
        ) {
            root.spouse?.let { PersonCard(it, Color.Magenta) }
            PersonCard(root, color = backgroundColor)
        }
​
        //Draw their children
        if (root.children.isNotEmpty()) {
            Row(horizontalArrangement = Arrangement.Center) {
                for (child in root.children) {
                    FamilyTree(
                        root = child.copy(name = "${child.name} sonOf ${root.name}"), //name change for debug purposes
                    )
                }
            }
        }
    }
}
​
@Composable
fun PersonCard(person: Person, color: Color) {
    Card(
        modifier = Modifier
            .padding(8.dp)
            .width(100.dp),
        colors = CardDefaults.cardColors(color)
    ) {
        Text(person.name)
    }
}

[whittede]

I'm on it! I'll take a crack at this in September and I'll ask some questions here as they come up. (I'm sure they will.)

Where is Geni's API endpoint that we are attempting to connect to with the SDK? That's where I will start plugging away at first to see if I can get some specific fields returned. Thanks!

[Sternbach-Software]

https://www.geni.com/platform/developer/help/reference?version=1

[whittede]

Hey @Sternbach-Software ! Just to post an update here in September, as promised:

I believe I'm 99% of the way there getting the Geni API to return specific values, but the last remaining issue I am running into is that on the final step, in which the API should be giving me an access token (according to here), I am instead getting the error {"error_description":"Invalid authorization code","error":"invalid_request"} returned. My scripts for executing the AJAX API calls are all in Javascript.

I've run out of time to work on this, so I will have to chase down this issue and fix it in October or November. I'll let you know when I resolve it. Should we move any future discussion about connecting to the API to a different thread?

Hope you're doing well! Thanks!

[Sternbach-Software]

Do you have an Android phone? I made an app that gets the access token using my registered Geni app id and callback URL, but the URL looks something like geniforandroid://access_token?ACCESS_TOKEN=ABC123. This works on Android because apps can intercept specific URLs and have them redirect to specific code which can read the URL, so when the browser that the user used to approve access tries to navigate to the above URL, our app is opened and the code I wrote saves the access token to a database. I am not sure how to reproduce that interception from the command line.

[whittede]

Hey @Sternbach-Software!

I set aside some time for myself to work on this in December as promised, but when the time came around, Christmas season was really packed and my wife asked if I could push my hobby project to January. Then I purchased a new laptop, so I reinstalled all of my web server setup onto this laptop when I started developing this month!

I am very close to getting the API to return a value that I want, but currently I am receiving a different error for each API endpoint that I try to access, so I need to choose a specific endpoint and just focus on that.

I've posted a question to the Geni Developers Forum regarding the /api/user/metadata endpoint, so hopefully I can get some assistance with the rate limits. If I don't receive a response there, I'll re-post the question onto Stack Overflow, and if that doesn't work then I was just recently on a video call with Geni's General Manager, so I will send him a send him a message and see if he can give me some help.

Anyway, I will keep plugging away at the API and I'll keep you updated on the responses I get!

[Sternbach-Software]

@whittede thanks so much! I have also been plugging away on the family tree component. Tried many different variations and resources so far, including looking at other languages. Turns out it is a very complicated problem.