What modification required to solve logically dead code issue

Question

0.00/5 (No votes)

See more:

Here is my block of code where i'm getting issue like -
"Execution cannot reach the expression void() inside this statement logContext?.Add(kvp.Key((..."

C#

var dict = JsonConvert.DeserializeObject<dictionary<string, object="">>(jsonString);                 
var logContext = new JObject();                 
foreach (var kvp in dict)                 
{                     
logContext?.Add(kvp.Key, kvp.Value as JToken ?? new JValue(kvp.Value));                 
}

What I have tried:

any idea, what i'm missing and to modify, plz suggest ?

Posted 5-May-21 0:26am

bholey

Updated 11-Jun-21 7:04am

PIEBALDconsult

v2

Add a Solution

2 solutions

Solution 2

logContext?.Add(kvp.Key, kvp.Value as JToken ?? new JValue(kvp.Value));

removing of ? null check from logContext?, resolved security risk vulnerabilities

Posted 11-Jun-21 7:04am

bholey

Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2021-05-05T00:45:00

When I try it (in a slightly "cut down" form as I don't have your JSON data) I get no such error:

C#

var dict = new Dictionary < string, object>();
var logContext = new JObject();
foreach (var kvp in dict)
    {
    logContext?.Add(kvp.Key, kvp.Value as JToken ?? new JValue(kvp.Value));
    }

I even tried "forcing" the conversion:

C#

logContext?.Add(kvp.Key, (kvp.Value as JToken) ?? new JValue(kvp.Value));

but it made no difference.
So ... we can't help you directly.
I'd start by changing all the var definitions to explicit types, to be sure of what I was working with (VS will do that for you) then break out your single line of code into multiple statements to try and see what part of it is doesn't like.

Sorry - but if we can't reproduce it, we can't solve it!

What modification required to solve logically dead code issue

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

What modification required to solve logically dead code issue

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

Existing Members

...or Join us