Yes, there are better ways to do that now (SEO wise).

Add this code to your ASCX skin:

<script runat="server" language="vb">
           
    Private Function GetPageLinkByName(ByVal PageName As String) As String
        
        Dim oTabC As New DotNetNuke.Entities.Tabs.TabController
        Dim oTab As DotNetNuke.Entities.Tabs.TabInfo = oTabC.GetTabByName(PageName, PortalSettings.PortalId)
        If Not oTab Is Nothing Then
            Return String.Format("<a href='{0}'>{1}</a>", oTab.FullUrl, PageName)
        Else
            Return ""
        End If
                
    End Function
    
</script>

 Then add this to your skin:

<%= GetPageLinkByName("Contact")%>

Of each link.
This used to be case sensitive, I think before DNN 5.3, but not anymore.

HTH