Witam!

Próbuję zrkbić sumowanie wybranych danych z sql

Przykładowa tabela:
ID|nazwa_produktu|energia_produktu|bialko_produktu
1 |Mąka | 2000 |100
2 |Woda | 2000 |100
3 |Margaryna | 2000 |100
4 |Marmolada | 2000 |100

chciałbym zsumować energie_produktu dla np. Mąki i Margaryny (dane porównuję za pomocoów post)

[SQL] pobierz, plaintext 
 
$s1=$_POST['skladnik'];
$s2=$_POST['skladnik0'];
$s3=$_POST['skladnik1'];
$s4=$_POST['skladnik2'];
$s5=$_POST['skladnik3'];
$s6=$_POST['skladnik4'];
 
 
 
 
 
	$zapisz = $_POST['zapisz'];
 
	IF(isset($zapisz)){
 
			$stmt = $pdo_db -> prepare('SELECT * FROM `produkty` WHERE `nazwa_produktu` = :nazwa_produktu '); // 2
			$stmt -> bindValue(':nazwa_produktu', $_POST['skladnik'], PDO::PARAM_INT);
			$stmt -> execute(); // 3
 
			IF($details = $stmt -> fetch()) // 4
			{
				$test=$details['energia_produktu'];
 
 
				echo '<hr/>
				<p><b>Energia produktu:</b> '.$suma.'</p>
';			
			}
			else
			{
				echo '<hr/><p>Przepraszamy, podany rekord nie istnieje!</p>';
			}
			$stmt -> closeCursor();
		}
 
 
 
[SQL] pobierz, plaintext 

Chcesz wypisać tylko sumę czy masz gdzieś listę tych wybranych produktów? Bo jeśli to pierwsze, to możesz zrobić zapytanie z sum(), a jeśli drugie, to możesz zsumować sobie w pętli, bo i tak masz potrzebne rekordy.

Tylko sume próbowałem z sum() ale nie wychodzi

Pokaż jak próbowałeś.

[SQL] pobierz, plaintext 
	IF(isset($zapisz)){
 
 
			$stmt = $pdo_db -> prepare('SELECT sum(energia_produktu) FROM `produkty` WHERE `nazwa_produktu` = :nazwa_produktu AND `nazwa_produktu2` = :nazwa_produktu AND `nazwa_produktu` = :nazwa_produktu3'); // 2
			$stmt -> bindValue(':nazwa_produktu', $_POST['skladnik'], PDO::PARAM_INT);
                        $stmt -> bindValue(':nazwa_produktu2', $_POST['skladnik0'], PDO::PARAM_INT);
                        $stmt -> bindValue(':nazwa_produktu3', $_POST['skladnik1'], PDO::PARAM_INT);
			$stmt -> execute(); // 3
 
			IF($details = $stmt -> fetch()) // 4
			{
				$suma = $details['sum(energia_produktu)'];
 
 
				echo '<hr/>
				<p><b>Energia produktu:</b> '.$suma.'</p>
';			
			}
			else
			{
				echo '<hr/><p>Przepraszamy, podany rekord nie istnieje!</p>';
			}
			$stmt -> closeCursor();
		}
[SQL] pobierz, plaintext 

Twoje zapytanie wyszukuje produkty, których nazwa to jednocześnie A, B i C. Czyli to się nigdy nie sprawdza, gdy A != B != C. Rozumiesz?

Zamiast tego, zrób zapytania na zasadzie:

[SQL] pobierz, plaintext 
SELECT sum(`energia_produktu`)
  FROM `produkty`
 WHERE `nazwa_produktu` IN ('Jabłko', 'Sernik', 'Czekolada')
[SQL] pobierz, plaintext 

Nie zabardzo tak mogę .

Juz pokazuję na czym to polega:

Wpisujemy do formularza dane np.woda i Mąka które sa filtrowane za pomoca autocompleat i sprawdzane czy sa w bazie jezeli sa w bazie skrypt przechodzi dalej.

Następnie muszę zliczyć ich energie ktora jest podana w bazie, a dane ma filtrowac z pomoca formularza czyli jak wpisze w formularzu Mąka i Woda to ma zliczyc energie mąki i wody.



A robiąc w ten sposób zlicza mi ze wszystkich wierszy kiedy j bym chcial tylko z wybranych

[SQL] pobierz, plaintext 
	IF(isset($zapisz)){
 
 
			$stmt = $pdo_db -> prepare('SELECT sum(energia_produktu) FROM `produkty` WHERE `nazwa_produktu` IN (`nazwa_produktu` = :nazwa_produktu, `nazwa_produktu` = :nazwa_produktu2, `nazwa_produktu` = :nazwa_produktu3)'); // 2
			$stmt -> bindValue(':nazwa_produktu', $_POST['skladnik'], PDO::PARAM_INT);
                        $stmt -> bindValue(':nazwa_produktu2', $_POST['skladnik0'], PDO::PARAM_INT);
                        $stmt -> bindValue(':nazwa_produktu3', $_POST['skladnik1'], PDO::PARAM_INT);
			$stmt -> execute(); // 3
 
			IF($details = $stmt -> fetch()) // 4
			{
				$suma = $details['sum(energia_produktu)'];
 
 
				echo '<hr/>
				<p><b>Energia produktu:</b> '.$suma.'</p>
';			
			}
			else
			{
				echo '<hr/><p>Przepraszamy, podany rekord nie istnieje!</p>';
			}
			$stmt -> closeCursor();
		}
 
 
[SQL] pobierz, plaintext 

Nie wiem z czym masz problem. Po prostu zrób tak, żeby zapytanie otrzymało odpowiednie IN (a, b, ...).

Tak też zrobiłem ale zlicza mi wszystkie wiersze a nie tylko 3 tak jak chce

obecnie mam w bazie 12 pozycji i w echo mi zwraca 12 zamiast 3 zakładajac ze dla kazdego energia jest rowna 1

Zrobiłem to w jeszcze inny sposób tylko nie działa ma ktos jakis pomysł

[PHP] pobierz, plaintext 
$skladniki = [];
for ($num=0; $num < 20; $num++) {
    if (!is_int($_POST['skladnik'.$num])) {
        break;
    }
 
    $skladniki[] = $_POST['skladnik'.$num];
}
if (isset($zapisz)) {
$stmt = $pdo_db -> prepare('SELECT energia_produktu FROM `produkty` WHERE `nazwa_produktu` IN (:skladniki)');
$stmt->bindValue(':skladniki', implode($skladniki, ','), PDO::PARAM_STR);
$stmt->execute(); // 3
$result = $stmt->fetch();
 
if ($result === null) {
    echo '<hr/><p>Przepraszamy, podany rekord nie istnieje!</p>';
    die();
}
 
$suma = 0;
foreach ($result as $item) {
    $suma+= $item['energia_produktu'];
}
echo '<p><b>Energia produktu:</b> '.$suma.'</p>';
}	
 
[PHP] pobierz, plaintext 

W chwili obecnej, Twoje zapytanie wygląda tak:

[SQL] pobierz, plaintext 
SELECT `energia_produktu` FROM `produkty` WHERE `nazwa_produktu` IN ('Jabłko,Banan,Czekolada')
[SQL] pobierz, plaintext 

Jest już prawie dobrze. Operator IN w SQL nie przyjmuje pojedynczego stringa ze słowami oddzielonymi przecinkami, ale osobnych stringów. Powinno być tak:

[SQL] pobierz, plaintext 
SELECT `energia_produktu` FROM `produkty` WHERE `nazwa_produktu` IN ('Jabłko', 'Banan', 'Czekolada')
[SQL] pobierz, plaintext 

Widzisz różnicę? Musisz dynamicznie podpiąć te parametry, to nie jest aż takie znowu trywialne. Dam Ci podpowiedź. Najłatwiej będzie Ci to zrobić w taki sposób:

[PHP] pobierz, plaintext 
$stmt = $pdo_db -> prepare('SELECT energia_produktu FROM `produkty` WHERE `nazwa_produktu` IN (?, ?, ?)');
$stmt->execute(['Jabłko', 'Banan', 'Czekolada']);
[PHP] pobierz, plaintext 

Najtrudniejsza część to dokleić te znaki zapytania w odpowiedniej liczbie, ale dasz radę.

Tu masz opisane jak to zrobić.

Dlaczego to durne array_fill przyjmuje indeks początkowy jako pierwszy parametr? Ciekawe w ilu przypadkach na 100 ktoś to ustawia na coś innego niż zero. Idiotyzm.

A dlaczego durne i od ilu ma mieć indeks? Na logikę skladnia jest dobra chociaz pamiętam ze jak wtedy szukałem najlepszego rozwiązania to właśnie sporo w tym bylo błędów. Ale że ludzie nie czytają dokumentacji to ich problem.

Zupełnie się z Tobą nie zgadzam, viking. A już tym zdaniem "Ale że ludzie nie czytają dokumentacji to ich problem." osiągnąłeś szczyt ignorancji.

Głupawa deklaracja array_fill nie jest niczym uzasadniona. Ta funkcja jest po prostu źle zaprojektowana. Jej deklaracja powinna wyglądać tak:

[PHP] pobierz, plaintext 
function array_fill(mixed $value, int $num, int $start_index = 0)
[PHP] pobierz, plaintext 

Lub tak:

[PHP] pobierz, plaintext 
function array_fill(int $num, mixed $value, int $start_index = 0)
[PHP] pobierz, plaintext 

Przy czym moim zdaniem bardziej naturalna jest pierwsza możliwość, bo najpierw chce się (ja chcę?) podać co ma być w tej tablicy, a potem ile. Nie na odwrót.

Dlaczego uważasz, że w porządku jest $start_index jako pierwszy parametr? Pokusiłbym się nawet o zdanie, że tego parametru mogłoby całkowicie nie być. Po co niby mam zaczynać od jakiegokolwiek indeksu numerycznego? Jedyny sensowny przykład jaki wymyśliłem, to taki:

[PHP] pobierz, plaintext 
$cats = array_fill(0, 'Franek', 3) + array_fill(3, 'Bubek', 5);
[PHP] pobierz, plaintext 

Ale i tak jest głupawy, bo lepiej by było zrobić to tak:

[PHP] pobierz, plaintext 
$cats = array_merge(array_fill('Franek', 3), array_fill('Bubek', 5));
[PHP] pobierz, plaintext 

Jak już mam mieć wpływ na indeks w array_fill to o wiele lepszy byłby argument typu callable:

[PHP] pobierz, plaintext 
function array_fill(mixed $value, int $num, callable $keys)
[PHP] pobierz, plaintext 

Otrzymywałby wartość i indeks jako parametry i żeby użyć go w durnowaty sposób zaproponowany przez array_fill, mielibyśmy na przykład:

[PHP] pobierz, plaintext 
$cats = array_fill('Bubek', 5, function ($value, $index) {
    return $index + 3;
});
[PHP] pobierz, plaintext 

viking naprawdę uważasz, że funkcja array_fill jest sensownie zaprojektowana? o0

Wszystkie argumenty są wymagane więc kolejność nie ma i tak żadnego znaczenia. Dla mnie obecna konstrukcja jest bardziej naturalna. Wolę indeksy numeryczne na początku a później dopiero wartości. Przykładowo:

[PHP] pobierz, plaintext 
array_fill(
  $this->getA(),
  (int) $this->getX(),
  długi callable
);
[PHP] pobierz, plaintext 

Zdecydowanie łatwiej ogarnąć krótkie pierwsze, długie kolejne

Podaj mi sensowne uzasadnienie tego kodu. Tworzysz tablicę rozpoczynającą się od indeksu getA() zawierającą kilka identycznych callable? Nie. Po prostu nie.

Co rozumiesz przez identycznych?

Nie wiem w sumie po co ta dyskusja, to jest jeden z tych przypadków, gdzie jestem pewien mojego punktu widzenia. Bardzo nie lubię, kiedy ktoś się wykazuje ignorancją i posługuje się argumentami na poziomie "o ich problem, że nie czytają dokumentacji". Jest to retoryka z rynsztoka.

Argumenty funkcji muszą być w sensownej kolejności, niestety PHP nie przestrzega tego paradygmatu, dlatego mamy np. array_map i array_filter, które mają je po prostu na odwrót. Takich przypadków jest o wiele więcej w PHP. Nie ma co tego bronić, bo to podpada pod fanatyzm i zaślepienie. Są to złe decyzje projektowe i oby kiedyś zostały naprawione.

Przykład z array_fill jest ewidentny. Tablica indeksowana n..m, gdzie n > 0, to w większości przypadków błąd projektowy aplikacji. Nie bez powodu w (prawie?) wszystkich innych językach nie ma nawet czegoś takiego jak tablica asocjacyjna, czasem są słowniki. Ale nikomu nie przychodzi do głowy słownik z kluczami np. 5, 6, 7 itd. Jest to egzotyczne zastosowanie, które owszem - może kiedyś znaleźć zastosowanie, ale nie należy dla tego osobliwego przypadku dawać - uwaga - obowiązkowego i pierwszego (!) argumentu funkcji. Te zera w array_fill pewnie mają już jakąś mafię.

Wybacz ale na pewno 90% problemów na tym forum wynika tylko i wyłącznie z tego że komuś nie chciało się zajrzeć do dokumentacji. To co funkcja zwraca to dla wielu osób już totalna abstrakcja. A i tak dokumentacja przeszła jakiś czas temu spory lifting i jest obecnie dużo bardziej czytelna. Lenistwa nie popieram więc dlaczego tego jednego zdania się po raz kolejny uczepiłeś? Błędy w nazewnictwie i kolejności znane są od wielu wielu (żeby daleko nie szukać mysql_query / pg_query) lat więc nic nowego. Nie ma w innych językach tablic asocjacyjnych / słowników? Python, C++, Perl. Jakoś we wszystkich występuje.

Uciekam z tego tematu, bo robi się niebezpiecznie idiotycznie.