周五我收到了一个面试问题,我认为我不及格了。问题是:
编写一个在PHP中处理双链表的类。
我理解这个概念,这是我给出的代码:
class element {
private $current;
public function __construct($e) {
$this->current = $e;
}
// method
// etc..
}
class doublelist
{
private $prev;
private $next;
private $current;
private $list;
public function add(element $e) {
if($this->current == NULL) {
$this->prev = $this->current;
}
$this->current = $e;
}
}
$list = new doublelist();
$list->add(new element('a'));
$list->add(new element('b'));
这最初是有效的,但如果我添加第二个元素,我会“失去”第一个元素,我不明白为什么。
答案 0 :(得分:12)
您需要跟踪$prev
上的$next
和element
,而不是列表。如果你想让它透明,你可以将每个element
包装在一个指向下一个和前一个指针的bean中,或者只是让element
按照定义包含它们。
你现在这样做的方式,列表只会知道哪一个是当前element
,哪一个是在此之前。但是你应该做的是从element
(或bean)中找出它将是下一个或之前的那个。
修改强>
由于这个问题偶尔会出现,我想我会添加一些代码来帮助解释这个问题。
class DoublyLinkedList {
private $start = null;
private $end = null;
public function add(Element $element) {
//if this is the first element we've added, we need to set the start
//and end to this one element
if($this->start === null) {
$this->start = $element);
$this->end = $element;
return;
}
//there were elements already, so we need to point the end of our list
//to this new element and make the new one the end
$this->end->setNext($element);
$element->setPrevious($this->end);
$this->end = $element;
}
public function getStart() {
return $this->start;
}
public function getEnd() {
return $this->end;
}
}
class Element {
private $prev;
private $next;
private $data;
public __construct($data) {
$this->data = $data;
}
public function setPrevious(Element $element) {
$this->prev = $element;
}
public function setNext(Element $element) {
$this->next = $element;
}
public function setData($data) {
$this->data = $data;
}
}
当然,您可以添加其他方法;如果有人对那些我感兴趣,我也可以添加它们。
答案 1 :(得分:5)
在编码之前,告诉他们它已经完成并包含在PHP标准库中。 http://php.net/manual/en/class.spldoublylinkedlist.php
此外,您的添加功能不应该采用元素。它应该只是$list->add('a');
你过多地暴露了你的实现。