Forum PHP.pl > [PHP]Skrócenie kodu

Pomoc - Szukaj - Użytkownicy - Kalendarz

Xart

14.04.2014, 13:29:30

Witam, da się może jakoś skrócić taki oto KOD ?

[PHP] pobierz, plaintext 
	$obrazek1 = $_POST['obrazek1'];
	$miniatura1 = $_POST['miniatura1'];
	$obrazek2 = $_POST['obrazek2'];
	$miniatura2 = $_POST['miniatura2'];
	$obrazek3 = $_POST['obrazek3'];
	$miniatura3 = $_POST['miniatura3'];
	$obrazek4 = $_POST['obrazek4'];
	$miniatura4 = $_POST['miniatura4'];
	$obrazek5 = $_POST['obrazek5'];
	$miniatura5 = $_POST['miniatura5'];
	$obrazek6 = $_POST['obrazek6'];
	$miniatura6 = $_POST['miniatura6'];
	$obrazek7 = $_POST['obrazek7'];
	$miniatura7 = $_POST['miniatura7'];
	$obrazek8 = $_POST['obrazek8'];
	$miniatura8 = $_POST['miniatura8'];
	$obrazek9 = $_POST['obrazek9'];
	$miniatura9 = $_POST['miniatura9'];
	$obrazek10 = $_POST['obrazek10'];
	$miniatura10 = $_POST['miniatura10'];
	$obrazek11 = $_POST['obrazek11'];
	$miniatura11 = $_POST['miniatura11'];
	$obrazek12 = $_POST['obrazek12'];
	$miniatura12 = $_POST['miniatura12'];
	$obrazek13 = $_POST['obrazek13'];
	$miniatura13 = $_POST['miniatura13'];
	$obrazek14 = $_POST['obrazek14'];
	$miniatura14 = $_POST['miniatura14'];
	$obrazek15 = $_POST['obrazek15'];
	$miniatura15 = $_POST['miniatura15'];
	$obrazek16 = $_POST['obrazek16'];
	$miniatura16 = $_POST['miniatura16'];
	$obrazek17 = $_POST['obrazek17'];
	$miniatura17 = $_POST['miniatura17'];
	$obrazek18 = $_POST['obrazek18'];
	$miniatura18 = $_POST['miniatura18'];
	$obrazek19 = $_POST['obrazek19'];
	$miniatura19 = $_POST['miniatura19'];
	$obrazek20 = $_POST['obrazek20'];
	$miniatura20 = $_POST['miniatura20'];
	$obrazek21 = $_POST['obrazek21'];
	$miniatura21 = $_POST['miniatura21'];
	$obrazek22 = $_POST['obrazek22'];
	$miniatura22 = $_POST['miniatura22'];
	$obrazek23 = $_POST['obrazek23'];
	$miniatura23 = $_POST['miniatura23'];
	$obrazek24 = $_POST['obrazek24'];
	$miniatura24 = $_POST['miniatura24'];
	$obrazek25 = $_POST['obrazek25'];
	$miniatura25 = $_POST['miniatura25'];
[PHP] pobierz, plaintext

john_doe

14.04.2014, 13:34:07

a co Ty wgrywasz przez form obraz i jego miniaturę?
Wgrywaj tylko fotkę a mini rób w locie to po 1sze.

po drugie użyj tablic.

Xart

14.04.2014, 13:37:11

@UP
W bazie będę miał same linki do fotek i miniatur więc nie mogę zrobić tego w locie ...
da radę ten kod pętlą jakoś ?

I kolejne ten kod może też się jakoś da mhm ?

[PHP] pobierz, plaintext 
$mysqli->query("INSERT INTO `galleries` (id, nazwa, opis, ilosc, linkz, obrazek1, obrazek2, obrazek3, obrazek4, obrazek5, obrazek6, obrazek7, obrazek8, obrazek9, obrazek10, obrazek11, obrazek12, obrazek13, obrazek14, obrazek15, obrazek16, obrazek17, obrazek18, obrazek19, obrazek20, obrazek21, obrazek22, obrazek23, obrazek24, obrazek 25, miniatura1, miniatura2, miniatura3, miniatura4, miniatura5, miniatura6, miniatura7, miniatura8, miniatura9, miniatura10, miniatura11, miniatura12, miniatura13, miniatura14, miniatura15, miniatura16, miniatura17, miniatura18, miniatura19, miniatura20, miniatura21, miniatura22, miniatura23, miniatura24, miniatura 25, data) VALUES....
[PHP] pobierz, plaintext

john_doe

14.04.2014, 13:40:15

Xart a możesz wyjaśnić co po co i dlaczego?
abyśmy wiedzieli co chcesz osiągnąć, jak ma działać
To podpowiemy optymalne rozwiązanie, bo to co tutaj pokazałeś jest raczej o kant d.... roztrzasnąć

Xart

14.04.2014, 13:42:50

Mam tabelę w bazie danych w której trzymam dane takie jak :

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `galleries` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `nazwa` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `opis` text COLLATE utf8_polish_ci NOT NULL,
  `ilosc` int(11) NOT NULL DEFAULT '1',
  `linkz` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek1` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek2` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek3` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek4` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek5` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek6` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek7` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek8` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek9` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek10` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek11` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek12` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek13` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek14` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek15` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek16` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek17` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek18` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek19` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek20` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek21` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek22` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek23` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek24` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `obrazek25` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura1` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura2` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura3` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura4` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura5` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura6` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura7` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura8` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura9` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura10` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura11` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura12` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura13` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura14` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura15` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura16` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura17` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura18` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura19` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura20` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura21` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura22` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura23` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura24` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `miniatura25` varchar(100) COLLATE utf8_polish_ci NOT NULL,
  `data` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext

Wgrywam dane do tej tabeli aby potem wyświetlić je skryptem takim:

[PHP] pobierz, plaintext 
<?php
include("config.php");
$id = $_GET['id'];
$zapytanie = $mysqli->query("SELECT * FROM `galleries` WHERE id='$id");
$wynik = $zapytanie->fetch_assoc();
$ilosc = $wynik['ilosc'];
for($i = 1; $i <= $ilosc; $i++){
$duzy = $wynik['obrazek'.$i.''];
$maly = $wynik['miniatura'.$i.''];
print '<a class="fancybox-buttons" data-fancybox-group="button" href="'.$duzy.'"><img src="'.$maly.'"/></a>';
}
?>
[PHP] pobierz, plaintext

nospor

14.04.2014, 13:45:09

Masz totalnie złą strukture bazy...
Mają byc dwie tabele:
Tabela GALERIA
ID
NAZWA
OPIS
LINKZ

Tabela OBRAZEK
ID
ID_GALERII - id galerii, do ktorej nalezy obrazek
SCIEZKA

I juz.

john_doe

14.04.2014, 13:46:15

w 1szej kolejności powinieneś utworzyć drugą tabelę np. pictures
która będzie przechowywała obrazy dla danej galerii w Twoim przypadku tabeli galleries

struktura tabela pictures
picture_id
gallery_id -> to id jest kluczem obcym z tabeli galleries
picture_file

Xart

14.04.2014, 13:47:40

Ale pytanie poco ?
Jak mogę te dane trzymać w jednej tabeli i będzie prościej ?

nospor

14.04.2014, 13:48:31

Po co? Bo tak sie wlasnie robi prawidlowo. To co ty zrobiles nie ma reki ani nogi.... I nie, to co zrobiles nie jest prostrze.
A co gdy nagle w galerii bedzie 30 zdjec a nie 24? Bedziesz dodawal kolejne pola? W schemaci co ci zaproponowalismy nie ma znaczenia ile bedzie zdjec w galerii

Xart

14.04.2014, 13:51:11

Nie ;p
Piszę takiego małego CMS'a dla strony szkoły

I wiem że w jeden galerii nie będzie więcej niż 25 zdjęć bo wtedy robią 2 galerie

nospor

14.04.2014, 13:52:30

Przychodzisz na forum z prosbą jak skrocic.... podajemy ci wiec jak to sie robi prawidlowo a Ty dalej swoje.... To po co marnujesz nasz czas? Naucz się pisać prawidlowo. To co Ty chcesz zrobic to marnowanie czasu na zle praktyki

Xart

14.04.2014, 13:55:14

Ale kodu itak to nie skróci nie prawdaż ?
Bo itak będę musiał dodać te zdjęcia tlyko tyle, że do innej tabeli i dodatkowo narobi mi się wiecej
$mysqli->query("INSERT INTO `pictures`
bo będę musiał każdy obrazek zapisywać oddzielnie bo chyba inaczej się nie da...

Kshyhoo

14.04.2014, 13:56:09

Cytat(Xart @ 14.04.2014, 14:29:30 )

[PHP] pobierz, plaintext 
$obrazek1 = $_POST['obrazek1'];
$miniatura1 = $_POST['miniatura1'];
[PHP] pobierz, plaintext

[PHP] pobierz, plaintext 
$o1 = $_POST['o1'];
$m1 = $_POST['m1'];
[PHP] pobierz, plaintext

nospor

14.04.2014, 13:56:29

Skroci kod i to bardzo.
W petli bedziesz dodawal zdjecia

Turson

14.04.2014, 13:56:52

O przebudowie bazy już pisałem w innym temacie, ale dalej upierasz się przy swoim.

Cytat

Ale kodu itak to nie skróci nie prawdaż ?
Bo itak będę musiał dodać te zdjęcia tlyko tyle, że do innej tabeli i dodatkowo narobi mi się wiecej

Tabela więcej, a kodu mniej

Kshyhoo hahaha

john_doe

14.04.2014, 13:58:28

robisz coś takiego

[PHP] pobierz, plaintext 
<input type="file" name="pic[]" />
[PHP] pobierz, plaintext

i takich inputów masz tyle ile chcesz wgrać fotek, u Ciebie 25 ( sporo

)

dalej robisz posta

i uruchom sobie var_dump($_POST)
zobaczysz co zawiera tablica POST i już wiesz wszystko

potem wystarczy w pętli np. for jechać po tej tablicy i zapodawać INSERT, czyli zamiast 25 "z buta" insertów generujesz je dynamicznie za pomocą php

Xart

14.04.2014, 14:01:22

@UP
Inputów mam tyle ile sobie ustawię podczas dodawania galerii
Mam pole select i w zależności od ilości wybranych np '12'
wyświetli mi 12 inputów na obrazek i 12 na miniaturki

mhm
tylko nie mam badego pojęcią jak to robić inaczej niż "z buta" bo nigdy nie robiłem ;p
dodawać pętlą o.O ?
Jeszcze takie pytanie z innej beczki mogę dodać sam obrazek chyba nie ? A jako miniaturke wyświetlać go w innych rozmiarach za pomocą HTML hmm ?

john_doe

14.04.2014, 14:09:32

do Ciebie jak grochem o ścianę, albo chcesz nauczyć się czegoś albo nie!
Miej sobie <select> gdzie ustawiasz ile ma być inputów, to bez znaczenia.

niech dla przykładu będą trzy

[PHP] pobierz, plaintext 
<input type="file" name="pic[]" />
<input type="file" name="pic[]" />
<input type="file" name="pic[]" />
[PHP] pobierz, plaintext

[PHP] pobierz, plaintext 
 
<?php
for($i=0;$i<count($_FILES["pic"]["name"]);$i++) 
{
if (file_exists("upload/" . $_FILES["pic"]["name"][$i]))
      {
      echo $_FILES["pic"]["name"][$i] . " już istnieje na serwerze. ";
      }
    else
      {
 
//I tutaj upload fotki i insert do tabeli
 
      move_uploaded_file($_FILES["pic"]["tmp_name"][$i],
      "upload/" . $_FILES["pic"]["name"][$i]);
 
$query = " INSERT INTO tabela (COL1, COL2, COL3) VALUES ( COS, COS, $_FILES["pic"]["tmp_name"][$i] ";
 
mysql_query($query);
 
      }
}
?>
 
[PHP] pobierz, plaintext

finish, BTW tak kopiuj wklej nie zadziała jak widzisz ale ogląd masz.

Xart

14.04.2014, 14:13:05

Zrobię to jeszcze troszkę inaczej i prościej ale ogólnie dzięki

-CuteOne-

14.04.2014, 14:18:27

Rozumiem Twój sceptycyzm ale uwierz mi Twój kod to sieczka

1. google -> php upload (poczytaj o wgrywaniu plików, tablicach FILES itp)
2. baza danych tak jak podpowiedzieli koledzy wyżej
3. możesz uploadować grafikę a następnie zmniejszyć ją w php'ie (google -> php image resize example) i zapisać pod taką samą nazwą+jakiś postfix np. file_1_mini.jpg
4. nie polecam skalować dużych obrazków w html, widziałem jak koleś robił miniatury z obrazka 1900x1000 px, strona ładowała się pół dnia

Xart

14.04.2014, 14:27:22

@UP
Ja nie potrzebuje wogole uploadu obrazków

Zakładamy, że obrazki mam już upnięte np na imgur i podaje tylko linki do wyświetlenia ich w galerii...

Napisałem takie coś:

[PHP] pobierz, plaintext 
<?php
	$nazwa = $_POST['nazwa'];
	$zdjecie = $_POST['zdjecie'];
	$ilosc = $_POST['ilosc'];
	$opis = $_POST['opis'];
	$mysqli->query("INSERT INTO `galerie` (nazwa, opis, zdjecie, data) VALUES ('$nazwa', '$opis', '$zdjecie', NOW())");
	// pętla do dodania obrazków
	for($i = 1; $i <= $ilosc; $i++){
	$obraz = $_POST['obraz'.$i.''];
	$mysqli->query("INSERT INTO `obrazki` (galeria, link) VALUES ('$galeria', '$obraz')");
	}
	echo '<meta http-equiv="refresh" content="3; URL=panel.php">';
?>
[PHP] pobierz, plaintext

I mam kolejne pytania....
1. Czy to zadziała i czy jest to w miarę optymalne
2. Da się jakoś łatwo wyciągnąć ID wiersza z tabeli galerie, który wcześniej dodałem a jest on ustawiony jako AUTO...
Czy muszę po wgraniu wiersza do galerie pobrać dane ?

nospor

14.04.2014, 14:30:33

Robiac nazwy z $i zamiast tablicy jak ci proponowano, wcale nie zrobiles proscie.... tablica byla o niebo bardziej uniwersalna i poprawna.

Ostatnie ID zwroci ci http://pl1.php.net/manual/en/mysqli-stmt.insert-id.php

Xart

14.04.2014, 14:42:06

No to jeszcze muszę spytać...
ale ja was męczę...

mógłby ktoś jakis link do tutka jak to na tych tablicach zrobić ten kod co mam ?

[PHP] pobierz, plaintext 
for($i = 1; $i <= $ilosc; $i++){
	$obraz = $_POST['obrazek'.$i.''];
	$mysqli->query("INSERT INTO `obrazki` (galeria, link) VALUES ('$galeria', '$obraz')");
	}
[PHP] pobierz, plaintext

// edit

mógłbym to zrobić np tak:

$obraz = $_POST['obrazek'][$i]; ?

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.