Relintai/pandemonium_engine
1
0
Fork 0
mirror of https://github.com/Relintai/pandemonium_engine.git synced 2024-11-23 09:28:07 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix split_floats behavior when spaces are used as separators

Browse Source
This commit is contained in:
Haoyu Qiu 2024-08-19 15:14:10 +08:00 committed by Relintai
parent 34608b314f
commit 4585110a29
2 changed files with 70 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
core/string/ustring.cpp
View File

@ -2731,13 +2731,16 @@ Vector<float> String::split_floats(const String &p_splitter, bool p_allow_empty)
int from = 0; int from = 0;
int len = length(); int len = length();
String buffer = *this;
while (true) { while (true) {
int end = find(p_splitter, from); int end = find(p_splitter, from);
if (end < 0) { if (end < 0) {
end = len; end = len;
} }
if (p_allow_empty || (end > from)) { if (p_allow_empty || (end > from)) {
ret.push_back(String::to_double(&get_data()[from])); buffer[end] = 0;
ret.push_back(String::to_double(&buffer.get_data()[from]));
buffer[end] = _cowdata.get(end);
} }
if (end == len) { if (end == len) {
@ -2755,6 +2758,7 @@ Vector<float> String::split_floats_mk(const Vector<String> &p_splitters, bool p_
int from = 0; int from = 0;
int len = length(); int len = length();
String buffer = *this;
while (true) { while (true) {
int idx; int idx;
int end = findmk(p_splitters, from, &idx); int end = findmk(p_splitters, from, &idx);
@ -2766,7 +2770,9 @@ Vector<float> String::split_floats_mk(const Vector<String> &p_splitters, bool p_
} }
if (p_allow_empty || (end > from)) { if (p_allow_empty || (end > from)) {
ret.push_back(String::to_double(&get_data()[from])); buffer[end] = 0;
ret.push_back(String::to_double(&buffer.get_data()[from]));
buffer[end] = _cowdata.get(end);
} }
if (end == len) { if (end == len) {

62
main/tests/test_string.cpp.old
View File

@ -1254,6 +1254,67 @@ bool test_37() {
return true; return true;
} }
bool test_38() {
#define CHECK_ARR_LEN(arr, len) \
if (arr.size() != len) { \
OS::get_singleton()->print("\tFAIL: Length of %s should be %d, got %d\n", #arr, len, arr.size()); \
return false; \
} else { \
OS::get_singleton()->print("\tPASS\n"); \
}
#define CHECK_ARR_ELEMENT(arr, i, expect) \
if (ABS(arr[i] - expect) > 0.00001) { \
OS::get_singleton()->print("\tFAIL: %s[%d] %f != %f\n", #arr, i, arr[i], expect); \
return false; \
} else { \
OS::get_singleton()->print("\tPASS\n"); \
}
OS::get_singleton()->print("\n\nTest 38: split_floats\n");
{
const String s = "1.2;2.3 4.5";
const float slices[3] = { 1.2, 2.3, 4.5 };
const Vector<float> d_arr = s.split_floats(";");
CHECK_ARR_LEN(d_arr, 2);
for (int i = 0; i < 2; i++) {
CHECK_ARR_ELEMENT(d_arr, i, slices[i]);
}
Vector<String> keys;
keys.push_back(";");
keys.push_back(" ");
const Vector<float> f_arr = s.split_floats_mk(keys);
CHECK_ARR_LEN(f_arr, 3);
for (int i = 0; i < 3; i++) {
CHECK_ARR_ELEMENT(f_arr, i, slices[i]);
}
}
{
const String s = " -2.0 5";
const float slices[10] = { 0, -2, 0, 0, 0, 0, 0, 0, 0, 5 };
const Vector<float> d_arr = s.split_floats(" ");
CHECK_ARR_LEN(d_arr, 10);
for (int i = 0; i < 10; i++) {
CHECK_ARR_ELEMENT(d_arr, i, slices[i]);
}
Vector<String> keys;
keys.push_back(";");
keys.push_back(" ");
const Vector<float> f_arr = s.split_floats_mk(keys);
CHECK_ARR_LEN(f_arr, 10);
for (int i = 0; i < 10; i++) {
CHECK_ARR_ELEMENT(f_arr, i, slices[i]);
}
}
return true;
}
typedef bool (*TestFunc)(); typedef bool (*TestFunc)();
TestFunc test_funcs[] = { TestFunc test_funcs[] = {
@ -1295,6 +1356,7 @@ TestFunc test_funcs[] = {
test_35, test_35,
test_36, test_36,
test_37, test_37,
test_38,
nullptr nullptr
}; };