Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

translation: Update index.md #1009

Merged
merged 3 commits into from
Jan 6, 2024
Merged

translation: Update index.md #1009

merged 3 commits into from
Jan 6, 2024

Conversation

K3v123
Copy link
Contributor

@K3v123 K3v123 commented Dec 27, 2023

improved the abstract.

If this PR is related to coding or code translation, please fill out the checklist and paste the console outputs to the PR.

  • I've tested the code and ensured the outputs are the same as the outputs of reference code.
  • I've checked the code (formatting, comments, indentation, file header, etc) carefully.
  • The code does not rely on a particular environment or IDE and can be executed on a standard system (Win, macOS, Ubuntu).

improved the abstract.
@krahets
Copy link
Owner

krahets commented Dec 27, 2023

Thanks for the PR! Could you set the PR's title as translation: Update index.md in the future? It will help classify different types of pull requests.

@K3v123
Copy link
Contributor Author

K3v123 commented Dec 27, 2023

oh wait sorry I was translating another md, and didnt see this message beforehand. yeah I will next time! sorry for the inconvenience.

@K3v123 K3v123 changed the title Update index.md translation: Update index.md Dec 27, 2023
@krahets krahets added translation English translation documents documents-related labels Dec 27, 2023
Copy link
Contributor

@yuelinxin yuelinxin left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, please consider the comment.


The bricks of an array are neatly arranged, each closely connected to the next. In contrast, the bricks of a linked list are scattered, with vines of connections freely weaving through the gaps between bricks.
In arrays, envision bricks standing in precise alignment, each tightly linked to its neighbor. In contrast, in linked lists, bricks scatter freely, and flexible vines weave effortlessly between them, creating a web-like pattern among the gaps.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would suggest sticking closer to the original text, for example:

"The bricks of an array align neatly, adjacent to one another. While the bricks of a linked list are scattered, with vines weaving freely to connect them."

Notably, try to avoid using the word "link" when describing an array to avoid confusion. Also, try to avoid adding text where it does not serve a clear purpose of adding clarity or addressing differences between English and Chinese, such as "creating a web-like pattern among the gaps".

redid the sentences, if this one is no good then I will just use what Yuelinxin has provided me.
thanks and sorry for the inconvenience, also happy new year!
Copy link
Contributor

@Phoenix0415 Phoenix0415 left a comment

Choose a reason for hiding this comment

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

Great work! thanks for you PR!

docs-en/chapter_array_and_linkedlist/index.md Outdated Show resolved Hide resolved
deleted sinuous.
@krahets krahets merged commit edf3f3e into krahets:main Jan 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documents documents-related translation English translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants