|
So, I'm looking at the code for one our apps, and all of the source files start with comments like:
OK, thanks for telling me that you've been following our file-naming convention. But what does the class do?
|
|
|
|
|
What about an automated tool that deletes all code having less than 10% of comments in?
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
[my articles]
|
|
|
|
|
Won't do. You can easily meet the 10% quota by writing things like:
void CFoo::LogMessage(const CString& strMsg)
|
|
|
|
|
what about an automated routine that deletes all code from incompentant programmers 
|
|
|
|
|
Gimme back my code!!!!!!!!!!!!!!!!!
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
[my articles]
|
|
|
|
|
This is very probably a template for new source files which was never changed.
For what the class does - Use the code, Luke!
Let's think the unthinkable, let's do the undoable, let's prepare to grapple with the ineffable itself, and see if we may not eff it after all.
Douglas Adams, "Dirk Gently's Holistic Detective Agency"
|
|
|
|
|
jhwurmbach wrote: For what the class does - Use the code, Luke!
Well, yeah. But such headers requires the reader having to scroll more, thus adding to the wear and tear on his/her finger joints. That is bad, and might result in a shortened life span as a programmer.
--
Kein Mitleid Für Die Mehrheit
|
|
|
|
|
What? You mean you don't have the audio/visual integration tool based on the Apache weapons officer helmet that tracks your eye movement and interprets you verbal commands? Man, life became much easier when I put on the helmet. I can't wait until they finish getting the jack put in my cerebelum so I can just think code.
|
|
|
|
|
Be very aware of this (or maybe it's common knowledge, and somehow I missed this vital info all these years).
If you accidentelly leave tracing enabled on a production site, be aware that anyone can see the details for the past xx requests. Now normally this info is fairly mundane - control hierarchy and server variables.
However, if you store connection strings in the HttpContext object, they will get output in the trace!! Could be very risky.
Of course normally you would use the connectionstrings helper which is available with ASP.NET 2.0 onwards, however if you have an older app, you may be doing this (e.g. when you Server.MapPath() to your local access database or something, and later upgrade to SQL server but keep the connection string in httpcontext object  )
"For fifty bucks I'd put my face in their soup and blow." - George Costanza
CP article: SmartPager - a Flickr-style pager control with go-to-page popup layer.
|
|
|
|
|
So, if you forget to turn off debugging code and do some pretty stupid stuff around your storing of connections, and some user manages to try this and figure it out, you could be in trouble if your network isn't properly secured.
Subtle.
|
|
|
|
|
Paddy Boyd wrote: forget to turn off debugging code
Yep, sloppy. Makes a stronger case for specifically scheduling code-reviews a while after initial development.
Paddy Boyd wrote: pretty stupid stuff around your storing of connections
OK, so apart from this potential scenario, what reason would you give me for NOT storing a connection string in the application state? (Bear in mind it has to call Server.MapPath.. so ConfigurationSettings.AppSettings is no good - and I'm talking .NET 1.1 here).
Paddy Boyd wrote: Subtle.
Yes it is. Especially for traditional ASP programmers - you just don't expect your Application variables to ever be visible to the public like that.
"For fifty bucks I'd put my face in their soup and blow." - George Costanza
CP article: SmartPager - a Flickr-style pager control with go-to-page popup layer.
|
|
|
|
|
Hi, I am in the trouber of the error while the project trys to saving the data to child table. The error message
is shown below:
NHibernate.ADOException: could not insert: [ConsoleApplication1.user][SQL: INSERT INTO user (UNAME, NID) VALUES (?, ?)] --->
So confused that why it cannot get values to save. NHibernate I used is version 1.2. The code files are list below, please help me out here.
Thanks in advance.
app.config:
<?xml version="1.0" encoding="utf-8" ?>
<configuration>
<configSections>
<section name="nhibernate" type="System.Configuration.NameValueSectionHandler, System, Version=1.2.0.4000,Culture=neutral, PublicKeyToken=b77a5c561934e089"/>
</configSections>
<nhibernate>
<add key="hibernate.connection.provider" value="NHibernate.Connection.DriverConnectionProvider"/>
<add key="hibernate.dialect" value="NHibernate.Dialect.MsSql2005Dialect"/>
<add key="hibernate.connection.driver_class" value="NHibernate.Driver.SqlClientDriver"/>
<add key="hibernate.connection.connection_string" value="Server=;initial catalog=;Persist Security Info=True;User ID=;Password="/>
</nhibernate>
</configuration>
user.hbm.xml:
<hibernate-mapping default-cascade="none" xmlns="urn:nhibernate-mapping-2.2">
<class name="ConsoleApplication1.user, ConsoleApplication1" table="user">
<id name="UID" type="System.Int32" column="UID" unsaved-value="0">
<generator class="native" />
</id>
<property name="UNAME" type="System.String" column="UNAME" not-null="false" />
<many-to-one name="Nationality" class="ConsoleApplication1.nationality, ConsoleApplication1" fetch="select" cascade="all">
<column name="NID" not-null="false" />
</many-to-one>
</class>
</hibernate-mapping>
user.hbm.cs:
namespace ConsoleApplication1 {
[System.SerializableAttribute()]
public class Abstractuser {
private int uID;
private string uNAME;
private ConsoleApplication1.nationality nationality;
public virtual int UID {
get {
return this.uID;
}
set {
this.uID = value;
}
}
public virtual string UNAME {
get {
return this.uNAME;
}
set {
this.uNAME = value;
}
}
public virtual ConsoleApplication1.nationality Nationality {
get {
return this.nationality;
}
set {
this.nationality = value;
}
}
}
[System.SerializableAttribute()]
public partial class user : Abstractuser {
}
}
nationality.hbm.xml:
<hibernate-mapping default-cascade="none" xmlns="urn:nhibernate-mapping-2.2">
<class name="ConsoleApplication1.nationality, ConsoleApplication1" table="nationality">
<id name="NID" type="System.Int32" column="NID" unsaved-value="0">
<generator class="native" />
</id>
<property name="NATIONALITY" type="System.String" column="NATIONALITY" not-null="false" />
<bag name="User" inverse="true" lazy="true" cascade="all">
<key>
<column name="NID" not-null="false" />
</key>
<one-to-many class="ConsoleApplication1.user, ConsoleApplication1" />
</bag>
</class>
</hibernate-mapping>
nationality.hbm.cs:
namespace ConsoleApplication1 {
[System.SerializableAttribute()]
[System.Xml.Serialization.XmlIncludeAttribute(typeof(ConsoleApplication1.user))]
[System.Xml.Serialization.SoapIncludeAttribute(typeof(ConsoleApplication1.user))]
public class Abstractnationality {
private int nID;
private string nATIONALITY;
private System.Collections.IList user;
public virtual int NID {
get {
return this.nID;
}
set {
this.nID = value;
}
}
public virtual string NATIONALITY {
get {
return this.nATIONALITY;
}
set {
this.nATIONALITY = value;
}
}
public virtual System.Collections.IList User {
get {
return this.user;
}
set {
this.user = value;
}
}
}
[System.SerializableAttribute()]
public partial class nationality : Abstractnationality {
}
}
namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
NHibernate.Cfg.Configuration cfg = new NHibernate.Cfg.Configuration();
cfg.AddAssembly("ConsoleApplication1");
ISessionFactory factory = cfg.BuildSessionFactory();
ISession session = factory.OpenSession();
ITransaction transaction = session.BeginTransaction();
nationality nat = (nationality)session.Get(typeof(nationality), 1);
user u = new user();
u.UNAME = "Pall";
u.Nationality = nat;
nat.User.Add(u);
try
{
if (!session.IsConnected)
{
session.Reconnect();
}
session.Save(u); // Error occured at this line!!
transaction.Commit();
session.Close();
}
catch (Exception e)
{
string s = e.ToString();
}
}
}
}
|
|
|
|
|
yeah, i see the horror in this 
(yes|no|maybe)*
|
|
|
|
|
Harry Sun wrote: Hi, I am in the trouber of the error while the project trys to saving the data to child table. The error message
is shown below:
You've hit the "Y Who Cares" Bug. It's much less advertised than the Y2K bug was, but much more prevalent. Can you feel the apathy? Can you?
|
|
|
|
|
Pete O'Hanlon wrote: You've hit the "Y Who Cares" Bug.
Hopefully said bug causes his system to overload, and a rm -rf /his/job/ must be preformed.
I'm going to become rich when I create a device that allows me to punch people in the face over the internet.
"If an Indian asked a programming question in the forest, would it still be urgent?" - John Simmons / outlaw programmer
|
|
|
|
|
First, you post the same question 3 times, and in different fourms as well. Well call that second, the cross-posting that is. Third, you don't know how to use a debugger. Fourth, you put 'help!' in your subject. Everyone that posts a question needs help, jackass. Fifth, you can't read the forum rules. This isn't for questions. It's for coding horrors, which yours is, but in a different context.
So basically, that makes you a 5x idiot. Quit crop-dusting our forums with your crap question. No one cares. Had you been civilized about it, you may have gotten some help. As a matter of fact, someone did answer you. They now wish they didn't, and I'm sure they wouldn't have if they'd seen your blatant disregard for the rules. Most people won't. Oh man I wish my device was ready.
I'm going to become rich when I create a device that allows me to punch people in the face over the internet.
"If an Indian asked a programming question in the forest, would it still be urgent?" - John Simmons / outlaw programmer
|
|
|
|
|
Nice
|
|
|
|
|
the dog and His Masters Voice.
Luc Pattyn [Forum Guidelines] [My Articles]
This month's tips:
- before you ask a question here, search CodeProject, then Google;
- the quality and detail of your question reflects on the effectiveness of the help you are likely to get;
- use PRE tags to preserve formatting when showing multi-line code snippets.
|
|
|
|
|
|
I took the lesson. Thank you.
|
|
|
|
|
I feel nice today, so I will help you once here, thats it though, next time, try the correct forum.
Harry Sun wrote: unsaved-value="0"
That seems like bogus.
|
|
|
|
|
Thanks for your kind. I removed that code you mentioned, but the problem still exists. In fact, I thought it is fine for define unsaved-value="0", because I have already make this attribute as incremental identiy in database. While debugging the code, the object which I intend to save in session.save(u) does have correct values. And it works fine if I just work with one table. Am I right? Please tell me if you got time.
|
|
|
|
|
You can't use user as a table name and have it work with NHibernate unless you do some special stuff in your mapping file...
In your user.hbm.xml do this:
//note the square brackets around the table name.
<class name="ConsoleApplication1.user, ConsoleApplication1" table="[user]">
That'll fix your problem...
Richard
http://www.BellaDev.net
|
|
|
|
|
Please check "Ignore HTML tags in this message" or do something about that <class..., it's not showing up
[ My Blog] "Visual studio desperately needs some performance improvements. It is sometimes almost as slow as eclipse." - Rüdiger Klaehn "Real men use mspaint for writing code and notepad for designing graphics." - Anna-Jayne Metcalfe
|
|
|
|
|
Dear Richard,
That is what the problem is. I really appreciate to what you did. It has been tough time for me. You saved me. Thank you again.
Harry
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
