Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chilkat ActiveX Libraries #8

Open
sancarn opened this issue Jun 3, 2022 · 0 comments
Open

Chilkat ActiveX Libraries #8

sancarn opened this issue Jun 3, 2022 · 0 comments
Labels
awesome-watch Watch library/resource for significant improvement before adding enhancement New feature or request

Comments

@sancarn
Copy link
Owner

sancarn commented Jun 3, 2022

After reddit user techjp indicated that it might be awesome enough. As a result we added it to:

Frameworks:

* [🖼][p_win][][a_all][💲](# "Some modules free; Some require $299+ license")[👽](# "Requires external dll") [Chilkat AXL](https://www.chilkatsoft.com/refdoc/activex.asp) - Numerous useful ActiveX libraries hosted from a single zip.

WebTools:

* From Frameworks
    * In `Chilkat AXL` find `HTTP`, `HTTPRequest`, `HTTPResponse`, `Auth*`, `OAuth*`, `REST`, `WebSocket` and numerous other useful web-related libraries.

This evening Kallum and I have been taking a look at it and we aren't certain it is awesome, so would like to put it to a vote. 5 votes and we will re-add it. I've added my vote to take into consideration techjps suggestion.

Reasons for removal:

The library does not follow any constructor naming conventions e.g. loadXYZ.

'Typical name convention
With new ChilkatCsv 
  .LoadFile(...)
End With

'Looks okay and expected
With ChilkatJson
  .LoadString(...)
End with

'would have prefered LoadFile... but ok
With new ChilkatXml
  .LoadXMLFile(...)   
End Wit

'Convention not followed
With new ChilkatGzip
  .ReadFile(...)
End With

'Convention not followed
With new ChilkatHttp
  .PostJson(...)
End With

It is important for a framework to remain internally consistent, as this will reduce the learning curve of the library. This is especially important when

Very few examples

Again difficult to learn what is correct to do without good and full examples.

Poor error checking(?)

In order to get a feeling for the library I tried to parse a csv and set the column "c" to the sum of "a" and "b". This code didn't error, but it didn't modify the csv either... I'm a little unsure whether I'm doing the right thing or not, but the lack of errors sure don't help.

Sub t()
  Dim x As New Chilkat_v9_5_0.ChilkatCsv
  Call x.LoadFile("C:\Users\sancarn\Desktop\tbd\chilkatax-9.5.0-x64\_test.csv")
  
  Dim i As Long: i = 1
  For i = 1 To x.NumRows
    Call x.SetCellByName(i, "c", x.GetCellByName(i, "a") + x.GetCellByName(i, "b"))
  Next
  
  Call x.SaveFile("C:\Users\sancarn\Desktop\tbd\chilkatax-9.5.0-x64\_test.csv")
End Sub
@sancarn sancarn added enhancement New feature or request awesome-watch Watch library/resource for significant improvement before adding labels Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awesome-watch Watch library/resource for significant improvement before adding enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant