[PHP] Warunkowe dodawanie zmiennych Opcje

[Therminus]

Mam taką tabelkę ze zmiennymi:

[PHP] pobierz, plaintext 
echo '<tr >
<td align="center">'.$wiersz2['Status_A'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_A'].'</td>
<td align="center">'.$wiersz2['Status_B'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_B'].'</td>
<td align="center">'.$wiersz2['Status_C'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_C'].'</td>
	...
<td align="right">'.($wiersz2['Pow_Uzyt_A'])+
		    ($wiersz2['Pow_Uzyt_B'])+
		    ($wiersz2['Pow_Uzyt_C'])+
	             ...
 
				.'</td>
			</tr>';
[PHP] pobierz, plaintext 

W sumie 8 pozycji (do litery "H")
Problem w tym, że na końcu muszę dodać te zmienne (wiersz 9 i dalsze w przedstawionym kodzie) pod warunkiem, że...
i tutaj zaczynają się schody.
Zmienna $wiersz2['Pow_Uzyt_A'] ma być brana pod uwagę w dodawaniu tylko wtedy, kiedy zmienna $wiersz2['Status_A'] = 'A' .
I analogicznie wartość ze zmiennej $wiersz2['Pow_Uzyt_B'] będzie dodawana, jeżeli zmienna $wiersz2['Status_B'] = 'B' i tak dalej.
W bazie pole w kolumnach "Status_[A...H]" po prostu zawiera tylko jedną literę (A...H) lub jest puste.
Jak to zrobić?

Ten post edytował Therminus 24.08.2022, 13:23:50

[trueblue]

Użyj funkcji if.

[Salvation]

[PHP] pobierz, plaintext 
<?php
 
declare(strict_types=1);
 
function summary(array $row): int
{
    $sum = 0;
 
    foreach(['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'] as $char) {
        $char = mb_strtoupper($char);
        $status = (string)($row['Status_' . $char] ?? '');
        $userPower = (int)($row['Pow_Uzyt_' . $char] ?? 0);
 
        if ($status === $char) {
            $sum += $userPower;
        }
    }
 
    return $sum;
}
[PHP] pobierz, plaintext 

Wykorzystanie (w Twoim przypadku):

[HTML] pobierz, plaintext 
<td align="right">' . summary($wiersz2) . '</td>
[HTML] pobierz, plaintext 

PS. Czy ktoś może wyłączyć parsowanie na emoty w blokach kodu?

Ten post edytował Salvation 24.08.2022, 16:26:56

[nospor]

PS. Czy ktoś może wyłączyć parsowanie na emoty w blokach kodu?

Niestety nie.
Ale mozesz zrobic to sam przed wyslaniem postu, WYLACZ EMOTKI czy jakos tak checkbox. To wylaczy dla calego postu

[Salvation]

Niestety nie.
Ale mozesz zrobic to sam przed wyslaniem postu, WYLACZ EMOTKI czy jakos tak checkbox. To wylaczy dla calego postu

Dzięki! Zapamiętam (IMG:style_emoticons/default/smile.gif)

[nospor]

ps: oj ta funkcja co naskrobales to chyba cos ci nie wyszla (IMG:style_emoticons/default/wink.gif) I to tak dosc mocno (IMG:style_emoticons/default/wink.gif)

[Salvation]

Niby dlaczego? Wszystko dobrze jest wg mnie: https://3v4l.org/4MkRf

Ten post edytował Salvation 24.08.2022, 16:26:17

[nospor]

A widzisz, nie doczytalem o co chodzilo autorowi. Zwracam honor (IMG:style_emoticons/default/smile.gif)

[Therminus]

[quote name='Salvation' date='24.08.2022, 16:52:52 ' post='1259713']
Coś mi się ta funkcja nie odpala z moim kodem.

[PHP] pobierz, plaintext 
declare(strict_types=1);
 
function summary(mysqli_fetch_array $row): int
{
    $sum = 0;
 
    foreach(['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'] as $char) {
        $char = mb_strtoupper($char);
        $status = (string)($row['Status_'.$char] (IMG:style_emoticons/default/questionmark.gif)  '');
        $userPower = (int)($row['Pow_Uzyt_'.$char] (IMG:style_emoticons/default/questionmark.gif)  0);
 
        if ($status === $char) {
            $sum += $userPower;
        }
    }
 
    return $sum;
}
 
$zapytanie2 = "SELECT * FROM `budynki` WHERE `Ulica` = '$wybor' and `Kategoria` LIKE 'LU%' and `Podzielone` = 'P'";
$wykonaj2 = mysqli_query($sql, $zapytanie2);
while($wiersz2=mysqli_fetch_array ($wykonaj2)) {
 
echo '<tr >
<td align="center">'.$wiersz2['Status_A'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_A'].'</td>
<td align="center">'.$wiersz2['Status_B'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_B'].'</td>
<td align="center">'.$wiersz2['Status_C'].'</td>
<td align="right">'.$wiersz2['Pow_Uzyt_C'].'</td>
	...
<td align="right">' . summary($wiersz2) . '</td>
 
			</tr>';
}
[PHP] pobierz, plaintext 

Nie jestem pewien wiersza nr 3. Być może tutaj jest błąd.

Ten post edytował Therminus 25.08.2022, 13:53:39

[nospor]

Czemu zrobiles
mysqli_fetch_array $row
?
Przeciez wyraznie miales napisane
array $row

Row to tablica

ps: I patrze nadal masz wywalone na to ze twoj skrypt jest podatny na sql injection. Zamiast to poprawic jak ci to juz dawno mowiono to ty wiesz "lepiej "

[Salvation]

Nie jestem pewien wiersza nr 3. Być może tutaj jest błąd.

Nie być może, a tak. Niepotrzebnie zmieniałeś deklarację funkcji. Miałeś zrobić jedynie copy-paste do siebie do kodu.

I popraw ten SQL Injection, bo Puchatek zaraz krwotoku dostanie (IMG:style_emoticons/default/haha.gif)

[Therminus]

ps: I patrze nadal masz wywalone na to ze twoj skrypt jest podatny na sql injection.

Mam wywalone, bo mnie w LAN nikt atakował nie będzie. (IMG:style_emoticons/default/oneeyedsmiley02.png)

Niepotrzebnie zmieniałeś deklarację funkcji.

Na początku rzeczywiście przekopiowałem tak jak było, ale nie działało. Potem kombinowałem i zmieniłem na próbę, a przed wysłaniem posta zapomniałem zmienić z powrotem.
Mam w przeglądarce biały ekran. Jak zakomentuję tę funkcję, to stronka się wyświetla (oczywiście z błędem o próbie użycia nie zadeklarowanej funkcji).
Wersja PHP: 7.4.29

I popraw ten SQL Injection, bo Puchatek zaraz krwotoku dostanie (IMG:style_emoticons/default/haha.gif)

(IMG:style_emoticons/default/haha.gif)

Ten post edytował Therminus 26.08.2022, 13:05:32

[nospor]

Mam wywalone, bo mnie w LAN nikt atakował nie będzie

Tyko kiedys zaczniesz pisac poza LAN i nagle ZONK bo zle nawyki zostaly. Jaki problem tam uzyc binodwania? Toz to 10 sekund roboty :/

Na początku rzeczywiście przekopiowałem tak jak było, ale nie działało. Potem kombinowałem i zmieniłem na próbę, a przed wysłaniem posta zapomniałem zmienić z powrotem.
Mam w przeglądarce biały ekran. Jak zakomentuję tę funkcję, to stronka się wyświetla (oczywiście z błędem o próbie użycia nie zadeklarowanej funkcji).

Pewnie uzywasz starego php. Salvation niestety lubi zakladac ze ludzie jada na najnowszym. Od mysli ze ktos uzywa starego to jemu krwotok grozi (IMG:style_emoticons/default/tongue.gif)

[Therminus]

Jaki problem tam uzyc binodwania? Toz to 10 sekund roboty :/

Jak się już wie co to jest bindowanie w tym przypadku. A ogarnięcie tematu, to dla mnie pewnie kilkanaście godzin. (IMG:style_emoticons/default/tongue.gif)

Pewnie uzywasz starego php. Salvation niestety lubi zakladac ze ludzie jada na najnowszym. Od mysli ze ktos uzywa starego to jemu krwotok grozi (IMG:style_emoticons/default/tongue.gif)

Tak pomyślałem, dlatego edytowałem poprzedniego posta
PHP 7.4.29 (cli) (built: Apr 14 2022 14:45:47) ( ZTS )
Nie taki bardzo stary, zatem...

EDIT:
Sprawdziłem na tej stronce gdzie Salvation umieścił kod i wybrałem moją wersję PHP i... tam działa. (IMG:style_emoticons/default/smile.gif)
A u mnie nie chce.

Ten post edytował Therminus 26.08.2022, 13:38:36

[nospor]

Skoro masz biala strone to sprawdz logi, jakis blad masz

Jak się już wie co to jest bindowanie w tym przypadku. A ogarnięcie tematu, to dla mnie pewnie kilkanaście godzin

https://www.php.net/manual/en/mysqli-stmt.bind-param.php

[Therminus]

Skoro masz biala strone to sprawdz logi, jakis blad masz

Wiemy coś więcej.
PHP Fatal error: strict_types declaration must be the very first statement in the script in /var/www/(...)/lok_uzytkowe.php on line 147,
U mnie ta linia to:

[PHP] pobierz, plaintext 
declare(strict_types=1);
[PHP] pobierz, plaintext 

Dzięki za linka.

[nospor]

PHP Fatal error: strict_types declaration must be the very first statement in the script in /var/www/(...)/lok_uzytkowe.php on line 147,

Komunikat dosc jasny...

Daj
declare(strict_types=1);
Na poczatku pliku jako pierwsza linia albo wywal to w ogole

[Therminus]

Daj
declare(strict_types=1);
Na poczatku pliku jako pierwsza linia albo wywal to w ogole

Wywaliłem to, ale przy dodawaniu nie bierze pod uwagę części dziesiętnych liczb.

Na początku pliku, to znaczy zaraz za <?php
?
Tak miałem od początku.

[nospor]

No bo w kodzie masz

$userPower = (int)($row['Pow_Uzyt_'.$char] questionmark.gif 0);

zamien na
$userPower = (float)($row['Pow_Uzyt_'.$char] questionmark.gif 0);


Dostales pare linije kkodu, no naprawde mogles sie zastanowic co robi kazda z nich. Branie z forum tez polega na nauce tego co sie dostaje

Na początku pliku, to znaczy zaraz za <?php
?
Tak miałem od początku.

Komunikat bledu mowil jednak co innego, mowil tez ze bylo to w linii 147 a nie na poczatku pliku

[Salvation]

Od mysli ze ktos uzywa starego to jemu krwotok grozi :P

xDDD dokładnie!

przy dodawaniu nie bierze pod uwagę części dziesiętnych liczb.

Bo masz rzutowanie na INT, no ale to już nospor wyjaśnił.

Na początku pliku, to znaczy zaraz za <?php
?
Tak miałem od początku.

Jak znam życie, to ten plik co umieściłeś w temacie include'ujesz do index.php albo do innego pliku, prawda?

[nospor]

Jak znam życie, to ten plik co umieściłeś w temacie include'ujesz do index.php albo do innego pliku, prawda?

Includowanie tu nie ma nic do znaczenia. declare ma byc na poczatku i tyle w temacie. Bardziej bym odstawial ze na poczatku tego pliku ma wpyte kodu HTML, a dokladnie 146 linii ( (IMG:style_emoticons/default/wink.gif) ) a dopiero potem otwiera <?php i daje declare.
A na poczatku znaczy na poczatku i tyle (IMG:style_emoticons/default/wink.gif)

[Therminus]

No bo w kodzie masz

$userPower = (int)($row['Pow_Uzyt_'.$char] questionmark.gif 0);

Aaaa... ślepota. (IMG:style_emoticons/default/haha.gif)

Komunikat bledu mowil jednak co innego, mowil tez ze bylo to w linii 147 a nie na poczatku pliku

Ale jesteś czujny, (IMG:style_emoticons/default/smile.gif)
W pliku mam trochę HTML-a, potem jedną sekcję PHP, potem znowu HTML, i drugą sekcję PHP.
I te 147 jest właśnie na początku tej drugiej sekcji.

Już dodaje dobrze. (IMG:style_emoticons/default/smile.gif)

Przy okazji mam pytanie nietypowe...
chcę w rzeczonej tabelce dać kolorowe litery, cyfry i jednocześnie pogrubione, ale nie mogę tego efektu uzyskać.
Albo mam bold, albo kolor
Coś takiego:

[HTML] pobierz, plaintext 
<td align="center"; style= "font-weight: bold", "color:blue">'.$wiersz2['Status_C'].'</td>
[HTML] pobierz, plaintext 

Jak wyrzucę z kodu "font-weight: bold" to kolor jest, jak wyrzucę "color:blue", to mam pogrubienie (czarne).
Zaklęcia jakie znalazłem w sieci nie działają. (IMG:style_emoticons/default/smile.gif)

[nospor]

nie
style= "font-weight: bold", "color:blue"

a:
style= "font-weight: bold,color:blue"

[Therminus]

style= "font-weight: bold,color:blue"

To było pierwsze, co zastosowałem, potem kombinowałem inaczej (IMG:style_emoticons/default/smile.gif)
Niestety, kolor działa, bold nie działa. (IMG:style_emoticons/default/thumbsdownsmileyanim.gif)

[trueblue]

style= "font-weight: bold; color:blue"

[nospor]

Oczywiscie sredni a nie przecinek jak raczyl poprawic trueblue.

[Therminus]

style= "font-weight: bold; color:blue"

Dzięki, a wydawało mi się, ze sprawdziłem wszystkie opcje z cudzysłowami i przecinkiem, średnikiem...

[gino]

@Salvation troszkę się przyczepię do tej funkcji.
Mnie uczono na studiach, że funkcja napisana dobrze poza oczywistymi przypadkami ma być nienaruszalna.
A co jeśli kodującemu przyjdzie ochota rozszerzyć zakres dajmy na to do K? Tablica ta powinna być argumentem funkcji. Po co też dodatkowa funkcja mb_strtoupper? Nie można od razu tablicy z dużymi literami zdefiniować?
Sorki, że się czepiam, ale jeśli chodzi o kod to troche pedantyczny jestem a że wywodzę się z C to operatory trójargumentowe jakoś same mi się piszą :)

Kod

function summary(array $chars, array $row): int
{
    $sum = 0;
    
    foreach($chars as $char) {
        $status = (string)($row['Status_' . $char] ?? '');
        $userPower = (int)($row['Pow_Uzyt_' . $char] ?? 0);
        
        $sum += ($status === $char) ? $userPower : 0;
    }
    
    return $sum;
}

$chars = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H'];
$array = [
    'Status_A' => 'A',
    'Pow_Uzyt_A' => 10,
    'Status_B' => 'B',
    'Pow_Uzyt_B' => 25,
    'Status_C' => '',
    'Pow_Uzyt_C' => 100,
];

echo summary($chars, $array);

Ten post edytował gino 27.08.2022, 06:48:14

[Salvation]

Mnie uczono na studiach, że funkcja napisana dobrze poza oczywistymi przypadkami ma być nienaruszalna.

Też kończyłem studia i takich głupot nie słyszałem na wykłądach.

A co jeśli kodującemu przyjdzie ochota rozszerzyć zakres dajmy na to do K? Tablica ta powinna być argumentem funkcji.

Jak dla mnie, niepotrzebne rozszerzenie o argument. Sama tablica powinna być "zamknięta" w funkcji, bo to jest baza funkcji.
+ Nie programuje się na zapas. Jest to jedna z zasad dobrych praktyk (IMG:style_emoticons/default/wink.gif)

Po co też dodatkowa funkcja mb_strtoupper? Nie można od razu tablicy z dużymi literami zdefiniować?

Można. Szybciej mi jednak było użyć małych liter i skorzystać z funkcji (IMG:style_emoticons/default/biggrin.gif) - z założenia też wychodzę, że to co "przyjmę" jest z małych liter.
Równie dobrze możesz użyć stringa i iterować po tym :shrug: Rozwiązanie zależy od autora (IMG:style_emoticons/default/wink.gif)

A żeby było prawilniej i z zachowaniem typowania od początku, to $row powinno być VO, ale wiadomo jak to jest z "projektami do szuflady" i nauką (IMG:style_emoticons/default/wink.gif)

Ten post edytował Salvation 28.08.2022, 22:40:16

[gino]

Może inaczej ujmę nienaruszalność, chodzi o to, że minimalizuje się czas do późniejszego grzebania w funkcjach, metodach ogólnie w bibliotekach współdzielonych lub pisanych dla konkretnego programisty, czasem mniej ogarniętego - do zera. Opis co ma robić ta funkcja nie narusza praktyki pisania na zapas w żadnym wypadku.
Coś należy lub nie do "bazy" funkcji? Sorki, nie znam takiego pojęcia. Może to jakiś skrót myślowy, niemniej domyślam się o co chodzi. Dane które mogą być w przyszłości zmienione lub rozszerzone nie należą do niej.

Pracowałeś kiedyś w projekcie w którym brało udział kilkunastu programistów wzwyż? Bo ja wiele razy. I jeśli widzieliśmy, pomimo opisu projektu często niedokładnego z racji tego, że ktoś kto to nadzorował miał delikatnie mówiąc wywalone, jakieś założenie nie było opisanie dokładnie np. jak tutaj zakres może ulec zmianie to się to pisało od razu. Uwierz, to czasem dobra praktyka. Później piszesz coś innego, ktoś sobie jednak przypomni że zakres może się zmienić i zawraca Ci głowę, którą masz zaprzątniętą czymś innym bo już piszesz coś innego. A uwierz mi też, że wszystko jest na wczoraj, cito, już, death line itd. Funkcja nie traci nic z opisu i działania i nic nie jest pisane na zapas.
Sorki, że trochę nieopatrznie wywołałem dyskusje zupełnie tutaj zbędną, więc to ostatni mój post w tym temacie.

[Salvation]

Nie wiem w ogóle skąd pomysł na CR w wątku (IMG:style_emoticons/default/biggrin.gif) to nie SO. Mogłeś dać swoje rozwiązanie i tyle, a nie czepiać się teraz czyjegoś kodu, bo wg Ciebie nie jest "piękny".
Jakoś w innych tematach nie widać Twoich postów z poprawianiem czyichś kodów, a też nie są idealne ;D

A odnośnie "bazy", to miałem na myśli to, że funkcja "summary" podsumowuje JEDYNIE kolumny A-H. Na chwilę obecną spełnia założenia. Trzeba będzie to rozszerzyć, przepisać lub delikatnie zmienić, to przyjdzie czas na to. Tak działa green field.
NIGDY nie pisze się idealnego kodu od razu, bo jedynie co się traci, to czas. A czas, to pieniądze klienta.

I dla Twojej wiadomości. Pracowałem i sam, i w zespołach po 15-20 osób. Także multi-narodowościowych (IMG:style_emoticons/default/wink.gif)