[FEATURE] improve the naming of variables #162

Closed
opened 2026-01-29 15:06:02 +00:00 by claunia · 7 comments
Owner

Originally created by @haru02w on GitHub (Jul 31, 2023).

Detailed description

The variable names used in sorting/merge_sort.c is really confusing. Please, use more useful names:
int *aux_array instead of int *b

Context

Sorry, I cannot understand what the code is doing

Possible implementation

No response

Additional information

No response

Originally created by @haru02w on GitHub (Jul 31, 2023). ### Detailed description The variable names used in `sorting/merge_sort.c` is really confusing. Please, use more useful names: `int *aux_array` instead of `int *b` ### Context Sorry, I cannot understand what the code is doing ### Possible implementation _No response_ ### Additional information _No response_
claunia added the Staleenhancement labels 2026-01-29 15:06:02 +00:00
Author
Owner

@realstealthninja commented on GitHub (Aug 3, 2023):

This seems like a very reasonable request and should be handled at review time to ensure no code with unintelligible variable name can get through.
as for ore-existing code we must refactor them if you want to you could start on it.
or point out some of the ones that need immediate attention and are very critical

@realstealthninja commented on GitHub (Aug 3, 2023): This seems like a very reasonable request and should be handled at review time to ensure no code with unintelligible variable name can get through. as for ore-existing code we must refactor them if you want to you could start on it. or point out some of the ones that need immediate attention and are very critical
Author
Owner

@BamaCharanChhandogi commented on GitHub (Aug 3, 2023):

Hey, I can contribute. Can you assign me?

@BamaCharanChhandogi commented on GitHub (Aug 3, 2023): Hey, I can contribute. Can you assign me?
Author
Owner

@alexpantyukhin commented on GitHub (Aug 6, 2023):

@BamaCharanChhandogi You are free to make a PR and link the issue on the PR. No need to assign for that.

@alexpantyukhin commented on GitHub (Aug 6, 2023): @BamaCharanChhandogi You are free to make a PR and link the issue on the PR. No need to assign for that.
Author
Owner

@BamaCharanChhandogi commented on GitHub (Aug 6, 2023):

Hey, @alexpantyukhin check #1294 PR

@BamaCharanChhandogi commented on GitHub (Aug 6, 2023): Hey, @alexpantyukhin check #1294 PR
Author
Owner

@dotdot0 commented on GitHub (Aug 29, 2023):

Is this issue still open??

@dotdot0 commented on GitHub (Aug 29, 2023): Is this issue still open??
Author
Owner

@github-actions[bot] commented on GitHub (Sep 29, 2023):

This issue has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions[bot] commented on GitHub (Sep 29, 2023): This issue has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Author
Owner

@github-actions[bot] commented on GitHub (Oct 7, 2023):

Please ping one of the maintainers once you add more information and updates here. If this is not the case and you need some help, feel free to ask for help in our Gitter channel or our Discord server. Thank you for your contributions!

@github-actions[bot] commented on GitHub (Oct 7, 2023): Please ping one of the maintainers once you add more information and updates here. If this is not the case and you need some help, feel free to ask for help in our [Gitter](https://gitter.im/TheAlgorithms) channel or our [Discord server](https://the-algorithms.com/discord/). Thank you for your contributions!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#162