Move methods

Author: JohanLarsson

WpfAnalyzers/CodeFixes/ImplementValueConverterFix.cs

@@ -79,7 +79,7 @@ semanticModel is { } &&
                     {
                         context.RegisterCodeFix(
                             "Implement IMultiValueConverter.ConvertBack for one way bindings.",
-                            (editor, _) => editor.AddMethod(classDeclaration, IMultiValueConverterConvertBack(classDeclaration.Identifier.ValueText)),
+                            (editor, _) => editor.AddMethod(classDeclaration, IMultiValueConverter.ConvertBack(classDeclaration.Identifier.ValueText)),
                             "Implement IMultiValueConverter",
                             diagnostic);
                     }
@@ -108,17 +108,6 @@ private static bool HasInterface(ClassDeclarationSyntax classDeclaration, Qualif
         return false;
     }
 
-    private static MethodDeclarationSyntax IMultiValueConverterConvertBack(string containingTypeName)
-    {
-        var code = StringBuilderPool.Borrow()
-                                    .AppendLine("        object[] System.Windows.Data.IMultiValueConverter.ConvertBack(object value, System.Type[] targetTypes, object parameter, System.Globalization.CultureInfo culture)")
-                                    .AppendLine("        {")
-                                    .AppendLine($"            throw new System.NotSupportedException($\"{{nameof({containingTypeName})}} can only be used in OneWay bindings\");")
-                                    .AppendLine("        }")
-                                    .Return();
-        return ParseMethod(code);
-    }
-
     private static TypeSyntax ParseTypeName(string text) => SyntaxFactory.ParseTypeName(text).WithSimplifiedNames();
 
     private static TypeSyntax Object(NullableContext nullableContext) => nullableContext switch
@@ -127,14 +116,6 @@ private static MethodDeclarationSyntax IMultiValueConverterConvertBack(string co
         _ => SyntaxFactory.PredefinedType(SyntaxFactory.Token(SyntaxKind.ObjectKeyword)),
     };
 
-    private static MethodDeclarationSyntax ParseMethod(string code)
-    {
-        return Parse.MethodDeclaration(code)
-                    .WithSimplifiedNames()
-                    .WithLeadingTrivia(SyntaxFactory.ElasticMarker)
-                    .WithTrailingTrivia(SyntaxFactory.ElasticMarker);
-    }
-
     private static class IValueConverter
     {
         internal static MethodDeclarationSyntax Convert(SyntaxGenerator generator, NullableContext nullableContext)
@@ -215,5 +196,24 @@ internal static MethodDeclarationSyntax Convert(SyntaxGenerator generator, Nulla
                 },
                 statements: new[] { generator.ThrowStatement(generator.ObjectCreationExpression(ParseTypeName("System.NotImplementedException"))) });
         }
+
+        internal static MethodDeclarationSyntax ConvertBack(string containingTypeName)
+        {
+            var code = StringBuilderPool.Borrow()
+                                        .AppendLine("        object[] System.Windows.Data.IMultiValueConverter.ConvertBack(object value, System.Type[] targetTypes, object parameter, System.Globalization.CultureInfo culture)")
+                                        .AppendLine("        {")
+                                        .AppendLine($"            throw new System.NotSupportedException($\"{{nameof({containingTypeName})}} can only be used in OneWay bindings\");")
+                                        .AppendLine("        }")
+                                        .Return();
+            return ParseMethod(code);
+
+            static MethodDeclarationSyntax ParseMethod(string code)
+            {
+                return Parse.MethodDeclaration(code)
+                            .WithSimplifiedNames()
+                            .WithLeadingTrivia(SyntaxFactory.ElasticMarker)
+                            .WithTrailingTrivia(SyntaxFactory.ElasticMarker);
+            }
+        }
     }
 }