[Jake.Nu]

Ursprungligen postat av Althalos

Eller if(!count($_POST)) eller if(!sizeof($_POST)) eller if(!end($_POST)) eller if(!next($_POST)) eller if(!current($_POST)) eller if(!count(array_keys($_POST)) eller if(array_reduce($_POST) != NULL)

etc.etc.etc. if($_POST) är fortfarande det kortaste alternativet.

Nja, jag skulle inte ta för givet att $_POST alltid är en array, som du gör.

[objx]

Det är ju bara om du själv lyckas skriva över $_POST med din egen kod som den inte är en array.
T.ex:
$_POST = null;

[Jonas]

Ursprungligen postat av ITisGood.se

Förstår inte riktigt poängen med det där exemplet om jag ska vara ärlig.
Du gör ju samma kontroll i funktionen som du lika gärna skulle kunna göra direkt i koden. Det hade ju till och med blivit kortare om du skrev den direkt i koden såhär:

PHP-kod:

<?php
if(isset($_POST['namn'])) {
$value = $_POST['namn'];
} else {
echo 'Inget namn satt';
}

Det där ger exakt samma resultat som din lösning.
Massa specialfunktioner i onödan gör bara koden onödigt komplex.

Eftersom den kontrollen sker på mer än ett ställe (vilket är troligt) så platsar den i en funktion.
All logik som görs på mer än ett ställe skall vara i en funktion för att underlätta.

Ursprungligen postat av emilv

Jonas själv har ett logiskt fel i exemplet.

Eh, var när hur? Koden är dessutom skriven direkt från huvudet, och var tänkt som exempel.

Ursprungligen postat av Althalos

if($_POST) är fortfarande det kortaste alternativet.

Som kommer att producera en NOTICE om $_POST inte är satt.

[emilv]

Ursprungligen postat av Jonas

Eh, var när hur?

Din jämförelse med false ska inverteras. Just nu säger du att om värdet inte är satt så ska du använda det.

Den logik du försöker låta bli att upprepa är rätt meningslös. Det blir ju lika mycket kod att skriva, bara att du skriver post() och skippar arrayens namn istället för isset(), och det är tveksamt om det kommer behöva göras ändringar i framtiden. Jag tycker det blir klarare att bara göra issetjämförelsen öppet istället för att lägga den i en funktion som potentiellt göra många fler saker.

Men samtidigt håller jag inte riktigt med i all kritik mot metoden. Om det fungerar för dig och dina projekt är det antagligen bra kod, och jag kan förstå att vissa inte vill skriva $_POST överallt. Och du har rätt i att vissa andra förslag ger notiser - syndigt värre!

[coredev]

Om jag förstår dig rätt så vill du göra följande:

[pseudokod]
if(är_satt($_POST)) {
//validera under-element till POST:en här samt gå vidare med hanteringen av dessa
}
[/pseudokod]

Detta vill du göra för att helt enkelt slippa gå in och ladda din inlänkade fil om du inte kommer från rätt formulär / om något bara klickat på "submit" / annan mystisk anledning?

Jag tror nog att du bör kolla på att revidera din design istället för att bara hitta en lösning som fungerar. Ditt pattern verkar vara ganska sårbart mot förändringar.

[pelmered]

Ursprungligen postat av Jonas

Eftersom den kontrollen sker på mer än ett ställe (vilket är troligt) så platsar den i en funktion.
All logik som görs på mer än ett ställe skall vara i en funktion för att underlätta.

Ja, absolut där håller jag med.
Men i det exemplet hjälper det dig ju inte alls eftersom du ändå tvingas göra i stort sett samma kontroll i koden ändå . Det gör ju att funktionen inte förenklar och gömmer undan kod vilket är hela anledningen till att man placerar sådan logik som används på flera ställen i en funktion.
Jämför:

Ursprungligen postat av Jonas

PHP-kod:

<?php
if( ($value = post('namn')) === false) {
echo 'Hej ' . $value;
} else {
echo 'Inget namn satt';
}
?>

med

PHP-kod:

<?php
if(isset($_POST['namn'])) {
echo 'Hej ' . $_POST['namn'];
} else {
echo 'Inget namn satt';
}

Din funktion fyller ju ingen funktion då man kan göra exakt samma kontroll direkt i koden och den är dessutom både enklare och kräver ett mindre antal tecken. Det andra exemplet har ju även fördelen att alla vet exakt vad som händer där utan att veta vad funktionen "post" gör.

[pelmered]

Ursprungligen postat av Jake.Nu

Nja, jag skulle inte ta för givet att $_POST alltid är en array, som du gör.

Är $_POST satt av servern är det en array, annars returnerar väl alla de där kontrollerna false, eller åtminstone något som evalueras till fasle i en if-sats. Har inte kolla på alla i huvudet dock så jag kan ha fel.

[Althalos]

Ursprungligen postat av Jonas

Som kommer att producera en NOTICE om $_POST inte är satt.

Testa kör följande på din dator:
error_reporting(-1);
if(!$_POST) echo 'test';

Det genererar i alla fall ingen E_NOTICE i min logg. Dessutom så används överallt i manualen på php.net...

Nja, jag skulle inte ta för givet att $_POST alltid är en array, som du gör.

$_POST är en s.k. "superglobal array" - man hänvisar till den på det sättet i manualen och det står tydligt att den endast definieras i form av en array.

[Jake.Nu]

Ursprungligen postat av Althalos

Testa kör följande på din dator:
error_reporting(-1);
if(!$_POST) echo 'test';

Det genererar i alla fall ingen E_NOTICE i min logg. Dessutom så används överallt i manualen på php.net...



$_POST är en s.k. "superglobal array" - man hänvisar till den på det sättet i manualen och det står tydligt att den endast definieras i form av en array.

Ja den definieras som en array men det finns inget som hindrar att skriva över den med valfritt värde.

[pelmered]

Ursprungligen postat av Jake.Nu

Ja den definieras som en array men det finns inget som hindrar att skriva över den med valfritt värde.

Finns ju ingen anledning att skiva över den. Om man skriver kod där det kan hända har man nog gjort något väldigt fel och då bör man nog tänka om lite i och för sig.