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 (2h) : LOL !
Tezzy (4h) : it's a trap
Zero2Cool (14h) : check your PM's please
Zero2Cool (14h) : steveishere
Zero2Cool (15h) : Great, Goodell was extended????
Smokey (21-Aug) : NOT A FORUM MEMBER ? JOIN US AND SHARE YOUR VIEWS !
Zero2Cool (20-Aug) : two slots open...time is ticking for those two.
Tezzy (20-Aug) : Did you fill?
Cheesey (20-Aug) : Dang Wayne, I'm trying not to cry. But it's a losing battle. My heart aches for you.
Zero2Cool (20-Aug) : You want in, you're in. :-)
Zero2Cool (20-Aug) : Need just one person to join PH fantasy league so we have ten. Not chasing everyone around anymore.
wpr (20-Aug) : Liver, pizza, brushed, short walk, ear scratches sat on my lap.
Smokey (19-Aug) : love, love, love .
gbguy20 (19-Aug) : lots of tasty people food, car rides, go to a lake, just something out of the usual and lots of attention
gbguy20 (19-Aug) : just make sure you give him an awesome weekend before Monday comes.
Cheesey (19-Aug) : We had to do that 2 years ago to our 15 year old beloved Seth. It's never easy. I'm here for you if you want to talk.
Cheesey (19-Aug) : Wayne, I know how hard that is. Loving them enough to let them go. I''m so sorry!
buckeyepackfan (19-Aug) : Sorry to hear wpr. I had to do the same several years ago. He also was 15. Just know you are easing his pain.
Smokey (19-Aug) : HELLO DAVE !
wpr (19-Aug) : He's a mix. Shepard, Aussy Blue, Border Collie
Porforis (18-Aug) : Good grief... You're the third person I know this week that's had to do that. I'm sorry :(
Zero2Cool (18-Aug) : damn that sucks Wayne... sorry buddy
beast (18-Aug) : Sorry wpr :(
Smokey (18-Aug) : I love dogs, what is the dogs breed ?
wpr (18-Aug) : I spoke with my vet. I am going to put my 16 yo dog down on Mon.
rabidgopher04 (18-Aug) : Coincidentally when I saw the Red Sox in Boston it was against the Brewers.
wpr (17-Aug) : Good idea Z.
wpr (17-Aug) : I tapped my savings to spend time with my son.
Zero2Cool (17-Aug) : I'll have to find a time when Brewers play Red Sox at Fenway. Both teams, one location!
wpr (17-Aug) : Why do you need to be a jerk? I live in a modest home. My kids are out of the house. My wife and I both work. My son paid for most of it. I
Smokey (17-Aug) : Yes, Insurance Agents do well . You must feel so superior .
rabidgopher04 (17-Aug) : Fenway is electric. Great place to watch baseball. I've been a couple times.
wpr (17-Aug) : A few have it already. Most have to save it to do things.
wpr (17-Aug) : Yes Smokey, life takes money to do anything.
Smokey (16-Aug) : IF THEY HAVE THE MONEY !
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
1h / Green Bay Packers Talk / Cheesey

1h / Green Bay Packers Talk / gbguy20

2h / Green Bay Packers Talk / beast

2h / Green Bay Packers Talk / beast

5h / Green Bay Packers Talk / sschind

8h / Green Bay Packers Talk / Smokey

15h / Green Bay Packers Talk / Zero2Cool

21-Aug / Green Bay Packers Talk / yooperfan

20-Aug / Green Bay Packers Talk / yooperfan

19-Aug / Green Bay Packers Talk / nerdmann

19-Aug / Green Bay Packers Talk / beast

19-Aug / Around The NFL / Zero2Cool

19-Aug / Green Bay Packers Talk / Smokey

18-Aug / Fantasy Sports Talk / Smokey

18-Aug / Green Bay Packers Talk / Cheesey

Headlines