Как занести диапазон ip адресов в таблицу базу данных Postgresql?

Question

[atm87]

Всю голову уже сломал. При нажатии на кнопку "OK" в таблицу базу данных должен записаться диапазон ip адресов согласно данным из переменных $ipaddress1 = "10.62.39.50" и $ipaddress2 = "10.62.39.55"
То есть, должно быть вот так:

При запуске кода, данные в таблицу не заносятся. Появляется ошибка:

function Connect-DB
{
    [CmdletBinding()]
    param
    (
        [string]$dbServer = 'localhost',
        [string]$dbName = 'postgres',
        [string]$dbUser = 'postgres',
        [string]$dbPass = 'postgres',
        [string]$port = '5432'
    )
    
    $szConnect = "Driver={PostgreSQL UNICODE(x64)};Server=$dbServer;Port=$port;Database=$dbName;Uid=$dbUser;Pwd=$dbPass;"
    $cnDB = New-Object System.Data.Odbc.OdbcConnection($szConnect)
    $dsDB = New-Object System.Data.DataSet
    $cnDB.Open()
    $adDB = New-Object System.Data.Odbc.OdbcDataAdapter
    $adDB.SelectCommand = New-Object System.Data.Odbc.OdbcCommand($commandDB, $cnDB)
    $adDB.Fill($dsDB)
    $cnDB.Close()
}

$ipaddress1 = "10.62.39.50"
$ipaddress2 = "10.62.39.55"
if (-not [string]::IsNullOrEmpty($ipaddress1) -and -not [string]::IsNullOrEmpty($ipaddress2))
{
    Write-Host "Успех"
	# проверка на валидность IP адресов
	$ipObj = [System.Net.IPAddress]::None
	if ([System.Net.IPAddress]::TryParse($ipaddress1, [ref]$ipObj) -and [System.Net.IPAddress]::TryParse($ipaddress2, [ref]$ipObj))
	{
        Write-Host "Проверка на валидность пройдена"
		# если проверка на валидность пройдена, то дальше проверяем на совпадения 3-х октетов
		$ipaddress1 = [System.Net.IPAddress]($ipaddress1) # 10.62.39.50
        Write-Host $ipaddress1
		$ipaddress2 = [System.Net.IPAddress]($ipaddress2) # 10.62.39.55
        Write-Host $ipaddress2
		$octets1 = $ipaddress1 -split '\.' # 10 62 39 50
        Write-Host $octets1
		$octets2 = $ipaddress2 -split '\.' # 10 62 39 55
        Write-Host $octets2
		$first3Octets1 = ($octets1 | Select-Object -First 3) -join '.' # 10.62.39
        Write-Host $first3Octets1
		$first3Octets2 = ($octets2 | Select-Object -First 3) -join '.' # 10.62.39
        Write-Host $first3Octets2
		if ($first3Octets1 -eq $first3Octets2)
		{
			if ([int]$octets1[3] -lt [int]$octets2[3]) # 50 < 55
			{                          
                # получаем диапазон ip-адресов
				$ipadr1 = $ipaddress1.GetAddressBytes()
                
				[Array]::Reverse($ipadr1)
				$ipadr1 = ([System.Net.IPAddress]($ipadr1 -join '.')).Address
                Write-Host $ipadr1
				
				$ipadr2 = $ipaddress2.GetAddressBytes()
				[Array]::Reverse($ipadr2)
				$ipadr2 = ([System.Net.IPAddress]($ipadr2 -join '.')).Address
				Write-Host $ipadr2
				
                for ($x = $ipadr1; $x -le $ipadr2; $x++)
				{
					$ip = ([System.Net.IPAddress]$x).GetAddressBytes()
					[Array]::Reverse($ip)
					$ip = $ip -join '.'
					Write-Host $ip
                    $commandDB = "INSERT INTO public.ipaddress (ip) VALUES ('$ip'::inet) returning id;"
                    Connect-DB($commandDB)
				}
			}
		}
		else
		{
			$textboxError = "Первые три октета IP адресов не совпадают."
		}
	}
	else
	{
		$textboxError = "Неверно введены IP адреса. В поле 'От' введите начальное значение IP адреса. В поле 'До' введите конечное значение IP адреса."
	}
}

Думаю, что нужно править код в цикле, но не знаю как. Уже 2-ой день бьюсь

Comments

[alexalexes]

Посмотрите, какие Constraint или Trigger закреплены за таблицей ipaddress в СУБД.
Какой-то из этих объектов СУБД слушает событие insert проверяет корректность поля ip и выдает сообщение в случае, если проверка не пройдена "Этот хост неизвестен".

[Akina]

Я ж на SO дал SQL-код, как всё внести в БД одним запросом... вот какой великий смысл трахаться и писать по одному адресу?

А соединение с БД и выполнение запроса, который вставляет данные в таблицу ВООБЩЕ - это отдельная и несвязанная задача. Походу именно с ней у тебя и проблема.

Answer 1

[MaxKozlov]

Ошибка вот тут
Connect-DB($commandDB)
Во-первых, уберите скобки, в PS они для параметров при вызове функций не нужны

и Где тут вообще место для команды ?

function Connect-DB
{
    [CmdletBinding()]
    param
    (
        [string]$dbServer = 'localhost',
        [string]$dbName = 'postgres',
        [string]$dbUser = 'postgres',
        [string]$dbPass = 'postgres',
        [string]$port = '5432'
    )

у вас там дальше по коду $commandDB используется, а в параметрах её нет
Поставьте первым параметром

function Connect-DB
{
    [CmdletBinding()]
    param
    (
        [Parameter(Mandatory)]
        [string]$CommandDB,
        [string]$dbServer = 'localhost',

и используйте как
Connect-DB $commandDB, а лучше Connect-DB -commandDB $commandDB

а вообще надо бы разделить connect-db на connect-db и invoke-db

Comments

[MaxKozlov]

atm87,

Ваш вопрос в комментарий надо бы перенести
а разделить, чтобы получилось примерно так

$connection = Connect-DB
...
Insert-DB -Connection $Connection -Command $CommandDB

а то у вас сейчас на каждую вставку открывается и закрывается коннект

[atm87]

MaxKozlov, И все-равно не могу додуматься. Как же поделить функцию, чтобы не подключаться много раз к БД в цикле for...
Помоги пожалуйста...

[MaxKozlov]

atm87, И где они эти попытки ?

[atm87]

MaxKozlov,
Я добавил функцию Insert-DB

function Connect-DB
{
	[CmdletBinding()]
	param
	(
		[Parameter(Mandatory)]
		[string]$dbServer = 'localhost',
		[string]$dbName = 'postgres',
		[string]$dbUser = 'postgres',
		[string]$dbPass = 'postgres',
		[string]$port = '5432'
	)
	
	$connect = "Driver={PostgreSQL UNICODE(x64)};Server=$dbServer;Port=$port;Database=$dbName;Uid=$dbUser;Pwd=$dbPass;"
	$cnDB = New-Object System.Data.Odbc.OdbcConnection($connect)
	$cnDB.Open()
}

function Insert-DB
{
	[CmdletBinding()]
	param
	(
		[Parameter(Mandatory)]
		[string]$commandDB
		
	)
	$adDB = New-Object System.Data.Odbc.OdbcDataAdapter
	$adDB.SelectCommand = New-Object System.Data.Odbc.OdbcCommand($commandDB, $cnDB)
	$adDB.Fill($dsDB) | out-null
	$cnDB.Close()
	$dsDB = New-Object System.Data.DataSet
}

Затем вызываю:

if ([int]$octets1[3] -lt [int]$octets2[3]) # 50 < 55
				{									
					# получаем диапазон ip-адресов
					$ipadr1 = $ipaddress1.GetAddressBytes()
					[Array]::Reverse($ipadr1)
					$ipadr1 = ([System.Net.IPAddress]($ipadr1 -join '.')).Address
					$ipadr2 = $ipaddress2.GetAddressBytes()
					[Array]::Reverse($ipadr2)
					$ipadr2 = ([System.Net.IPAddress]($ipadr2 -join '.')).Address
					
					$connection = Connect-DB
					for ($x = $ipadr1; $x -le $ipadr2; $x++)
					{
						$ip = ([System.Net.IPAddress]$x).GetAddressBytes()
						[Array]::Reverse($ip)
						$ip = $ip -join '.'
						$commandDB = "INSERT INTO public.ipaddress (ip) VALUES ('$ip'::inet) returning id;"
					}
					Insert-DB -Connection $Connection -Command $CommandDB
				}
				else
				{
					$textboxError.Text = "Неверно введено значение IP-адреса второго поля. Четвертый октет после последней точки должен быть больше четвертого октета в первом поле."
				}

Появляется ошибка:


Если стереть параметр -Connection, то программа предлагает следующие параметры:

[MaxKozlov]

Ваш Connect-DB не возвращает
$cnDB
добавьте это последней строкой
а в Insert-DB Надо добавить соответствующий параметр
$Connection
и именно его использовать вместо $cnDB
и, кстати, .Close() не делать в Insert-DB, зачем коннект закрывать?

ну и вызов Insert-DB внутри цикла ведь нужен, нет ?

[atm87]

Спасибо тебе огромное. Добавил в функцию параметр [string]$connection

function Insert-DB
{
	[CmdletBinding()]
	param
	(
		[Parameter(Mandatory)]
		[string]$commandDB,
		[string]$connection
	)
	
	$adDB = New-Object System.Data.Odbc.OdbcDataAdapter
	$adDB.SelectCommand = New-Object System.Data.Odbc.OdbcCommand($commandDB, $connection)
	$adDB.Fill($dsDB) | out-null
}

Изменил код:

if ([int]$octets1[3] -lt [int]$octets2[3]) # 50 < 55
				{									
					# получаем диапазон ip-адресов
					$ipadr1 = $ipaddress1.GetAddressBytes()
					[Array]::Reverse($ipadr1)
					$ipadr1 = ([System.Net.IPAddress]($ipadr1 -join '.')).Address
					$ipadr2 = $ipaddress2.GetAddressBytes()
					[Array]::Reverse($ipadr2)
					$ipadr2 = ([System.Net.IPAddress]($ipadr2 -join '.')).Address
					
					for ($x = $ipadr1; $x -le $ipadr2; $x++)
					{
						$ip = ([System.Net.IPAddress]$x).GetAddressBytes()
						[Array]::Reverse($ip)
						$ip = $ip -join '.'
						$commandDB = "INSERT INTO public.ipaddress (ip) VALUES ('$ip'::inet);"
					}
					$Connection = Connect-DB
					Insert-DB -Connection $Connection -Command $CommandDB
				}

Теперь появляется другая ошибка......

[atm87]

Смысл в том, что мне в базу данных нужно просто добавить диапазон ip адресов.

[MaxKozlov]

atm87,
Ну, во-первых $Connection, конечно не [string]
если не знаете тип, не указывайте вообще
во-вторых, я не понимаю, зачем переносить вызов connect-db после цикла, вместо того чтобы внести insert-db внутрь цикла
Ну и в-третьих, вы вызываете $adDB.Fill($dsDB)
а никакого присваивания $dsDB я не вижу в последней версии.
в предыдущей оно почему-то было после
.Fill()

вам бы процедурку отладить построчно в чистом новом окошке, а не просто запускать, раз не можете ошибки интерпретировать.