Finally my article viginity is lost.....
-
I'm a big boy now! :) Just posted my first ever article. Couldn't have done it without the help from several folk here at Codeproject. especially Rick Shaub[^] who helped me get to grips with winforms threading. Check it out here. Batch Image Resizer[^] Any constructive critisism is always welcome....
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
I fail to see the point of the article. It's just a bunch of so-so code. It also doesn't work properly with XP. The Windows7Tools class instantiates an instance of TaskbarManager outside the constructor if block (before you know whether Windows 7 is there.) SaveRecentFolders doesn't check if _recentFolders is null. The article doesn't display properly in Firefox 2.x
-
I fail to see the point of the article. It's just a bunch of so-so code. It also doesn't work properly with XP. The Windows7Tools class instantiates an instance of TaskbarManager outside the constructor if block (before you know whether Windows 7 is there.) SaveRecentFolders doesn't check if _recentFolders is null. The article doesn't display properly in Firefox 2.x
Joe Woodbury wrote:
I fail to see the point of the article. It's just a bunch of so-so code.
How does that phrase go?..... If you've nothing nice to say....... Firefox 2.x...... Seriously?? Take a deep breath......Firefox 3.6[^]
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
-
Joe Woodbury wrote:
I fail to see the point of the article. It's just a bunch of so-so code.
How does that phrase go?..... If you've nothing nice to say....... Firefox 2.x...... Seriously?? Take a deep breath......Firefox 3.6[^]
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
Way to keep your cool. I thought Joe's opinion was a bit harsh. You're article could use a little polish, sure, but it seems like you are well on your way. Just look at the feedback people leave at the bottom of your article and I'm sure you'll get your article looking great in no time. :)
-
Joe Woodbury wrote:
I fail to see the point of the article. It's just a bunch of so-so code.
How does that phrase go?..... If you've nothing nice to say....... Firefox 2.x...... Seriously?? Take a deep breath......Firefox 3.6[^]
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
So you think asking for a critique means all you want is someone to kiss your ass? I gave you specific glaring problems with your code and ignored all the structural problems. An article, whether here or elsewhere, should have an explicit point, not just be a code dump. What principle are you teaching? Why should someone suggest this article to someone else? Edit: To add from Code Project guidelines: - Is the goal of the article clear? - Could the article help someone in their day-to-day work?
modified on Monday, February 1, 2010 9:32 PM
-
Way to keep your cool. I thought Joe's opinion was a bit harsh. You're article could use a little polish, sure, but it seems like you are well on your way. Just look at the feedback people leave at the bottom of your article and I'm sure you'll get your article looking great in no time. :)
I fail to see how I am being a bit harsh. Coddling someone does them no good and in this case does the code project community no good. There are already too many code-dump articles, we don't need more. The Code Project guidelines include: - Is the goal of the article clear? - Could the article help someone in their day-to-day work? This article satisfies neither of these goals and pretending they do won't help anyone who is visiting Code Project to learn a specific algorithm or technique.
-
I fail to see how I am being a bit harsh. Coddling someone does them no good and in this case does the code project community no good. There are already too many code-dump articles, we don't need more. The Code Project guidelines include: - Is the goal of the article clear? - Could the article help someone in their day-to-day work? This article satisfies neither of these goals and pretending they do won't help anyone who is visiting Code Project to learn a specific algorithm or technique.
Pretty sure I read that the goal was to describe his application, and he did so while describing a few of the technical problems and solutions along the way. Being that he categorized it under "Applications", seems like a perfectly suitable goal and a fair execution. And this seems like it could be a very useful application to some people, which he has provided free of charge, so even better. Code Project has many sections and many purposes; perhaps you should explore them a bit more.
-
Pretty sure I read that the goal was to describe his application, and he did so while describing a few of the technical problems and solutions along the way. Being that he categorized it under "Applications", seems like a perfectly suitable goal and a fair execution. And this seems like it could be a very useful application to some people, which he has provided free of charge, so even better. Code Project has many sections and many purposes; perhaps you should explore them a bit more.
I have explored them and find the increasing net of Code Project to make is less useful, not more. Having poorly written, poorly designed applications doesn't help anyone (and it is poorly written and poorly designed.) Now, if I'm mistaken and Code Project is now Mediocre Project then I stand corrected and apologize.
-
I fail to see how I am being a bit harsh. Coddling someone does them no good and in this case does the code project community no good. There are already too many code-dump articles, we don't need more. The Code Project guidelines include: - Is the goal of the article clear? - Could the article help someone in their day-to-day work? This article satisfies neither of these goals and pretending they do won't help anyone who is visiting Code Project to learn a specific algorithm or technique.
Joe Woodbury wrote:
I fail to see how I am being a bit harsh.
Joe, for what it's worth, I agree with your assessment of the article not being up to the standards we'd like to see at CP, but really find your word choice off-putting. The point of criticism is, I believe, to help someone do better both by noting where he did poorly and by noting where he did well. The way you phrased things, I felt you went out of your way to find the most pejorative way of saying what you wanted to say. Unless of course, the point of your criticism was to insure he never contributes another article.
Jon "You're the kind of people the import police come asking about later. I don't like talking to the import police. They make me nervous, and my pseudopods sweat." ~ Bog, Factor for the Interstellar Trading Co. Soap Box 1.0: the first, the original, reborn troll-less
-
Joe Woodbury wrote:
I fail to see how I am being a bit harsh.
Joe, for what it's worth, I agree with your assessment of the article not being up to the standards we'd like to see at CP, but really find your word choice off-putting. The point of criticism is, I believe, to help someone do better both by noting where he did poorly and by noting where he did well. The way you phrased things, I felt you went out of your way to find the most pejorative way of saying what you wanted to say. Unless of course, the point of your criticism was to insure he never contributes another article.
Jon "You're the kind of people the import police come asking about later. I don't like talking to the import police. They make me nervous, and my pseudopods sweat." ~ Bog, Factor for the Interstellar Trading Co. Soap Box 1.0: the first, the original, reborn troll-less
Pretty much my feeling exactly. New authors should be shown a little courtesy and encouragement, not beratement for the terrible piece of work they've inflicted upon us all. That will only scare away authors with potential and ensure that first time authors stay that way, thereby reducing the overall quality of articles on Code Project.
-
Pretty much my feeling exactly. New authors should be shown a little courtesy and encouragement, not beratement for the terrible piece of work they've inflicted upon us all. That will only scare away authors with potential and ensure that first time authors stay that way, thereby reducing the overall quality of articles on Code Project.
Well, I can't give you a 5, or even a 4 because I do think Joe has a point about standards being the same. It's just that there are ways of helping someone see the errors that don't make him feel like he's under attack. But I'll give you a 3. ;)
Jon "You're the kind of people the import police come asking about later. I don't like talking to the import police. They make me nervous, and my pseudopods sweat." ~ Bog, Factor for the Interstellar Trading Co. Soap Box 1.0: the first, the original, reborn troll-less
-
Well, I can't give you a 5, or even a 4 because I do think Joe has a point about standards being the same. It's just that there are ways of helping someone see the errors that don't make him feel like he's under attack. But I'll give you a 3. ;)
Jon "You're the kind of people the import police come asking about later. I don't like talking to the import police. They make me nervous, and my pseudopods sweat." ~ Bog, Factor for the Interstellar Trading Co. Soap Box 1.0: the first, the original, reborn troll-less
Oakman wrote:
But I'll give you a 3.
Oy! Beware the dangers of discussing voting standards. Never again, I tells you, never again.
Oakman wrote:
I do think Joe has a point about standards being the same
I hold all authors to the same standards. I don't give less feedback because they are beginners. I just don't yell at them.
-
Excuse me, but what is a "viginity"? :laugh: Now, and more seriously, congrats for the first article, but I've not been able to read the article as the link is taking you to a non correct place... :sigh:
-
Well, I can't give you a 5, or even a 4 because I do think Joe has a point about standards being the same. It's just that there are ways of helping someone see the errors that don't make him feel like he's under attack. But I'll give you a 3. ;)
Jon "You're the kind of people the import police come asking about later. I don't like talking to the import police. They make me nervous, and my pseudopods sweat." ~ Bog, Factor for the Interstellar Trading Co. Soap Box 1.0: the first, the original, reborn troll-less
You made me laugh. :thumbsup:
-
I fail to see the point of the article. It's just a bunch of so-so code. It also doesn't work properly with XP. The Windows7Tools class instantiates an instance of TaskbarManager outside the constructor if block (before you know whether Windows 7 is there.) SaveRecentFolders doesn't check if _recentFolders is null. The article doesn't display properly in Firefox 2.x
Firefox 2.x? Are you serious? :~
-
I'm a big boy now! :) Just posted my first ever article. Couldn't have done it without the help from several folk here at Codeproject. especially Rick Shaub[^] who helped me get to grips with winforms threading. Check it out here. Batch Image Resizer[^] Any constructive critisism is always welcome....
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
"Quantization"?!? What user group are you targeting? If you don't want to have to include a lengthy explanation of what something is, use a term that can be understood easily, and skip any explanation -- try "Trim" or "Enhance", depending on what the actual, visible outcome from the matrix is (but stick with "quantization" if you allow custom matrices to be added). Or "Confumibrigate". I like "Confumibrigate". Confumibrigation rocks. Today.
I wanna be a eunuchs developer! Pass me a bread knife!
-
I fail to see the point of the article. It's just a bunch of so-so code. It also doesn't work properly with XP. The Windows7Tools class instantiates an instance of TaskbarManager outside the constructor if block (before you know whether Windows 7 is there.) SaveRecentFolders doesn't check if _recentFolders is null. The article doesn't display properly in Firefox 2.x
Joe Woodbury wrote:
The Windows7Tools class instantiates an instance of TaskbarManager outside the constructor if block (before you know whether Windows 7 is there.)
And no exception handler. Naughty, naughty. Cancel QA's bonus payments.
I wanna be a eunuchs developer! Pass me a bread knife!
-
Joe Woodbury wrote:
I fail to see the point of the article. It's just a bunch of so-so code.
How does that phrase go?..... If you've nothing nice to say....... Firefox 2.x...... Seriously?? Take a deep breath......Firefox 3.6[^]
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
Sorry I'm contacting you here, but I saw no other way than to find one of your old posts. Why did you delete all your messages regarding "Trouble deleting items using generic xml provider" from the C# forum?
-
Sorry I'm contacting you here, but I saw no other way than to find one of your old posts. Why did you delete all your messages regarding "Trouble deleting items using generic xml provider" from the C# forum?
Please accept my sincerest apologies. I thought that there was no interest after your initial reply and I didn't really fancy leaving all that source code out there for the world to steal. (Selfish I know but I'd put a lot of work into it). I figured out a simple solution to the problem. I could remove items from my list as long as there were more than one item present. If there was only one List.Remove() just ignored me silently (I still haven't figured out why. I had already written a Delete() method into my Persistable class so I just checked to see if there was more than one item in my list and if not just deleted the file since it would be created automtically the next time the provider was called. I can let you see the full source code if you like. I designed it in mind to use it for simple low to medium use websites as a datastore rather than have clients shell out for a database with their hosting plan and it seems to run quite quickly with the tests I've done. Anyway thanks for your assistance the other day. Cheers
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
-
Please accept my sincerest apologies. I thought that there was no interest after your initial reply and I didn't really fancy leaving all that source code out there for the world to steal. (Selfish I know but I'd put a lot of work into it). I figured out a simple solution to the problem. I could remove items from my list as long as there were more than one item present. If there was only one List.Remove() just ignored me silently (I still haven't figured out why. I had already written a Delete() method into my Persistable class so I just checked to see if there was more than one item in my list and if not just deleted the file since it would be created automtically the next time the provider was called. I can let you see the full source code if you like. I designed it in mind to use it for simple low to medium use websites as a datastore rather than have clients shell out for a database with their hosting plan and it seems to run quite quickly with the tests I've done. Anyway thanks for your assistance the other day. Cheers
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
Thanks for the explanation. In the future, do not delete your posts like that. If you really think the code is so valuable that others would steal it and you'd suffer from it, then I'd just post a simple example that demonstrates the problem rather than the more sensitive code. I answer questions not only for you but for others who come across similar problems, so deleting posts like that prevents others from finding solutions. For example, you could have posted your above solution as a reply to your own question and that may have helped somebody else. By the way, just a guess here, but the problem may have been that your empty list serialized to nothing (i.e., an empty or null string) and maybe somewhere along the line there is code that says "if there was nothing to persist to a file, then don't persist that to the file" (i.e., leave the contents of the file intact). Maybe in your code or maybe in the .Net Framework code. Not sure, but I thought I'd throw that out there.
-
Please accept my sincerest apologies. I thought that there was no interest after your initial reply and I didn't really fancy leaving all that source code out there for the world to steal. (Selfish I know but I'd put a lot of work into it). I figured out a simple solution to the problem. I could remove items from my list as long as there were more than one item present. If there was only one List.Remove() just ignored me silently (I still haven't figured out why. I had already written a Delete() method into my Persistable class so I just checked to see if there was more than one item in my list and if not just deleted the file since it would be created automtically the next time the provider was called. I can let you see the full source code if you like. I designed it in mind to use it for simple low to medium use websites as a datastore rather than have clients shell out for a database with their hosting plan and it seems to run quite quickly with the tests I've done. Anyway thanks for your assistance the other day. Cheers
JimBob SquarePants ******************************************************************* "He took everything personally, including our royalties!" David St.Hubbins, Spinal Tap about Ian Faith, their ex-manager *******************************************************************
JimBob SquarePants wrote:
I designed it in mind to use it for simple low to medium use websites as a datastore rather than have clients shell out for a database with their hosting plan and it seems to run quite quickly with the tests I've done.
By the way, have you heard of SQL Server CE? It's basically a database in a file and I'm pretty sure it'd run on any computer (e.g., a web server) that has .Net Framework (2.0?). I recently used it to make a time tracking tool (each entry is stored to the database). Since it doesn't require an extra install (like the full SQL Server does), it can be used on client computers or web servers (though I haven't tested it on a web server).