DOCSP-49279: $convert stage LINQ method #604
Conversation
✅ Deploy Preview for docs-csharp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
source/fundamentals/linq.txt
Outdated
| .. code-block:: csharp | ||
|
|
||
| var query = queryableCollection | ||
| .Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>())); |
There was a problem hiding this comment.
Hi @papafe would you prefer if we showed a more complex example that uses the options?
There was a problem hiding this comment.
I think the example is fine, it's difficult to make a more complex example using the dataset we have in this page. But at least you can show OnError or OnNull. So it could be:
var query = queryableCollection
.Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>( OnError = -1, OnNull = 0 )));
source/fundamentals/linq.txt
Outdated
| $convert | ||
| ~~~~~~~~ | ||
|
|
||
| You can use the ``Mql.Convert()`` method in a ``Select()`` projection to |
There was a problem hiding this comment.
The use of Mql.Convert is not limited only to Select, even though it could be the most common.
source/fundamentals/linq.txt
Outdated
| .. code-block:: csharp | ||
|
|
||
| var query = queryableCollection | ||
| .Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>())); |
There was a problem hiding this comment.
I think the example is fine, it's difficult to make a more complex example using the dataset we have in this page. But at least you can show OnError or OnNull. So it could be:
var query = queryableCollection
.Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>( OnError = -1, OnNull = 0 )));
source/fundamentals/linq.txt
Outdated
| $convert | ||
| ~~~~~~~~ | ||
|
|
||
| You can use the ``Mql.Convert()`` method in a ``Select()`` projection to |
There was a problem hiding this comment.
I can see from the other operators in the page that usually we first introduce briefly the operator, and then show how to use it in LINQ. Maybe we should follow that pattern?
There was a problem hiding this comment.
I moved some sections around to distinguish between stages/operators
source/fundamentals/linq.txt
Outdated
| The ``Convert()`` method takes the following parameters: | ||
|
|
||
| - Value to convert. | ||
| - ``ConvertOptions<Tto>`` instance that specifies the type to convert to |
There was a problem hiding this comment.
Here we can omit TTo, as we did not introduce it before.
source/fundamentals/linq.txt
Outdated
| .. code-block:: csharp | ||
|
|
||
| var query = queryableCollection | ||
| .Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>( OnError = -1, OnNull = 0 ))); |
There was a problem hiding this comment.
| .Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int>( OnError = -1, OnNull = 0 ))); | |
| .Select(r => Mql.Convert(r.RestaurantId, new ConvertOptions<int> { OnError = -1, OnNull = 0 })); |
My bad, I used the wrong parenthesis in the example 🤦
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-49279
Staging Links
Self-Review Checklist