ต่อไป »

[ninwatza]

ขอถาม php หน่อยครับผมงง 

เมื่อผมใส่ isset เข้าไปเพื่อที่จะไม่ให้เกิด notice



มันก็ฟ้องว่า undefined ครับ



แต่ถ้าผมไม่ใส่ isset มันก็รันได้ปรกติครับ แต่มันจะขึ้นโชว์ notice ก่อน
ถ้าผมไปปิด notices ใน php ได้ไหมครับ จะมีปัญหาอะไรรึเปล่าครับ 

[sbaydee]

ควรเอา function calculator ไปไว้ข้างนอกหรือบนๆก็ได้ครับ

[smapan]

ไม่เกี่ยวกับ isset ครับ ที่จริงมันก็เกี่ยวเอาเป้นว่า มันอยู่ที่ความตั้งใจนั่นล่ะครับ
มันซับซ้อนครับถ้าคิดให้สมเหตูตามที่เขียนเอาไว้
...แต่ผมเชื่อว่าคุณคงไม่ได้คิดซับซ้อนขนาดนั้น

เอาเป้นว่าวางลำดับของ ฟังชั่นผิดที่ละกันครับ

[esanza.com]

ควรเอา function calculator ไปไว้ข้างนอกหรือบนๆก็ได้ครับ

ตามท่านนี้ครับ 

[maridb204]

ลองเปลี่ยนเป็น ตัวแปล perprice  แทนครับ   ผ่าน 100 %

[iC1assicTh]

ผมงงว่าเขียนให้มันซับซ้อนทำไมหว่า - -* รึทั่วไปเข้าเขียนแบบนี้กันนะ ไม่ค่อยได้ดูวิธีคนอื่นเขียน แฮ่ะๆ 

[ninwatza]

ขอบคุณครับ พอดีผมทำตาม หนังสือ

แต่ในหนังสือมันไม่มี isset ผมทำตามแล้วมันก็ขึ้น notice ผมก็เลยเอา isset เข้าไปใส่ก็เลยเป็นแบบนั้นครับฮ่าๆ

[max30012540]

ผมแนะนำให้แยกไฟล์ออกมา เป็นไฟรวมเป็นฟังก์ชั่นเลยดีกว่าครับ ถ้าจะใช้ค่อย include เข้ามา จะทำให้เรียกใช้งานได้หลายครั้งโดยแก้ไขครั้งเดียว

[ossytong]

ผมแนะนำให้เข้าไปที่ Control Panel -> Uninstall -> เลือก NetBeans แล้วกด UnInstall ออกครับ

 

จริงๆให้เปลี่ยนจาก isset  เป็น !empty แทนครับ

การใช้ isset จะทำให้เกิด Notice ขึ้นมาครับ

[xvlnw.com]

ผมแนะนำให้เข้าไปที่ Control Panel -> Uninstall -> เลือก NetBeans แล้วกด UnInstall ออกครับ

 

จริงๆให้เปลี่ยนจาก isset  เป็น !empty แทนครับ

การใช้ isset จะทำให้เกิด Notice ขึ้นมาครับ

เข้าใจอะไรระหว่าง isset กับ empty ผิดไปหรือเปล่าครับ

โค๊ด:

<?php
$data = '';
if(empty($data)) echo '$data IS NULL<br>';
if(isset($data)) echo '$data is create, This not Error Notic<br>';
?>

[ossytong]

ผมแนะนำให้เข้าไปที่ Control Panel -> Uninstall -> เลือก NetBeans แล้วกด UnInstall ออกครับ

 

จริงๆให้เปลี่ยนจาก isset  เป็น !empty แทนครับ

การใช้ isset จะทำให้เกิด Notice ขึ้นมาครับ

เข้าใจอะไรระหว่าง isset กับ empty ผิดไปหรือเปล่าครับ

โค๊ด:

<?php
$data = '';
if(empty($data)) echo '$data IS NULL<br>';
if(isset($data)) echo '$data is create, This not Error Notic<br>';
?>

ผมเข้าใจดีครับ เรื่องนี้

การทำงานของ empty ภายในจะเป็น (!isset ... || ... == "")
เราสามารถใช้ !empty แทน isset ได้อยู่แล้วครับ

เพราะเวลาใช้  isset เฉยๆกับตัวแปร array มันจะฟ้อง notice ขึ้นมาครับ ว่าไม่เจอ index ของ array นั้นก่อน ที่จะมีการ pass &value เข้า function
ประสบการณ์เลยสอนผมว่า ทางเลี่ยงคือใช้  !empty กับกรณีแบบนี้แทน จะทำให้ ไม่เกิด notice ขึ้นมา

เพรายังไง การส่งค่าจาก form $_POST['submit'] มันจะต้องมีค่าในฟอร์มอยู่แล้ว (บรรทัดที่ 22)
จุดประสงค์ของ code ที่ จขกท. เขียนก็เพื่อเช็คว่า มีการ POST ใช่มั้ย ??

เลยเขียนแบบนั้น ดังนั้นการใช้ !empty ก็ตอบโจทย์จุดประสงค์การทำงานได้เช่นเดียวกันครับ

ส่วนเพิ่มเติมสำหรับ จขกท. function Cal... ที่อยู่ใน if นั้น ให้เอาออกมาไว้ข้างนอกได้เลยครับ ไม่จำเป็นต้องอยู่ในนั้น จะสวยกว่า แนะนำทำเป็น functions.php แล้วเอา function ไปใส่ไว้ในนั้นแล้ว require มาใช้งาน ตั้งแต่หัวไฟล์ หรือ ในไฟล์ config ได้เลย

จะได้ถูกตาม Concept DRY + SRP ^^

[xvlnw.com]

^
^
^

สิ่งที่ผมต้องการจะสื่อคือ มันใช้งานคนละอย่างกันครับ
isset = สร้างหรือยัง
empty = มีค่าหรือเปล่า ไม่ได้สนใจว่าจะสร้างหรือไม่สร้าง เพราะว่ามันก็ต้องสร้างอยู่แล้ว ถึงจะมีค่า

ในการใช้งานจริงๆ แน่นอนว่ามันก็ใช้งานแทนกันได้ในบางลักษณะ แต่สิ่งที่ผมอยากจะแนะนำก็คือ !empty vs isset ผมแนะนำ isset ครับ เพราะว่ามัน processing เพียงแค่ครั้งเดียว
แต่หากต้องการจะเช็คด้วย ว่าค่ามันว่างหรือเปล่า การเขียน if(isset($data) and $data != '') แน่นอนว่ามันก็ไม่ได้ต่างอะไรกับ !empty แต่จากโค๊ดที่ จขกท. เขียนมา ก็เพียงแค่เช็คว่ามีการ POST หรือเปล่าแค่นั้นเองครับ

ส่วนประเด็นที่สำคัญของ Notic มันคือ Error ที่บอกถึง "การไม่ได้สร้างตัวแปรนั้นๆ" ถ้าจะกล่าวถึง Error Notic ก็ต้องใช้ isset ดีที่สุดครับ เพราะเช็คว่าสร้างหรือยัง?

[ninwatza]

อ๋อครับผมลงใช้ !empty มันก็ใช้ได้น่ะครับ : )

โอเคครับ ผมใช้ isset ต่อไป  

แล้วผมจะต้องใช้ !empty หรือ isset ในโอกาสไหนดีครับ  

[Synchronize]

empty มันจะตรวจสอบได้ลึกกว่าอยู่หน่อยนึง แค่นั้น น่ะครับ

ก็เอาไว้ใช้ในโอกาศที่ ต้องการ หน่อยนึงที่ว่า  เช่น

$a = array();

มันจะได้ผล แบบนี้

isset($a) = true
!empty($a) = false

และพวก string "" ว่างๆ หรือ ค่า 0 ถ้าเข้า empty ก็เป็น true หมด เหมือนกัน เพราะมันถือเป็นค่าว่าง

แต่ปกติ ผมจะตรวจสอบว่า ตัวแปล ที่จะตรวจมีค่า 0 รึป่าว  ผมก็ $a == 0 ไปเลย ตรงๆ นะ

แต่ถ้าเราใช้ empty ตรวจ  ถึงแม้ ตัวแปลนั้น ยังไม่ได้ถูกสร้างขึ้นมา มันก็ จะไม่ warn

(ซึ่ง ผมยังไม่เคยเจอกรณี ที่ต้องตรวจแบบนี้)

 

[ossytong]

อ๋อครับผมลงใช้ !empty มันก็ใช้ได้น่ะครับ : )

โอเคครับ ผมใช้ isset ต่อไป   

แล้วผมจะต้องใช้ !empty หรือ isset ในโอกาสไหนดีครับ 

อยู่ที่ประสบการณ์ของแต่ละคนครับ เจอเคสมามากแค่ไหน เห็นปัญหาจากแต่ละเคสมาแค่ไหน
ในมิติตื้นๆ การท่องจำ ตามที่เค้าเขียนๆมากัน มันก็อาจจะเห็นปัญหาของการใช้งานไม่ครบถ้วน...
บางเหตุการณ์ มันมี ตัวแปรอื่นๆที่จะบอกเราจะสอนเราว่าเราควรจะใช้อะไร...  ในสถานการร์แบบไหน

แต่เพื้นฐานราก็จะต้องรู้ก่อนว่า แต่ละอันเบื้องหลังมันทำงานยังไง ....


โดยวิธีที่ จขกท. เขียน ผมก็ใช้แนวนี้เป็นตัวดักการ เช็ค $_POST เวลาผม HardCode แล้ว โยนค่าเข้าไฟล์ตัวเองผมก็จะใช้วิธินี้ครับ
แต่ปัญหาคือ เมื่อ up ขึ้น Production บางเคสเจอประเด็นที่ว่าการใช้ isset กับตัวแปล $_POST ในลักษณะการกำหนด key(แบบของ จขกท.) จะส่งผลให้เกิด Notice warning ขึ้นมา
ดังนั้นการใช้ !empty จึงสามารถป้องกัน  case นี้ได้

ประสบการณ์ตรงนี้เลยบอกว่าผม ในเมื่อมันใช้ได้ เหมือนกัน แต่ไม่ส่งผลอะไรกับความเร็วของ php เลย การใช้ตัวที่ Safety ที่สุดในจุดประสงค์ของ code เราที่สุดก็น่าจะโอเครกว่าการใช้ isset แล้วต้องมาระแวงว่า Production Server นี้มันจะเกิด Notice มั้ยว้าาา... (มีผลมาจาก php.ini รวมถึง version ของ php การปิด error และ การ control ไฟล์นี้ หลายๆที่ Production Server เราเองไม่มีสิทจะเข้าไปกำกับตรงนี้ได้ทั้งหมด)

ยังไงการใช้ isset หรือ !empty ไม่ส่งผลอะไรกับความเร็วที่ควรจะสนใจเลยครับ มันเล็กน้อย หยุมหยิมมากๆ มีเรื่องอื่นที่ใหญ่กว่านี้ ควรจะกังวลกว่าอีกเยอะครับ 

แต่การใช้ววิธีแบบนี้ผมก็ใช้สำหรับการเขียนไฟล์เดียวสั้นๆ เวลา HardCode ไรงี้มันง่ายและเร็ว ผมเลยชอบการใช้วิธีแบบนี้ ^^

แต่เดี๋ยวนี้ ถ้า app เล็กๆ ไม่กี่หน้าผมก็เลือกใช้ slim framework + eloquent ORM เอา
App ใหญ่ๆก็ใช้ Laravel

ก็จะไม่เจอ case แนวนี้และ เพราะใช้ ความสามารถของ famework แต่ละตัวไป
แล้วจะ get มากขึ้นว่ามันมีเรื่องอื่นๆให้ กังวลกว่าเรื่อง isset , !empty เยอะครับ ^^

ป.ล. เรื่อง isset , !empty มันไม่ได้คนละอย่างกันแบบสุดขั้วถึงขนาดบอกว่า แบบนั้นแบบนี้ได้หรอกครับ ทำความเข้าใจ เบื้องหลังการทำงานแล้วเลือก apply ให้เข้ากับจุดประสงค์ อัลกอริธึม เราเหมาะสมที่สุดครับ ไม่ต้องซีเรียสประเด็นอะไรพวกนี้มาก มีอีกหลายๆเรื่องที่ควรกังวลมากกว่าเยอะ

[xvlnw.com]

^
^
^
ผมเข้าใจกับสิ่งที่ท่านพยายามอธิบายนะครับ เพราะมันก็เป็นส่วนเล็กๆจริงๆอย่างที่ว่านั่นแหละ

แต่ที่ผมอยากถามคือ Notice Warning เมื่อใช้กับ isset ผมขอตัวอย่างโค๊ดมาศึกษา จะขอบคุณมากครับ

จากประสบการณ์ของผมคือ ไม่เคยเจอเมื่อใช้ isset แล้วจะมี Notice Warning ครับ

 

[MapTwoZa]

ทั้ง empty และ isset มันทำให้ไม่แสดง notice ทั้ง 2 แบบแหละครับ

ประเด็นคือ ควรใช้อย่างไหนมากกว่ากัน

ถ้าถามผม ผมตอบว่า empty ครับ

เพราะอะไร เพราะ empty ใช้ได้กับทั้ง string แล้วก็ตัวแปลอื่นๆด้วยครับ
"" (an empty string)
0 (0 as an integer)
"0" (0 as a string)
NULL
FALSE
array() (an empty array)
var $var; (a variable declared, but without a value in a class)


การที่มานั่งไล่ code แล้วเจอ if(isset($data) and $data != '')  หรือ if(isset($data) and count(data) > 0 )  หรือเคสอื่นๆอีก ตามประเภทด้านบน
เทียบกับเจอ if ( empty($data) )
ผมเลือกอย่างหลัง

เหตุผลประกอบมี 2 เหตุผลครับ
1. code convention
2. ทำแบบนี้ อาจจะมี bug แอบแฝงมาได้ครับ

[xvlnw.com]

^
^
^

ในลักษณะเช็คค่าของตัวแปรด้วย ผมก็จะเลือกใช้ empty ครับ

แต่ถ้าไม่เช็คค่ามัน ก็ isset ครับ (ตั้งแต่ผมพิมพ์มาตั้งแต่แรก ผมบอกอยู่แล้วว่าเอาไว้เช็คว่าตัวแปรมันถูกสร้างหรือยัง? แค่นั้นครับ)

แน่นอนว่าผมก็เลือกใช้ แต่ก็ยังมีข้อสงสัยจากคุณ ossytong บอกว่า isset จะทำให้เกิด Error Notice ในบางกรณี ซึ่งผมไม่เคยเจอจริงๆนะครับ  ><

[smapan]

อ๋อครับผมลงใช้ !empty มันก็ใช้ได้น่ะครับ : )

โอเคครับ ผมใช้ isset ต่อไป  

แล้วผมจะต้องใช้ !empty หรือ isset ในโอกาสไหนดีครับ  

น่าจะใช่ที่ถาม
http://stackoverflow.com/quest...why-check-both-isset-and-empty

[ossytong]

อ๋อครับผมลงใช้ !empty มันก็ใช้ได้น่ะครับ : )

โอเคครับ ผมใช้ isset ต่อไป   

แล้วผมจะต้องใช้ !empty หรือ isset ในโอกาสไหนดีครับ 

น่าจะใช่ที่ถาม
http://stackoverflow.com/quest...why-check-both-isset-and-empty

อธิบายชัดเจนดีครับ ผมมองเหมือนกับ User : diEcho ที่ตอบในความเห็นสุดท้ายครับ สำหรับ case ของ จขกท. คือ case นี้เลยครับ (โยนเข้า page แล้ว submit มีค่าแน่นอน)

ป.ล. ผมยืนยันว่า มันมี case ที่ ใช้ isset กับ $_POST แบบถูกกำหนด index มาแล้วส่งผลให้เกิด Notice ได้ครับ แต่จำไม่ได้ว่าเกิดใน php version ไหน แล้วอยู่ในรูปแบบ env อะไรบ้าง นานมากแล้วสำหรับการเขียนแบบ HardCode แบบนี้ ด้วยความที่ Production มาหลาย env ทั้ง Ubuntu, CentOS, IIS ver, other  เก่าๆบ้างใหม่ๆบ้าง  ติดตั้งทั้ง เอกชน - ราชการ มาหลายองกรค์ เคยปรากฏเคสนี้ขึ้นจน เป็นเหตุให้ผม เปลี่ยจาก isset มาเป็น !empty สำหรับกรณี code ประมาณนี้

 ไม่แน่ใจว่าเป็นที่ bug ของ php version นั้นๆ หรือเป็นที่ env หรือเป็นที่ php.ini แต่ยังไงซะ สำหรับผมผม make sure ที่สุดคือการเลือกใช้ !empty ก็เป็นการ make sure สำหรับสถานการณ์ code แบบนี้ได้มากกว่าอยู่ดี 

กรณีนี้ จะใช้ isset หรือ จะใช้ !empty มันก้ไม่มีอะไรผิดอยู่ดี ก็ทำงานได้ทั้ง 2 แบบ
ถ้าเข้าใจการทำงานของมัน ก็ไม่เห็นว่าจะมีประเด็นอะไร... ที่จะต้องพูดให้มากมาย
(ถ้าเรื่องแค่นี้ยังยาวขนาดนี้ สงสัย Repository Concept, Facade, PSR-0,4 การเลือกใช้ OOP Pattern จะขนาดไหน ^^)
มันก็แค่ function นึงที่มาตอบสนองอัลกอ หรือเจตนาของการใช้ code  อยู่ดี

จะเลือกใช้อะไรก็ จะ isset, !empty มันก็คือเครื่องๆมือที่มาตอบสนองอัลกอของเราทั้งนั้น ก็ตามถนัด
ตาม ประสบการณ์ของที่เจอมา ถ้าเข้าเข้าใจ การทำงานของมัน ก็ไม่มีประเด็นอะไรหรอก
มันตอบโจทย์ อัลกอได้ สามารถเขียน Test Case ได้ ก็ เวิคทั้งนั้น วันนึงเจอปัญหา เจออะไรมากขึ้น เลือก เครื่องมือมาใช้ มาแทนที่ได้ ก็ โอเคร

[ninwatza]

ขอบคุณทุกท่านมากครับ

ผมรู้สึกปลื้ม ที่มี เทพphp หลายๆท่านมาร่วมตอบข้อสงสัย

เอาเป็นว่าผมใช้ !empty ในโอกาสที่ผมต้องการเช็คว่า ก่อนจะ $_POST ไปกดปุ่ม submit มีค่าแล้วหรือยังถ้ายังไม่มีก็ไม่ต้องทำ แต่ถ้ามีก็ทำ

ถ้า isset ไว้ตรวจสอบว่ามันสร้างมาแล้วหรือยังตัวแปรนี้มีค่าหรือไม่มีค่าก็เท่ากันแค่อยากรู้ว่ามันสร้างมาแล้วหรือยัง
ถ้าสร้างแล้วก็ทำ ถ้ายังไม่สร้างก็ไม่ต้องทำ



ฉะนั้นส่วนใหญ่จึงเลือกใช้ !empty มากกว่า เพราะมันตรวจสอบได้ลึกกว่า

ผมเข้าใจถูกไหมครับ