Станислав Макаров

Нет, это не признак дурного тона. Auto -- замечательная фича, позволяющая сделать код более читабельным. Но, как и практически любая хорошая фича C++, она может сделать вам только хуже, если применять её без надобности там, где она не нужна. Поэтому поддерживаю уже высказанное здесь мнение

если вы без указания типа и так УЖЕ ИМЕЕТЕ достаточно информации, чтобы уверенно работать со значением - тогда можно использовать auto. Если чувствуете, что не имеете, или сомневаетесь, что имеете - лучше тип указать.

При объявлениях функций auto в качестве типа возвращаемого значения вы так просто не напишете. Это должны быть инлайн функции. Как правило, такая потребность может встретиться разве что при написании шаблонов, которые реализуются прямо в хедере и для методов в которых порой вывести тип возвращаемого значения без auto крайне тяжело. Если вы школьник и не так давно программируете, на ближайшее время вообще забудьте про такую возможность. Она вам не нужна. А для разработчиков всяких хитроумных шаблонных библиотек это может быть очень даже полезной фичей.

И ещё один аргумент за использование auto при определении переменных. Большинство современных IDE нормально справляются с тем, чтобы сообщить вам тип переменной, когда вы просто наводите мышкой на её имя, даже если она объявлена как auto. Это во многих случаях смягчает негативные стороны использования auto, если таковые вообще имели место в данном куске кода.

А для более "продвинутых" программистов рекомендую к прочтению вот эту статью Герба Саттера. Всё прекрасно разложил по полочкам насчёт использования auto и объяснил, почему это в большинстве случаев не так страшно, как некоторым кажется на первый взгляд. Конечно, предложенная в конце идиома "almost always auto" спорная, C++ это всё-таки не джаваскрипт. Но остальное написано очень разумно.

auto в заголовке функции - дурной тон.
В объявлении переменной - стандартная практика.

посмотрите на какой-нибудь NAS. судя по всему Вам этого "за глаза"

event так относится к экземпляру делегата ( Станислав Макаров : мультикаст ), как property относится к значению поля.

event - это синтаксический сахар, обертка для методов add и remove, как property является оберткой к методам get и set.

Таким образом, вопрос некорректен, поскольку event не тип.

Тела шаблонных методов должны находится в *.h

1) Унаследовались от std::iterator вы неверно. Второй параметр это тип который должен получится после разыменования. Если оператор * возвращает Т, то ставьте Т параметром.

2)

MyIterator подтягивает из std::iterator определенные в нем операторы (* и ->),

Какие определённые операторы? std::iterator определяет несколько алиасов типов, вот и всё.

Вам нужно:
а) Хранить в итераторе указатель на ноду.
б) Операторы ++ и -- переводят указатель на следующую/предыдущую ноду
в) Операторы * и -> обращаются к данным в текущей ноде.

Как-то так:

template <typename T>
struct Node
{
    T data;
    Node* next;
    Node* prev;
};

template <typename T>
class list_iterator : public std::iterator<std::bidirectional_iterator_tag, T> 
{
    private:
    
    Node<T>* current;
    
    public:
    
    list_iterator& operator++()
    {
        current = current->next;
        return *this;
    }
    
    T& operator*()
    {
        return current->data;   
    }
};