Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. Database & SysAdmin
  3. Database
  4. Dynamic Query Question

Dynamic Query Question

Scheduled Pinned Locked Moved Database
questiondatabasehelp
16 Posts 7 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • K Kevin Marois

    But if I declare it in the @Command string, I cannot use @voter_id outside the string, correct? I still get the same error message because @voter_id only exists in the string.

    Everything makes sense in someone's mind

    S Offline
    S Offline
    smcnulty2000
    wrote on last edited by
    #6

    Absolutely. Dasblinkenlight's solution is a nice one to the problem, BTW. I normally build a temp table and just run the data into that. Using an insert or an update. I know it seems (both suggested solutions) like a lot of overhead to get one piece of data out but that's really what it takes.

    create table #Value (
    val int
    )

    set @command = 'insert into #value (val) SELECT top 1 voterid FROM tblCamp_CT WHERE ' +@Query

    exec(@command)

    set @voter_id=(select top 1 val from #Value)
    delete #Value

    Like that. You could also research sp_executesql for your project. this works, for example:

    declare @voter_id int , @query varchar(max), @command nvarchar(4000)
    ,@parm nvarchar(20)

    set @query='1=1'

    SET @Command = 'SELECT top 1 @voter_id = voterid FROM tblCamp_CT WHERE ' + @Query

    set @parm ='@voter_id int output '

    exec sp_executesql @command,@parm ,@voter_id out

    select @voter_id test

    As you can see the sp_executesql gives you another option. Some of this depends on what your personal flavor is toward a given solution.

    _____________________________ Give a man a mug, he drinks for a day. Teach a man to mug...

    1 Reply Last reply
    0
    • K Kevin Marois

      When I run this:

      DECLARE @voter_id INT
      SET @Command = 'SELECT @voter_id = voterid FROM tblCamp_CT WHERE ' + @Query
      EXEC (@Command)

      I get the error

      Must declare the scalar variable "@voter_id".

      Why is this happening????

      Everything makes sense in someone's mind

      P Offline
      P Offline
      PIEBALDconsult
      wrote on last edited by
      #7

      That would be another reason not to use stored procedures. Things like that are so much simpler when the SQL is in your DAL.

      M S 2 Replies Last reply
      0
      • P PIEBALDconsult

        That would be another reason not to use stored procedures. Things like that are so much simpler when the SQL is in your DAL.

        M Offline
        M Offline
        Mycroft Holmes
        wrote on last edited by
        #8

        PIEBALDconsult wrote:

        That would be another reason not to use stored procedures

        Boo, hissss, brrrppp and other sundry rude noises. Oh wait we've been round this tree before.

        Never underestimate the power of human stupidity RAH

        1 Reply Last reply
        0
        • P PIEBALDconsult

          That would be another reason not to use stored procedures. Things like that are so much simpler when the SQL is in your DAL.

          S Offline
          S Offline
          S Douglas
          wrote on last edited by
          #9

          PIEBALDconsult wrote:

          That would be another reason not to use stored procedures. Things like that are so much simpler when the SQL is in your DAL.

          I'll bite, this is easy in a proc. Seems like it would be way more difficult embedded in the DAL. I'm interested in how you would accomplish it?

          EXEC sp_executesql @DYNA_SQL, N'@NUM_ROWS int out'


          Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

          P 1 Reply Last reply
          0
          • S S Douglas

            PIEBALDconsult wrote:

            That would be another reason not to use stored procedures. Things like that are so much simpler when the SQL is in your DAL.

            I'll bite, this is easy in a proc. Seems like it would be way more difficult embedded in the DAL. I'm interested in how you would accomplish it?

            EXEC sp_executesql @DYNA_SQL, N'@NUM_ROWS int out'


            Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

            P Offline
            P Offline
            PIEBALDconsult
            wrote on last edited by
            #10

            What does it do? :confused:

            S 1 Reply Last reply
            0
            • P PIEBALDconsult

              What does it do? :confused:

              S Offline
              S Offline
              S Douglas
              wrote on last edited by
              #11

              Pretty much the same thing that smcnulty2000 suggested doing in the second suggestion. sp_executesql allows for parameters to be fed in / out the dynamic SQL being executed. http://msdn.microsoft.com/en-us/library/ms188001.aspx[^]


              Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

              P 2 Replies Last reply
              0
              • S S Douglas

                Pretty much the same thing that smcnulty2000 suggested doing in the second suggestion. sp_executesql allows for parameters to be fed in / out the dynamic SQL being executed. http://msdn.microsoft.com/en-us/library/ms188001.aspx[^]


                Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

                P Offline
                P Offline
                PIEBALDconsult
                wrote on last edited by
                #12

                Here are a couple of simple examples. Not using my usual data access classes. I prefer the ExecuteScalar, it was designed for this sort of thing.

                        System.Data.IDbConnection dbc = new System.Data.SqlClient.SqlConnection
                        (
                            @"Server='localhost\\SQLEXPRESS'; Database='Rubbish'; Trusted\_Connection='True'"
                        ) ;
                
                        System.Data.IDbCommand cmd = dbc.CreateCommand() ;
                
                        cmd.CommandText = "SELECT @voter\_id=Id FROM Account WHERE " + "Name = 'Two'" ;
                
                        System.Data.IDbDataParameter prm = cmd.CreateParameter() ;
                
                        prm.ParameterName = "@voter\_id" ;
                        prm.DbType = System.Data.DbType.Int32 ;
                        prm.Direction = System.Data.ParameterDirection.Output ;
                
                        cmd.Parameters.Add ( prm ) ;
                
                        dbc.Open() ;
                
                        cmd.ExecuteNonQuery() ;
                
                        int id = (int) prm.Value ;
                
                        cmd.Parameters.Clear() ;
                        cmd.CommandText = "SELECT Id FROM Account WHERE " + "Name = 'Three'" ;
                
                        object o = cmd.ExecuteScalar() ;
                
                        if ( o != System.DBNull.Value )
                        {
                            id = (int) o ;
                        }
                
                        dbc.Close() ;
                
                S 1 Reply Last reply
                0
                • S S Douglas

                  Pretty much the same thing that smcnulty2000 suggested doing in the second suggestion. sp_executesql allows for parameters to be fed in / out the dynamic SQL being executed. http://msdn.microsoft.com/en-us/library/ms188001.aspx[^]


                  Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

                  P Offline
                  P Offline
                  PIEBALDconsult
                  wrote on last edited by
                  #13

                  Here's another take on it:

                      public static T
                      GetAccountId<T>
                      (
                          this System.Data.IDbConnection dbc
                      ,
                          string Filter
                      ,
                          params System.Tuple<string,object>\[\] Parameters
                      )
                      {
                          T result = default(T) ;
                          
                          System.Data.IDbCommand cmd = dbc.CreateCommand() ;
                          cmd.CommandText = "SELECT Id FROM Account WHERE " + Filter ;
                          if ( Parameters != null )
                          {
                              foreach ( System.Tuple<string,object> p in Parameters )
                              {
                                  System.Data.IDbDataParameter prm = cmd.CreateParameter() ;
                                  prm.ParameterName = p.Item1 ;
                                  prm.Value = p.Item2 ;
                                  cmd.Parameters.Add ( prm ) ;
                              }
                          }
                          dbc.Open() ;
                          object o = cmd.ExecuteScalar() ;
                          if ( o != System.DBNull.Value )
                          {
                              result = (T) o ;
                          }
                          dbc.Close() ;
                          return ( result ) ;
                      }
                  
                  1 Reply Last reply
                  0
                  • P PIEBALDconsult

                    Here are a couple of simple examples. Not using my usual data access classes. I prefer the ExecuteScalar, it was designed for this sort of thing.

                            System.Data.IDbConnection dbc = new System.Data.SqlClient.SqlConnection
                            (
                                @"Server='localhost\\SQLEXPRESS'; Database='Rubbish'; Trusted\_Connection='True'"
                            ) ;
                    
                            System.Data.IDbCommand cmd = dbc.CreateCommand() ;
                    
                            cmd.CommandText = "SELECT @voter\_id=Id FROM Account WHERE " + "Name = 'Two'" ;
                    
                            System.Data.IDbDataParameter prm = cmd.CreateParameter() ;
                    
                            prm.ParameterName = "@voter\_id" ;
                            prm.DbType = System.Data.DbType.Int32 ;
                            prm.Direction = System.Data.ParameterDirection.Output ;
                    
                            cmd.Parameters.Add ( prm ) ;
                    
                            dbc.Open() ;
                    
                            cmd.ExecuteNonQuery() ;
                    
                            int id = (int) prm.Value ;
                    
                            cmd.Parameters.Clear() ;
                            cmd.CommandText = "SELECT Id FROM Account WHERE " + "Name = 'Three'" ;
                    
                            object o = cmd.ExecuteScalar() ;
                    
                            if ( o != System.DBNull.Value )
                            {
                                id = (int) o ;
                            }
                    
                            dbc.Close() ;
                    
                    S Offline
                    S Offline
                    S Douglas
                    wrote on last edited by
                    #14

                    Interesting, those are not very far off from I have used in code. Though most of my work is all database level work now (data warehousing, stored procs, SSIS, SSAS and SSRS), so I've come to use stored procs for everything. Just seems easier to me.


                    Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

                    P 1 Reply Last reply
                    0
                    • S S Douglas

                      Interesting, those are not very far off from I have used in code. Though most of my work is all database level work now (data warehousing, stored procs, SSIS, SSAS and SSRS), so I've come to use stored procs for everything. Just seems easier to me.


                      Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

                      P Offline
                      P Offline
                      PIEBALDconsult
                      wrote on last edited by
                      #15

                      S Douglas wrote:

                      I've come to use stored procs for everything

                      They are rarely the right tool for the job.

                      S 1 Reply Last reply
                      0
                      • P PIEBALDconsult

                        S Douglas wrote:

                        I've come to use stored procs for everything

                        They are rarely the right tool for the job.

                        S Offline
                        S Offline
                        S Douglas
                        wrote on last edited by
                        #16

                        PIEBALDconsult wrote:

                        They are rarely the right tool for the job.

                        No other tool suits the needs, :). I don't disagree with you sediments. However, in my world its all that exists.


                        Common sense is admitting there is cause and effect and that you can exert some control over what you understand.

                        1 Reply Last reply
                        0
                        Reply
                        • Reply as topic
                        Log in to reply
                        • Oldest to Newest
                        • Newest to Oldest
                        • Most Votes


                        • Login

                        • Don't have an account? Register

                        • Login or register to search.
                        • First post
                          Last post
                        0
                        • Categories
                        • Recent
                        • Tags
                        • Popular
                        • World
                        • Users
                        • Groups