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.

Sign up for GitHub

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

Jump to bottom

Dimension subscripts for two dimensional array #29181

Closed
Soran-M opened this issue Apr 26, 2022 · 2 comments · Fixed by #36782
Closed

Dimension subscripts for two dimensional array #29181

Soran-M opened this issue Apr 26, 2022 · 2 comments · Fixed by #36782
Assignees
@BillWagner
Labels
doc-enhancement Improve the current content [org][type][category] dotnet-csharp/svc fundamentals/subsvc Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.

Comments

@Soran-M
Copy link

Soran-M commented Apr 26, 2022 •
edited by github-actions bot
Loading

[Enter feedback here]

Hi,
It would be helpful if you specify which one of the dimensions is the row and which one is the column, especially for 2 dimension arrays like int[,] array = new int[4, 2];

Based on the info in https://www.tutorialspoint.com/csharp/csharp_multi_dimensional_arrays.htm, it is int[row,column]

Also it would help if we know how the array data is saved in memory, by row or by column.

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Associated WorkItem - 148957
@PRMerger14 PRMerger14 added fundamentals/subsvc dotnet-csharp/svc Pri1 High priority, do before Pri2 and Pri3 labels Apr 26, 2022
@dotnet-bot dotnet-bot added the ⌚ Not Triaged Not triaged label Apr 26, 2022
@BillWagner BillWagner added the doc-enhancement Improve the current content [org][type][category] label Apr 27, 2022
@dotnet-bot dotnet-bot removed the ⌚ Not Triaged Not triaged label Apr 27, 2022
@BillWagner
Copy link
Member

Thanks for raising this @Soran-M

I've added this to our backlog to address.

@BillWagner BillWagner self-assigned this Aug 21, 2023
@BillWagner BillWagner added the 🗺️ reQUEST Triggers an issue to be imported into Quest. label Aug 21, 2023
@BillWagner
Copy link
Member

Fixing as part of the work on collection expressions #36356

@github-actions github-actions bot added 📌 seQUESTered Identifies that an issue has been imported into Quest. and removed 🗺️ reQUEST Triggers an issue to be imported into Quest. labels Aug 22, 2023
BillWagner added a commit to BillWagner/docs that referenced this issue Aug 25, 2023
@BillWagner
Edit pass on Arrays article
3067d01 
Edit and update the Arrays article. Remove older outdated samples, update style (both text and code), and fix two open issues:

- Fix dotnet#27181 Clarify the default value for array elements.
- Fix dotnet#29181 Clarify the row, column for a 2D array.
@BillWagner BillWagner mentioned this issue Aug 25, 2023
Merged
@ghost ghost added the in-pr This issue will be closed (fixed) by an active pull request. label Aug 25, 2023
BillWagner added a commit to BillWagner/docs that referenced this issue Aug 31, 2023
@BillWagner
Edit pass on Arrays article
8ff6a29 
Edit and update the Arrays article. Remove older outdated samples, update style (both text and code), and fix two open issues:

- Fix dotnet#27181 Clarify the default value for array elements.
- Fix dotnet#29181 Clarify the row, column for a 2D array.
BillWagner added a commit that referenced this issue Aug 31, 2023
@BillWagner @gewarren
Add Collection expressions to the language reference (#36782)
b8f805b 
* Update TOC

Move programming guide articles on arrays and collections into the language reference section. This set of content will be updated and edited, then collection expressions will be added.

* Move array content

Move all array content into the language reference.

* Move all snippets

Move snippets to the language reference location.

* Update internal links

Run cleanrepo to update redirected links with the new target, and replace absolute links with relative links.

* Edit pass on collections article

Fix open issues on the collection articles, and perform an edit pass to update.

Simplify examples and remove outdated examples. Update text. Instead of listing all collections, refer to articles in the .NET fundamentals and API reference where all collections are listed.

* Edit pass on Arrays article

Edit and update the Arrays article. Remove older outdated samples, update style (both text and code), and fix two open issues:

- Fix #27181 Clarify the default value for array elements.
- Fix #29181 Clarify the row, column for a 2D array.

* rough draft of collection expressions.

* grammar check and build issues

Do a grammar check on the new article, fix build warnings, and add links.

* modify title

The language reference and spec titles were duplicated

* Apply suggestions from code review

Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>

* respond to feedback.

---------

Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@ghost ghost removed the in-pr This issue will be closed (fixed) by an active pull request. label Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BillWagner BillWagner

Labels
doc-enhancement Improve the current content [org][type][category] dotnet-csharp/svc fundamentals/subsvc Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.
Projects
No open projects
dotnet/docs August 2023 sprint
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Collection expressions to the language reference BillWagner/docs
4 participants
@BillWagner @dotnet-bot @Soran-M @PRMerger14