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 intoworkattech:masterfrom
tusharanand29:master
Open

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

Conversation

@tusharanand29
Copy link
Copy Markdown

Please review it
Thanking you
Tushar

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[][])
Copy link
Copy Markdown
Collaborator

@gcnit gcnit Feb 20, 2020

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


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?

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
Copy link
Copy Markdown
Collaborator

gcnit commented Feb 20, 2020

@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
Copy link
Copy Markdown
Author

tusharanand29 commented Feb 21, 2020 via email

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