【发布时间】:2017-10-09 20:28:16
【问题描述】:
我在实现一个偶尔必须对齐的循环缓冲区时遇到了问题。
假设我有两个数组,leftArr 和 rightArr。我想将右数组移动到byteArr,将左数组移动到byteArr + 右数组的长度。 leftArr 和 rightArr 都大于 byteArr,rightArr 大于 leftArr。 (这和rotating a circular buffer不太一样,因为左数组不需要从byteArr开始)虽然左右数组不重叠,但byteArr处存储的组合数组可能与当前数组重叠,存储在leftArr 和rightArr。从byteArr 到rightArr + rightArrLen 的所有内存都可以安全地写入。一种可能的实现是:
void align(char* byteArr, char* leftArr, int leftArrLen, char* rightArr, int rightArrLen) {
char *t = malloc(rightArrLen + leftArrLen);
// form concatenated data
memcpy(t, right, rightArrLen);
memcpy(t + rightArrLen, left, leftArrLen);
// now replace
memcpy(byteArr, t, rightArrLen + leftArrLen);
free(t);
}
但是,我必须以恒定的内存复杂度来完成此操作。
我目前的样子是这样的:
void align(char* byteArr, char* leftArr, int leftArrLen, char* rightArr, int rightArrLen)
{
// first I check to see if some combination of memmove and memcpy will suffice, if not:
unsigned int lStart = leftArr - byteArr;
unsigned int lEnd = lStart + leftArrLen;
unsigned int rStart = rightArr - byteArr;
unsigned int rEnd = rStart + rightArrLen;
unsigned int lShift = rEnd - rStart - lStart;
unsigned int rShift = -rStart;
char temp1;
char temp2;
unsigned int nextIndex;
bool alreadyMoved;
// move the right array
for( unsigned int i = 0; i < rEnd - rStart; i++ )
{
alreadyMoved = false;
for( unsigned int j = i; j < rEnd - rStart; j-= rShift )
{
if( lStart <= j + rStart - lShift
&& j + rStart - lShift < lEnd
&& lStart <= (j + rStart) % lShift
&& (j + rStart) % lShift < lEnd
&& (j + rStart) % lShift < i )
{
alreadyMoved = true;
}
}
if(alreadyMoved)
{
// byte has already been moved
continue;
}
nextIndex = i - rShift;
temp1 = byteArr[nextIndex];
while( rStart <= nextIndex && nextIndex < rEnd )
{
nextIndex += rShift;
temp2 = byteArr[nextIndex];
byteArr[nextIndex] = temp1;
temp1 = temp2;
while( lStart <= nextIndex && nextIndex < lEnd )
{
nextIndex += lShift;
temp2 = byteArr[nextIndex];
byteArr[nextIndex] = temp1;
temp1 = temp2;
}
if( nextIndex <= i - rShift )
{
// byte has already been moved
break;
}
}
}
// move the left array
for( unsigned int i = lStart; i < lShift && i < lEnd; i++ )
{
if( i >= rEnd - rStart )
{
nextIndex = i + lShift;
temp1 = byteArr[nextIndex];
byteArr[nextIndex] = byteArr[i];
while( nextIndex < lEnd )
{
nextIndex += lShift;
temp2 = byteArr[nextIndex];
byteArr[nextIndex] = temp1;
temp1 = temp2;
}
}
}
}
此代码在lStart = 0, lLength = 11, rStart = 26, rLength = 70 的情况下有效,但在lStart = 0, lLength = 46, rStart = 47, rLength = 53 的情况下失败。我可以看到的解决方案是添加逻辑来确定正确数组中的字节何时已被移动。虽然这对我来说是可能的,但我想知道是否有更简单的解决方案来解决这个问题,它以恒定的内存复杂性运行并且没有额外的读写操作?
这是一个测试实现的程序:
bool testAlign(int lStart, int lLength, int rStart, int rLength)
{
char* byteArr = (char*) malloc(100 * sizeof(char));
char* leftArr = byteArr + lStart;
char* rightArr = byteArr + rStart;
for(int i = 0; i < rLength; i++)
{
rightArr[i] = i;
}
for(int i = 0; i < lLength; i++)
{
leftArr[i] = i + rLength;
}
align(byteArr, leftArr, lLength, rightArr, rLength);
for(int i = 0; i < lLength + rLength; i++)
{
if(byteArr[i] != i) return false;
}
return true;
}
【问题讨论】:
-
所以
leftArr和rightArr总是 将somewhere 指向以byteArr开头的同一个内存块? -
看起来代码可以以
void foo(char* byteArr, unsigned lStart, unsigned rStart, unsigned leftArrLen, unsigned rightArrLen);开头 -
@chux 推断得很好(在您删除的答案中),但是当不知道
char* byteArr的长度时存在危险 - 缺少函数参数。 -
@WeatherVane 虽然
byteArr确实没有长度,但对于我的目的来说,它确实“足够长”。 -
可以将
left+left_length假定为<= right吗?否则为什么叫他们左,右?