WN - Kolla om någon $

WN (https://www.wn.se/forum/index.php)

- Serversidans teknologier (https://www.wn.se/forum/forumdisplay.php?f=4)

- - Kolla om någon $_POST isset (https://www.wn.se/forum/showthread.php?t=1047931)

Kolla om någon $_POST isset

Går det på något sätt att kolla om någon $_POST isset med php?

Då menar jag inte att kolla en specifik $_POST som t.ex. if (isset($_POST['submit'])) utan kolla om det finns någon $_POST över huvudtaget som är isset.

Hittade svaret på ett annat forum, så tänkte dela med mig.

if ($_POST) kollar ifall någon $_POST över huvudtaget har skickats med sidan. Detta är skitbra då jag har samlat alla mina händelser som ska triggas i fall något formulär har postats till sidan i en inlänkad php fil.

Oh happy day!

Annars är det mer korrekt med:

PHP-kod:


		
			
<?php
function post($value) {
   if(isset($_POST[$value])) {
      return $_POST[$value];
   } else {
      return false;
   }
}
?>

Man bör göra liknande funktioner med $_GET, $_COOKIE, $_SERVER & $_SESSION.

fördelen är om man får bort repetetiv kod från sidorna.
Tex.

PHP-kod:


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

Citat:

Ursprungligen postat av Jonas (Inlägg 20403980)

Annars är det mer korrekt med:

PHP-kod:


		
			
<?php

function post($value) {

   if(isset($_POST[$value])) {

      return $_POST[$value];

   } else {

      return false;

   }

}

?>

Man bör göra liknande funktioner med $_GET, $_COOKIE, $_SERVER & $_SESSION.

fördelen är om man får bort repetetiv kod från sidorna.
Tex.

PHP-kod:


		
			
<?php

if( ($value = post('namn')) === false) {

echo 'Hej ' . $value;

} else {

echo 'Inget namn satt';

}

?>

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.

Citat:

Ursprungligen postat av ITisGood.se (Inlägg 20403981)

Massa specialfunktioner i onödan gör bara koden onödigt komplex.

Jonas själv har ett logiskt fel i exemplet.

Tack för tipset robincox! Brukar ganska ofta göra något när formulär postats, så nu slipper jag hålla koll på variabelnamn.

Anledningen till att jag vill ha if ($_POST) och inte if ($_POST['something']) är att jag har en php sida som jag includar om någon $_POST finns över huvud taget. Finns det någon $_POST så kommer massor av if (isset($_POST['differentValues'])) att kollas. Jag sparar en massa laddningstid genom att hoppa över alla $_POST kollningar om det inte finns någon $_POST över huvud taget.

Citat:

Ursprungligen postat av robincox (Inlägg 20404004)

Det verkligt rätta sättet är att enkapsulera $_POST i ett objekt och sedan ha en metod, så skriver man typ som såhär:
if($obj->isset()) include('file.php');

Och om man vill hämta ett värde så skriver man
$obj->post('key');

Motsvarande metoder ska också finnas för $_GET etc. Kolla hur ramverk såsom Codeigniter har implementerat detta.

Men vill man inte programmera objektorienterat så skulle jag föredra Jonas lösning under alla omständigheter. Jag vet inte hur din kod ser ut men jag antar att du ändå kollar vilka specifika värden som är tillagda, innan du försöker göra något med dessa. Så du borde väl egentligen göra det typ såhär:
if($obj->validate()) include('file.php');

validate antas här vara en funktion som validerar din formdata och laddar in filen bara om formulärdatan går att använda till någonting. Det vore en bra lösning. Kör man inte objektorienterat så skulle jag ha gjort likadant d.v.s. validerat först och sedan baserat på valideringen inkluderat filen. Inte kollat om någon data finns, ladda in filen och sedan låta den göra valideringen. Men jag hade inte programmerat på det här sättet överhuvudtaget så vad vet jag...

En sista sak bara. Det är inte den här typen av optimeringar som sparar "massa med laddningstid" så Don't Repeat Yourself-principen och god struktur trumfar att minska antalet operationer, i det här sammanhanget, varje gång. Enligt min åsikt.

Varför inte bara kolla om _POST är tom?

PHP-kod:


		
			
if (!empty($_POST)) {
    echo "Något finns i _POST";
}

Citat:

Ursprungligen postat av Jake.Nu (Inlägg 20404088)

Varför inte bara kolla om _POST är tom?

PHP-kod:


		
			
if (!empty($_POST)) {

    echo "Något finns i _POST";

}

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.