Comments by pt1401 (Top 42 by date)

And what are the contents of your textboxes?
I'm guessing that either txtAge or txtIC dont have integers in them?

Did you have any other SQL versions installed previously?
CTP versions that haven't been fully uninstalled can cause this.

Re. "I am expecting query from you".
That's not going to happen without telling us how the data is stored - what's the database schema? Tables? Columns?

Also, please tabulate the data in your question properly.
If it's easy to read, people will be more likely to offer solutions.

I think we need some more information. Such as
Is it always the first six files you want?
Or do you perhaps need all the files starting with

20170505

?
Are the files always in sets of three? xxxF_nnn.jpg, xxxR_nnn.jpg and xxx_yyy.tif?

It always helps to have some info on the logic requirements...

Why do you think you should only get 4 records?
SELECT * INTO #TEMP FROM TEST_DEL WHERE 1=0 inserts nothing, it just creates an empty temp table with the correct schema.

Me neither, what do you mean?
only asking
looking some clues

I think you need to rewrite your question :-)

This is a homework question.
There is short-term benefit, but no long-term benefit, in us doing your homework for you.
Long-term benefit is what's important.
It's *your* homework - *you* need do do it so that *you* can learn programmimg.

If you really need to create from an interface, perhaps because you want to inject it, you could create composite interfaces, then create a D using interface ID

public interface ID : IA
{
int p { get; set; }
int F4();

}
public class D : ID
{
}

Depends on whether you need to inject it.
If not, this would work (and has the advantage of being mind-numbingly simple).
Sometimes the simple way is best.

var myD = new D();

The trick is to read the requirement and identify the objects, operations and variables.
Something like this :-

1) At the start, the elevator is situated at a given floor. Its first action is to load on some passengers
Objects: Elevator, Passenger
Operations: Elevator.Load(int passengerCount)
Variables: currentFloor, passengersToLoad

2) Elevator moves from current floor to destination floor:
- it announces every floor it passes. // Would this part be a loop? Depending how many floors the building has?
-It will not have to stop at any of these floors just the destination floor
Operations: Elevator.MoveTo(int destinationFloor), Elevator.Announce(int floorNumber)

3) At the destination floor it performs two actions:
- offloads a certain amount of riders (not necessarily all of them)
- onload a certain number of people (but do not overload the elevator) // Would these two actions also be loops as well for loading and unloading riders?
Operations: Elevator.Unload(int passengersToUnload), Elevator.Unload(int passengersToLoad)
Variables: maxPassengers

That should give you a starting point for the objects you need (just one, Elevator), and the methods.
Then you need to code the methods by translating the requirements into logic and then code.
What you have is good so far, good luck...

You're auto-generating a C# class based on input created by the user?
Presumably at some point, that class will get compiled & executed.

My initial thought is 'Why?'
It seems dangerous and I feel there must be a better way.
But without knowing what you application is, or what it's supposed to achieve, I can't tell you what that 'better way' might be.

I'm guessing your solution is targeting 'x32' instead of 'AnyCPU' ?

You need to try harder to help people to help you.

You don't give enough information - for a starter, what error are you getting?
Here's a possibility:-
Your sql string 'sql2' isn't used - your command uses sql1, which you give no details of.

As a side issue, it's not a good idea to build sql with string concatenation.
Do some googling on sql injection and sql parameters.

Confucious he say "Man who ask 'Please reply me soon' who doesn't acknowledge replies soon, not get reply so soon next time..."

What has this got to do with coding?

I'm sorry, really?
What I have tried:
"I need...."

So you have tried nothing?
We are not your slaves/servants.

Oh come on.
If you expect people to help you, at least give us a chance.
I get that you want to parse these files using C#, but what language is it that is in the text files that you want to parse?

"What I have tried: I have been struggling for the right code"
But you can't give us any more detailed insight into your struggles?
Cut & paste your attempts below...

> Connect as late as possible, query as fast as possible, and disconnect and Dispose as soon as possible.
Excellent mantra. Developers should have this tattooed on their foreheads in mirror writing :-)

I hope you added a 'Add shortcut to desktop?" option in your installer?

I don't understand why you'd want to do this.
Surely whether a program is pinned to the start menu should be decided by the user, not by the program that he's running?

Let the user decide - if any program did this to me I'd immediately uninstall it.

I've used both, but have now settled with NLog. They're very similar, but NLog has 5 logging levels vs 4 for Log4Net.

Really?
Your question: i need the best and efficient way to save image into database
You've tried: i need the best and efficient way to save image into database

It seems you can't even make the effort to tell us what database you're using.
If you want people to help you, you should make some effort to help them help you.

Have you tried running the proc as above and comparing the resultset to the original?

You don't need strSql, use either or the proc definitions above

So I assume the cursor is needed, and there will be something the --PROCESS loop?
If that's the case, I'd write the proc as
CREATE PROCEDURE [dbo].[sp_empInfo]
@empID int =null,
@TesterName varchar(250)=null
AS

DECLARE @emp INT
DECLARE @tester VARCHAR(250)

DECLARE users_cursor CURSOR FAST_FORWARD READ_ONLY FOR
SELECT @emp=ID, @tester=TesterName
FROM emp
WHERE Status=1
AND (@empID IS NULL OR @empId='' OR ID=@empID)
AND (@TesterName IS NULL OR @TesterName='' OR @TesterName=TesterName)

OPEN users_cursor
FETCH NEXT FROM users_cursor INTO @emp, @tester

WHILE @@FETCH_STATUS = 0
BEGIN
-- PROCESS
FETCH NEXT FROM users_cursor INTO @emp, @tester
END

CLOSE users_cursor
DEALLOCATE users_cursor


If the cursor really isn't needed, it reduces to
CREATE PROCEDURE [dbo].[sp_empInfo]
@empID int =null,
@TesterName varchar(250)=null
AS

SELECT @emp=ID, @tester=TesterName
FROM emp
WHERE Status=1
AND (@empID IS NULL OR @empId='' OR ID=@empID)
AND (@TesterName IS NULL OR @TesterName='' OR @TesterName=TesterName)

There's a place for dynamic sql, but it's better no used if it can be avoided.
UPDATE: SELECT @emp=ID, @tester=TesterName
SHOULD BE SELECT ID, TesterName

Because the question was 'why am I getting an invalid column name error' :)

It isn't the right way, because it's horribly inefficient.
Try:-

IF @empID IS NOT NULL AND LEN(@empID) > 0
SELECT ID, TesterName FROM emp Where Status=1 AND ID=@empID
ELSE IF @TesterName IS NOT NULL AND LEN(@TesterName) > 0
SELECT ID, TesterName FROM emp Where Status=1 AND TesterName=@TesterName


Ah ok, that doesn't work when you supply both @empID & @TesterName

Does the WHERE clause return any records?
Does it return the record(s) that you were intending to delete?
Try changing the DELETE to SELECT * and running in in SQL Management Studio to confirm.

A side issue is PLEASE don't construct SQL like this, it leaves your code wide open to SQL injection attacks. It may just a noddy program for your personal use, but it's a very bad and dangerous habit to get into. Use parameterised queries instead.

1. Please format your code listings.
2. Please give the full error message, ideally with a stack trace.

In summary, put a little effort into helping others to help you.

Without any details of your code, which service provider you're using, or the API details, I doubt that anyone is going to be able to help.

It seems that what you have tried is exactly what you have been set. So how did you get on? What particular problems did you hit, and what have you tried to get around those problems?
Nothing? Guessed not...

Very perceptive of you...

Reason for my vote of 5 \n Problems publishing images don't deserve downvotes. Hava a 5 in compensation...

Reason for my vote of 2
This is just a rant. If you're going to give a tip, you should give reasons why your suggestion is worth following - the only reason you give is a single example of someone using Select * in a way that broke *your* code because they used it in a particular way. Select * can be used safely and there are many situations where it's use is valid - though I agree it's generally wasteful on bandwidth and the number of rows returned should be limited. And using it to transfer data between tables is a *BAD* idea.