The Weird and The Wonderful

Maximilien wrote:

They are hints.

Maximilien wrote:

In that case, it is not useful

raddevus wrote:

AI you have failed me.

M.D.V.

If something has a solution... Why do we have to worry about?. If it has no solution... For what reason do we have to worry about?
Help me to understand what I'm saying, and I'll explain it better to you
Rating helpful answers is nice, but saying thanks can be even nicer.

Nelek wrote:

Problem found.

Nelek wrote:

BTW... I am with you in this case.

Jon McKee wrote:

I'm curious if this error still pops up if you write more code below the if statement.

static void Main(string[] args)
        {
            if (args.Length < 1){
                Console.WriteLine("Please provide 1 argument to indicate the command you want to run.\nUsage: getInfo <command-name>");
                return;
            }

            switch (args[0].ToLower()){
                case "os":{
                    Console.WriteLine($"OS : {Environment.OSVersion}");
                    break;
                }
                case "pwd":{
                    Console.WriteLine($"The current directory is: {Environment.CurrentDirectory}");
                    break;
                }
                case "cl":{
                    Console.WriteLine($"Command line was: {Environment.CommandLine}");
                    break;
                }
                case "sysdir":{
                    Console.WriteLine($"System dir: {Environment.SystemDirectory}");
                    break;
                }
                case "mname":{
                    Console.WriteLine($"Machine name: {Environment.MachineName}");
                    break;
                }
            }
        }

static void Main(string[] args)
        {
            if (args.Length >= 1)
            {
                switch (args[0].ToLower())
                {
                    case "os":
                        {
                            Console.WriteLine($"OS : {Environment.OSVersion}");
                            break;
                        }
                    case "pwd":
                        {
                            Console.WriteLine($"The current directory is: {Environment.CurrentDirectory}");
                            break;
                        }
                    case "cl":
                        {
                            Console.WriteLine($"Command line was: {Environment.CommandLine}");
                            break;
                        }
                    case "sysdir":
                        {
                            Console.WriteLine($"System dir: {Environment.SystemDirectory}");
                            break;
                        }
                    case "mname":
                        {
                            Console.WriteLine($"Machine name: {Environment.MachineName}");
                            break;
                        }
                }
            }
            else
            {
                Console.WriteLine("Please provide 1 argument to indicate the command you want to run.\nUsage: getInfo <command-name>");
                return;
            }
        }

Jon McKee wrote:

seems like the purpose of this rule (from some short googling) is to reduce conditional nesting and/or cyclomatic complexity.

static void Main(string[] args)
{
    if (args.Length == 0)
    {
        Console.WriteLine("Please provide 1 argument to indicate the command you want to run.\nUsage: getInfo <command-name>");
        return;
    }
    
    foreach (string arg in args)
    {
        Console.WriteLine(arg.ToLowerInvariant() switch
        {
            "os" => $"OS : {Environment.OSVersion}",
            "pwd" => $"The current directory is: {Environment.CurrentDirectory}",
            "cl" => $"Command line was: {Environment.CommandLine}",
            "sysdir" => $"System dir: {Environment.SystemDirectory}",
            "mname" => $"Machine name: {Environment.MachineName}",
            _ => $"Unknown command: '{arg}'",
        });
    }
}

Richard Deeming wrote:

I don't like the args.Length < 1 test

Bernhard Hiller wrote:

Here's a hammer. Now you have to use it, even when there's no nail.

M.D.V.

If something has a solution... Why do we have to worry about?. If it has no solution... For what reason do we have to worry about?
Help me to understand what I'm saying, and I'll explain it better to you
Rating helpful answers is nice, but saying thanks can be even nicer.

Greg Utas wrote:

Because of pre-fetching, many CPUs perform better when the non-jump case is taken. If the tool is guessing that the non-jump case is actually the rare one, it might be making the suggestion on those grounds.

Greg Utas wrote:

worked on a product where the tags <usual> and <rare> could be used to tag code paths to assist the compiler with optimization.

StringBuilder sb = new StringBuilder(120);

var sb = new StringBuilder(120);

StringBuilder sb = new StringBuilder(120);