So far i'm doing right. is that what you say sir?
To be honest, I can't answer that without more knowledge of the actual problem. But just FYI, showing the query grid, because it tends to truncate things, is not the best choice to show what you have done. If you switch that to SQL view and then cut/paste the SQL string for posting, that would tell us what you are doing.
Having said that, I inferred the desire to group things for summation, but the query grid you showed us doesn't look like a proper summation query. It is missing a row that would describe whether a field was a grouping element, summation element, or criterion.
Learn how to use an aggregate function in Access to sum the data in a query result set.
support.microsoft.com