Timeline for Get users from groups in active directory, disposing of everything
Current License: CC BY-SA 3.0
6 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jul 18, 2016 at 19:50 | answer | added | Malachi | timeline score: 4 | |
| Jul 18, 2016 at 19:31 | answer | added | extremecsharper | timeline score: 5 | |
| Jul 18, 2016 at 18:20 | comment | added | Servy | Then don't intent the code so much; clearly it's not aiding readability. Of course, if you really want to keep them, then just refactor that portion out into a method. | |
| Jul 18, 2016 at 18:16 | comment | added | Michael Brandon Morris |
@Servy Bear in mind that this code in the file is indented twice, once for the namespace and once for the class. After that, the line foreach (var groupPrincipal in principalSearchResult.GetGroupPrincipals()) is indented six times and with the names spelled out like that, is 98 chars long.
|
|
| Jul 18, 2016 at 18:11 | comment | added | Servy | I don't see any lines in that snippet that wouldn't fit on the screen if you actually used meaningful variable names. You only use one or two variables per line, and you've still got lots of free horizontal space. | |
| Jul 18, 2016 at 16:55 | history | asked | Michael Brandon Morris | CC BY-SA 3.0 |