Don't throw when out of accessible expressions

Author: koenbeuk

src/EntityFrameworkCore.Projectables.Generator/ExpressionSyntaxRewriter.cs

@@ -112,34 +112,34 @@ public ExpressionSyntaxRewriter(INamedTypeSymbol targetTypeSymbol, NullCondition
 
         public override SyntaxNode? VisitMemberBindingExpression(MemberBindingExpressionSyntax node)
         {
-            if (_conditionalAccessExpressionsStack.Count == 0)
+            if (_conditionalAccessExpressionsStack.Count > 0)
             {
-                throw new InvalidOperationException("Expected at least one conditional expression on the stack");
-            }
+                var targetExpression = _conditionalAccessExpressionsStack.Pop();
 
-            var targetExpression = _conditionalAccessExpressionsStack.Pop();
+                return _nullConditionalRewriteSupport switch {
+                    NullConditionalRewriteSupport.Ignore => SyntaxFactory.MemberAccessExpression(SyntaxKind.SimpleMemberAccessExpression, targetExpression, node.Name),
+                    NullConditionalRewriteSupport.Rewrite => SyntaxFactory.MemberAccessExpression(SyntaxKind.SimpleMemberAccessExpression, targetExpression, node.Name),
+                    _ => node
+                };
+            }
 
-            return _nullConditionalRewriteSupport switch {
-                NullConditionalRewriteSupport.Ignore => SyntaxFactory.MemberAccessExpression(SyntaxKind.SimpleMemberAccessExpression, targetExpression, node.Name),
-                NullConditionalRewriteSupport.Rewrite => SyntaxFactory.MemberAccessExpression(SyntaxKind.SimpleMemberAccessExpression, targetExpression, node.Name),
-                _ => node
-            };
+            return base.VisitMemberBindingExpression(node);
         }
 
         public override SyntaxNode? VisitElementBindingExpression(ElementBindingExpressionSyntax node)
         {
-            if (_conditionalAccessExpressionsStack.Count == 0)
+            if (_conditionalAccessExpressionsStack.Count > 0)
             {
-                throw new InvalidOperationException("Expected at least one conditional expression on the stack");
-            }
+                var targetExpression = _conditionalAccessExpressionsStack.Pop();
 
-            var targetExpression = _conditionalAccessExpressionsStack.Pop();
+                return _nullConditionalRewriteSupport switch {
+                    NullConditionalRewriteSupport.Ignore => SyntaxFactory.ElementAccessExpression(targetExpression, node.ArgumentList),
+                    NullConditionalRewriteSupport.Rewrite => SyntaxFactory.ElementAccessExpression(targetExpression, node.ArgumentList),
+                    _ => Visit(node)
+                };
+            }
 
-            return _nullConditionalRewriteSupport switch {
-                NullConditionalRewriteSupport.Ignore => SyntaxFactory.ElementAccessExpression(targetExpression, node.ArgumentList),
-                NullConditionalRewriteSupport.Rewrite => SyntaxFactory.ElementAccessExpression(targetExpression, node.ArgumentList),
-                _ => Visit(node)
-            };
+            return base.VisitElementBindingExpression(node);
         }
 
         public override SyntaxNode? VisitThisExpression(ThisExpressionSyntax node)