Method inside Execute()

I was wondering why I don’t see anyone use methods inside Execute().

 

(I'm assuming you're talking about nested functions here)
If there is no technical reason(like burst not liking it, I haven't tried it yet), then I'd assume it's because it's a new-ish feature and most people aren't aware it's now possible with C# 7. Or they just think it's bad practice or not necessary in most cases.

On the other hand, if you're just talking about having other methods in the job struct that you call from Execute(), if it makes your code any nicer/more readable then I don't see why not.

 

I used methods inside my Execute including local function feature from C#7. It works great and got inlined in the asm.

 

Can someone point me to what you are talking about with C# 7? I use static methods inside Execute() all the time. Burst will compile it great too.

 

Code (CSharp):

1. public void Execute()
2. {
3.     bool SomeMethod()
4.     {
5.         return true;
6.     }
7.  
8.     bool blabla = SomeMethod();
9. }

https://docs.microsoft.com/en-US/dotnet/csharp/programming-guide/classes-and-structs/local-functions

 

@eizenhorn Oh nice, I had no idea. That seems like it may get confusing though....

 

I pile up the methods at the bottom. When you do this enough it feels like you could turn the code into an entirely english sentence. I think it is best if you could define local function as exp. bodied => one liner so it doesn't look as unwieldly, and also great for recursion.

 

Do you have a code snippet, I’m curious