Solved

Problem on winsock connection

Posted on 2001-07-03
14
319 Views
Last Modified: 2013-11-13
I'm using winsock component (tcpip) for communication between pc to pc. One end as winsock client and other end as winsock server. Everything find, i can connect, send/receive data and close connection.

Problem arise when, multiple winsock client try to connect to winsock server. Only one client can connect at one time.

How to make multiple connection possible?
0
Comment
Question by:azsoft
  • 4
  • 2
  • 2
  • +6
14 Comments
 
LVL 5

Expert Comment

by:dredge
ID: 6249550
i just solved this same problem about 2 weeks ago.

it's easy.

create your WSock object, and set the Index value to 0 (zero) (this creates a Control Array)

in your code:

Private Sub Form_Load()
  wsock(0).localPort=80 'or whatever port you want
  wsock(0).listen
End Sub

'this will allow multiple connections
Private Sub WSock_ConnectionRequest(Index as Integer, RequestID as Integer)

'make sure that wsock 0 is listening
if index=0 then
  dim counter as integer
  counter = wsock.ubound+1
  load WSock(counter)
  wsock(counter).localport=0
  wsock(counter).accept(requestID)
end if



from now on, in any of your event codes, you will have the Index value to use. just make sure you always specify the index when using the WSock object.

WSock(Index).sendata "hello"

etc. etc.
0
 
LVL 5

Expert Comment

by:dredge
ID: 6249555
if you post your email address, I'll send you over an actual VB6 source code that does all this.
0
 

Author Comment

by:azsoft
ID: 6249603
showmefirst@usa.net
0
 
LVL 1

Expert Comment

by:Wippie
ID: 6249681
Just wondered:
Why use the 0-index control to listen to the port?
Why not use one control to listen to the port and another with a object array to handle all connections ??
Wouldn't that give some more controlable code?

BUG:
First bug i saw i the code printed was that the winsock-control never was unloaded!
Let's say we made a nice little http-server out of this.
It runs without a problem in, let's say, a week.
Imaging the number of winsock-controls which is loaded into the memory by now.
Anyway, it's pretty easy to solve this.
Just unload the control then connection closes and then you a new control, loop throw the object array and reload the first one missing.
0
 
LVL 5

Expert Comment

by:dredge
ID: 6249703
I just sent it.
0
 
LVL 5

Expert Comment

by:dredge
ID: 6249712
wippie:

by using the 0 index, you save on memory and other cpu resources.

that was just an example, of course i didn't close the winsock. there's not even any code in there to do anything aside from verify that you actually made a connection.


azsoft:

load the Host onto your computer and run it, then goto a couple of other computers and load the Client, and have the clients connect to the host at the same time.
0
 
LVL 16

Accepted Solution

by:
Richie_Simonetti earned 100 total points
ID: 6249823
I can't post the url here (Later i will do). It is from Planet source code i think.
This is a complete form:

VERSION 5.00
Object = "{248DD890-BB45-11CF-9ABC-0080C7E7B78D}#1.0#0"; "MSWINSCK.OCX"
Begin VB.Form Form1
   AutoRedraw      =   -1  'True
   Caption         =   "Form1"
   ClientHeight    =   3195
   ClientLeft      =   60
   ClientTop       =   345
   ClientWidth     =   4680
   LinkTopic       =   "Form1"
   ScaleHeight     =   3195
   ScaleWidth      =   4680
   StartUpPosition =   3  'Windows-Standard
   Begin VB.Timer Timer1
      Interval        =   200
      Left            =   1560
      Top             =   1440
   End
   Begin MSWinsockLib.Winsock Winsock1
      Index           =   0
      Left            =   2160
      Top             =   1440
      _ExtentX        =   741
      _ExtentY        =   741
      _Version        =   393216
   End
End
Attribute VB_Name = "Form1"
Attribute VB_GlobalNameSpace = False
Attribute VB_Creatable = False
Attribute VB_PredeclaredId = True
Attribute VB_Exposed = False
Dim i As Long
Dim Con As Long

Private Sub Form_Load()
i = 0
Con = 0
Winsock1(0).LocalPort = 80
Winsock1(0).Listen
End Sub

Private Sub Timer1_Timer()
Me.Caption = "Verbindungen: " & Con
End Sub

Private Sub Winsock1_Close(Index As Integer)
If Index = 0 Then
Winsock1(0).Close
DoEvents
Winsock1(0).Listen
DoEvents
Else
Winsock1(Index).Close
DoEvents
End If
Con = Con - 1
End Sub

Private Sub Winsock1_ConnectionRequest(Index As Integer, ByVal requestID As Long)
If Index = 0 Then
i = i + 1
Con = Con + 1
Load Winsock1(i)
Winsock1(i).LocalPort = 0
DoEvents
Winsock1(i).Accept requestID
DoEvents
End If
End Sub

Private Sub Winsock1_DataArrival(Index As Integer, ByVal bytesTotal As Long)
Winsock1(Index).GetData newData$
DoEvents
End Sub

Private Sub Winsock1_Error(Index As Integer, ByVal Number As Integer, Description As String, ByVal Scode As Long, ByVal Source As String, ByVal HelpFile As String, ByVal HelpContext As Long, CancelDisplay As Boolean)
Winsock1(Index).Close
DoEvents
Con = Con - 1
End Sub
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 

Expert Comment

by:evilTeach
ID: 6252931
I've done this myself a couple of times...

I've always done what Wippie is talking about.

Have one control: wskListen that receives ONLY connection requests.  Once a machine requests a connection I create a new control from a wskConnection Control Array.  This is where all future communication occurs.

To save on resources, when a user disconnects, I have a string that holds the comma-seperated indices of available controls.  This way I can see if there's an available unused control so I don't have to create another.

There is ONE thing that hasn't been mentioned yet.  All of this should work, IN THEORY.  But there's a problem with the control.  If you try to send out information over all of the controls you will find that it will not be received by everyone.  There's an easy way to resolve this.  A couple of well placed DoEvents will take care of it.

Here's an example:

(If playerID = 0 then a message is sent to all users, otherwise it's sent only to 1 player.  Note the use of DoEvents)

Public Sub SendMessage(strMessage As String, Optional _
                      playerID As Integer = 0, Optional _
                      newLine As Boolean = True)
    If newLine Then
        strMessage = strMessage & vbCrLf
    End If
   
    strMessage = vbCrLf & strMessage
   
    If playerID <> 0 Then
        If player(playerID).Status >= PLR_NAME Then
            frmMain.wskPlayer(playerID).SendData strMessage
            DoEvents
        End If
    Else
        Dim i As Integer
        For i = 1 To frmMain.wskPlayer.UBound
            If player(i).Status >= PLR_NAME Then
                frmMain.wskPlayer(i).SendData strMessage
                DoEvents
            End If
        Next
    End If

End Sub

Hope that helps!
0
 

Expert Comment

by:evilTeach
ID: 6252938
Oops...sorry Richie, just noticed you mentioned the DoEvents in your sourcecode first :)
0
 
LVL 16

Expert Comment

by:Richie_Simonetti
ID: 6252970
That's OK!
First, it is not my code.
Second, indentation, indentation, indentation,...!
0
 
LVL 2

Expert Comment

by:TravisHall
ID: 6253896
Just as an aside...

With at least some versions of the Winsock control, that component leaks memory when removed from a control array. There's an MSDN article about it somewhere. It may have been corrected in recent versions of the control, for all I know, but it is something to watch out for.

When I needed to do something with a dynamic number of winsock controls a while back, I created my app as an MDI app, put winsock controls on MDI child windows and loaded and unloaded those windows as required. With the unloading of the child window, all the resources of its controls are cleaned up, and the memory leak doesn't happen. It worked very well.

In that case, I had other reasons to use that structure, though. I didn't actually find out about the memory leak problem until later. When I did, I stopped and looked carefully at my program's structure, and determined that I had already sidestepped the problem.
0
 
LVL 1

Expert Comment

by:Moondancer
ID: 6979338
ADMINISTRATION WILL BE CONTACTING YOU SHORTLY.  Moderators Computer101, Netminder or Mindphaser will return to finalize these if they are still open in 7 days.  Experts, please post closing recommendations before that time.

Below are your open questions as of today.  Questions which have been inactive for 21 days or longer are considered to be abandoned and for those, your options are:
1. Accept a Comment As Answer (use the button next to the Expert's name).
2. Close the question if the information was not useful to you, but may help others. You must tell the participants why you wish to do this, and allow for Expert response.  This choice will include a refund to you, and will move this question to our PAQ (Previously Asked Question) database.  If you found information outside this question thread, please add it.
3. Ask Community Support to help split points between participating experts, or just comment here with details and we'll respond with the process.
4. Delete the question (if it has no potential value for others).
   --> Post comments for expert of your intention to delete and why
   --> YOU CANNOT DELETE A QUESTION with comments; special handling by a Moderator is required.

For special handling needs, please post a zero point question in the link below and include the URL (question QID/link) that it regards with details.
http://www.experts-exchange.com/jsp/qList.jsp?ta=commspt
 
Please click this link for Help Desk, Guidelines/Member Agreement and the Question/Answer process.  http://www.experts-exchange.com/jsp/cmtyHelpDesk.jsp

Click you Member Profile to view your question history and please keep them updated. If you are a KnowledgePro user, use the Power Search option to find them.  

Questions which are LOCKED with a Proposed Answer but do not help you, should be rejected with comments added.  When you grade the question less than an A, please comment as to why.  This helps all involved, as well as others who may access this item in the future.  PLEASE DO NOT AWARD POINTS TO ME.

To view your open questions, please click the following link(s) and keep them all current with updates.
http://www.experts-exchange.com/questions/Q.20107188.html
http://www.experts-exchange.com/questions/Q.20131426.html
http://www.experts-exchange.com/questions/Q.20134727.html
http://www.experts-exchange.com/questions/Q.20141831.html
http://www.experts-exchange.com/questions/Q.20145275.html
http://www.experts-exchange.com/questions/Q.20145277.html
http://www.experts-exchange.com/questions/Q.20196746.html
http://www.experts-exchange.com/questions/Q.20249690.html
http://www.experts-exchange.com/questions/Q.20255224.html
http://www.experts-exchange.com/questions/Q.20287293.html
http://www.experts-exchange.com/questions/Q.20287915.html
http://www.experts-exchange.com/questions/Q.20292953.html
http://www.experts-exchange.com/questions/Q.20293816.html

To view your locked questions, please click the following link(s) and evaluate the proposed answer.
http://www.experts-exchange.com/questions/Q.20187315.html

*****  E X P E R T S    P L E A S E  ******  Leave your closing recommendations.
If you are interested in the cleanup effort, please click this link
http://www.experts-exchange.com/jsp/qManageQuestion.jsp?ta=commspt&qid=20274643 
POINTS FOR EXPERTS awaiting comments are listed in the link below
http://www.experts-exchange.com/commspt/Q.20277028.html
 
Moderators will finalize this question if in @7 days Asker has not responded.  This will be moved to the PAQ (Previously Asked Questions) at zero points, deleted or awarded.
 
Thanks everyone.
Moondancer
Moderator @ Experts Exchange
0
 
LVL 49

Expert Comment

by:DanRollins
ID: 7147983
Hi azsoft,
It appears that you have forgotten this question. I will ask Community Support to close it unless you finalize it within 7 days. I will ask a Community Support Moderator to:

    Accept Richie_Simonetti's comment(s) as an answer.

azsoft, if you think your question was not answered at all or if you need help, just post a new comment here; Community Support will help you.  DO NOT accept this comment as an answer.

EXPERTS: If you disagree with that recommendation, please post an explanatory comment.
==========
DanRollins -- EE database cleanup volunteer
0
 
LVL 1

Expert Comment

by:Computer101
ID: 7182112
Comment from expert accepted as answer

Computer101
E-E Moderator
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Go is an acronym of golang, is a programming language developed Google in 2007. Go is a new language that is mostly in the C family, with significant input from Pascal/Modula/Oberon family. Hence Go arisen as low-level language with fast compilation…
This article is meant to give a basic understanding of how to use R Sweave as a way to merge LaTeX and R code seamlessly into one presentable document.
The goal of this video is to provide viewers with basic examples to understand and use conditional statements in the C programming language.
The viewer will learn how to clear a vector as well as how to detect empty vectors in C++.

911 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

18 Experts available now in Live!

Get 1:1 Help Now