[PHP][MySQL] Problem z warning i notice Opcje

[eminiasty]

Bledy:

[PHP] pobierz, plaintext 
Warning: trim() expects parameter 1 to be string, array given in C:\xampp\htdocs\M\inc\inc_other\functions.php on line 36
Notice: Undefined index: active in C:\xampp\htdocs\M\inc\script\categories.php on line 64
Notice: Use of undefined constant active - assumed 'active' in C:\xampp\htdocs\M\inc\script\categories.php on line 82
Notice: Undefined offset: 0 in C:\xampp\htdocs\M\inc\script\categories.php on line 101
[PHP] pobierz, plaintext 

Chce dodawac do bazy kategorie o kilku typach.


Robie to w nastepujacy sposob:

[PHP] pobierz, plaintext 
$query2="
INSERT INTO ".$prefix."categories (`categories_id` ,`users_id`,`categories_name`,`categories_colorText`, `categories_colorBg`,`categories_active`, `categories_dateAdd`) 
VALUES('','".$user."','".$name."','".$colortext."','".$colorbg."','".active."', NOW())
"; 
 
$result= mysqli_query($link,$query2) or die(mysqli_error());
 
 
$categories = mysqli_insert_id($link);
 
if(!empty($_POST['type'])) {
$type = array();
$work = 0;		
foreach($_POST['type'] as $i ) { 
    $type[$i] = filtrujDane($link,$_POST['type'][$i]);
	$work=$work+1;
}	
 
for($i=0;$i<$work;$i++){
$query3="
INSERT INTO ".$prefix."categoriestype (`categoriestype_id`, `categories_id`) 
VALUES('".$categories."','".$type[$i]."')";
$result = mysqli_query($link,$query3) or die(mysqli_error());
	}
}
[PHP] pobierz, plaintext 

Kod niby dziala, aczkowlwiek, wystepuja powyze bledy, chcialbym wiedziec jak je wyeliminowac?

w name formularza mam name='type()'

Ten post edytował eminiasty 25.06.2016, 11:44:39

[viking]

Czy to nie tobie juz z miesiąc temu mówiłem żebyś zrezygnował z nic nie wynoszącej funkcji filtrujDane?
Błędy wystarczy przetłumaczyć. Do trim dajesz tablicę zamiast stringa. Nie ma indeksu active ani stałej. Tablica nie ma klucza 0.

[eminiasty]

Nie ja gdzieś na necie wyczaiłem ta funkcje.

dlaczego taka funkcja jest nic nie wnosi?

'".active."' - tu był błąd z active.

Tablica nie ma klucza 0.

Czyli powinienem zdefiniowac tablice jako null na poczatku? nie pomga to

Do trim dajesz tablicę zamiast stringa

Dodanie tablicy wiaze sie z jakas petla tak po mojemu, a jak zaimplementowac takie cos w funkcji, prosil bym o jakis przyklad

[viking]

Nie czaj gdzieś w necie tylko szukaj w dokumentacji http://php.net/manual/pl/mysqli-stmt.bind-param.php
Twoja funkcja nie zabezpieczała przed sql injection i do tego niepotrzebne dawała trim jak dobrze pamiętam. Przez to masz jeszcze kolejne problemy. Jak zrobisz to w końcu dobrze połowa tego kodu nie będzie potrzebna.

Jak decydujesz się iterować po $_POST['type'] to najpierw sprawdź czy jest to tablica.

[eminiasty]

[PHP] pobierz, plaintext 
foreach($_POST['type'] as $i => $key) { 
 
$temp = implode('', $_POST['type'][$i]);
 
$type[$i] = mysqli_real_escape_string($link,trim($temp));
$work=$work+1;
}	
[PHP] pobierz, plaintext 

Probuje to rozwiazac w ten sposob ale chyba jezcze bardziej komplikuje.

[PHP] pobierz, plaintext 
Warning: trim() expects parameter 1 to be string, array given ...
[PHP] pobierz, plaintext 

Ten post edytował eminiasty 25.06.2016, 13:01:17

[viking]

Co masz w $_POST['type']?

[eminiasty]

[PHP] pobierz, plaintext 
array(4) { [0]=> string(1) "1" [1]=> string(1) "2" [2]=> string(1) "3" [3]=> string(1) "4" } string(1) "1" 
[PHP] pobierz, plaintext 

No i jest to tablica.., bo tak sie chyba to sprawdza..

[PHP] pobierz, plaintext 
if(is_array($_POST['type'])){
var_dump($_POST['type']);	
[PHP] pobierz, plaintext 

Ten post edytował eminiasty 25.06.2016, 13:10:48

[viking]

[PHP] pobierz, plaintext 
if ($stmt = mysqli_prepare($link, "INSERT INTO ".$prefix." categories (`users_id`,`categories_name`,`categories_colorText`, `categories_colorBg`,`categories_active`, `categories_dateAdd`) VALUES(?, ?, ?, ?, ?, NOW())")) {
mysqli_stmt_bind_param($stmt, "issss", $user, itd....);
if (mysqli_stmt_execute($stmt)) {
  $category = mysqli_insert_id($link);
  $sql = "INSERT INTO ".$prefix."categoriestype (`categoriestype_id`, `categories_id`) VALUES(?, ?)";
  if (!empty($_POST['type'] && is_array($_POST['type'])) {
    $stmt2 = mysqli_prepare($link, $sql);
    foreach ($_POST['type'] as $type) {
        mysqli_stmt_bind_param($stmt2, "is", $category, $type);
        mysqli_stmt_execute($stmt2);
    }
  }
}
}
[PHP] pobierz, plaintext 

Czy jakoś tak. Polecam użyć stylu obiektowego i try/catch + commit / rollback bo się można pociąć tak pisząc.

[eminiasty]

Mgógłbyś to rozjaśnic? Pierwszy raz widze cos pokroju mysqli_stmt..

[viking]

Masz wszystko w dokumentacji z dużą ilością przykładów.