Issue #5570 Add doc for C# String.Split() #5581
Conversation
|
|
mamtawardhani
added
new entry
There was a problem hiding this comment.
Hey @mushahidq, thank you for contributing to Codecademy Docs, the entry is well written! 😄
I've requested some changes, could you please review and modify those at your earliest convenience? Thank you! 😃
| // Splits a string using a character as a delimeter | ||
| string[] subs = String.Split(char) | ||
|
|
||
| // Splits a string using a string as a delimeter | ||
| string[] subs = String.Split(string) | ||
|
|
||
| // Splits a string using a character as a delimeter into maximum substrings of the number specified in the second argument. | ||
| string[] subs = String.Split(char, Int) | ||
|
|
||
| // Splits a string using a character as a delimeter into maximum substrings of the number specified in the second argument. | ||
| string[] subs = String.Split(string, Int) |
There was a problem hiding this comment.
Can we separate these as follows:
Using a single character as a delimiter:
string[] subs = String.Split(char)
Using a string as a delimiter:
string[] subs = String.Split(string)
and others...
| `.Split()` is a method of class `String`. This method can take upto 3 parameters: | ||
| 1. The first parameter is the delimeter and it can be a `Char`, an array of characters `Char[]`, or a `String` | ||
| 2. The second parameter is optional and could be the manximum number of desired substrings which is specified as an integer of type `Int32` | ||
| 3. The third parameter which is also optional could contain the split options `StringSplitOptions` |
There was a problem hiding this comment.
The parameters should be described in the following format:
char:string:
and other parameters
| @@ -0,0 +1,116 @@ | |||
| --- | |||
| Title: '.Split()' | |||
| Description: 'Used to break string into substrings by specifying a delimeter.' | |||
There was a problem hiding this comment.
| Description: 'Used to break string into substrings by specifying a delimeter.' | |
| Description: 'Divides a string into an array of substrings based on specified delimiters.' |
There was a problem hiding this comment.
Description should begin with a verb
| - 'paths/computer-science' | ||
| --- | ||
|
|
||
| The `.Split()`** method breaks a string into substrings based on a delimeter which can be a character or another string and returns an array of strings |
There was a problem hiding this comment.
| The `.Split()`** method breaks a string into substrings based on a delimeter which can be a character or another string and returns an array of strings | |
| The **`.Split()`** method breaks a string into substrings based on a specified delimiter, which can be a character, an array of characters, or a string. It returns an array of strings containing the split substrings. |
There was a problem hiding this comment.
Reframed for better clarity
mamtawardhani
added
status: review 1️⃣ completed
status: waiting for author
and removed
status: under review
|
Hey @mushahidq are you still working on this? |
|
Yes, I am. I'll make the requested changes by EOD |
minor fixes
There was a problem hiding this comment.
Thank you for contributing to Codecademy Docs @mushahidq 😄
The entry looks good for a second round of review! 🚀
| Console.WriteLine($"Substring: {sub}"); | ||
| } | ||
|
|
||
| // To remove spaces too, we can specify ', ' as the delimeter |
There was a problem hiding this comment.
| // To remove spaces too, we can specify ', ' as the delimeter | |
| // To remove spaces, specify ', ' as the delimiter. |
| Console.WriteLine($"Substring: {sub}"); | ||
| } | ||
|
|
||
| // To limit our substring to 2 we can specify the optional parameter |
There was a problem hiding this comment.
| // To limit our substring to 2 we can specify the optional parameter | |
| // To limit the number of substrings to 2, specify the optional parameter. |
There was a problem hiding this comment.
Thank you for contributing to Codecademy Docs @mushahidq 😄
The entry looks good for a second round of review! 🚀
mamtawardhani
added
status: ready for next review
and removed
status: waiting for author
labels
PragatiVerma18
added
status: under review
Description
Added a new entry for
String.Split()method in C#Issue Solved
Issue #5570
Type of Change
Checklist
mainbranch.Issues Solvedsection.