SQL Komut Dosyası - Bir IP'nin listelenip listelenmediğini kontrol edin

Question

SQL üzerinde çalışıyorum ve bu kodun düzgün bir şekilde yazıldığını veya daha iyi bir şekilde yapılıp yapılmadığını görmek için tavsiyenizi istiyorum. Bu komut dosyası oluşturun:

• bir tablo 4 sayı grubuna

• kontrol etmesine izin vermek bir SPROC bir IP adresi tükürme izin Siyah

• bir sproc Kote Edildiği IP'ler adresini saklamak için bir IP Siyah ise Listelenen veya değil Lütfen bana bildirin! Teşekkürler!

  -- Black Listed Table 
  CREATE TABLE UtlBlacklistedIPs 
       ( 
        octet1 TINYINT, 
        octet2 TINYINT, 
        octet3 TINYINT, 
        octet4 TINYINT 
       ); 
  

1 SPROC

-- Convert an IP address in 4 octet for db storing 
    CREATE PROCEDURE dbo.storeIPoctetsv1 
    @ip CHAR(15) 
    AS 
    BEGIN 
    SET NOCOUNT ON 
    INSERT UtlBlacklistedIPs(octet1, octet2, octet3, octet4) 
    SELECT 
    CONVERT(TINYINT, PARSENAME(@ip, 4)), 
    CONVERT(TINYINT, PARSENAME(@ip, 3)), 
    CONVERT(TINYINT, PARSENAME(@ip, 2)), 
    CONVERT(TINYINT, PARSENAME(@ip, 1)) 
    END 

2 SPROC

-- SPROC check if an IP address has been black listed 
CREATE PROCEDURE dbo.sprocCheckIp 
    @ip CHAR(15) 
    AS 
    BEGIN 
    SET NOCOUNT ON 
    DECLARE 
    @octet1 tinyint, 
    @octet2 tinyint, 
    @octet3 tinyint, 
    @octet4 tinyint; 

    SET @octet1 = CONVERT(tinyint, PARSENAME(@ip, 4)); 
    SET @octet2 = CONVERT(tinyint, PARSENAME(@ip, 3)); 
    SET @octet3 = CONVERT(tinyint, PARSENAME(@ip, 2)); 
    SET @octet4 = CONVERT(tinyint, PARSENAME(@ip, 1)); 

    IF EXISTS ( 
     SELECT octet1, octet2, octet3, octet4 
     FROM UtlBlacklistedIPs 
     WHERE 
     octet1 =@octet1 
     AND octet2 =@octet2 
     AND octet3 =@octet3 
     AND octet4 =@octet4 
    ) 
     BEGIN 
     PRINT 'Ip: '+@ip+' is black listed' 
     END 

     ELSE 
     BEGIN  
     PRINT 'Ip: '+@ip+' is NOT black listed' 
     END  
    END 

bazı değerler

-- Insert dummy values 
DECLARE @i INT 
SET @i = 0 
WHILE @i < 2000 
BEGIN 

    EXEC dbo.storeIPoctetsv1 @ip = '204.71.34.21' 
    EXEC dbo.storeIPoctetsv1 @ip = '12.38.145.32' 
    EXEC dbo.storeIPoctetsv1 @ip = '127.0.0.1' 

    SET @i = @i + 1 
END 

kullanımı SPROC

-- Use SPROc 
    EXEC dbo.sprocCheckIp '125.254.125.111' -- NOT black listed Ip 
    EXEC dbo.sprocCheckIp '204.71.34.21' -- black listed Ip 

Answer 1

Veritabanının dışında bunu muhtemelen .NET'te kullanacağınızı farz ediyorum. Bu durumda, yeniden yapıya olur böyle:

• sproc dört girdi olarak oktet ve 0 ya da 1 ile, herhangi bir IP kontrol etmek için, giriş olarak dört oktet ile
• sproc siyah listelenen IP depolamak çıktı olarak (RETURN, değil PRINT)
• Bunun yerine .NET'te IP adreslerinin bölünmesini yapın. SQL Server çok zayıf bir dize işlemeye sahiptir, bu yüzden bir Ip sınıfını, sizin için işleyen statik bir yöntem olan int[] ToOctets(string ip) ile yazmaktan çok daha iyisiniz. Bunu şöyle kullanırsınız: var octs = Ip.ToOctets("213.82.158.93")
• Bir IP'nin kara listeye alınmış olup olmadığını kontrol ettiğinizde, ExecuteScalar'ı çağırırsınız (veya saklı yordamdan skaler değer elde etmek için kullandığınız yöntem, tercih ettiğiniz bağlaçla aranabilir)) sonuca ulaşmak için.

Answer 2

Thomas ile aynı fikirdeyim ancak bir sorum var: IP'yi neden 4 ayrı sütunda saklıyorsunuz? Eğer egzersizin bütün noktası "kara listeye alınmış IP adreslerini" takip etmekteyse, o zaman ne zaman IP adresinin bir parçasını incelemeniz gerekecekti - her zaman her şeye bakmıyor muydunuz? Böylece bölmeleri ayırmak/yeniden birleştirmek gereksiz görünüyor. Çok fazla fazla T-SQL yazmanıza da neden oluyor.