- Reading empty parameter values

- Parameter name as a single character
- Generating an exception when accessing a non-existent section
This commit is contained in:
Alexander Zhirov 2023-06-07 23:16:59 +03:00
parent bc65396f71
commit ff1bc5db66
11 changed files with 61 additions and 41 deletions

View File

@ -1,5 +1,16 @@
# Changelog # Changelog
## [v0.4.0](https://git.zhirov.kz/dlang/readconf/compare/v0.3.1...v0.4.0) (2023.04.30)
### New
- Reading empty parameter values
- Parameter name as a single character
### Bug fixes
- Generating an exception when accessing a non-existent section
## [v0.3.1](https://git.zhirov.kz/dlang/readconf/compare/v0.3.0...v0.3.1) (2023.04.30) ## [v0.3.1](https://git.zhirov.kz/dlang/readconf/compare/v0.3.0...v0.3.1) (2023.04.30)
- Windows OS support - Windows OS support

View File

@ -34,15 +34,15 @@ void main()
{ {
rc.read("./tests/settings.conf"); rc.read("./tests/settings.conf");
foreach (key, param; rc.sn.keys()) foreach (key, param; rc.cf.sn.keys())
writefln("%s => %s", key, param); writefln("%s => %s", key, param);
writeln(rc.sn.key("value1")); writeln(rc.cf.sn.key("value1"));
foreach (key, param; rc.sn("part2").keys()) foreach (key, param; rc.cf.sn("part2").keys())
writefln("%s => %s", key, param); writefln("%s => %s", key, param);
writeln(rc["part2"]["value1"]); writeln(rc[]["part2"]["value1"]);
} }
``` ```
@ -54,6 +54,7 @@ value2 => Yes!
value3 => value in apostrophes value3 => value in apostrophes
value4 => 1000 value4 => 1000
value5 => 0.000 value5 => 0.000
value6 =>
value7 => //path value7 => //path
value8 => "Hey!" value8 => "Hey!"
text without quotes text without quotes
@ -68,14 +69,14 @@ The unittests provide [examples](examples/) of configuration files and the `sett
```sh ```sh
Running bin/readconf-test-unittest Running bin/readconf-test-unittest
✓ test __unittest_L111_C1
✓ test __unittest_L26_C1
✓ test __unittest_L52_C1
✓ test __unittest_L4_C1 ✓ test __unittest_L4_C1
✓ test __unittest_L106_C1
✓ test __unittest_L25_C1
✓ test __unittest_L51_C1
Summary: 4 passed, 0 failed in 7 ms Summary: 4 passed, 0 failed in 7 ms
``` ```
## DUB ## DUB
Add a dependency on `"readconf": "~>0.3.1"` Add a dependency on `"readconf": "~>0.4.0"`

View File

@ -17,22 +17,14 @@
"name": "unittest", "name": "unittest",
"targetPath": "bin", "targetPath": "bin",
"dependencies": { "dependencies": {
"silly": "~>1.1.1" "silly": "~>1.2.0-dev.2"
}, },
"importPaths": ["source","tests"], "importPaths": ["source","tests"],
"sourcePaths": ["tests"] "sourcePaths": ["tests"]
},
{
"name": "executable",
"targetType": "executable",
"targetPath": "bin",
"targetName": "app",
"importPaths": ["source","tests"],
"sourcePaths": ["tests"]
} }
], ],
"targetName": "readconf", "targetName": "readconf",
"dependencies": { "dependencies": {
"singlog": "~>0.3.0" "singlog": "~>0.4.0"
} }
} }

View File

@ -2,7 +2,7 @@
"fileVersion": 1, "fileVersion": 1,
"versions": { "versions": {
"datefmt": "1.0.4", "datefmt": "1.0.4",
"silly": "1.1.1", "silly": "1.2.0-dev.2",
"singlog": "0.3.0" "singlog": "0.4.0"
} }
} }

View File

@ -5,7 +5,8 @@ parameter3 => value3 // This will be a comment
parameter4 => value4 /* This will be a comment parameter4 => value4 /* This will be a comment
parameter5 => value5;This will not be a comment parameter5 => value5;This will not be a comment
parameter6 => value6// This will also be a whole value parameter6 => value6// This will also be a whole value
parameter7 => //value7 ;The value will not be read parameter7 => //value7 ;The value will be read as empty value
parameter8 => "//value8" # Now the value is correctly parameter8 => "//value8" # Now the value is correctly
parameter9 => ';value9' // The value is correctly too parameter9 => ';value9' // The value is correctly too
parameter10 => '"value10"' // Quotes inside parameter10 => '"value10"' // Quotes inside
parameter11 => ; empty value

View File

@ -19,4 +19,4 @@ parameter4 = value10
[_section] # Creating the new section [_section] # Creating the new section
parameter1 = value11 parameter1 = value11
parameter2 = value12 parameter2 = value12

View File

@ -13,4 +13,5 @@ parameter-10 =>value10
parameter11 = value11 parameter11 = value11
parameter12_ => value12 parameter12_ => value12
parameter13 =

Binary file not shown.

Before

Width:  |  Height:  |  Size: 137 KiB

After

Width:  |  Height:  |  Size: 119 KiB

View File

@ -21,12 +21,12 @@ class Config
private: private:
enum { enum {
GROUP_PARAMETER = 4, GROUP_PARAMETER = 4,
GROUP_VALUE_1 = 11, // string GROUP_VALUE_1 = 12, // string
GROUP_VALUE_2 = 14, // "strin" GROUP_VALUE_2 = 15, // "strin"
GROUP_VALUE_3 = 16, // 'string' GROUP_VALUE_3 = 17, // 'string'
GROUP_SECTION_OTHER_OUTER = 17, // [string] GROUP_SECTION_OTHER_OUTER = 19, // [string]
GROUP_SECTION_OTHER_INNER = 18, // string GROUP_SECTION_OTHER_INNER = 20, // string
GROUP_SECTION_MAIN = 20, // [] GROUP_SECTION_MAIN = 23, // []
} }
static Config config; static Config config;
@ -34,9 +34,9 @@ private:
bool readed = false; bool readed = false;
ConfigFile[string] configs; ConfigFile[string] configs;
const string pattern = "^( |\\t)*(((\\w(\\w|-)+)(( |\\t)*(=>|=){1}" const string pattern = "^( |\\t)*((((\\w(\\w|-)?)+)(( |\\t)*(=>|=){1}"
~ "( |\\t)*)(?!\\/(\\/|\\*))(([^ >\"'=\\n\\t#;].*?)|(\"(.+)\")" ~ "( |\\t)*)(?!\\/(\\/|\\*))(([^ >\"'=\\n\\t#;].*?)|(\"(.+)?\")"
~ "|('(.+)')){1})|(\\[(\\w(\\w|-)+)\\])|(\\[\\]))( |\\t)*" ~ "|('(.+)?')|()){1})|(\\[((\\w(\\w|-)?)+)\\])|(\\[\\]))( |\\t)*"
~ "(( |\\t)(#|;|\\/\\/|\\/\\*).*)?$"; ~ "(( |\\t)(#|;|\\/\\/|\\/\\*).*)?$";
/** /**
@ -84,10 +84,12 @@ private:
int group = GROUP_VALUE_1; int group = GROUP_VALUE_1;
if (match[group][0] == '\"') if (match[group].length) {
group = GROUP_VALUE_2; if (match[group][0] == '\"')
else if (match[group][0] == '\'') group = GROUP_VALUE_2;
group = GROUP_VALUE_3; else if (match[group][0] == '\'')
group = GROUP_VALUE_3;
}
this.configs[configName].add(sectionName, ConfigParameter(match[GROUP_PARAMETER], match[group])); this.configs[configName].add(sectionName, ConfigParameter(match[GROUP_PARAMETER], match[group]));
} }
@ -207,7 +209,10 @@ struct ConfigFile
return sections[sections.byKey.front]; return sections[sections.byKey.front];
if (section.length == 0) if (section.length == 0)
section = mainSection; section = mainSection;
return section in sections ? sections[section] : ConfigSection(section); if (section !in sections)
throw new Exception("The selected section does not exist");
return sections[section];
} }
/** /**
@ -234,7 +239,10 @@ struct ConfigFile
return sections[sections.byKey.front]; return sections[sections.byKey.front];
if (section.length == 0) if (section.length == 0)
section = mainSection; section = mainSection;
return section in sections ? sections[section] : ConfigSection(section); if (section !in sections)
throw new Exception("The selected section does not exist");
return sections[section];
} }
} }

View File

@ -5,8 +5,8 @@ value2=>"value in quotation marks"
value3=>'value in apostrophes' ; and here is the first comment value3=>'value in apostrophes' ; and here is the first comment
value4 => 1000 // free space in spaces and tabs value4 => 1000 // free space in spaces and tabs
value5 => 0.000 /* different form of commenting value5 => 0.000 /* different form of commenting
value6 = // the string will not be read because the parameter value is missing value6 = // the string will be read as empty value
value7 = '' ; limiting empty characters will also not be taken into account value7 = '' ; limiting empty characters will also be taken into account
[part2] ; a new section! [part2] ; a new section!
value1 = this value will be in the new section value1 = this value will be in the new section
value 2 = 200 ; The parameter name is incorrect value 2 = 200 ; The parameter name is incorrect

View File

@ -20,6 +20,7 @@ unittest
assert(mainSection["parameter-10"] == "value10"); assert(mainSection["parameter-10"] == "value10");
assert(mainSection.key("parameter11") == "value11"); assert(mainSection.key("parameter11") == "value11");
assert(mainSection["parameter12_"] == "value12"); assert(mainSection["parameter12_"] == "value12");
assert(mainSection["parameter13"].empty);
} }
unittest unittest
@ -98,9 +99,13 @@ unittest
assert(comConMaiSec["parameter4"] == "value4"); assert(comConMaiSec["parameter4"] == "value4");
assert(comConMaiSec["parameter5"] == "value5;This will not be a comment"); assert(comConMaiSec["parameter5"] == "value5;This will not be a comment");
assert(comConMaiSec["parameter6"] == "value6// This will also be a whole value"); assert(comConMaiSec["parameter6"] == "value6// This will also be a whole value");
assert(comConMaiSec["parameter7"] == "");
assert(comConMaiSec["parameter7"].empty);
assert(comConMaiSec["parameter8"] == "//value8"); assert(comConMaiSec["parameter8"] == "//value8");
assert(comConMaiSec["parameter9"] == ";value9"); assert(comConMaiSec["parameter9"] == ";value9");
assert(comConMaiSec["parameter10"] == "\"value10\""); assert(comConMaiSec["parameter10"] == "\"value10\"");
assert(comConMaiSec["parameter11"] == "");
assert(comConMaiSec["parameter11"].empty);
} }
unittest unittest
@ -112,6 +117,7 @@ unittest
assert(rc.cf.sn.key("value3") == "value in apostrophes"); assert(rc.cf.sn.key("value3") == "value in apostrophes");
assert(rc[][]["value4"] == "1000"); assert(rc[][]["value4"] == "1000");
assert(rc.cf.sn["value5"] == "0.000"); assert(rc.cf.sn["value5"] == "0.000");
assert(rc.cf.sn["value6"].empty);
assert(rc[][].key("value7") == "//path"); assert(rc[][].key("value7") == "//path");
assert(rc.cf.sn.key("value8") == "\"Hey!\""); assert(rc.cf.sn.key("value8") == "\"Hey!\"");
assert(rc[]["part2"]["value1"] == "this value will be in the new section"); assert(rc[]["part2"]["value1"] == "this value will be in the new section");