Code cleanup of Rider vs ReSharper

Answered

as far as I looked into it, In resharper you can set way more fine details about your cleanup. When I use the cleanup built in in Rider, It seems to have way less options. Or am I missing something?

I thought about writing code in Rider and cleaning it up in vs with resharper but that seems odd...

6 comments
Comment actions Permalink

Hello,

R# and Rider's code cleanup profiles and available options for change should be identical, as you can see in the corresponding Help section for both products:

https://www.jetbrains.com/help/resharper/Code_Cleanup__Index.html

https://www.jetbrains.com/help/rider/Code_Cleanup__Index.html

I would appreciate you giving me an example of the difference you have noticed so I could check it and please let me know the versions of both products you are currently using. Thank you in advance!

0
Comment actions Permalink

Hey, i just stumbled upon this:

https://youtrack.jetbrains.com/issue/RIDER-15021

so it seems what I've been missing, which is mainly File Layout Settings, will soon be available with Rider, too (I know it is already, but I'd wait for the UI Version)
Or I'll check if I can export the XAML file from VS with Resharper and import it in Rider...!

Thanks for your answer!

0
Comment actions Permalink

I have tested clean-up with both Rider and ReSharper (latest versions) and have had much better results with ReSharper. Twice I have seen Rider clean-up actually make source code errors (corruptions) which break a build. It's also slow.

I share a clean-up profile between RIder and ReSharper using a dotSettings file (https://github.com/finlabsuk/open-banking-connector-csharp/blob/master/FinnovationLabs.OpenBanking.Library.Connector.sln.DotSettings) and Rider always tries to add annoying "SettingsMigration/IsMigratorApplied" lines. This happens even when creating fresh new dotSettings files in Rider or ReSharper which suggests something is wrong and I'm reluctant to commit such additions. They are not required on ReSharper which is where I now do all my cleanup.

 

0
Comment actions Permalink

Note on above: I have noticed ReSharper creating same IsMigratorApplied lines now too (not sure when this started or if I somehow missed it before?). But main good thing is ReSharper (Windows) and Rider (macOS) are now doing the same thing so I have committed these lines. I will start to report this and other issues I find systematically to JetBrains to try to help make Rider/ReSharper better products. (I realise this particular issue may be design behaviour but I think should be checked as I would prefer not to have noise like this in files under revision control.)

NB: issue can be reproduced by setting "Place property/indexer/event attribute on the same line" to "Never" in ReSharper and saving setting update to shared DotSettings file. You will then end up with 

<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation"> <s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_ACCESSORHOLDER_ATTRIBUTE_ON_SAME_LINE_EX/@EntryValue">NEVER</s:String> <s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpKeepExistingMigration/@EntryIndexedValue">True</s:Boolean> <s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpPlaceEmbeddedOnSameLineMigration/@EntryIndexedValue">True</s:Boolean> <s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpUseContinuousIndentInsideBracesMigration/@EntryIndexedValue">True</s:Boolean> <s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateBlankLinesAroundFieldToBlankLinesAroundProperty/@EntryIndexedValue">True</s:Boolean></wpf:ResourceDictionary>

 

 

0
Comment actions Permalink

Hello Mark,

Thank you for your report. There is an existing record on our tracker dedicated to the behaviour described: RSPR-473323. We would appreciate it if you would upvote it in order to bring increased awareness to the issue. Also please click to Watch it for monitoring the status.

0
Comment actions Permalink

Thanks Olga, I've upvoted and commented on the issue you referenced.

0

Please sign in to leave a comment.