Unparsing skips arguments with values equal to the argument type's default value
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 45/100
Direção de pesquisa
Reproduza o problema com o app Program e TestArgs fornecido, concentrando-se em Parser.FormatCommandLine com settings.SkipDefault = true. Compare a saída para os valores 0, 1 e 2 com as linhas de comando esperadas e, em seguida, verifique se cada resultado pode ser analisado novamente em um round-trip para os valores originais.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Using CommandLineParser 2.9.1, the unparsing behaviour with SkipDefault = true seems to be incorrect. Here is a test app to reproduce the behaviour.
using CommandLine;
namespace UnparsingTest
{
public class Program
{
public class TestArgs
{
[Option('c', Required = true)]
public int CompulsoryWithoutDefault { get; set; }
[Option('d', Required = true, Default = 1)]
public int CompulsoryWithDefault { get; set; } = 1;
[Option('o', Required = false, Default = 1)]
public int Optional { get; set; } = 1;
}
static void Main()
{
Parser parser = new Parser();
for (int i = 0; i <= 2; ++i)
{
TestArgs args = new TestArgs()
{
CompulsoryWithoutDefault = i,
CompulsoryWithDefault = i,
Optional = i,
};
string commandLine = parser.FormatCommandLine(args, settings => settings.SkipDefault = true);
System.Console.WriteLine($"CompulsoryWithDefault: {args.CompulsoryWithDefault}, CompulsoryWithoutDefault: {args.CompulsoryWithoutDefault}, Optional: {args.Optional}, Command line: \"{commandLine}\"");
}
}
}
}
There are 2 compulsory arguments, one with a Default value and one without a Default value. CompulsoryWithDefault and Optional have their Default values set to any value that is not default(int).
The test app's console output shows the unparsed command line for the 3 sets of arguments
CompulsoryWithDefault: 0, CompulsoryWithoutDefault: 0, Optional: 0, Command line: ""
CompulsoryWithDefault: 1, CompulsoryWithoutDefault: 1, Optional: 1, Command line: "-c 1"
CompulsoryWithDefault: 2, CompulsoryWithoutDefault: 2, Optional: 2, Command line: "-c 2 -d 2 -o 2"
The unparsed command line for the first case will fail round trip parsing because the compulsory -c and -d arguments are missing and -o is missing so Optional will have a value of 1 instead of 0. The unparsed command line for the second case will also fail round trip parsing because the compulsory -d argument is missing. Only the third case is giving correct output.
The bug appears to be that the unparser is skipping Required = true arguments, arguments with value equal to Default = {value} and arguments with value equal to default(typeof(argument)).
The correct behaviour would be to only skip arguments that do not have Required = true and whose value equals the value specified in Default = {value}.
The expected console output would be
CompulsoryWithDefault: 0, CompulsoryWithoutDefault: 0, Optional: 0, Command line: "-c 0 -d 0 -o 0"
CompulsoryWithDefault: 1, CompulsoryWithoutDefault: 1, Optional: 1, Command line: "-c 1 -d 1"
CompulsoryWithDefault: 2, CompulsoryWithoutDefault: 2, Optional: 2, Command line: "-c 2 -d 2 -o 2"
These command lines would then correctly round trip parse/unparse.
- Linguagem predominante
- C#
- Estrelas
- 4.8k
- Forks
- 478
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de commandlineparser/commandline
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
commandlineparser/commandline#951 · 1 comentário ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 70/100
commandlineparser/commandline#880 · 2 reações ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 45/100
commandlineparser/commandline#952 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
commandlineparser/commandline#948 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
commandlineparser/commandline#947 ·
Todas as issues de commandlineparser/commandline
Issues semelhantes
-
CS0162 "Unreachable code detected" warning from a MSBuildTemp .tmp file in every game project Abertabug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
-
Type: enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
apache/arrow-adbc#4809 ·
-
type/automation type/tech-debt
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
microsoft/vscode-azurefunctions#5197 · 1 comentário ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
microsoft/microsoft-ui-reactor#1274 ·