gift to all networking noobs (as i am)...

[daemonna]

after finding lot of helpfull people here, i've decided to give something back (actually my own mutant-fork of closed source project), so i gave birth to

http://code.google.com/p/elementalserver/

and i hope this code will help many noobs to learn how does qt networking work.

[wysota]

Yes, of course it's threaded.... Geeeeez people, what's with you!?

[daemonna]

if you don't like threads, you can help write single-threaded version

[wysota]

I do like threads but not in this context. By giving such a code to people you are encouraging them to learn bad habits, this code is just wrong (both conceptually and technically), overbloated and unnecessary. If you want to give someone an example then make it simple.

[daemonna]

this code is just wrong (both conceptually and technically), overbloated and unnecessary.

this code is just wrong (both conceptually and technically), overbloated and unnecessary.

To copy to clipboard, switch view to plain text mode 

how can i know if something is wrong? only from people like you... but just to say 'it's wrong' it's not enough. What is wrong? How can it be improved? Do you have specific workaround for this and that?

...if NOT, it's just useless non-productive talk from your side.

By giving such a code to people you are encouraging them to learn bad habits

By giving such a code to people you are encouraging them to learn bad habits

To copy to clipboard, switch view to plain text mode 

No! this is GOOGLE CODE! i dont encourage anyone to anything... you can take it or leave it.... or help improve it (best choice). I'm not saying "this is best software on planet", but saying "this is my piece of code, please, take a look and help improve.. and if you like it, share it."... coz this will be software for YOU people, not only for me....

so please, dont forget you have to get hands dirty, before talking dirty... :P

[wysota]

this code is just wrong (both conceptually and technically), overbloated and unnecessary.

this code is just wrong (both conceptually and technically), overbloated and unnecessary.

To copy to clipboard, switch view to plain text mode 

how can i know if something is wrong?

If you don't feel confident enough that your code is not wrong, don't post it as an "example" to others.

only from people like you...

You should know it based on your knowledge.

Do you have specific workaround for this and that?

It's not a matter of workarounds, it's a matter of concept.

...if NOT, it's just useless non-productive talk from your side.

I already told you my doubts in another thread. And my (and not only mine) opinions about such network programming are scattered all over this forum. If that's not constructive enough for you, you'll have to wait about a month.

By giving such a code to people you are encouraging them to learn bad habits

By giving such a code to people you are encouraging them to learn bad habits

To copy to clipboard, switch view to plain text mode 

No! this is GOOGLE CODE!

No, this is QtCentre forum:

and i hope this code will help many noobs to learn how does qt networking work.

[daemonna]

i got your point, but as i said before..

without your contribution , your comments are useless for me...

without your contribution , your comments are useless for me...

To copy to clipboard, switch view to plain text mode 

looks like you're coming from windows userbase and have no clue about contribution and sharing for FREE... do you?

my (and not only mine) opinions about such network programming are scattered all over this forum

my (and not only mine) opinions about such network programming are scattered all over this forum

To copy to clipboard, switch view to plain text mode 

that's right, and that's why these opinions are useless, coz they are scattered all over and are just opinions without showing something 'that works' and is 'not wrong and bloated'. Or maybe it's problem with search engine, because if i put there 'single thread server', i'll find just questions without actuall answers. i thought this suppose to be forum TO HELP, not to flame, argue and blame someone 'your design is wrong'....

if you have at least little will to help, download code... fix what you think can be fixed or make a 'new design' document (or whatever) , and mail me changes. dont forget it's your choice to IMPROVE IT, CONTINUE with your useless 'it's wrong'-attitude (coz saying 'it's wrong' is no help at all) or simply IGNORE it.... so make your choice!

[wysota]

looks like you're coming from windows userbase

No, not really.

and have no clue about contribution and sharing for FREE... do you?

Forgive me, I will not answer this question. You should be able to find the answer to this one on your own.

[nish]

i always like ppl who post their code on to the web. so gud job @daemonna. But the guy u r talking to is very experienced, so take his comments as positive.

[daemonna]

Forgive me, I will not answer this question.

there's nothing to forgive wysota... it's your choice to contribute or not.. and i dont blame if you will not contribute. but i simply dont accept "this is wrong" statement without proper explanation and piece of code that WORKS good way. Open source community is live only thanks to people that come and give SOLUTION, not a people that come, complain and go...

so dont take me wrong, i'm only gives here something that works (maybe wrong way, but it works) and can be changed by anyone to be better and to prosper to community.

People! i'm not trying to put blame/flame/p*** on wysota head... NO! i know he's VERY experienced and probably knows more than anyone else, but saying only "THIS IS NOT GOOD" doesnt make him any better than any flamer. I ask for help, not personal help which gains me money, but help for OPEN SOURCE project to make it better (coz i'm not guru at all) and saying only "it's wrong" is like telling african children "go and find some food if you're hungry"... it's NO HELP AT ALL.

[wysota]

Forgive me, I will not answer this question.

there's nothing to forgive wysota... it's your choice to contribute or not..

Yes, it is my choice to contribute or not I'm happy I made the right choice.

but i simply dont accept "this is wrong" statement without proper explanation

I already gave you a proper explanation.

and piece of code that WORKS good way.

There is such a piece of code already in Qt docs.

so dont take me wrong, i'm only gives here something that works (maybe wrong way, but it works) and can be changed by anyone to be better and to prosper to community.

I'm sorry, but in my opinion each existing wrong solution puts people one more step away from finding the proper solution. I just don't share your point of view that a poor solution is better than no solution. Sooner or later this poor solution will explode in your face and you'll waste more time trying to patch it up than you'd spend thinking how to do it properly in the first place.

saying only "THIS IS NOT GOOD" doesnt make him any better than any flamer.

Suit yourself, I didn't want to make you feel bad but if you insist:
- your application is leaking memory
- your server is prone to DOS attacks
- you are using shared memory for something that is completely irrelevant to the server and thus it makes no sense to put it in this 'example'
- your shared memory thread has no exit condition (so your program is likely to never finish)
- using a thread for your shared memory stuff is a pure waste of resources, the same could be achieved without using sleep, i.e using idle processing
- you are using signals and slots with QThread in a wrong way
- you are needlessly subclassing QThread
- I have no idea why you operate on byte arrays in such a complicated way
- you are putting literally everything into threads, even single method calls (even the non-blocking QTcpServer::listen()), God only knows why
- you are using arbitrary values for.... yes, I have no idea why you are doing that...
- you are using global/extern variables instead of encapsulating them properly (and you are using them across threads)
- your archive has the size of almost 1.5MB, a bit much as for an "elementary example"
- you should really read this: Keeping your GUI Responsive (just be careful, it's written by someone who doesn't contribute)
- and this one too: You're doing it wrong

If you want to share code -- good, I don't mind. But don't call it an "example". An example should be short, to the point and flawless.

[daemonna]

finally Guru said something wise....


THANKS!

[JohannesMunk]

Daemonna, it's obscene to tell wysota with 19k messages and >2k thanks, that he doesn't know about contribution and sharing.

Put that attitude back where it came from!

Johannes