没有重复的链表

Question

我有以下代码（对于我的简单测试是正确的）用于没有重复的链表，但我认为它有点难看。

谁能推荐一种更简洁的方法来处理重复代码？ 当前有问题的是：

if( (val == cur->val) || (cur->next && (val == cur->next->val)) )

但我认为使用不同的比较运算符可能存在更好的解决方案（我没有看到）。

另外，有人可以给我一个“有用的”断言的建议或在这里。很难判断何时断言，特别是如果你有一个 if 语句为你做这件事。

struct Node 
{
    Node(int v):val(v),next(NULL){}
    int val;
    Node * next;
};

void insert(Node ** ppHead, const int val)
{
    if(ppHead == NULL)
        return;
    if(*ppHead == NULL || val < (*ppHead)->val)
    {
        Node * tmp = new Node(val); // new throws
        tmp->next = *ppHead;
        *ppHead = tmp;
    }
    else
    {
        Node * cur = *ppHead;
        while(cur->next && (val > cur->next->val))
            cur = cur->next;

        if( (val == cur->val) || (cur->next && (val == cur->next->val)) )
            return;

        Node * tmp = new Node(val); // new throws
        tmp->next = cur->next;
        cur->next = tmp;
    }
    return;
}


int _tmain(int argc, _TCHAR* argv[])
{
    Node * list = NULL;
    int x[] = { 5, 4, 6, 7, 1, 8, 1, 8, 7, 2, 3, 0, 1, 0, 4, 9, 9 };
    int size = sizeof(x) / sizeof(x[0]);
    for(int i = 0; i < size; i++)
        insert(&list, x[i]);
    Node * cur = list;
    while(cur) {
        printf (" %d", cur->val);
        cur = cur->next;
    }
    printf("\n");
    return 0;
}

Answer 1

我会这样写：

void insert(Node ** ppHead, const int val)
{
    if (ppHead == NULL)
        return;
    while (*ppHead && (*ppHead)->val < val)
        ppHead = &(*ppHead)->next;
    if (*ppHead && (*ppHead)->val == val)
        return;
    Node * tmp = new Node(val); // new throws
    tmp->next = *ppHead;
    *ppHead = tmp;
}

Answer 2

这行得通吗？

// Note the change from > to >=
while(cur->next && (val >= cur->next->val))
{   cur = cur->next;
}

if (val == cur->val)
{   return;
}

Answer 3

首先，如果您将其用于生产代码，您可能应该使用std::（如果可以的话）。

我对您的代码思考得越多，我就越认为您应该保留两个指针。基本上一个到当前的Node 和一个到以前的Node。如果cur == NULL，则在prev 之后插入。如果cur->value == val，返回。然后您可以检查是否cur->value < val，如果是，则推进两个节点。

您目前有特殊代码来处理*ppHead == NULL。但是，如果您将 prev 改为 Node** curPtr，则不需要这样做。所以从curPtr=ppHead 和cur=*curPtr 开始。那么上面的算法应该适用于整个事情。

或者作为刚刚发布代码的人，您可以将 ppHead 用作 curPtr 变量本身，将 (*ppHead) 用作 cur。不确定哪个更具可读性。

Answer 4

我会向 Node 添加一个方法来执行查找操作（*未经测试的代码）：

Node* Find(int value)
{
    if( this.val == value ) return this;
    if( this.next == null ) return null;
    return next.Find(value);
}

然后在插入之前要检查是否存在：

if( null == _head || null == _head.Find(value) )
    ... add value ...