You are not logged in.

  • Login
Bitte besucht unser neues Forum unter https://forum.nitrado.net | Please visit our new Forum at https://forum.nitrado.net

BananaCraftingLP

User / Kunde

Posts: 836

Thanks: 51

  • Send private message

21

Sunday, October 21st 2012, 10:36am

Problem

Hm ganz nette Idee, aber wegen dem Port Problem werden alle Server als offline angezeigt. Benutzt du ganz normal fsockopen() ? Weil was ist denn das Problem mkt dem Port?

miner345

User / Kunde

  • "miner345" is male
  • "miner345" started this thread

Posts: 361

Location: NRW

Occupation: Auszubildender

Thanks: 19

  • Send private message

22

Sunday, October 21st 2012, 10:43am

Ich versteh das auch nicht so ganz....
Der Standardport 25565 wird ja angezeigt.
Der dazu passende Teil des Codes:

PHP Source code

1
2
3
4
if(isset($_GET['port'])) $port $_GET['port'];
    else $port 25565;

fsockopen($ip$port$errno$errstr4);

BananaCraftingLP

User / Kunde

Posts: 836

Thanks: 51

  • Send private message

23

Sunday, October 21st 2012, 11:42am

Da fehlt was :D

PHP Source code

1
2
3
4
5
6
7
if(isset($_GET['port'])) {
    $port $_GET['port'];
} else {
    $port '25565';
}

fsockopen($ip$port$errno$errstr4);

CybroX

User / Kunde

  • "CybroX" is male

Posts: 6,249

Occupation: Gott

Thanks: 737

  • Send private message

24

Sunday, October 21st 2012, 1:37pm

Wieso in Anfängermethode ? :P

PHP Source code

1
$port = (!empty($_GET['port'])) ? $_GET['port'] : '25565' ;


Ausserdem muss man @fsockopen nutzen um eine sichtbare Fehlermeldung zu unterbinden.
You depend on our protection, yet you feed us lies from the tablecloth.

This post has been edited 1 times, last edit by "CybroX" (Oct 21st 2012, 4:54pm)


BananaCraftingLP

User / Kunde

Posts: 836

Thanks: 51

  • Send private message

25

Sunday, October 21st 2012, 4:49pm

PS: Ich würde den timeout höher stellen :D Viele Server, also vor allem Minecraft Server haben extrem schlechte Pings, aber ich glaube das ist Geschmackssache.

miner345

User / Kunde

  • "miner345" is male
  • "miner345" started this thread

Posts: 361

Location: NRW

Occupation: Auszubildender

Thanks: 19

  • Send private message

26

Sunday, October 21st 2012, 6:10pm

Da fehlt was
Nö tut es nicht...

Wieso in Anfängermethode ?
Wieso nicht? :D
Ich würde den timeout höher stellen
Also bisher gab es damit keine Probleme, ich guck mal ;)

BananaCraftingLP

User / Kunde

Posts: 836

Thanks: 51

  • Send private message

27

Sunday, October 21st 2012, 7:23pm

Quoted

Da fehlt was
Nö tut es nicht...


Doch, die Klammern. Ich weiß nicht ob man die hier unbedingt bei nur einer Zeile braucht, aber allein der übersichtlichkeit halber :D

miner345

User / Kunde

  • "miner345" is male
  • "miner345" started this thread

Posts: 361

Location: NRW

Occupation: Auszubildender

Thanks: 19

  • Send private message

28

Sunday, October 21st 2012, 10:05pm

Man braucht es auf jeden Fall nicht und ich glaube was übersichtlich ist, muss jeder Programmierer für sich selbst entscheiden.
Zu langer Code durch unnötige Zeilen kann auch durchaus zu Unübersichtlichkeit führen, jedoch zu kurzer Code ebenso.
Hier gibt es ja ganz schön 3 Möglichkeiten, wie man diese Codezeile darstellen kann. (1-5 Zeilen)
Wie man das nun letzendlich schreibt, muss jeder für sich entscheiden.
Vor allem bei diesem Code habe ich mir dabei nicht allzuviele Gedanken gemacht, deshalb ist es auch so unordentlich ^^.

xama

User / Kunde

  • "xama" is male

Posts: 3,639

Location: Zürich, Schweiz

Occupation: Webdesigner

Thanks: 18

  • Send private message

29

Sunday, October 21st 2012, 10:29pm


fensterbrett

User / Kunde

  • "fensterbrett" is male

Posts: 1,838

Location: ::1, sweet ::1

  • Send private message

30

Sunday, October 21st 2012, 10:33pm

-entfernt-
Kein Support per PN.

This post has been edited 1 times, last edit by "fensterbrett" (Oct 21st 2012, 10:39pm)


miner345

User / Kunde

  • "miner345" is male
  • "miner345" started this thread

Posts: 361

Location: NRW

Occupation: Auszubildender

Thanks: 19

  • Send private message

31

Tuesday, October 23rd 2012, 10:02pm

#push

Bitte besucht unser neues Forum unter https://forum.nitrado.net| Please visit our new Forum at https://forum.nitrado.net

Similar threads