OT von hpzeller aus Thema "MODUL: Artikel Anfrage" ausgegliedert

[hpzeller]

Hallo Community

Bei diesem Thread handelt es sich um eine Ausgliederung des off-topic Beitrags hier von hpzeller  aus dem Thread "MODUL: Artikel Anfrage".

Gruss
Hanspeter

Linkback: https://www.modified-shop.org/forum/index.php?topic=42767.0

[hpzeller]

Folgende Antwort bezieht sich auf diese Antwort hier.

Ich denke isset($articelId) in if (isset($articelId) && $articelId != '') wurde genau deswegen eingefügt, um eine etwaige Fehlermeldung bei Nichtvorhandensein von Variablen zu verhindern.

Begründung:
In der Datei modified-shop_1.06-r4642_SP3_install\modified-shop_1.06-r4642_SP3_install\shoproot\account_edit.php Zeile 88 steht noch dieser Code,

Code: PHP  [Auswählen]

if($vatID->vat_info['error']==1){

aber in der Datei modified-shop_2.0.7.2-r14622_install\shoproot\account_edit.php Zeile 111 dann dieser.

Code: PHP  [Auswählen]

if (isset($vatID->vat_info['error']) && $vatID->vat_info['error']==1){

Gruss
Hanspeter

[noRiddle (revilonetz)]

Da du auf einen Post meiner Wenigkeit verlinkst, auf welchen du sagst geantwortet zu haben, hier noch eine kurze Stellungnahme:
Mir ist das klar was du schreibst. Ich habe auf RobinTheHoods  Einlassungen reagiert, welcher in Artikel Anfrage | Antwort #491 Folgendes empfahl:

Hallo und guten Abend,

da ich das hier gesehen habe und gern etwas dazu sage würde, weil mir das öfters mal auffällt. Ich persönlich würde auch

Code: PHP  [Auswählen]

if (!empty($article_id)) {
    ...
}

nicht empfehlen. Natürlich erfüllt es hier seine Aufgabe, aber ich würde gerne einmal folgendes zeigen, um es als Anregung zu verwenden:

Code: PHP  [Auswählen]

<?php

$articleId = 42;

...
// do a lot of things
...


if (!empty($articelId)) {
    echo 'Ein rotes Fahrrad';
}

Der Text "Ein rotes Fahrrad" wird uns NIE angezeigt egal welche $articleId wir wählen und wir haben mit der Funktion empty() sogar noch zusätzlich dafür gesorgt, dass uns PHP jetzt nicht einmal mehr mitteilt, was wir falsch gemacht haben.

Mit besten Grüßen und einen schönen Abend
Robin

Ich habe versucht ihm nachträglich in dem von dir zitierten Post zu widersprechen.

Gruß,
noRiddle

[hpzeller]

Hallo noRiddle,

dann habe ich deine Antwort wohl falsch verstanden.

Inzwischen habe ich die Antwort #491 von RobinTheHood noch mal genau durchgelesen und dabei festgestellt, dass er ja beide Varianten, also sowohl diese,

Code: PHP  [Auswählen]

if (isset($article_id) && $article_id != '') {

als auch diese,

Code: PHP  [Auswählen]

if (!empty($article_id)) {

nicht empfiehlt.

Gruss
Hanspeter

[noRiddle (revilonetz)]

Schließt du das aus dem "auch" bei "würde auch .... nicht empfehlen" ?
Ich sehe so recht nichts worauf sich "auch" bezieht.
Wenn es so gemeint war wie du sagst, was soll denn dann eine Lösung sein ?

Gruß,
noRiddle

[RobinTheHood]

Guten Abend,

ich selbst verwende isset() und empty() auch viel. In Programmiersprachen wie C, C++ und Java kann ich keine Variable verwenden, die nicht vorher deklariert wurde. Hier lassen sich Fehler bereits statisch vor der Ausführung des Codes finden. Deswegen finde ich es spannend drüber nachzudenken, wie man eine ähnliche formale Korrektheit mit PHP erreichen könnte.

Ich kann das selbst gar nicht so gut beschreiben. Die folgenden Links zeigen vielleicht, was ich über die Funktion empty() als Anregung geben wollte.

• Don't use isset and empty in PHP - https://youtu.be/QKi-tFNP9jw
  
• When to use empty in PHP? I'd say never - https://www.beberlei.de/post/when_to_use_empty_in_php_i_say_never
• Using isset() and empty() hurts your code (mit spannender Diskussion) - https://dev.to/aleksikauppila/using-isset-and-empty-hurts-your-code-aaa/comments

Wie bereits erwähnt, bin ich nicht gegen die Verwendung. Ich finde es nur nicht schlecht, wenn man mögliche Fallstricke kennt.

Mit besten Grüßen und einen schönen Sonntag
Robin

[hpzeller]

Hallo noRiddle,

ja genau, aus dem Wort "auch" schliesse ich, dass er beide Varianten nicht empfiehlt.

Gruss
Hanspeter

[noRiddle (revilonetz)]

@hpzeller
Verstehe.

@RobinTheHood
Bzgl. deiner geposteten Links:
Interessante Ideen und Diskussionen.

Es düfte keine allgemeingültige Lösung geben, man sollte schlicht wissen was man tut bei Verwendung irgendeiner der Varianten.
Expliziterer Code
!== NULL
!== 0
!== ''
!== false
ist aussagekräftiger und weniger fehleranfällig.

In der Tat habe ich mich oft bei der Verwendung von !empty()  unwohl gefühlt weil es so viel bedeuten kann.

Vielen Dank jedenfalls für das Aufbringen des Themas und für das Bewußtmachen, das ist immer mal wieder erhellend.

Das Thema erinnert mich übrigens an dieses Problem:
>> Seltsames PHP-Problem bzgl. typen-sicherem Vergleich
Seit dem achte ich mehr und mehr darauf ganz genau zu sein und wo möglich Operatoren zu verwenden die auch auf Typengleichheit gehen. Deshalb erinnert es mich an das was ich oben bei "Expliziter Code..." geschrieben habe.

Gruß,
noRiddle