Discussion Board
Welcome Guest! You can login or register. Login or Register.
2 Pages12>

Options
Go to last post Go to first unread
Zero2Cool  
#1 Posted : Friday, October 22, 2010 2:59:22 PM(UTC)
I'm trying to clean up the database.

My goal is to remove the posts from members that have been deleted.

I want a select statement that shows me how many of them there are before deleting.


I think I'll be using three tables with this.

table (field)
[ul]users (user_id)
posts (poster_id)
posts_text (post_id)[/ul]

Every post is given and id number. Then that id number is assigned to the post text.

The relationship between users and posts is the user_id and poster_id.
The relationship between posts and posts_text is the post_id.

I tried this, but it didn't work so well.

[php]SELECT *
FROM posts LEFT JOIN users
ON posts.poster_id != users.user_id[/php]


I think I caused some site load errors with this query too, lol.
dhpackr  
#2 Posted : Friday, October 22, 2010 3:09:19 PM(UTC)
bummer you can't delete posts
dhpackr  
#3 Posted : Friday, October 22, 2010 3:17:12 PM(UTC)
don't you want to use count

SELECT posts, COUNT(*) FROM posts WHERE Posterid=poster_id GROUP BY poster_id;

you have to play around with the table names, and you add quite a bit to a query with 'where' & 'group by' as well as 'orderby'

just running a select or count query should not break the site
Zero2Cool  
#4 Posted : Friday, October 22, 2010 3:22:57 PM(UTC)
I dunno, do I?
dhpackr  
#5 Posted : Friday, October 22, 2010 3:25:30 PM(UTC)
you know the answer, just play around till you get the query.

wanna let me in?

i'll try
otherwise this will help
Nonstopdrivel  
#6 Posted : Friday, October 22, 2010 3:31:11 PM(UTC)
I'm confused. Are you wanting to clean up deleted posts or posts by deleted members? The owner of the other board I used to administrate did the latter routinely; unfortunately, some of the most prolific members (e.g., instructors) were getting deleted. This resulted in some of our best threads being rendered nearly indecipherable, with members responding to phantom statements and questions. Threads came to resemble Swiss cheese, with much of the best content missing, rendering them all but useless. If this is what you want to do, I'd urge caution in implementing it.
Zero2Cool  
#7 Posted : Friday, October 22, 2010 3:36:19 PM(UTC)
627 posts returned.


Every post should have a valid relationship to a member.
Every topic should have a valid relationship to a post.
Every post should have a valid relationship to a topic.

I don't want rogue data out there.
Pack93z  
#8 Posted : Friday, October 22, 2010 3:44:48 PM(UTC)
Yes use the count, cleaner approach for your request in the first post.. as a review.
Pack93z  
#9 Posted : Friday, October 22, 2010 3:45:50 PM(UTC)
Okay.. with the timeout error, more than likely due to query gen, my comment now seems out of place. lol.
Zero2Cool  
#10 Posted : Friday, October 22, 2010 3:52:22 PM(UTC)
627 is what I got back, which is just a list of members who have 1 or more posts.

The member list shows 618 with a post.

That to me means we have posts for 9 members that have been deleted.
Zero2Cool  
#11 Posted : Friday, October 22, 2010 3:56:11 PM(UTC)
In the posts table, I have 13,155 posts.
The software shows 12,989.

I have 166 posts that were deleted, but not removed from the database??

Edit, actually it means I have 13,155 post ID's, but only 12,989 posts with a text.
Zero2Cool  
#12 Posted : Friday, October 22, 2010 4:11:55 PM(UTC)
[php]
SELECT _fpro_posts_text.post_id
FROM _fpro_posts, _fpro_posts_text
WHERE _fpro_posts.poster_id='2'
AND _fpro_posts_text.post_id != _fpro_posts.post_id
[/php]

Shouldn't this display a list of posts_text.post_id's that do not have a corresponding posts.posts_id for the poster_id 2?
Zero2Cool  
#13 Posted : Friday, October 22, 2010 4:33:07 PM(UTC)
This is hopeless. There's a ton of NULL's under id 2, okay, 136 or so, but I can't delete them records!
djcubez  
#14 Posted : Friday, October 22, 2010 9:47:14 PM(UTC)
From what you say this forum software works like other forum software with a POSTS table, a MEMBERS/USERS table and a TOPICS table. Each of them refer to each other by an auto-incremented value (topicID, postID and memberID). If you want to delete all the posts from members that don't exist you'd have to use multiple queries. For example:

Code:

<?php

// establish variables
$memberposts = "0";
$nonmemberposts = "0";

// first query all posts
$post_query ="SELECT * FROM posts";
$post_result = mysql_query($post_query);
while($post_row = mysql_fetch_array($post_result)) {
	// now we can address each post individually
	// we should set some variables we need
	$postID = $post_row["id"];				// The post ID
	$memberID = $post_row["memberid"];		// The member that made this post
	$isMember = true;						// boolean for whether the member exists or not
	// now we need to figure out if the member with that id still exists
	// to do that you need to do a num_rows query
	$member_query = "SELECT * FROM members WHERE id='$memberID'";
	$member_count = mysql_num_rows($member_query);
	// check if it returns a result
	if($member_count < "1") {
		// the member does not exist
		$isMember = false;
	}
	// now we return the result of our script
	if($isMember) {
		// post verified
		$memberposts++;
	} else {
		// delete the post?
		// I wouldn't
		// $delete_query = "DELETE FROM posts WHERE id='$postID'";
		// mysql_query($delete_query);
		$nonmemberposts++;	
	}
}

// display a report at the end
echo "Total posts by members: " . $membersposts . "<br />total posts by non-members: " . $nonmemberposts;

?>


Obviously you'll have to replace the references I make to mysql rows and tables because I don't know whats yours are exactly. But if I do understand what you want this code should work, although I haven't tested it.

EDIT: Please don't run the script with the delete query in it until you've verified that it works. I would not want you to accidentally delete all the posts in this forum because of my bad code. I've already commented the delete query out so that if you want to you can just comment it back in.
Zero2Cool  
#15 Posted : Friday, October 22, 2010 10:48:33 PM(UTC)
Yeah, I don't want to delete them just yet. I want to see them first, that's why I was using the SQL query in the phpAdmin.

I can provide the three tables for you guys if that'll help.
dhpackr  
#16 Posted : Friday, October 22, 2010 10:53:44 PM(UTC)
Code:

<?php

// $delete_query = "DELETE ?????FROM posts WHERE id='$postID'";
		// mysql_query($delete_query);
		$nonmemberposts++;	
?>


wouldn't work if it wasn't commented out, this would cause an error, not telling what you want to delete.

why don't you just delete id #2, the whole row? if you are using auto increment for the id field, it wouldn't let you delete the row unless you delete the id
Zero2Cool  
#17 Posted : Friday, October 22, 2010 10:58:03 PM(UTC)
I've deleted members in the users table, but not their posts. Therefore there's posts out there that cause errors while searching.
dhpackr  
#18 Posted : Friday, October 22, 2010 11:23:58 PM(UTC)
imo, the best way to maintain your site would be to manually go into the database through php admin and delete the rows that way.

a script would be nice for maintenance, but you are risking wiping out data you want to keep.

just wondering, did you join the mysql forum. you should, and post questions in the forum.

you'll get your solution...if you are driven.

BTW..I find it humorous, a .NET Developer is running his forum using PHP, Microsoft's main nemesis.

Isn't that like sayin you work at Ford but drive a Toyota?

:icon_smile: just sayin!
Zero2Cool  
#19 Posted : Saturday, October 23, 2010 12:08:55 AM(UTC)
I was looking into developing my own software using ASP-MVC2 with SQL Server, but it seems like too much work. I'd be reinventing a lot of the options here.

Might as well stick with this and enhance it, instead of recreating everything, right?
dhpackr  
#20 Posted : Saturday, October 23, 2010 2:55:39 AM(UTC)
yes, stick with the PHP site. Classic ASP is an old technology. You have to write a ridiculous amount of "spaghetti" code to do even a simple database call.

Microsoft updated to .NET for a reason!
Rss Feed
Users browsing this topic
Guest
2 Pages12>
Forum Jump
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

Notification

Icon
Error

Fan Shout
Smokey (22-Jul) : After 11 days with no Cable/Internet , I'm pleased to be back !
Porforis (21-Jul) : Misery + Money = Misery
Zero2Cool (21-Jul) : He left SIX kids fatherless. He had resources to get help, but sadly felt he was without hope.
Zero2Cool (21-Jul) : Has not*
Zero2Cool (21-Jul) : Money has and never will be a measure of happiness. Never.
Cheesey (21-Jul) : Another one kills himself. I don't get it. I guess money doesn't solve it all.
Zero2Cool (20-Jul) : Linkin Park Frontman Chester Bennington, 41, Dead
DarkaneRules (20-Jul) : RIP chester bennington :(
Zero2Cool (15-Jul) : Today's Birthdays: Since69 (52)
Cheesey (11-Jul) : Thanks Yoop! Smokey...I am "aged cheese!"LOL!
yooperfan (11-Jul) : Happy belated Birthday Cheesey
Smokey (10-Jul) : That's Gouda Cheesey ! LOL
Cheesey (10-Jul) : As Groucho Marx once said, "I'm gonna join a club, and beat you with it!"LOL!
Smokey (10-Jul) : Jolly Good Show , Welcome to the Club !
Cheesey (9-Jul) : Thanks guys. Officially a geezer now.
Zero2Cool (9-Jul) : Today's Birthdays: Cheesey (60) , DanJustDan29 (30)
wpr (9-Jul) : Happy birthday cheesey.
Zero2Cool (9-Jul) : Yo Alan!
Zero2Cool (9-Jul) : Today's Birthdays: Cheesey (60)
Smokey (5-Jul) : A good night to "You Tube" an old Football Game !
Smokey (5-Jul) : Lots of rain tonight across the nation !
Smokey (4-Jul) : Hello DoddPower
Smokey (3-Jul) : I recall the 200th as well, it was Spectacular !
Cheesey (3-Jul) : I remember the 200th....doesn't seem that long ago.
Smokey (3-Jul) : Happy 241st Birthday America !
Cheesey (3-Jul) : "Great taste, less filling!"LOL!
Smokey (3-Jul) : Mrs. Guion ? LOL
Cheesey (3-Jul) : I don't know...who hear wants to taste Guion???
Zero2Cool (1-Jul) : Today's Birthdays: Grabacr (29)
Smokey (30-Jun) : Guion did wrong , but he may also have good taste .
Porforis (30-Jun) : Guion probably went out to get something better than meatloaf.
Smokey (30-Jun) : So would you like a metal or a chest to hang it on ?
Zero2Cool (30-Jun) : Stupid emoji. It's not two question marks. It's a winky face!! Guess I need to run this house gooder lol
Zero2Cool (30-Jun) : And I run this house. Understand that. ??
Zero2Cool (30-Jun) : Bahahahhaha basement
Please sign in to use Fan Shout
2017 Packers Schedule
Sunday, Sep 10 @ 3:25 PM
SEAHAWKS
Sunday, Sep 17 @ 7:30 PM
at Falcons
Sunday, Sep 24 @ 3:25 PM
BENGALS
Thursday, Sep 28 @ 7:25 PM
BEARS
Sunday, Oct 8 @ 3:25 PM
at Cowboys
Sunday, Oct 15 @ 12:00 PM
at Vikings
Sunday, Oct 22 @ 12:00 PM
SAINTS
Sunday, Oct 29 @ 12:00 AM
- BYE -
Monday, Nov 6 @ 7:30 PM
LIONS
Sunday, Nov 12 @ 12:00 PM
at Bears
Sunday, Nov 19 @ 12:00 PM
RAVENS
Sunday, Nov 26 @ 7:30 PM
at Steelers
Sunday, Dec 3 @ 12:00 PM
BUCCANEERS
Sunday, Dec 10 @ 12:00 PM
at Browns
Sunday, Dec 17 @ 12:00 PM
at Panthers
Saturday, Dec 23 @ 7:30 PM
VIKINGS
Sunday, Dec 31 @ 12:00 PM
at Lions
Think About It
Think About It
Recent Topics
33m / Green Bay Packers Talk / gbguy20

2h / Green Bay Packers Talk / Smokey

6h / Green Bay Packers Talk / Smokey

8h / Around The NFL / Smokey

21-Jul / Green Bay Packers Talk / Poppa San

21-Jul / Green Bay Packers Talk / beast

21-Jul / Green Bay Packers Talk / Poppa San

21-Jul / Green Bay Packers Talk / Poppa San

21-Jul / Around The NFL / Poppa San

21-Jul / Around The NFL / Poppa San

19-Jul / Green Bay Packers Talk / dhazer

19-Jul / Green Bay Packers Talk / steveishere

19-Jul / Green Bay Packers Talk / isocleas2

19-Jul / Green Bay Packers Talk / DoddPower

18-Jul / Green Bay Packers Talk / Cheesey

Headlines