更好地捕獲會話從PHP中的會員代碼ID

Question

有人可以幫我清理這個，並使其更合乎邏輯嗎？我現在炒，似乎無法得到一個很好的代碼行:)

我試圖從網址捕捉會員ID像？aid = 3056677。這個想法是IF aff id在GET中被設置，優先，會話，最後是cookie。另外，我們不希望設置不存在的aff標識。

你知道更多的嘗試和做法嗎？

session_start(); // start session 

// affiliate id 
$g_aid = (isset($_GET['aid']) && $_GET['aid'] != '') ? trim($_GET['aid']) : false; 
$s_aid = (isset($_SESSION['aid']) && $_SESSION['aid'] != '') ? trim($_SESSION['aid']) : false; 
$c_aid = (isset($_COOKIE['aid']) && $_COOKIE['aid'] != '') ? trim($_COOKIE['aid']) : false; 

if($g_aid !== false) // use get if set 
    $aid = $g_aid; 
elseif($s_aid !== false) // next use session if get not set 
    $aid = $s_aid; 
elseif($c_aid !== false) // cookie 
    $aid = $c_aid; 
else 
    $aid = ''; // leave it empty 

// if $aid is set is it in the $affiliates array? 
//If not use the first key from that array 
$aid = (isset($affiliates[$aid])) ? $aid : key($affiliates); 

// save it and set it 
// (maybe shouldn't be done if already stored? 
setcookie('aid', $aid); 
$_SESSION['aid'] = $aid; 

Answer 1

謝謝你們，這看起來越來越好。有一點可以爲你澄清，如果在GET中給出了一個aff標識，它必須是在我們可能消滅別人的aff標識之前存在的有效標識。每筆交易都涉及金錢，我們希望會員儘可能長時間獲得信用。

關於empty它不是太有用，因爲空白愚弄它。所以除非你在使用之前修剪，否則我覺得它不夠準確。所以我不知道GET的空白。對其他人來說沒問題，因爲我們已經檢查過了。

這裏是我到目前爲止，從你的幫助（在這裏所做的複雜三元打破時，發現真我不希望它繼續執行行？）：

session_start(); // start session 

    $aid = !empty($_GET['aid'])  ? trim($_GET['aid']) : 
     !empty($_SESSION['aid']) ? $_SESSION['aid'] : 
     !empty($_COOKIE['aid']) ? $_COOKIE['aid'] : 
     ''; 

    // use first key from array if aid not set 
    if(!isset($a[$aid])) $aid = key($a); 

    if(!isset($_SESSION['aid']) || $aid != $_SESSION['aid']) 
    { 
    setcookie('aid', $aid); 
    $_SESSION['aid'] = $aid; 
    } 

Answer 2

session_start(); 

// checks if a field is valid 
function isValid($aid) { 
    return (!empty($aid) && trim($aid) != ''); 
} 

// set the affiliate ID 
$aid = isValid($_GET['aid'])  ? $_GET['aid'] : 
     isValid($_SESSION['aid']) ? $_SESSION['aid'] : 
     isValid($_COOKIE['aid']) ? $_COOKIE['aid'] : 
     ''; 

// use first key from array if aid not set 
if (!isset($affiliates[$aid])) $aid = key($a); 

// save and set 
setcookie('aid', $aid); 
$_SESSION['aid'] = $aid; 

Answer 3

1. 爲什麼你會測試會話和餅乾，如果你有從$ _GET數組有效AFFILIATEID？ ==>使其漸進式，以便只檢查會話，如果沒有找到GET並且僅在沒有找到會話時檢查cookie。

2. 不要重複驗證affiliateID。 ==>編寫一個驗證函數並重用它，您可能希望稍後添加更多規則。

3. 使用大括號，使代碼更易讀

4. $援助或$ AFF是壞的變量名，$ AFFILIATEID反而是一個很好的！你不會因爲編寫簡短的變量名稱而贏得任何東西，但是你通過編寫不言自明的代碼贏得很多。

壞例如不說話

if (validate($aff)) 

好例如跟你說話

if (isValid($affiliationID)) 

所以我爲核心的變化提案組件：

if (isValid($_GET['aid'])) 
{ 
    $affiliationID = trim($_GET['aid']; 
} 
else if (isValid($_SESSION['aid'])) 
{ 
    $affiliationID = trim($_SESSION'aid']; 
} 
else if (isValid($_COOKIE['aid'])) 
{ 
    $affiliationID = trim($_COOKIE['aid']; 
} 
else 
{ 
    throw new Exception('No affiliation ID defined'); 
} 

function isValid($affiliationID) 
{ 
    if (empty($affiliationID)) 
    { 
     return false; 
    } 
    else 
    { 
     return true; 
    } 
}