我试图创建一个扫描网络以查找ARP请求并列出所有现有网络设备的应用。目前,我使用SharpPcap
和PacketDoNet
。
根据给定的IP解析主机名时,在解析"未知"时,我会得到SocketException
。主办。所以我把它放在try / catch中。由于我认为这是不好的风格(忽略例外),我正在寻找不同的解决方案。
以下是一些代码:
// Button for scanning the network
private void btnStartScanningForClients_Click(object sender, RoutedEventArgs e)
{
// Check for correct interface
// [...]
// Start scanning process
if (!this.netWorkItOut.Startet)
{
// Dis-/Enable visual controls
// [...]
// Start scanning
var index = this.cbNetworkInterface.SelectedIndex
this.netWorkItOut.StartDevice(index);
this.netWorkItOut.Scanner.StartScanningNetwork(resolveHostnames);
}
}
这是控制对象,它保存扫描程序,处理事件,接受数据包并将它们放入队列
public void StartDevice(int deviceIndex)
{
this.Startet = true;
// [...]
this.Device = WinPcapDeviceList.Instance[deviceIndex];
// Activate Scanner
this.Scanner = new Scanner(this.DeviceInfo);
// Subscribe Events
// [...]
this.Device.Open(DeviceMode.Promiscuous, 1);
this.Device.Filter = "(arp || ip || ip6)";
this.Device.OnPacketArrival += device_OnPacketArrival;
this.Device.StartCapture();
}
private void device_OnPacketArrival(object sender, CaptureEventArgs e)
{
//PacketDoNet
Packet packet;
try
{ packet = Packet.ParsePacket(LinkLayers.Ethernet, e.Packet.Data); }
catch (Exception)
{ return; }
if (packet is EthernetPacket)
{
var arp = ARPPacket.GetEncapsulated(packet);
if (arp != null)
{
if (this.Scanner.Started)
{
lock (this.Scanner.PacketQueueARP)
{
this.Scanner.PacketQueueARP.Add(arp);
}
}
}
}
}
这是Controlling对象和Scanner类。 scanner类用于处理ARP请求并解析主机名
public void StartScanningNetwork(bool resolveHostnames)
{
// [...]
this.ResolveHostnames = resolveHostnames;
// start worker to listen for ARP packets
this.workerARP = new Thread(WorkerARP);
this.workerARP.Name = "Scanner thread (ARP)";
this.workerARP.Start();
this.Started = true;
}
private void WorkerARP()
{
List<IPAddress> processedIps = new List<IPAddress>();
// copy packets from storage queue to thread queue for processing
while (Started)
{
// [...]
if (this.threadQueueARP.Count > 0)
{
foreach (var packet in this.threadQueueARP)
{
// [...]
if (!processedIps.Contains(ip))
{
// [...]
if (this.ResolveHostnames)
{
var resolveHostnamesTask = Task.Factory.StartNew(ResolveHostnamesWorker, ip);
}
}
// [...]
}
// [...]
}
// [...]
}
}
private void ResolveHostnamesWorker(object data)
{
if (data is IPAddress)
{
var ip = (IPAddress)data;
var hostname = "";
try
{
hostname = Dns.GetHostEntry(ip).HostName;
}
catch { }
// Raise Event for hostname resolved
}
}
关于hostname = Dns.GetHostEntry(ip).HostName
那么:在通过Dns.GetHostEntry()解析HostEntry时,如何避免使用try / catch?如果没有已知的主机,是否有一个仅返回null
的函数?
提前致谢!
答案 0 :(得分:3)
由于https://github.com/Microsoft/referencesource/blob/master/System/net/System/Net/DNS.cs下的班级Dns
的代码可用,因此您只能使用相关部分。 (当然,您必须检查MIT许可证是否可以在您的项目中使用)
实现在方法InternalGetHostByAddress
中,抛出异常。在那里你可以返回一个值(bool,enum ...),以便在查询成功时提供信息。
答案 1 :(得分:1)
据我所知,没有像TryGetHostName()这样的方法不会抛出异常。
但在我看来,根据您的预期捕捉异常是明显的。因此,您应该限制捕获您期望的异常:
private void ResolveHostnamesWorker(object data)
{
if (data is IPAddress)
{
var ip = (IPAddress)data;
var hostname = "";
try
{
hostname = Dns.GetHostEntry(ip).HostName;
}
catch(SocketException socketException)
{
// maybe limit handling based on data in socketException and
// call throw; to rethrow exception if not the expected one
}
// Raise Event for hostname resolved
}
}