[MySQL][PHP]import danych z .csv Opcje

[Cromwell]

Importuję dane z pliku csv do bazy danych. Skrypt wygląda tak:

[PHP] pobierz, plaintext 
//formularz
echo '<form action="dodaj.php" method="get">
      <input type="hidden" name="add" value="dodaj" />
      <input type="submit" value="dodaj" />
      </form>';
 
// jesli add = dodaj, wykonaj skrypt
$add = $_GET['add'];
 
if ($add == 'dodaj')
{
 
//definiowanie atrybutow
$fieldseparator = ";";
$lineseparator = "\n";
$csvfile = "firmy.csv";
 
 
if(!file_exists($csvfile)) {
	echo "Plik nie istnieje. Upewnij się, że umieściłeś go w katalogu skryptu.\n";
	exit;
}
 
$file = fopen($csvfile,"r");
if(!$file) {
	echo "Błąd podczas otwierania pliku.\n";
	exit;
}
 
$size = filesize($csvfile);
if(!$size) {
	echo "Plik jest pusty.\n";
	exit;
}
 
$csvcontent = fread($file,$size);
 
fclose($file);
 
conn();
 
$lines = 0;
$queries = "";
$linearray = array();
 
foreach(split($lineseparator,$csvcontent) as $line) {
 
	$lines++;
	$line = trim($line," \t");	
	$line = str_replace("\r","",$line);	
	$line = str_replace("'","\'",$line);
	$linearray = explode($fieldseparator,$line);
	$linemysql = implode("','",$linearray);	
	$query = "insert into firmy values('$linemysql');";	
	$queries .= $query . "\n";
	@mysql_query($query);
}
 
echo "Znaleziono $lines rekordów w pliku CSV. Wszystkie zostały dodane do bazy poprawnie.\n";
}
 
?>
[PHP] pobierz, plaintext 

Skrypt działa bardzo dobrze.
Chciałbym żeby rekordy które wgrywam o jakimś ID zastępowały rekordy które są już w bazie z takim samym ID.

Ten post edytował Cromwell 30.11.2009, 16:12:11

[jasin]

W takim razie powinno Ci pomóc INSERT ... ON DUPLICATE KEY UPDATE.

Ten post edytował jasin 30.11.2009, 16:24:51

[Cromwell]

Ok, przeczytałem fragment w manualu i albo ja tego nie rozumiem, albo nie o to mi chodzi.
Z tego co wywnioskowałem, dzięki temu poleceniu mogę zmienić określone dane, aby rekordy nie były takie same.
A mi chodzi o to, żeby w przypadku takiego samego ID, nowy rekord nadpisywał stary.

Nie wiem, pewnie coś źle rozumiem.
Zmieniłem w kodzie tak:

[PHP] pobierz, plaintext 
$query = "insert into firmy values('$linemysql') ON DUPLICATE KEY UPDATE lp=VALUES(lp);";
[PHP] pobierz, plaintext 

I nie działa.

[jasin]

Ok. Zawsze możesz zrobić zapytanie:

[SQL] pobierz, plaintext 
UPDATE ... WHERE id = x
[SQL] pobierz, plaintext 

i jeśli to zapytanie zwróci zero zaktualizowanych rekordów to robisz INSERT.

[Cromwell]

Wprowadziłem zmiany:

[PHP] pobierz, plaintext 
$linearray = explode($fieldseparator,$line);
	$pierw = $linearray[0];
	$linemysql = implode("','",$linearray);
 
	$query = "UPDATE firmy SET values('$linemysql') WHERE lp = $pierw;";
 
	$queries .= $query . "\n";
 
	@mysql_query($query);
	if ($query < 1)
	{ $query = "insert into firmy values('$linemysql');";
	  @mysql_query($query); }
[PHP] pobierz, plaintext 

Jednak efekt jest ten sam - dodaje wszystkie rekordy raz jeszcze..

[jasin]

Poczytaj o mysql_affected_rows.
Powinno u Ciebie wyglądać to jakoś tak:

[PHP] pobierz, plaintext 
      $linearray = explode($fieldseparator,$line);
 
      $pierw = $linearray[0];
 
      $linemysql = implode("','",$linearray);
 
      $query = "UPDATE firmy SET values('$linemysql') WHERE lp = $pierw;";
 
      $queries .= $query . "\n";
 
      @mysql_query($query);
 
      if (mysql_affected_rows() == 0)
      { 
          $query = "insert into firmy values('$linemysql');";
          @mysql_query($query); 
      }
[PHP] pobierz, plaintext 

Ten post edytował jasin 30.11.2009, 20:05:10

[Cromwell]

Po wprowadzeniu zmian, nic się nie dodaje..
Aby zobaczyć, gdzie jest błąd, zmieniłem na:

[PHP] pobierz, plaintext 
$pierw = $linearray[0];
 
	$linemysql = implode("','",$linearray);
 
	$query = "UPDATE firmy SET values('$linemysql') WHERE lp = '$pierw';";
 
	$queries .= $query . "\n";
 
	mysql_query($query) or die('1'.mysql_error());
 
	if (mysql_affected_rows() == 0)
	{ 
	$query = "insert into firmy values('$linemysql');";
	mysql_query($query) or die('2'.mysql_error());  
	}
[PHP] pobierz, plaintext 

Teraz dostaję komunikat

1You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near

Zastanawia mnie to, ponieważ danych, które są wczytywane nie zmieniałem, tak jak wyglądały tak dalej wyglądają, więc to nie z nimi problem.
Składnia zapytania.. wydaje mi się, że jest dobrze.
Nie wiem w sumie, gdzie teraz jest błąd.

[darko]

wyprintuj całe zapytanie

[Cromwell]

Ok, całe zapytanie wygląda tak:

[SQL] pobierz, plaintext 
UPDATE firmy SET VALUES('10000001','1','2','3','','4','5','6','7','8','9','10','11','12','13','14','15','16','17','18','19','20','21',22','23','24','25',' 26','27','28','29','','30','31','32','33','34','35','36','37','38','39','40','41','42','43','44','45','46','47','48','49','50','51','52', '53','54','55','56') WHERE lp = '10000001';
[SQL] pobierz, plaintext 

Zmieniłem w nim wartości na liczby, gdyż po prostu nie mogę ich pokazać publicznie (wewnątrz zapytania po przecinkach dodałem jeszcze dwie spacje, aby się ładnie wyświetliło tutaj). Same wartości na pewno wchodzą do bazy dobrze, gdyż skryptem, który umieściłem w pierwszym poście dodawałem dokładnie te same rekordy.

Zapytanie ma prawo się nie wykonać, jeśli w bazie danych nie ma danego rekordu. Jednak wtedy skrypt powinien zejść niżej i wykonać to drugie. To, które jest identyczne jak to z pierwszego postu.

Może chodzi o to, że jeśli skrypt nie wykona zapytania, to już dalej nie robi tylko raz jeszcze zaczyna od foreach z nową wartością?

EDIT
Jak usunąłem fragment, który informował o błędzie w pierwszym zapytaniu, wtedy skrypt niby cały poszedł, dostałem info, że rekordy zostały dodane poprawnie, jednak w bazie ich nie ma.

Ten post edytował Cromwell 1.12.2009, 12:07:48

[darko]

Wyrzuć warunek where

Ten post edytował darko 1.12.2009, 12:08:53

[Cromwell]

Zmieniłem na:

[SQL] pobierz, plaintext 
UPDATE firmy SET (lista pól, oddzielona przecinkami) VALUES('10000001','1','2','3','','4','5','6','7','8','9','10','11','12','13','14','15','16','17','18','19','20','21',22','23','24','25',' 26','27','28','29','','30','31','32','33','34','35','36','37','38','39','40','41','42','43','44','45','46','47','48','49','50','51','52', '53','54','55','56') WHERE lp = '10000001';
[SQL] pobierz, plaintext 

I dalej to samo. Zapytanie nie przechodzi, ale myślę, że to dlatego, bo w bazie danych nie ma rekordu z taką liczbą lp.
Tylko że w takim wypadku skrypt powinien iść dalej, wykonać zapytanie INSERT INTO, a tego nie robi..

@darko, bez warunku WHERE dalej nie idzie..

Ten post edytował Cromwell 1.12.2009, 12:29:53

[darko]

W moim ostatnim poście napisałem: Wyrzuć warunek where, a widzę, że dalej jest...

Nie myślałeś o tym, żeby użyć REPLACE W Twoim przypadku bardziej by się sprawdziło.

[ziqzaq]

Z ciekawości zapytam.
Skąd taka składnia "updatea"?

[SQL] pobierz, plaintext 
UPDATE tabela SET (kolumny) VALUES (wartosci);
[SQL] pobierz, plaintext 

Mysql update vs. insert.

[Cromwell]

Zmieniłeś post po tym, jak ja napisałem swój.
Wyrzuciłem i dodałem część postu od "@darko, bez warunku".

A co do REPLACE - szczerze mówiąc, nie wiem jak sensownie zbudować wyrażenie używając REPLACE. Żeby jeśli rekord z takim LP już jest w bazie tylko podmieniało, a jeśli nie ma, to wstawiało nowy.

[darko]

REPLACE works exactly like INSERT, except that if an old row in the table has the same value as a new row for a PRIMARY KEY or a UNIQUE index, the old row is deleted before the new row is inserted

To chyba wszystko mówi, na przykład:

[SQL] pobierz, plaintext 
REPLACE INTO nazwa_bazy.nazwa_tabeli VALUES('10000001','1','2','3','','4','5','6','7','8','9','10','11','12','13','14','15','16','17','18','19','20','21',22','23','24','25',' 26','27','28','29','','30','31','32','33','34','35','36','37','38','39','40','41','42','43','44','45','46','47','48','49','50','51','52', '53','54','55','56')
[SQL] pobierz, plaintext 

to dokładnie spełnia swoją rolę, sprawdziłem na przykładowej tabeli.

[Cromwell]

Działa idealnie

Początkowo nie zrozumiałem dokładnie REPLACE (chociaż używałem go już kiedyś), bo szukałem jeszcze w google dla pewności różnych zastosowań i trafiłem na kilka bardzo rozbudowanych przykładów.

Wielkie, wielkie dzięki