Skip to content
This repository was archived by the owner on May 18, 2022. It is now read-only.

Splitwise Machine coding#19

Open
tusharanand29 wants to merge 1 commit into
workattech:masterfrom
tusharanand29:master
Open

Splitwise Machine coding#19
tusharanand29 wants to merge 1 commit into
workattech:masterfrom
tusharanand29:master

Conversation

@tusharanand29

Copy link
Copy Markdown

Please review it
Thanking you
Tushar

Comment thread Main.java
It keep records from the begining meaning if there is no borrow between the users still we maintain the amount.
**/

public static void printTable(int Matrix[][])

@gcnit gcnit Feb 20, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

variable name should start with lowercase

Comment thread Main.java

public static void updateTransctionTableEqual(int Matrix[][],String payer, String whom[] , int amount)
{
int split_wise = (amount)/whom.length;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either use camelCase or underscore_separated based on the language’s convention. Please do not use both in your solution.
Source: How to prepare for machine coding round?

Comment thread Main.java
updateTransctionTableEqual() method updates the Transaction Matrix on 'EQUAL' option
**/

public static void updateTransctionTableEqual(int Matrix[][],String payer, String whom[] , int amount)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you need a 'Table' suffix?

@gcnit

gcnit commented Feb 20, 2020

Copy link
Copy Markdown
Collaborator

@tusharanand29 sorry for the delay in review. I've added a few comments. Please go through How to prepare for Machine Coding Round?

I would recommend you to do a self-review from my solution for Splitwise design so as to get a better idea on how to structure your code. You can push again after making the changes for another review.

Would recommend you to try the other problems as well.

Snake and Ladder Design: https://workat.tech/machine-coding/practice/snake-and-ladder-problem-zgtac9lxwntg
Trello Design: https://workat.tech/machine-coding/practice/trello-problem-t0nwwqt61buz

@tusharanand29

tusharanand29 commented Feb 21, 2020 via email

Copy link
Copy Markdown
Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants