Press n or j to go to the next uncovered block, b, p or k for the previous block.
| 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 | 1x 1x 1x 1x 1x 1x 1x 1x 7273x 21659x 7008x 114x 137x 137x 137x 12x 125x 37x 37x 37x 30x 7x 7x 1x 6x 6x 1x 5x 5x 4x 1x 1x 1x 125x | import {Issue} from "../issue";
import {ABAPRule} from "./_abap_rule";
import * as Statements from "../abap/2_statements/statements";
import * as Expressions from "../abap/2_statements/expressions";
import {BasicRuleConfig} from "./_basic_rule_config";
import {RuleTag} from "./_irule";
import {ABAPFile} from "../abap/abap_file";
export class PreferReturningToExportingConf extends BasicRuleConfig {
}
export class PreferReturningToExporting extends ABAPRule {
private conf = new PreferReturningToExportingConf();
public getMetadata() {
return {
key: "prefer_returning_to_exporting",
title: "Prefer RETURNING to EXPORTING",
shortDescription: `Prefer RETURNING to EXPORTING. Generic types cannot be RETURNING.`,
extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-returning-to-exporting
https://docs.abapopenchecks.org/checks/44/`,
tags: [RuleTag.Styleguide, RuleTag.SingleFile],
};
}
public getConfig() {
return this.conf;
}
public setConfig(conf: PreferReturningToExportingConf) {
this.conf = conf;
}
public runParsed(file: ABAPFile): Issue[] {
const ret: Issue[] = [];
const stru = file.getStructure();
if (stru === undefined) {
return [];
}
for (const def of stru.findAllStatements(Statements.MethodDef)) {
Iif (def.findFirstExpression(Expressions.MethodDefChanging)) {
continue;
}
const exporting = def.findFirstExpression(Expressions.MethodDefExporting);
if (exporting === undefined) {
continue;
}
const returning = def.findFirstExpression(Expressions.MethodDefReturning);
if (returning !== undefined) {
continue;
}
const params = exporting.findDirectExpressions(Expressions.MethodParam);
if (params.length !== 1) {
continue;
}
const concat = params[0].concatTokens().toUpperCase();
if (concat.endsWith("TYPE ANY")
|| concat.endsWith("TYPE ANY TABLE")
|| concat.endsWith("TYPE C")
|| concat.endsWith("TYPE CLIKE")
|| concat.endsWith("TYPE CSEQUENCE")
|| concat.endsWith("TYPE DATA")
|| concat.endsWith("TYPE DECFLOAT")
|| concat.endsWith("TYPE HASHED TABLE")
|| concat.endsWith("TYPE INDEX TABLE")
|| concat.endsWith("TYPE N")
|| concat.endsWith("TYPE NUMERIC")
|| concat.endsWith("TYPE OBJECT")
|| concat.endsWith("TYPE P")
|| concat.endsWith("TYPE SIMPLE")
|| concat.endsWith("TYPE SORTED TABLE")
|| concat.endsWith("TYPE STANDARD TABLE")
|| concat.endsWith("TYPE TABLE")
|| concat.endsWith("TYPE X")
|| concat.endsWith("TYPE XSEQUENCE")) {
continue;
}
const token = params[0].getFirstToken();
const issue = Issue.atToken(file, token, "Prefer RETURNING to EXPORTING", this.getMetadata().key, this.conf.severity);
ret.push(issue);
}
return ret;
}
}
|