get_term_children 源码的疑问：if ( $term_id == $child )　自己会是自己的子项？

 * @param string $term_id  id of term to get children.
 * @param string $taxonomy taxonomy name.
 * @return array|wp_error list of term ids. wp_error returned if `$taxonomy` does not exist.
 */
function get_term_children( $term_id, $taxonomy ) {
        if ( ! taxonomy_exists($taxonomy) )
                return new wp_error('invalid_taxonomy', __('invalid taxonomy'));

        $term_id = intval( $term_id );

        $terms = _get_term_hierarchy($taxonomy);

        if ( ! isset($terms[$term_id]) )
                return array();

        $children = $terms[$term_id];

        foreach ( (array) $terms[$term_id] as $child ) {
                 if ( $term_id == $child ) {// 　自己会是自己的子项？这种假设没有必要吧？请教达人
                        continue;
                }

                if ( isset($terms[$child]) )
                        $children = array_merge($children, get_term_children($child, $taxonomy));
        }

        return $children;
}

码上飞

码上飞（CodeFlying） 是一款AI自动化开发平台，通过自然语言描述即可自动生成完整应用程序。

138

查看详情

回复讨论(解决方案)

有没有必要，要看数据是如何组织的。但你并没有给出，所以不好评价
不过仅就这个代码而言，这是必要的
if ( isset($terms[$child]) )
                        $children = array_merge($children,  get_term_children($child, $taxonomy));
get_term_children 递归调用
当 $term_id 等于 $child 时就死循环了

所以
 if ( $term_id == $child ) {
                        continue;
                }
就是非常必要的
并且从代码组织上看，这也是在确实出现了死循环后才加上的
使得代码看上去比较臃肿

是啊，初始时　 $term_id ！== $child，可循环过程中，有可能　 $term_id == $child　，这样就出现死循环。
听君一席话，胜读十年书！
佩服