Правильно ли сделан заголовок в коде

Что Вы имеете в виду под "правильно" ? Если с точки зрения валидности html, то правильно.

для оптимизации заголовка. не правильно сформулировал в начале

<div> никак не влияет - просто блок, если конечно не "display:none"

В плане оптимизации, лучше просто

<dic class="content">

<h1>Яблоки в новый год</h1>

<p>Текст про яблоки...</p>

<p>Текст про яблоки...</p>

<h2>Яблоки в подарок</h2>

<p>Текст про яблоки...</p>

</div>

А все то, что вы написали можно спрятать в css.

.content p { text-align: justify; }
.content h1 { font-size: 20px; }
.content h2 { font-size: 18px; }

.content p { text-align: justify; }

.content h1 { font-size: 20px; }

.content h2 { font-size: 18px; }

если не прятать так как написал пойдет?

или лучше все таки прятать?

Да пофиг, в принципе. Второй вариант современнее.