Refactor encode_action_body

[bernardnormier]

This code:

icerpc-csharp/tools/slicec-cs/src/encoding.rs

Line 349 in b06a0d0

fn encode_action_body(

should be simplified. It's confusing to compute value and then most of the time not use it but instead hardcode "value".

It would be nice to add some comments too. For example, explain why we create this strange helper variable:

let is_optional = type_ref.is_optional && !is_tagged;

[InsertCreativityHere]

This function got some simplification in #3916.
There is still more opportunity for simplification, but it would require untangling the other functions that call into it first.