[DLL] Elitepvpers Library v2

[Sedrika]

Originally Posted by Che​

In welcher Sprache hast du die Bibliothek denn geschrieben? Wie sendest du die Anfragen ab? (über nen XMLRPC-Client, also über die Tapatalk API oder ganz normal?)

Werd mir das ganze morgen oder so mal genauer ansehen, die Idee finde ich jedoch nicht schlecht.

Da ich geschrieben habe, dass es mit C# nicht geht, sollte man von Visual Basic ausgehen.

Ich nutze weder die Tapatalk API (noch nie angesehen, wär eig mal nicht schlecht) oder den XML...-Client. Das meiste sind einfache WebRequests, die ich nutze.

[Che]

Originally Posted by Sedrika

Da ich geschrieben habe, dass es mit C# nicht geht, sollte man von Visual Basic ausgehen.

In der Regel würde man eher von Hochsprache ausgehen, anhand deiner Beispiele hatte ichs mir schon fast gedacht, deswegen die Nachfrage.

Originally Posted by Sedrika

Ich nutze weder die Tapatalk API (noch nie angesehen, wär eig mal nicht schlecht) oder den XML...-Client. Das meiste sind einfache WebRequests, die ich nutze.

XMLRPC ist das Protokoll, welches für die Tapatalk-API verwendet wird.

Das wäre dann ein Verbesserungsvorschlag meinerseits, denn so geht es bedeutend schneller als über WebRequests (Shoutbox ausgenommen). Kannst es dir ja mal ansehen, ne Dokumentation findest du hier:

Beispiele für die Verwendung des XMLRPC-Protokolls unter VB findest du leicht über Google. Das wäre echt ne nette Verbesserung

[Sedrika]

Werd ich mir ansehen, könnte mir sicherlich helfen, wenn ich aus prinzip die Lib neu schreibe, da ich dann unnötiges meist raushaue usw.

[Furkan]

Könntest du noch einfügen, dass man sehen kann wieviele ungelesene PN's man hat und am besten auch noch den Titel + Text davon.

[-PinkiWinki-]

Zum C# Problem

[STAThread]
static void Main(string[] args)
{
   xLibrary.xLibrary a = new xLibrary.xLibrary();
   Console.WriteLine(a.IsEpvpUser());
}

Wieso killt IsEpvpUser() den Prozess wenn das Ergebnis false ist? Sowas sollte eine Library niemals machen, außer es ist WIRKLICH nötig. Außerdem sollte man sowieso niemals sowas wie .Kill() anwenden. Genauso wie die HWID zu generieren und zu kopieren + MessageBox sind komplett unnötig, sowas sollte man immer der Anwendung überlassen!

Ansonsten, Fehlerbehandlung scheint hier zu fehlen.
.IsUser(xLibrary.xLibrary.UserGroup.Banned) ->
"Ungültige Konvertierung von der Zeichenfolge in Typ Double.", evtl. weil ich meine HWID nicht eingetragen habe .

[Sedrika]

Originally Posted by -PinkiWinki-

Zum C# Problem

[STAThread]
static void Main(string[] args)
{
   xLibrary.xLibrary a = new xLibrary.xLibrary();
   Console.WriteLine(a.IsEpvpUser());
}

Wieso killt IsEpvpUser() den Prozess wenn das Ergebnis false ist? Sowas sollte eine Library niemals machen, außer es ist WIRKLICH nötig. Außerdem sollte man sowieso niemals sowas wie .Kill() anwenden. Genauso wie die HWID zu generieren und zu kopieren + MessageBox sind komplett unnötig, sowas sollte man immer der Anwendung überlassen!

Ansonsten, Fehlerbehandlung scheint hier zu fehlen.
.IsUser(xLibrary.xLibrary.UserGroup.Banned) ->
"Ungültige Konvertierung von der Zeichenfolge in Typ Double.", evtl. weil ich meine HWID nicht eingetragen habe .

Ich habe in der Nacht die Lib komplett neu geschrieben und in Klassen unterteilt.
Und das mit dem Kill, war eine sehr alte funk noch, wo ich das ganze getestet hatte.

Ich hau bald ein Update rein mit weiteren Funktionen.

[Lumi]

Du könntest zu deiner .dll noch "Premium Ablauf" Check hinzufügen.
Wenn User = Premium, dann "Premium ablauf: 05-24-2013" oder sowas. ^^

[~.ScoiL]

Und für die die es in AutoIt verwenden möchten:

Ihr öffnet Notepad und kopiert euch den Code.
Dann speichert ihr die Datei in euer Scriptverzeichnes und benennt sie zum Beispiel hello.vb .

 Spoiler

Imports System.Collections.Generic
Imports System.Text
Imports System.Runtime.InteropServices

Namespace myDotNetLibrary
  <ClassInterface(ClassInterfaceType.AutoDual)> _
  Public Class myDotNetClass
	Private myProperty As String

	Public Sub New()
	End Sub

	Public Function myDotNetMethod(input As String) As String
	  Return "Hello " & input
	End Function

	Public Property myDotNetProperty() As String
	  Get
		Return myProperty
	  End Get
	  Set(ByVal value As String)
		myProperty = value
	  End Set
	End Property

  End Class
End Namespace

Hier noch ein Scriptbeispiel:

 Spoiler

$vbc = "C:\WINDOWS\Microsoft.NET\Framework\v4.0.30319\vbc.exe" ;  ; check the path of your version
$RegAsm = "C:\WINDOWS\Microsoft.NET\Framework\v4.0.30319\RegAsm.exe" ; check the path of your version

RunWait($vbc & " /target:library hello.vb", @ScriptDir, @SW_HIDE) ; compile the .net DLL
RunWait($RegAsm & " /codebase hello.dll", @ScriptDir, @SW_HIDE) ; register the .net DLL

$obj = ObjCreate("myDotNetLibrary.myDotNetClass")
$obj.myDotNetProperty = "AutoIt-World"
ConsoleWrite("! " & $obj.myDotNetMethod($obj.myDotNetProperty) & @CRLF)

RunWait($RegAsm & " /unregister hello.dll", @ScriptDir, @SW_HIDE) ; unregister the .net DLL

Quelle:

[Sedrika]

Habe mal die Lib neu geschrieben und in Klassen unterteilt usw.

Habe den Thread auch neugeschrieben und beispiele für Funktionen rausgenommen, da diese nun in der Lib selber vorhanden sind.

Mfg Sedrika

[-PinkiWinki-]

+

xLibraryEx.xLibraryEx.OpenFuncIndex() = Totaler Schwachsinn, für was bietet Microsoft direkt eine XML Dokumentation an in der IDE? Das sind simple Kommentare im Code, die IDE erzeugt daraus dann eine XML...

Wieso muss man von jeder Klasse eine Instanz anlegen? Wie wär es mit static/shared?

Kann es sein das du nirgends .Dispose aufrufst? Das ist ein Memory Leak.

[Sedrika]

Meinst du damit solche kommentare im Source

    ''' <summary>
    ''' Anmelden auf Elitepvpers
    ''' </summary>
    ''' <param name="Username">Benutzername</param>
    ''' <param name="Password">Passwort</param>
    ''' <returns>True = Erfolgreich. False = Fehlgeschlagen.</returns>
    ''' <remarks></remarks>

Die sind drin.

Und wozu soll ich den Inhalt der Post Variable immer wieder disposen, wenn ich sie jedesmal überschreibe ?

Wenn es dich glücklich macht, schreibe ich ruhig vor jeder funk noch ein shared.

[-PinkiWinki-]

Ja die Kommentare meine ich, du musst aber auch die XML Datei mit senden(Und in den Einstellungen erstmal einschalten).

Es ist egal ob du die Variable überschreibst, Dispose MUSS immer aufgerufen werden, das hat schon seinen Sinn. Oft verwenden Klassen native Resourcen, dann sollte man auf jeden Fall Dispose aufrufen, sonst hast du einen schönen Memory Leak im Programm .

[Sedrika]

Ich schreib grad den kram nochmal um.

Edit: Hab es mal nun verändert neu hochgeladen und die v1.11 entfernt.

Ach und Dispose ist nicht nötig um Speicher freizugeben, da .NET dies von alleine dank der Garbagecollection welches ein Mehrstufiger Prozess ist und das ganze von alleine macht.
Dabei werden Objekte die nicht mehr referenziert werden aus dem Speicher geräumt und die, die noch referenziert werden, werden in eine zweite Ebene der Garbagecollection verschoben.

Das was ich jetzt gemacht habe ist alles zu sharen, damit man keine Instanzen mehr braucht.

Jemand der mit einem .NET Code ankommt (in dem fall du mit C#) und sich etwas darüber informiert hat, sollte es eigendlich wissen.

Zudem gibt bsp. Windows 7 alle Speicher vom Programm nach dem beenden wieder frei.

Mfg Sedrika

[-PinkiWinki-]

Eben nicht. Der GC verwaltet nur managed Objekte und ruft auch nicht Dispose auf. Dispose muss selber aufgerufen werden(Oder Using Pattern).

Klassen die das IDisposable Interface implementieren nutzen oft unmanaged Resourcen, welche dann mit Dispose freigegeben werden. Wenn du das Objekt einfach überschreibst bleiben damit die unmanaged Resourcen im Speicher und werden nie gelöscht, was am Ende ein Memory Leak ist . Du solltest dich besser informieren, ich weiß wie der GC funktioniert.

Klar, logisch, nachdem das Programm beendet wird wird alles gelöscht. Aber was wenn die Anwendung länger läuft und öfter die Funktionen nutzt? Am Ende hast du dann einen RAM Verbrauch von 100 MB bzw bekommst eine OutOfMemory Exception im Programm oder hast massig File/Stream/... Handles offen die eigentlich geschlossen werden sollen. Wie gesagt Dispose hat seinen sinn.

Bei manchen Klassen könnte man es theoretisch weglassen, aber das ist einfach unsauber und ein schlechter Stil.

[Sedrika]

Wozu sollte ich IDisposable in meine Subs einbringen um zwischen durch Manuell Resourcen freizugeben, wenn die GC das für mich macht.

Und nur weil mein Stil nicht deinem entspricht ist es och lange nicht schlecht.