|
SqlConnection con = new SqlConnection("Data Source=.\\sqlexpress;Initial Catalog=MLM;Integrated Security=True");
con.Open();
SqlCommand com = new SqlCommand("select Pin_Password from RegistrationForm where Pin_Password = '" + TextBox3.Text + "'", con);
SqlDataReader rs = com.ExecuteReader();
DataSet ds = new DataSet();
while (rs.Read())
{
if (rs["Pin_Password"].ToString().Trim().Equals(TextBox3.Text.Trim()))
{
DateTime dtStart;
DateTime.TryParse(TextBox1.Text, out dtStart);
DateTime dtEnd;
DateTime.TryParse(TextBox1.Text, out dtEnd);
SqlDataAdapter adp = new SqlDataAdapter("select RequestedDate,RequestedPins,ApprovedPins,Action from AdminPins where RequestedDate between '" + dtStart + "' and '" + dtEnd + "'", con);
adp.Fill(ds);//It is giving error at this place
ds.Clear();
GridView1.DataSource = ds.Tables[0];
GridView1.DataBind(); 
}
else
{
Response.Write("Enter Correct Password");
}
}
rs.Close();
com.ExecuteNonQuery();
con.Close();
|
|
|
|
|
You are getting an error here because you have an active operation being performed on this connection, i.e. you are already performing a DataReader operation when you attempt to perform the fill. Your Fill operation needs to operate on a different connection, or you need to enable MARS (Multiple Active Result Sets) on your connection.
BTW - your application should be rewritten to cope with SQL Injection Attacks. You have a very vulnerable design here.
"WPF has many lovers. It's a veritable porn star!" - Josh Smith As Braveheart once said, "You can take our freedom but you'll never take our Hobnobs!" - Martin Hughes.
My blog | My articles | MoXAML PowerToys | Onyx
|
|
|
|
|
I have a config file in xml format which specifies where in the Windows registry to look for the location of my appliation. For example, the xml would specify "HKEY_LOCAL_MACHINE\SOFTWARE\MyTestApp\InstallDir".
If haven't yet created the MyTestApp key in the registry and the below line of code is executed, I get "Object reference not set to an instance of an object" exception.
String filePath =(string) Registry.GetValue("HKEY_LOCAL_MACHINE\SOFTWARE\MyTestApp", "InstallDir", "");
I know I could test for the existence of the RegistryKey first(see below code) but I don't want to hardcode this to the LocalMachine hive. Is there a way to get to the registry hive by name?
RegistryKey rk = Registry.LocalMachine.OpenSubKey("SOFTWARE\\MyTestApp");
|
|
|
|
|
You're getting the exception because Registry.GetValue returns null if the key doesn't exist, and you are casting this null to a string. Instead you should check for null first.
Object keyValue = Registry.GetValue("HKEY_LOCAL_MACHINE\SOFTWARE\MyTestApp", "InstallDir", "");
String filePath;
if(keyValue != null)
{
filePath =(string)keyValue;
}
else
{
}
(By the way, the registry is best avoided now for most stuff. You should consider looking at .net settings or isolated storage instead unless you have to support the registry for backwards compatibility or something like that)
Simon
|
|
|
|
|
Simon Stevens wrote: // Now you know you have a value, so you can cast it to a string.
Not necessarily. You may have a non-null string object but still not have a value, i.e. an empty string. A better approach may be to use String.IsNullOrEmpty
only two letters away from being an asset
|
|
|
|
|
I see your point, and normally I would recommend using .isNullOrEmpty, but in this case the data retrived from the registry is an Object. The check for null is to find out if a non-null value was returned from the registry. If the object is non-null (including empty string) (and assuming that you created the key, so you know that the registry data type is a String type), then you can cast it to a String. Then, you can do String.IsNullOrEmpty to see if the registry value is an empty string if you want, but the point is that the registry key existed, and you have data, even if it's an empty string.
(I suppose, depending how critical the app is, you might want to do a non-failing cast instead just incase someone has changed the registry data type, but this is going a bit beyond the scope of the OP now)
Simon
|
|
|
|
|
Thank you everyone. Checking for null solves my problem.
|
|
|
|
|
Hi fellows,
I´m using the class "BaseEntityWithState" as the base class for lot of aur bussines classes.
Basically it has lots of methods to trak object state (stripped off so only relevanto ones to the problem are shown).
When a child class has one of its porperties modified it changes the state to "modified".
I need to clone the entity and I´m doing that serializing and deserializing the instance.
Supouse I need to clone a child entity wich currentStatus is "unchanges". During the deserialization part of the cloning method, the members of the base class are assigned it´s values properly. Then whe the "Value" property of the child class is set, it changes the currentStatus property of the base class. So the original instance has currentStatus "unchanged" and the cloned one has "modified".
What I need is to be able to avoid that.
I tried setting the Order property of XMLElement atributte to change the serialization order, but the serializer complains that I should specify order for all member of the base an child class. This would be to hard to mantain. Remember there are lots of child classes, each one with several members.
So, the only solution that is working is declaring in the child class, after all other declarations a "currentStatus2" property (commented in the code).
That way this is the last property to get serialized and also the las to get deserialized, propperly setting the currentStatus atributte to the original value.
For this solution to work requires me to add the ficticious property to EVERY child class. Not too much work, but I´m worried that this is not a good solution, cause it requiere other programmers (less techies than me) not to forget to add that property.
I´d like to provide a better solution to this problem.
Thanks in advance
Sebastián Streiger
Imports System.IO
Imports System.Xml
Imports System.Xml.Serialization
Imports System.Reflection
Public Enum estados
unchanged
modified
End Enum
Public Class BaseEntityWithState
Private Shared Function getXmlTypeMapping(ByVal tipo As System.Type) As XmlTypeMapping
Return (New XmlReflectionImporter).ImportTypeMapping(tipo)
End Function
Public Function serialize() As String
Dim writer As New StringWriter
With New XmlSerializer(Me.GetType())
.Serialize(writer, Me)
End With
Return writer.ToString
End Function
Private Shared Function getXmlSerializer(ByVal tipo As System.Type) As XmlSerializer
Return New XmlSerializer(getXmlTypeMapping(tipo))
End Function
Public Function DesSerializar(ByVal tipo As System.Type, ByVal xml As String) As Object
Return New XmlSerializer(tipo).Deserialize(New StringReader(xml))
End Function
Public Function clone() As BaseEntityWithState
Dim aux As BaseEntityWithState = Me.DesSerializar(Me.GetType(), Me.serialize)
Return aux
End Function
Private status As estados = estados.unchanged
Public Property currentStatus() As estados
Get
Return Me.status
End Get
Set(ByVal value As estados)
Me.status = value
End Set
End Property
End Class
Public Class childEntityWithState
Inherits BaseEntityWithState
Private _value As Integer
Public Property value() As Integer
Get
Return Me._value
End Get
Set(ByVal value As Integer)
Me._value = value
Me.currentStatus = estados.modified
End Set
End Property
'Public Property currentStatus2() As estados
' Get
' Return MyBase.currentStatus
' End Get
' Set(ByVal value As estados)
' MyBase.currentStatus = value
' End Set
'End Property
End Class
|
|
|
|
|
What I'm about to propose is not good design, but it may be a simple solution: use a private boolean to inhibit changing the status property while deserializing.
2+2=5 for very large amounts of 2
(always loved that one hehe!)
modified on Wednesday, June 3, 2009 8:22 AM
|
|
|
|
|
Moreno:
How could I realize during property assigment if it´s being done by serialization or by a line of programmer code.
I don´t think it´s good design also, but I don´t know if it´s possible.
Keep in mind that this class can be serialized within other classes serializacion, so I can´t make something like:
original.TrackStatus = false
cloned = original.clone
original.TrackStatus = true
cloned.TrackStatus = true
Anyway, thank you very much
Sebastián Streiger
|
|
|
|
|
Just pretend I didn't post anything ok ? What I said up to now is just too stupid haha. 
But really, I couldn't come up with any serious way to do what you need. Tweaking the DesSerializar or Clone routines is sure the right way to go, but I can't come up with a good way of storing the serialized object's status and then pasting that into the deserialized object.
Hope some good guy here will offer a good solution.
2+2=5 for very large amounts of 2
(always loved that one hehe!)
|
|
|
|
|
Moreno:
He, He,
I also hope we can get a clean solution to this problem.
Your feedback is highy appreciated.
If I manage to find a bettter way I´ll let you know.
Thank you very much.
Sebastián
|
|
|
|
|
Hummm I either was too dumb yesterday or I'm missing something now. 
I think what you need is simply:
Public Function clone() As BaseEntityWithState
Dim statusbuffer As estados = Me.status
Dim aux As BaseEntityWithState = Me.DesSerializar(Me.GetType(), Me.serialize)
aux.status = statusbuffer
Return aux
End Function
Hope this time it's useful to you.
2+2=5 for very large amounts of 2
(always loved that one hehe!)
|
|
|
|
|
Moreno:
Is that your Name?
What you suggest is the first thing I tried, but It only worked with the object that is getting called clone.
It doesnt work if that object contains references to other chils of the same base class. I mean, it works for the object being cloned, but not for the references it has.
I´m working now in a solution suggested by a collegue by implementing the IXMLSerializable interface.
I´ll let you know what comes up.
Sebastián
|
|
|
|
|
Yep it's my name, I know it's a common family name for Spanish and latinos.
It sure works only for the object on which you call clone, and not for references.
Best of luck with implementing the interface then.
At first I also reasoned on the line of parsing the XML and getting the status directly from there, in order to apply it to the cloned object(s) after de-serialization, but I find it too much of a patchy solution.
Well, best of luck again, and I'll be waiting to hear how it goes. 
2+2=5 for very large amounts of 2
(always loved that one hehe!)
|
|
|
|
|
Hi
I have a problem. I am developing a VoIP application using DirectSound to capture and play voice. Everything is OK with the capturing, but I can't play the received data smoothly when it is a small piece. It should be small for preventing latency. Currently I capture 320 bytes and send it via UDP. The wave format is 8000 Hz and 8000 B/S.
For reproducing the problem I use the following code:
int step = 320, c = 0, N = 2;
offset = data.Length % step;
// (SecondaryBuffer) buf size is N * step;
buf.Play(0, BufferPlayFlags.Looping);
byte[] smallArr = new byte[step];
double sleep = (double)step / waveFormat.AverageBytesPerSecond * 1000;
//data is a captured sound saved to a file previously
while (data.Length > offset)
{
buf.SetCurrentPosition(((c + 1) % N) * step);
Array.Copy(data, offset, smallArr, 0, step);
buf.Write(c*step, smallArr, LockFlag.None);
Thread.Sleep((int)sleep);
offset += step;
c++;
c%=N;
}
While debugging I found out that the PlayPosition cursor doesn't reach the expected point. I move it forward and the result is that not all data is played and the sound is not smooth.
There is no problem when the playing portion is bigger but I can't afford latency.
When I use a lot higher quality(and play bigger portion of data) there is no problem again but the traffic is large!
I can't use more than one buffers because the noise is awful when play/stop/mute.
So, would you help me to solve the problem with the playing buffer?
Thank you in advance!
|
|
|
|
|
Andrey U wrote: While debugging I found out that the PlayPosition cursor doesn't reach the expected point. I move it forward and the result is that not all data is played and the sound is not smooth.
That's not a good idea. Any change to the play position will affect the output. The results from GetCurrentPosition may be incorrect / inaccurate due to the way DirectSound works (especially on XP).
Your basic idea of double buffering is good, but you need to manage it in a more reliable way. The "Sleep" in particular, since it's the same length as the time to play one buffer-full, doesn't allow any time for the data copying. (You should also try to only have one copy / write for efficiency.)
My approach is similar, but I check when the play cursor switches from one buffer to the next (either half way along the double buffer, or looping back to the start) and then I copy into the other half. Use a smaller sleep time, and check regularly.
I should say "my approach was similar..." though, as I've now switched over to using XAudio2, which is much better for managing streaming.
Andrey U wrote: I can't use more than one buffers because the noise is awful when play/stop/mute.
That shouldn't happen - it sounds like something else is going wrong in that case.
There are three kinds of people in the world - those who can count and those who can't...
|
|
|
|
|
molesworth wrote: That's not a good idea. Any change to the play position will affect the output. The results from GetCurrentPosition may be incorrect / inaccurate due to the way DirectSound works (especially on XP).
I know that the play position affects the output, but in my real application I receive packets and I can't wait.
I use PlayPosition property only for finding out what happens, so I don't rely on it.
molesworth wrote: Your basic idea of double buffering is good, but you need to manage it in a more reliable way. The "Sleep" in particular, since it's the same length as the time to play one buffer-full, doesn't allow any time for the data copying. (You should also try to only have one copy / write for efficiency.)
I've tried different types of buffering and the only way to improve the output is to use bigger buffer, but this means more latency...
molesworth wrote: That shouldn't happen - it sounds like something else is going wrong in that case.
It happens! I've tried with an empty buffer and when I call play,stop,etc. I hear this awful pop.
I've just read in another forum that this problem migth be caused by "The Windows resampler", which works better at certain rates and bad at 8 kHz. But how could I avoid this?
|
|
|
|
|
Andrey U wrote: I know that the play position affects the output, but in my real application I receive packets and I can't wait.
I use PlayPosition property only for finding out what happens, so I don't rely on it.
Yes, but when you call "SetCurrentPosition" you're moving the play cursor, which will cause jumps / glitches in your audio output. If you're playing at the same rate you're receiving data (which you should be) then you should be able to let the buffer play continually without having to move the cursor.
Andrey U wrote: I've tried different types of buffering and the only way to improve the output is to use bigger buffer, but this means more latency...
Well, your buffers are very small, and about 40ms (1/25th sec) long. For a voice chat or VoIP application you should be able to go up to 100ms or even 200ms without any problem. Nobody is going to notice those kinds of delays in voice only communications.
I'm not sure why you're getting popping on playing empty buffers - if you've zero-filled them they should play silence. Starting and stopping waves which aren't on zero-crossings can cause pops, but dealing with that is another matter.
Andrey U wrote: I've just read in another forum that this problem migth be caused by "The Windows resampler", which works better at certain rates and bad at 8 kHz. But how could I avoid this?
There shouldn't be any problem playing at 8 KHz, and I've just run a quick test in my app which works fine. It sounds pretty bad playing music, but it does play smoothly. I suspect your problems are more to do with your Sleep timer and moving the play cursor.
I'd go for something more like :-
while (stillPlaying)
{
CheckForInputData();
playPos = GetCurrentPosition();
if (playPos just crossed half-buffer)
{
FillOtherHalfBuffer();
}
Sleep(10);
}
There are three kinds of people in the world - those who can count and those who can't...
|
|
|
|
|
I can pretty much guarantee you will have problems with this approach.
First, looping and using Sleep for synchronization is going to be nothing
but problematic. You can use SetNotificationPositions() to have an event
signaled when the playback buffer reaches certain positions - that event can wake
a waiting thread which can take appropriate action to provide more playback data.
Much more efficient and no Sleep required.
Second, on the playback end, you need to deal with jitter from at least two main sources:
1) Network. If you're transferring audio over a network it is NOT always going to
arrive in an evenly timed stream - it will come in chunks and at different intervals,
often with variations larger than 25ms, which I'm assuming is your "packet" time.
2) Clock drift. Clocks on different machines are never the same. Close, but close
doesn't count here. As one machine streams data at its 8000KHz, the playback machine
tries to play the data at its 8000Hz, but with different clocks, they (usually pretty
rapidly) get out of sync - the playback machine either gets data faster than it is consuming
it, or consumes data faster than it is receiving it. This HAS to be accounted for, unless
all conversations are going to be very short - like a few seconds.
I would really recommend redesigning your playback end. Use directsound buffer notifications
and use threads smartly. I personally have data read from the network on one thread which is
pushed into a jitter buffer. Another thread waits on buffer notifications, and when it wakes,
pulls data from the jitter buffer and copies it to the playback buffer. The jitter buffer
can be implemented any way you want - it just needs to provide data when none is available, or
deal with too much data available.
Good luck.
Mark Salsbery
Microsoft MVP - Visual C++
![Java | [Coffee]](https://codeproject.global.ssl.fastly.net/script/Forums/Images/coffee.gif)
|
|
|
|
|
Thank you very much!
Let me show you what I did.
Playing method:
(I've set the buffer to notify me at the half and at the end)
public void PlayingInThread()
{
NotifyPlaying();
playBuffer.Play(0, BufferPlayFlags.Looping);
bool flag = true;
byte[] data;
while (true)
{
arePlaying.WaitOne();
data = jitter.Get();
if (flag)
{
playBuffer.Write(playBufferBytes / 2, data, LockFlag.None);
}
else
{
playBuffer.Write(0, data, LockFlag.None);
}
flag = !flag;
}
}
And I wrote a very simple jitter class, which uses a queue:
class JitterBuffer
{
Queue<byte[]> mBuffer = new Queue<byte[]>();
int mItemLength;
public JitterBuffer(int length)
{
mItemLength = length;
}
public void Put(byte[] data)
{
mBuffer.Enqueue(data);
}
public byte[] Get()
{
if (mBuffer.Count == 0)
return new byte[mItemLength];
else
return mBuffer.Dequeue();
}
}
Am I right up to this point?
Testing this code I saw that there is aproximately one more packet (which can't be played because of the slower playing!) received every second.
So my next question is how to deal with 'too much data available'?
To be honest, I'm not very good at threads and I don't know if it is a good solution but I've tried the following code:
public void CatchUp()
{
if (playBuffer.PlayPosition >= playBufferBytes / 2)
playBuffer.SetCurrentPosition(playBufferBytes-2);
else
playBuffer.SetCurrentPosition(playBufferBytes / 2-2);
}
And I call this method from JitterBuffer.Put() when there are more then 2 packets in the queue. As you can guess in this case I face my initial problems with 'not smooth output'. Maybe I should only drop a packet from the queue when there are a lot?
(What bothers me about the threads is that I manipulate playBuffer from 2 threads in this case. Am I supposed to do it?)
And my last question: should I rely on the notification from a secondary buffer? I've read several times that there are bugs with it.
Regards
|
|
|
|
|
Andrey U wrote: Am I right up to this point?
Yep, that looks like a good way to do things. As Mark says, threading is definitely the right approach (I previously just posted a simplistic method since your original post just had a simple closed play loop). Threading may look complicated, but once you've done a couple of trials you'll probably see that it's fairly simple really 
You should perhaps zero-fill your empty buffer in the case where there's no received data, however. It may be cleared in debug builds, but probably will be full of random data (noise) in release.
Andrey U wrote: Maybe I should only drop a packet from the queue when there are a lot? (What bothers me about the threads is that I manipulate playBuffer from 2 threads in this case. Am I supposed to do it?)
You definitely shouldn't operate on the play buffer in anything other than your play thread. I'd also advise against moving the play cursor, since this will definitely cause glitches, and may also cause other problems in playback.
For overruns you can either drop packets, which will also give glitches, although more manageable ones, or you can just play all data - ideally your transmitting app should detect silence and stop sending data, giving the receiver time to catch up during those pauses.
Your buffer size and the reliability of the connection will determine how often you get overruns or underruns, and it's a trade-off between several factors to determine the best size to use. As Mark mentioned, you need to deal with all of these to make a reliable application.
Andrey U wrote: And my last question: should I rely on the notification from a secondary buffer? I've read several times that there are bugs with it.
I've had problems with notifications myself. My last DirectSound system used GetCurrentPosition to check the play cursor at regular intervals, and I found this was much more reliable. (My current project is using XAudio2, which seems to be a lot better.)
There are three kinds of people in the world - those who can count and those who can't...
|
|
|
|
|
Will all modern versions of Windows honor small changes in playback speed, or are they apt to be rounded up to larger increments? If the sampling PC samples at a constant 8,000Hz, would it be possible to have the playback PC output at a rate slightly faster or slower based upon whether it is getting ahead of or behind the data stream? One would have to be careful, of course, to avoid varying the speed so much as to cause warbling, but I would expect two system clocks should be easily within 0.1% of each other.
|
|
|
|
|
Playback frequency on DSound buffers can be set to integer accuracy, and they'll play properly at whatever pitch is selected.
I'm not sure this is a good approach however. Accurately measuring the rate at which network data is arriving could be pretty difficult, especially since it's arriving in reasonable sized packets, and making continual changes to the frequency could, as you say, cause noticeable unpleasant artefacts.
I've never written a VoIP app myself*, so can only advise in a theoretical way, but I'd go with the approach of playing all data as it arrives, and using silence detection on the transmitting side to signal gaps in the stream during which the playing side can "catch up".
That would be for small (e.g. < 250ms) gaps - for larger ones I'd probably drop some packets, and live with the glitches. VoIP isn't trying to achieve CD quality sound, just reasonably comprehensible speech. Sampling at 8 bit, 8KHz is going to reduce quality quite a lot anyway...
[* My current system is dealing with horrible latencies, multiple streams and multiple buffers, but I'm taking a different approach to dealing with these problems.]
There are three kinds of people in the world - those who can count and those who can't...
|
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
![Java | [Coffee]](https://www.codeproject.com/script/Forums/Images/coffee.gif)