new feature: printing the position info while printing the logs ?

[MienDev]

Hi, @langley-agm
is there any plan for adding feature on printing the position info while printing the logs ?

the outputs in debugger console is only show position like logger.ts:79. this position info in logger.ts is useless for locating the error while debugging. but position info file > class > function > line such as app.module.ts line 78 would be very very helpful, even if a function level position would be better.

[langley-agm]

@MienDev
"is useless for locating the error while debugging"
If you log an error you do see see stack trace don't you?

[MienDev]

@langley-agm thank you, sorry for replaying late.

sometimes we just want to log business logic errors, etc. but not throw program logic exception.
and no stack trace will be provided in browser console.

[langley-agm]

There's a difference between log business logic errors (console.error) and throw program logic exception (throw).

logger.error IS logging a business logic error, all its doing at the end is console.error which is exactly what you want to do.

[zjx20]

@langley-agm I think you misunderstood the question. Every line in the console has a position (please see the attached image). But logs logged by angular2-logger just don't have the current position. It's not representing the position in the business logic, that makes the position info less useful.

Here is the solution:

class Logger {
  // copied from angular2-logger
  log1(message?: any, ...optionalParams: any[]) {
    console.log.apply( console, arguments );
  }

  get log2(): Function {
    return console.log.bind(console);
  }
}

const logger = new Logger();

logger.log1("hello");  // give a wrong position

logger.log2("world");  // give a correct position

console.log("I'm here");  // as a reference

I can create a PR if you feel ok with this solution.

[langley-agm]

sometimes we just want to log business logic errors, etc. but not throw program logic exception.
and no stack trace will be provided in browser console.

I think you misunderstood the question.

Which question? lol

[langley-agm]

@zjx20 Yea if you add a PR I'll take a look at it, i'll do some testing and think on the implications of the change.

I'm curious is there a reason for the get and return in log2() ?

[zjx20]

Which question? lol

Well, I think he was wanting every log to have a current position info, but you told him just to use logger.error() all the time.

Why are you doing a return?

Because I am returning a function.

Can you show me how are you calling log2()?

I had already showed you: logger.log2("world");. By the way, the image I attached was just the output of the code I provided. You can give it a try though.

[langley-agm]

but you told him just to use logger.error() all the time

No I didn't. He said he wanted to log errors, and errors do show the stack trace.

Because I am returning a function.

Ok, I'll rephrase my question, why are you returning a function?

I had already showed you

Yea I saw that you put it down there. I edited my message.

[zjx20]

I'm curious is there a reason for the get and return in log2() ?

First of all, console.log.bind(console) is just a function object, you can think it's equivalent to console.log (google can give you more details about the strange syntax, and why we need it). So we can call console.log.bind(console)("hello world") to log a message (just like calling console.log("hello world")).

And the get modifier is a syntax sugar. Without get, we have to rewriter the code as logger.log2()("world").

Hope I was explaining well :)

[langley-agm]

I know what get and return are.
The question is why are you using them?

When you call logger.log2("world"); you get back the function that bind() creates but you never call it, in order to call it you'd have to do something like logger.log2()("world"); similar to your example: console.log.bind(console)("hello world").

It's not a weird syntax, its just a method that returns another method:
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_objects/Function/bind

[langley-agm]

you can think it's equivalent to console.log

They are not equivalent.

console.log its a void function that prints the parameters it receives and doesn't return anything.

console.log.bind(console) bind its a function that returns another function, it applies to any functions not just console.log, and it doesn't print anything on its own, you'd have to call the function that you get back from it like you are doing in console.log.bind(console)("hello world") do try to create the PR so you can test it and it gets clearer.

[zjx20]

When you call logger.log2("world"); you get back the function that bind() creates but you never call it

No, that's not true. logger.log2 is already the function created by bind(), and ("world") is for invoking the returned function.

http://jsbin.com/larafawucu/1/edit?html,js,output

[zjx20]

They are not equivalent.

I was not meaning console.log.bind(console) === console.log, I just said their functionality is the same.

[langley-agm]

I just said their functionality is the same.

Their functionality is not the same.