Skip to content

Commit b6082e4

Browse files
committed
REF extract ParseBoolOption method
1 parent f71f391 commit b6082e4

File tree

2 files changed

+20
-16
lines changed

2 files changed

+20
-16
lines changed

ClientLib/OptionsParser.cs

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,15 @@ public void PrintHelp()
5252
Console.WriteLine("<type_of_client> [opts]");
5353
this.WriteOptionDescriptions(Console.Out);
5454
}
55+
56+
protected static bool ParseBoolOption(string durable)
57+
{
58+
if (durable == "yes" || durable == "true" || durable == "True")
59+
return true;
60+
if (durable == "no" || durable == "false" || durable == "False")
61+
return false;
62+
throw new ArgumentException();
63+
}
5564
}
5665

5766
/// <summary>
@@ -319,14 +328,7 @@ public SenderOptions() : base()
319328
this.Add("property-type=", "specify message property type (overrides auto-cast feature)",
320329
(string propertyType) => { this.PropertyType = propertyType; });
321330
this.Add("msg-durable=", "send durable messages yes/no",
322-
(string durable) => {
323-
if (((durable == "yes") || (durable == "true")) || (durable == "True"))
324-
this.Durable = true;
325-
else if (((durable == "no") || (durable == "false")) || (durable == "False"))
326-
this.Durable = false;
327-
else
328-
throw new ArgumentException();
329-
});
331+
(string durable) => { this.Durable = ParseBoolOption(durable); });
330332
this.Add("msg-ttl=", "message time-to-live (ms)",
331333
(uint ttl) => { this.Ttl = ttl; });
332334
this.Add("msg-priority=", "message priority",

ClientLibNetCore/OptionsParser.cs

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,15 @@ public void PrintHelp()
5252
Console.WriteLine("<type_of_client> [opts]");
5353
this.WriteOptionDescriptions(Console.Out);
5454
}
55+
56+
protected static bool ParseBoolOption(string durable)
57+
{
58+
if (durable == "yes" || durable == "true" || durable == "True")
59+
return true;
60+
if (durable == "no" || durable == "false" || durable == "False")
61+
return false;
62+
throw new ArgumentException();
63+
}
5564
}
5665

5766
/// <summary>
@@ -319,14 +328,7 @@ public SenderOptions() : base()
319328
this.Add("property-type=", "specify message property type (overrides auto-cast feature)",
320329
(string propertyType) => { this.PropertyType = propertyType; });
321330
this.Add("msg-durable=", "send durable messages yes/no",
322-
(string durable) => {
323-
if (((durable == "yes") || (durable == "true")) || (durable == "True"))
324-
this.Durable = true;
325-
else if (((durable == "no") || (durable == "false")) || (durable == "False"))
326-
this.Durable = false;
327-
else
328-
throw new ArgumentException();
329-
});
331+
(string durable) => { this.Durable = ParseBoolOption(durable); });
330332
this.Add("msg-ttl=", "message time-to-live (ms)",
331333
(uint ttl) => { this.Ttl = ttl; });
332334
this.Add("msg-priority=", "message priority",

0 commit comments

Comments
 (0)