[2008] warning in my code for gethostbyname

[smelf1111]

Hi when i use the following code i get a warning
'Public Shared Function GetHostByName(hostName As String) As System.Net.IPHostEntry' is obsolete: 'GetHostByName is obsoleted for this type, please use GetHostEntry instead.

Dim host As System.Net.IPHostEntry = System.Net.Dns.GetHostByName(System.Net.Dns.GetHostName) 'gets the machines current IP

now the code works and shows me the ip

[Hack]

now the code works and shows me the ip

So, what is your question?

[smelf1111]

how to get rid of the warning, it asys use gethostentry but if i use that it displays the mac and not the ip

[jmcilhinney]

An obsolete method is one that still exists in the current version of the Framework but is intended to be removed at some later time. Given that the error message says:

please use GetHostEntry instead

I think it's fairly obvious what you should do. Don't you? In the next version of the Framework GetHostByName may be removed and your code will stop working. GetHostEntry is guaranteed to be available.

[Atheist]

how to get rid of the warning, it asys use gethostentry but if i use that it displays the mac and not the ip

Well, it returns an IPHostEntry, which contains a property "AddressList" which is an array of IP addresses for the host.

[jmcilhinney]

how to get rid of the warning, it asys use gethostentry but if i use that it displays the mac and not the ip

That's not a MAC address. It's an IPv6 address. The IPAddress class has properties that tell you whether an IP address is IPv6 or IPv4.

[smelf1111]

ah right so how do i just get it to display the ipv4 address?

[chris128]

That's not a MAC address. It's an IPv6 address. The IPAddress class has properties that tell you whether an IP address is IPv6 or IPv4.

Does it? I cant see such a property
Or are you referring to IsIPv6LinkLocal and IsIPv6Multicast etc? in which case, would you have to test every one of those IsIPv6.. properties to ensure it wasnt a v6 address?

PS I know you could in theory just check to see if the string was a certain length as IPv6 addresses are longer but I'm curious to see if there is a more 'proper' way of doing it

[Atheist]

The AddressFamily property will be Net.Sockets.AddressFamily.InterNetworkV6 if the the address is an IPv6 address.

[chris128]

Ahh that will be why I didnt spot it then, because microsoft decided that instead of naming it IPv4/v6 or something intuitive they called it InterNetwork and InterNetworkv6 lol thanks

[smelf1111]

right i tried this

Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click
        Dim myip As String = System.Net.Dns.GetHostAddresses(My.Computer.Name)(1).ToString().Trim
        ComboBox1.Items.Add(myip)
    End Sub

but it return fe80::5efe:192.168.0.19%10 i just wanted 192.168.0.19,
how do i get it to show that and not this fe80::5efe:192.168.0.19%10

[chris128]

Well why are you accessing the second item in the list of addresses? By using (1) in your code you refer to the second item in the array because the array is "zero based" (meaning the first item is item 0 and the second item is item 1 etc etc).

Also, I thought you were going to use GetHostEntry instead now?

One moment and I'll write an example to help you learn, not just to copy and paste and forget about

[chris128]

OK here we go:

'Loop through each IPAddress object in the list of IP addresses that is returned
'by the AddressList property
For Each IP As IPAddress In Dns.GetHostEntry(My.Computer.Name).AddressList
    'Make sure its an IPv4 address (thanks Athiest!)
    If IP.AddressFamily = Sockets.AddressFamily.InterNetwork Then
         'Do something with the IP string, in this case show it in a messagebox
         MessageBox.Show(IP.ToString)
    End If
Next

[smelf1111]

if you put a 0 it will only return the ipv6 address to get the ipv4 i use the 1,

chris your code returns 3 items for me
fe80::3894:6b0e:ed6c:7a1c%8
fe80::5efe:192.168.0.19%10
and
192.168.0.19

[chris128]

OK maybe the IF statement needs changing then to say NOT InterNetworkv6 instead of just IS InterNetwork

if you put a 0 it will only return the ipv6 address to get the ipv4 i use the 1

You said that your code returned the ipv6 address anyway, and you are using the 1 instead of the 0, so the above statement is not true is it
In your situation using the 3rd item in the array (a (2) )would get you the v4 address because the v4 address is the third IP that is in your list of IP addresses... but that is only because your PC has 3 IP addresses, it would be different on other computers so you have to loop through them all and check instead of just grabbing one straight away like you are trying to do.

[smelf1111]

cool, ipv6 must be on by default in vista.

[chris128]

Yeah it is