הוסטס - פורום אחסון האתרים הגדול בישראל - שאלה

- פורום תיכנות (https://hosts.co.il/forums/forumdisplay.php?f=14)

- - שאלה | שאילתא (https://hosts.co.il/forums/showthread.php?t=93405)

שאלה | שאילתא

HTML קוד:

SELECT DISTINCT COUNT(ID) AS number, PageCode

FROM entries

WHERE PageType = 2

ORDER BY number DESC

למה זה מציג לי רק שורה אחת ?
אני רוצה להשיג את 10 הדפים הכי נצפים מסוג 2 (PageType)
השאילתא מחזירה לי רק שורה אחת וגם היא לא הדף הכי נצפה

מה המטרה של ה- DISTINCT וה- COUNT כאן?

על פי השאילתה אני מניח שכל שורה מציינת דף ומס' הצפיות של אותו דף שמור בשדה number,
אז תשלוף את השדות שאתה צריך, תסנן שורות עם WHERE ותעשה ORDER BY number DESC בדיוק כמו שעשית - זה מספיק.

לא הבנתי בכלל מה עשית בקוד, הקוד צריך להיות משהו כמו

PHP קוד:


		
			
SELECT * FROM entries WHERE PageType = 2 ORDER BY number DESC LIMIT 0,10

המטרה שלי היא להגיע לדפים הכי נצפים מסוג דף 2 (pagetype) , ה-count נועד לספור את מספר ה-ID של הכניסות לדף מספר X
הטבלה בנויה כך :
ID
IP
TIME
PageType
PageCode

אני צריך לשלוף את הקוד דף של 10 דפים מסוג 2 שיש להם הכי הרבה צפיות (יותר שורות בטבלה)

השאילתא שרשמתי כאן דיי מאולתרת כי ניסיתי המון ניסוחים שונים ולא הצלחתי להגיע לתוצאה שאני צריך

אני אנחש ששדה ה-ID זה מזהה ייחודי של העמוד.
אני לא רואה באיזה שדה אתה שומר את כמות הצפיות שלך.

הטבלה עצמה היא כמות הצפיות

שיטה די עקומה לספור צפיות לפי דעתי.. אבל זה צריך להיות ככה:

קוד:

SELECT COUNT(*) AS number, PageCode FROM entries WHERE PageType=2 GROUP BY ID ORDER BY number DESC

ציטוט:

נכתב במקור על ידי Itay (פרסם 824919)

שיטה די עקומה לספור צפיות לפי דעתי.. אבל זה צריך להיות ככה:

קוד:

SELECT COUNT(*) AS number, PageCode FROM entries WHERE PageType=2 GROUP BY ID ORDER BY number DESC

מוזמן להציע דרך משלך אני פתוח לרעיונות שונים

ציטוט:

נכתב במקור על ידי Somdir (פרסם 824928)

a

מוזמן להציע דרך משלך אני פתוח לרעיונות שונים

נראה לי שאיתי התכוון לכך (ואני מסכים איתו) שזה בזבזני על כל צפיה להכניס רשומה חדשה ל-DB, ואח"כ לכמת אותן.
לחילופין היה כדאי פשוט להחזיק מונה שאתה מוסיף לו +1 כל צפיה.

מה שכן, צורת עבודה כזאת הכרחית במצבים בהם אתה רוצה לאגור גם מידע על הצפיה עצמה (כמו מה שאתה עושה: IP, תאריך ושעה וכו').

ציטוט:

נכתב במקור על ידי IgalSt (פרסם 824936)

לא רק לאגור מידע שכזה אלא גם לסנן כניסות זהות מאותו IP

והשאילתא של איתי לא עבדה כמו שרציתי

משהו כזה לדעתי:

קוד:

Select *, count(*) as viewsCount

From entries e

Where e.PageType=2

Group By e.ID

Order By viewsCount DESC

Limit 10

שזה מאוד דומה למה שאיתי הציע.

מה הבעיה עם השאילתה שלו/שלי?

ציטוט:

נכתב במקור על ידי IgalSt (פרסם 824936)

ציטוט:

נכתב במקור על ידי Somdir (פרסם 824949)

לא רק לאגור מידע שכזה אלא גם לסנן כניסות זהות מאותו IP

והשאילתא של איתי לא עבדה כמו שרציתי

גם כדי לדעת את כל הפרמטרים האלה לא חייב לעשות לכל אחד רשומה חדשה.
פשוט בכל דף תתן גם עמודה "IP" שתפריד כל IP בסימן "|" לדוגמא הטבלה תראה כך:
ID - מזהה העמוד (לדוגמא - 3)
VIEWS - מספר צפיות בעמוד (לדוגמא - 2)
IP - הIPים שצפו כבר בעמוד (לדוגמא - "123.256.23.31|88.62.642.12)
pageType - מה שאמרת, "סוג הדף" (נניח שכמו שרצית הסוג הוא 2)

עכשיו פשוט בכל פעם לפני שאתה מוסיף לעמוד +1 בכמות הצפיות (VIEWS) אתה בודק אם הIP של מי שצופה בדף שווה לאחד מהIPים שנשמרו בעמודה "IP".
אתה יכול לבצע את הכנסת הIP והבדיקה אם הוא כבר קיים ע"י EXPLODE פשוט של העמודה, הנה קוד דוגמא (נניח שהערכים שהבאתי לעיל הם ערכי הטבלה):

PHP קוד:


		
			
$ip = $_SERVER['REMOTE_ADDR'];

$pageID = $_GET['id'];

$upView = '1';

$query = mysql_query("SELECT * FROM table WHERE id='$pageID' ")

$row = mysql_fetch_array($query);

$views = $row['VIEWS']+1;

$exp = explode("|", $row['IP']) //מפרקים את הערך שנמצא בעמודה IP בין הסימנים "|" (מה שמפריד את רשימות הIP

for($i='0';$i<count($exp);$i++) { //נתחיל לולאת FOR כדי לבדוק את כל רשימות הIP שקיימים בעמודה

    if($i=='0') //אם לולאה רצה פעם ראשונה...

        $updateIP = $exp[$i]; //נקבע משתנה חדש שמעתה יכיל את כל הIP של הטבלה על מנת לסדר IP מחדש במידה והIP הצופה עכשיו הוא ייחודי (ההכנס לטבלה לאחר הסידור מחדש תעשה רק אם הIP ייחודי)

    else

        $updateIP .= "|".$exp[$i]; //יתחיל להפריד IP הסימן | בין IP לIP

    if($exp[$i]==$ip) //אם הוא מוצא IP בעמוד שכבר שווה לIP של מי שצופה אז הוא שנה את המשתנה ל0

        $upView='0';

}

if($upView=="1") { //בודק אם המשתנה שווה ל1, אם כן, כלומר אם לא בוצע שינוי אז הIP לא קיים כבר בעמוד ולכן זו צפייה ייחודית ואז...

    $updateIP .= "|".$ip; //אם הIP שצופה הוא ייחודי ולא קיים בטבלה נשרשר אותו למשתנה שיעדכן את הIP הייחודים ונכניס לטבלה את הכל...

    mysql_query("UPDATE table SET VIEWS='$views', IP='$updateIP' WHERE id='$pageID'"); //פשוט, לאחר הבדיקה שביצענו אם הIP שצופה הוא ייחודי ואינו נמצא בטבלה, אם אמת נכניס מחדש את רשומות הIP וגם את מספר הצפיות בעמוד

}

בקשה, זה הקוד שבעצם יבצע לך את הבדיקה אם הIP שצופה בעמוד כבר צפה בעמוד בעבר או לא, במידה והוא ייחודי ואינו צפה בעמוד עדיין כמות הצפיות בדף תגדל ב1 ורשומות הIP יעדוכנו ויכללו מעתה את המזהה של הצופה העכשווי.
כמובן שיש עוד דרכים לכתוב אותו, אולי אפילו בדרכים יעילות יותר אבל אני לא מאמין שזה משמעותי כל כך..

עכשיו את הקוד שביקשת בהתחלה הרבה יותר פשוט לכתוב, כמובן שחוץ מזה שיותר פשוט לכתוב מעכשיו לא תכנס לך כל פעם רשומה חדשה למסד עם כל צפייה ייחודית...
אני די בטוח שאת הקוד שביקשת בהתחלה לאחר השינוי אתה תדע לכתוב אבל בכל זאת, שנעשה עבודה מלאה (:

PHP קוד:


		
			
$query = mysql_query("SELECT * FROM table WHERE pageType='2' ORDER BY VIEWS DESC LIMIT 0,10");

while($row=mysql_fetch_array($query)){

    print 'יציג לך את 10 העמודים הכי נצפים מסוג pageType ששווים ל2';

}

מקווה שעזרתי

ציטוט:

נכתב במקור על ידי City-Games.co.il (פרסם 824974)

PHP קוד:


		
			
$ip = $_SERVER['REMOTE_ADDR'];

$pageID = $_GET['id'];

$upView = '1';

$query = mysql_query("SELECT * FROM table WHERE id='$pageID' ")

$row = mysql_fetch_array($query);

$views = $row['VIEWS']+1;

$exp = explode("|", $row['IP']) //מפרקים את הערך שנמצא בעמודה IP בין הסימנים "|" (מה שמפריד את רשימות הIP

for($i='0';$i<count($exp);$i++) { //נתחיל לולאת FOR כדי לבדוק את כל רשימות הIP שקיימים בעמודה

    if($i=='0') //אם לולאה רצה פעם ראשונה...

        $updateIP = $exp[$i]; //נקבע משתנה חדש שמעתה יכיל את כל הIP של הטבלה על מנת לסדר IP מחדש במידה והIP הצופה עכשיו הוא ייחודי (ההכנס לטבלה לאחר הסידור מחדש תעשה רק אם הIP ייחודי)

    else

        $updateIP .= "|".$exp[$i]; //יתחיל להפריד IP הסימן | בין IP לIP

    if($exp[$i]==$ip) //אם הוא מוצא IP בעמוד שכבר שווה לIP של מי שצופה אז הוא שנה את המשתנה ל0

        $upView='0';

}

if($upView=="1") { //בודק אם המשתנה שווה ל1, אם כן, כלומר אם לא בוצע שינוי אז הIP לא קיים כבר בעמוד ולכן זו צפייה ייחודית ואז...

    $updateIP .= "|".$ip; //אם הIP שצופה הוא ייחודי ולא קיים בטבלה נשרשר אותו למשתנה שיעדכן את הIP הייחודים ונכניס לטבלה את הכל...

    mysql_query("UPDATE table SET VIEWS='$views', IP='$updateIP' WHERE id='$pageID'"); //פשוט, לאחר הבדיקה שביצענו אם הIP שצופה הוא ייחודי ואינו נמצא בטבלה, אם אמת נכניס מחדש את רשומות הIP וגם את מספר הצפיות בעמוד

}

PHP קוד:


		
			
$query = mysql_query("SELECT * FROM table WHERE pageType='2' ORDER BY VIEWS DESC LIMIT 0,10");

while($row=mysql_fetch_array($query)){

    print 'יציג לך את 10 העמודים הכי נצפים מסוג pageType ששווים ל2';

}

מקווה שעזרתי

חייב לציין שהרעיון שלך חכם במיוחד , אולי אפשר לייעל יותר אבל בסיס הרעיון הוא טוב .
אני חושב ליישם זאת תודה על העזרה אפשר לנעול אני אסתדר מכאן

ציטוט:

נכתב במקור על ידי City-Games.co.il (פרסם 824974)

PHP קוד:


		
			
$ip = $_SERVER['REMOTE_ADDR'];

$pageID = $_GET['id'];

$upView = '1';

$query = mysql_query("SELECT * FROM table WHERE id='$pageID' ")

$row = mysql_fetch_array($query);

$views = $row['VIEWS']+1;

$exp = explode("|", $row['IP']) //מפרקים את הערך שנמצא בעמודה IP בין הסימנים "|" (מה שמפריד את רשימות הIP

for($i='0';$i<count($exp);$i++) { //נתחיל לולאת FOR כדי לבדוק את כל רשימות הIP שקיימים בעמודה

    if($i=='0') //אם לולאה רצה פעם ראשונה...

        $updateIP = $exp[$i]; //נקבע משתנה חדש שמעתה יכיל את כל הIP של הטבלה על מנת לסדר IP מחדש במידה והIP הצופה עכשיו הוא ייחודי (ההכנס לטבלה לאחר הסידור מחדש תעשה רק אם הIP ייחודי)

    else

        $updateIP .= "|".$exp[$i]; //יתחיל להפריד IP הסימן | בין IP לIP

    if($exp[$i]==$ip) //אם הוא מוצא IP בעמוד שכבר שווה לIP של מי שצופה אז הוא שנה את המשתנה ל0

        $upView='0';

}

if($upView=="1") { //בודק אם המשתנה שווה ל1, אם כן, כלומר אם לא בוצע שינוי אז הIP לא קיים כבר בעמוד ולכן זו צפייה ייחודית ואז...

    $updateIP .= "|".$ip; //אם הIP שצופה הוא ייחודי ולא קיים בטבלה נשרשר אותו למשתנה שיעדכן את הIP הייחודים ונכניס לטבלה את הכל...

    mysql_query("UPDATE table SET VIEWS='$views', IP='$updateIP' WHERE id='$pageID'"); //פשוט, לאחר הבדיקה שביצענו אם הIP שצופה הוא ייחודי ואינו נמצא בטבלה, אם אמת נכניס מחדש את רשומות הIP וגם את מספר הצפיות בעמוד

}

PHP קוד:


		
			
$query = mysql_query("SELECT * FROM table WHERE pageType='2' ORDER BY VIEWS DESC LIMIT 0,10");

while($row=mysql_fetch_array($query)){

    print 'יציג לך את 10 העמודים הכי נצפים מסוג pageType ששווים ל2';

}

מקווה שעזרתי

לדעתי זה פתרון לא טוב.
בצורה כזאת מגיעים למצב שבו מעדכנים שדות מאוד ארוכים של טקסטים הרבה פעמים.
ל-DB יותר קל להוסיף רשומות חדשות מאשר לעדכן שדה טקסט גדול מאוד.

כמו כן לבצע חיפוש על טקסט זו גם פעולה יחסית יקרה.

אני הייתי הולך על רשומה ייחודית לכל אחד. ככה בעתיד גם ניתן לבצע חיתוכים מעניינים במידת הצורך (לדוגמה לפי תאריך או במה צפה כל IP וכו')

ציטוט:

נכתב במקור על ידי Somdir (פרסם 824982)

פתרון טוב יותר יהיה ליצור שדה שיכיל את מס' הצפיות הכולל של הדף בטבלה של הדפים,
וטבלה נוספת למידע מפורט של כל צפייה (אם אתה צריך).

ציטוט:

נכתב במקור על ידי xPerfection (פרסם 824997)

בדיוק

למה שאתה מתאר באמת נראה לי הכי נכון לפעול כמו שעבדת, אבל אין סיבה סתם להפעיל שאילתות שמעמיסות כמו זאת שכתבתי קודם, השאילתה בעצם תעבור (אם מדובר על צפיות באתר חיי) על אלפי רשומות ותבצע סינונים, זה דבר שמעמיס ולוקח זמן
מצד שני, כשאתה מבצע את השאילתה שמכניסה את הערכים האלו, לעדכן שדה נפרד ב +1 זה לא כזה נורא. ככה רק כשאתה צריך את הנתונים האלה ספציפית תשתמש בהם, בשאר הזמן תשתמש בשדה שמתעדכן בפלוס 1 בכל פעם